Re: [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision

From	Clemens Gruber <clemens.gruber@pqgruber.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision
Date	2016-06-04 11:10 +0200
Message-ID	<rGfKF-6Wc-7@gated-at.bofh.it> (permalink)
References	<rFAeu-5Zh-19@gated-at.bofh.it> <rFD2G-7GN-11@gated-at.bofh.it> <rG08W-5za-11@gated-at.bofh.it> <rGca6-4Ez-15@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

Hi Eric,

On Sat, Jun 04, 2016 at 07:18:28AM +0200, Eric Nelson wrote:
> AFAIK, the SGTL5000 versions < 0x11 are like Sasquatch: I've seen
> no real proof of their existence. I tried to chase down when this
> code was introduced, but it seems to have been around since
> the dawn of the driver.

> 
> Clemens, if you're really trying to control an external regulator,
> I think you'll need lots more than this patch. There are some
> fundamental flaws in regulator handling and I put together some
> RFC patches to address them in February 2015:
> 	http://mailman.alsa-project.org/pipermail/alsa-devel/2015-February/thread.html#88353
> 
> The long and short of it is that at least one of the regulators
> would need to be initialized before the SGTL5000 probes.

Thank you very much for your input on this. So we can assume that no
user of the sgtl5000 is controlling an external VDDD regulator, because
this does not work without your patch series from Februar 2015.

As far as not enabling the internal LDO when using external VDDD and
avoiding the misleading "Using internal LDO..." message at boot, do you
think my patch would be OK?

Regards,
Clemens

Thread

[PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision Clemens Gruber <clemens.gruber@pqgruber.com> - 2016-06-02 14:50 +0200
  Re: [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision Fabio Estevam <festevam@gmail.com> - 2016-06-02 17:50 +0200
    Re: [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision Mark Brown <broonie@kernel.org> - 2016-06-02 19:00 +0200
      Re: [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision Clemens Gruber <clemens.gruber@pqgruber.com> - 2016-06-03 18:00 +0200
    Re: [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision Eric Nelson <eric@nelint.com> - 2016-06-03 18:30 +0200
      Re: [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision Eric Nelson <eric@nelint.com> - 2016-06-04 07:20 +0200
        Re: [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision Clemens Gruber <clemens.gruber@pqgruber.com> - 2016-06-04 11:10 +0200
          Re: [alsa-devel] [PATCH] ASoC: sgtl5000: only check VDDD-supply, not  revision Clemens Gruber <clemens.gruber@pqgruber.com> - 2016-06-06 01:20 +0200
            Re: [alsa-devel] [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision Fabio Estevam <festevam@gmail.com> - 2016-06-06 03:30 +0200
              Re: [alsa-devel] [PATCH] ASoC: sgtl5000: only check VDDD-supply, not  revision Clemens Gruber <clemens.gruber@pqgruber.com> - 2016-06-06 18:50 +0200
                Re: [alsa-devel] [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision Fabio Estevam <festevam@gmail.com> - 2016-06-06 19:20 +0200
  Re: [PATCH] ASoC: sgtl5000: only check VDDD-supply, not revision Mark Brown <broonie@kernel.org> - 2016-06-06 20:00 +0200

csiph-web