Re: [PATCH 4.6 000/121] 4.6.2-stable review

Mark Brown
2016-06-06T23:30:34Z

[-- Attachment #1: Type: text/plain, Size: 1361 bytes --]

On Mon, Jun 06, 2016 at 06:43:47PM -0400, Javier Martinez Canillas wrote:
> On 06/06/2016 06:32 PM, Tyler Baker wrote:

> > I have bisected[1] these two failures down to 5e3ca2b349b1
> > ("regulator: Try to resolve regulators supplies on registration") and
> > confirmed that reverting this patch on top of this stable queue fixes
> > the boot issue.

> I'm not familiar with the apq8016-sbc board or Qualcomm platforms in general
> but this patch is known to cause deadlocks when looking up regulators that
> were not registered via OF. This was an issue on at least some Nvidia boards.

The Qualcomm platforms are running into trouble because the qcom_smd
driver is broken and apparently nobody checks -next works for their
boards since they only noticed after -rc1 in spite of the change having
been in -next for a good few weeks. You need at least "regulator:
qcom_smd: add list_voltage callback" and "regulator: qcom_smd: add
regulator ops for pm8941 lnldo" from my tree which I didn't send to
Linus yet since we only got them in at the end of last week. I think
that's probably OK for v4.6 TBH, the platform is fairly incomplete
upstream and the original patch fixes issues for other systems.

> So since this patch made it to stable, commit a21513742308 ("regulator: Fix
> deadlock during regulator registration") [0] should also be included.

Please.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]