From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nicolin Chen Subject: Re: [PATCHv2 5/8] ASoC: SGTL5000: Enhance the SGTL5000 codec driver about regulator. Date: Fri, 1 Nov 2013 18:02:03 +0800 Message-ID: <20131101100201.GF28401@MrMyself> References: <1383289495-24523-1-git-send-email-Li.Xiubo@freescale.com> <1383289495-24523-6-git-send-email-Li.Xiubo@freescale.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: Content-Disposition: inline In-Reply-To: <1383289495-24523-6-git-send-email-Li.Xiubo-KZfg59tc24xl57MIdRCFDg@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Xiubo Li Cc: r65073-KZfg59tc24xl57MIdRCFDg@public.gmane.org, timur-N01EOCouUvQ@public.gmane.org, lgirdwood-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, r64188-KZfg59tc24xl57MIdRCFDg@public.gmane.org, rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org, pawel.moll-5wv7dgnIgG8@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org, ian.campbell-Sxgqhf6Nn4DQT0dZR+AlfA@public.gmane.org, rob-VoJi6FS/r0vR7s880joybQ@public.gmane.org, linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org, perex-/Fr2/VpizcU@public.gmane.org, tiwai-l3A5Bk7waGM@public.gmane.org, grant.likely-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, fabio.estevam-KZfg59tc24xl57MIdRCFDg@public.gmane.org, LW-bxm8fMRDkQLDiMYJYoSAnRvVK+yQ3ZXh@public.gmane.org, oskar-fYPSZ7JpQqsAvxtiuMwx3w@public.gmane.org, shawn.guo-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org, b18965-KZfg59tc24xl57MIdRCFDg@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-doc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, alsa-devel-K7yf7f+aM1XWsZ/bQMPhNw@public.gmane.org, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org List-Id: devicetree@vger.kernel.org On Fri, Nov 01, 2013 at 03:04:52PM +0800, Xiubo Li wrote: > On VF610 series there are no regulators used, and now whether the > CONFIG_REGULATOR mirco is enabled or not, for the VF610 audio micro? or macro? > patch series, the board cannot be probe successfully. > And this patch will solve this issue. > I finally got your idea about what this patch does. But it seems to be more likely a work around. Maybe I here can't think it through comprehensively, but I think you can try to add a fixed regulator to sgtl5000 in the devicetree, rather than disabling common functions even if not breaking others. > Signed-off-by: Xiubo Li > --- > sound/soc/codecs/sgtl5000.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/sound/soc/codecs/sgtl5000.c b/sound/soc/codecs/sgtl5000.c > index 1f4093f..c2f6d86 100644 > --- a/sound/soc/codecs/sgtl5000.c > +++ b/sound/soc/codecs/sgtl5000.c > @@ -61,6 +61,7 @@ static const struct reg_default sgtl5000_reg_defaults[] = { > { SGTL5000_DAP_AVC_DECAY, 0x0050 }, > }; > > +#ifdef CONFIG_REGULATOR > /* regulator supplies for sgtl5000, VDDD is an optional external supply */ > enum sgtl5000_regulator_supplies { > VDDA, > @@ -93,6 +94,9 @@ static struct regulator_init_data ldo_init_data = { > .num_consumer_supplies = 1, > .consumer_supplies = &ldo_consumer[0], > }; > +#else > +#define SGTL5000_SUPPLY_NUM 0 > +#endif > > /* > * sgtl5000 internal ldo regulator, > @@ -112,7 +116,9 @@ struct sgtl5000_priv { > int master; /* i2s master or not */ > int fmt; /* i2s data format */ > struct regulator_bulk_data supplies[SGTL5000_SUPPLY_NUM]; > +#ifdef CONFIG_REGULATOR > struct ldo_regulator *ldo; > +#endif > struct regmap *regmap; > struct clk *mclk; > }; > @@ -879,6 +885,7 @@ static int ldo_regulator_remove(struct snd_soc_codec *codec) > return 0; > } > #else > +#ifndef CONFIG_SND_SOC_FSL_SGTL5000_VF610 Checking a platform or SoC specific define here is just a bit.... Could you pls find a better solution? Like I said at first, use fixed regulator or figure out why your System would hang with current sgtl5000 driver. If it really has a critical bug in its regulator-related code, I think you can then make a greater contribution by fixing the bug rather than bypassing it. Best regards, Nicolin Chen > static int ldo_regulator_register(struct snd_soc_codec *codec, > struct regulator_init_data *init_data, > int voltage) > @@ -886,6 +893,7 @@ static int ldo_regulator_register(struct snd_soc_codec *codec, > dev_err(codec->dev, "this setup needs regulator support in the kernel\n"); > return -EINVAL; > } > +#endif > > static int ldo_regulator_remove(struct snd_soc_codec *codec) > { > @@ -1137,6 +1145,7 @@ static int sgtl5000_resume(struct snd_soc_codec *codec) > #define sgtl5000_resume NULL > #endif /* CONFIG_SUSPEND */ > > +#ifdef CONFIG_REGULATOR > /* > * sgtl5000 has 3 internal power supplies: > * 1. VAG, normally set to vdda/2 > @@ -1373,6 +1382,7 @@ err_regulator_free: > return ret; > > } > +#endif > > static int sgtl5000_probe(struct snd_soc_codec *codec) > { > @@ -1387,6 +1397,7 @@ static int sgtl5000_probe(struct snd_soc_codec *codec) > return ret; > } > > +#ifdef CONFIG_REGULATOR > ret = sgtl5000_enable_regulators(codec); > if (ret) > return ret; > @@ -1395,6 +1406,7 @@ static int sgtl5000_probe(struct snd_soc_codec *codec) > ret = sgtl5000_set_power_regs(codec); > if (ret) > goto err; > +#endif > > /* enable small pop, introduce 400ms delay in turning off */ > snd_soc_update_bits(codec, SGTL5000_CHIP_REF_CTRL, > -- > 1.8.4 > -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html