RE: [PATCHv1] ASoC: SGTL5000: Fix kernel failed while getting regulator consumers

2013-11-27 Thread Li Xiubo
> Subject: Re: [PATCHv1] ASoC: SGTL5000: Fix kernel failed while getting > regulator consumers > > On Wed, Nov 27, 2013 at 08:13:03AM +, Li Xiubo wrote: > > Please fix your mailer to word wrap within paragraphs, it makes your mail > much more legible. > Yes, I

Re: [PATCHv1] ASoC: SGTL5000: Fix kernel failed while getting regulator consumers

2013-11-27 Thread Mark Brown
On Wed, Nov 27, 2013 at 08:13:03AM +, Li Xiubo wrote: Please fix your mailer to word wrap within paragraphs, it makes your mail much more legible. > There is one dependency patch: "regulator: core: Provide a dummy regulator > with full constraints". > From the dependency patch, we can see t

RE: [PATCHv1] ASoC: SGTL5000: Fix kernel failed while getting regulator consumers

2013-11-27 Thread Li Xiubo
> > +static int sgtl5000_external_vddd_used(struct snd_soc_codec *codec) > > +{ > > + struct regulator *consumer; > > + struct sgtl5000_priv *sgtl5000 = snd_soc_codec_get_drvdata(codec); > > + > > + consumer = regulator_get(codec->dev, sgtl5000- > >supplies[VDDD].supply); > > + if (IS_ERR(c

RE: [PATCHv1] ASoC: SGTL5000: Fix kernel failed while getting regulator consumers

2013-11-26 Thread Li Xiubo
> > else { > > ret = sgtl5000_replace_vddd_with_ldo(codec); > > > You could fix the coding style issue (braces on both branches of the if > clause) here too. > > Yes, I will. Thanks. N�r��yb�X��ǧv�^�)޺{.n�+{zX����ܨ}���Ơz�&j:+v���zZ+��+zf���h���~i���z�

Re: [PATCHv1] ASoC: SGTL5000: Fix kernel failed while getting regulator consumers

2013-11-26 Thread Mark Brown
On Tue, Nov 26, 2013 at 05:55:13PM +0800, Xiubo Li wrote: > +static int sgtl5000_external_vddd_used(struct snd_soc_codec *codec) > +{ > + struct regulator *consumer; > + struct sgtl5000_priv *sgtl5000 = snd_soc_codec_get_drvdata(codec); > + > + consumer = regulator_get(codec->dev, sgtl

Re: [PATCHv1] ASoC: SGTL5000: Fix kernel failed while getting regulator consumers

2013-11-26 Thread Lothar Waßmann
Hi, Xiubo Li wrote: > SGTL5000 codec allows to use the internal LDO instead of VDDD, if the > internal LDO is used, this caused the following kernel failed while trying > to get the external VDDD regulator consumer. > > Adding sgtl5000_external_vddd_used() to fix it. And this has been tested on >