lkml.org 
[lkml]   [2010]   [Jun]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [RFC][PATCH 21/26] alsa: ASoC: Add JZ4740 ASoC support
    On Wed, Jun 02, 2010 at 09:12:27PM +0200, Lars-Peter Clausen wrote:

    Again, overall very good.

    > +static int jz4740_i2s_set_clkdiv(struct snd_soc_dai *dai, int div_id, int div)
    > +{
    > + struct jz4740_i2s *i2s = jz4740_dai_to_i2s(dai);
    > +
    > + switch (div_id) {
    > + case JZ4740_I2S_BIT_CLK:
    > + if (div & 1 || div > 16)
    > + return -EINVAL;
    > + jz4740_i2s_write(i2s, JZ_REG_AIC_CLK_DIV, div - 1);
    > + break;
    > + default:
    > + return -EINVAL;
    > + }
    > +
    > + return 0;
    > +}

    You can probably figure out the bit clock automatically by default...

    > + if (dai->active) {
    > + conf = jz4740_i2s_read(i2s, JZ_REG_AIC_CONF);
    > + conf &= ~JZ_AIC_CONF_ENABLE;
    > + jz4740_i2s_write(i2s, JZ_REG_AIC_CONF, conf);
    > +
    > + clk_disable(i2s->clk_i2s);
    > + }
    > +
    > + clk_disable(i2s->clk_aic);

    Might make sense to manage this clock dynamically at runtime too for a
    little extra power saving?

    > + i2s->clk_aic = clk_get(&pdev->dev, "aic");
    > + if (IS_ERR(i2s->clk_aic)) {
    > + ret = PTR_ERR(i2s->clk_aic);
    > + goto err_iounmap;
    > + }
    > +
    > + i2s->clk_i2s = clk_get(&pdev->dev, "i2s");
    > + if (IS_ERR(i2s->clk_i2s)) {
    > + ret = PTR_ERR(i2s->clk_i2s);
    > + goto err_iounmap;
    > + }

    Ideally you'd free the AIC clock when unwinding (and later stop it after
    it was enabled). Though since you don't do any error checking after
    this point it's kind of academic :)


    \
     
     \ /
      Last update: 2010-06-03 19:57    [W:4.114 / U:0.572 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site