Re: [PATCH 1/2] ASoC: sgtl5000: Fix define for SGTL5000_BIAS_R_MASK

From: Wolfram Sang
Date: Wed Oct 19 2011 - 06:06:08 EST


On Wed, Oct 19, 2011 at 09:06:57AM +0100, Girdwood, Liam wrote:
> On 19 October 2011 03:54, Axel Lin <axel.lin@xxxxxxxxx> wrote:
> > According to the datasheet:
> > CHIP_MIC_CTRL 0x002A
> >
> > BITS[9:8] BIAS_RESISTOR
> >        0x0 = Powerd off
> >        0x1 = 2.0 kohm
> >        0x2 = 4.0 kohm
> >        0x3 = 8.0 kohm
> >
> > Thus SGTL5000_BIAS_R_MASK should be defined as 0x0300 instead of 0x0200.
> >
> > Signed-off-by: Axel Lin <axel.lin@xxxxxxxxx>
> > ---
> >  sound/soc/codecs/sgtl5000.h |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> >
> > diff --git a/sound/soc/codecs/sgtl5000.h b/sound/soc/codecs/sgtl5000.h
> > index eec3ab3..8a9f435 100644
> > --- a/sound/soc/codecs/sgtl5000.h
> > +++ b/sound/soc/codecs/sgtl5000.h
> > @@ -280,7 +280,7 @@
> >  /*
> >  * SGTL5000_CHIP_MIC_CTRL
> >  */
> > -#define SGTL5000_BIAS_R_MASK                   0x0200
> > +#define SGTL5000_BIAS_R_MASK                   0x0300
> >  #define SGTL5000_BIAS_R_SHIFT                  8
> >  #define SGTL5000_BIAS_R_WIDTH                  2
> >  #define SGTL5000_BIAS_R_off                    0x0
> > --
> > 1.7.5.4
> >
> >
> >
> >
>
> Both
>
> Acked-by: Liam Girdwood <lrg@xxxxxx>

And

Acked-by: Wolfram Sang <w.sang@xxxxxxxxxxxxxx>

--
Pengutronix e.K. | Wolfram Sang |
Industrial Linux Solutions | http://www.pengutronix.de/ |

Attachment: signature.asc
Description: Digital signature