| Message ID | 20171212074208.30753-1-javierm@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Tue, Dec 12, 2017 at 8:42 AM, Javier Martinez Canillas <javierm@redhat.com> wrote: > Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x > Mixer nodes") disabled the Mixer node by default in the DTSI and enabled > for each Exynos 542x DTS. But unfortunately it missed to enable it for the > Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC variant. > > Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x Mixer nodes") > Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> > Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> > > --- > > Changes in v2: > - Remove RFT tag. Thanks guys! However I still would like to see a tested-by for this on Peach Pi (AFAIU, Marek's only acked the code/solution). Best regards, Krzysztof > - Add Marek's Acked-by tag. > - Add fixes tag. > > arch/arm/boot/dts/exynos5800-peach-pi.dts | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts > index b2b95ff205e8..0029ec27819c 100644 > --- a/arch/arm/boot/dts/exynos5800-peach-pi.dts > +++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts > @@ -664,6 +664,10 @@ > status = "okay"; > }; > > +&mixer { > + status = "okay"; > +}; > + > /* eMMC flash */ > &mmc_0 { > status = "okay"; > -- > 2.14.3 >
On Tue, Dec 12, 2017 at 10:55 AM, Krzysztof Kozlowski <krzk@kernel.org> wrote: > On Tue, Dec 12, 2017 at 8:42 AM, Javier Martinez Canillas > <javierm@redhat.com> wrote: >> Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x >> Mixer nodes") disabled the Mixer node by default in the DTSI and enabled >> for each Exynos 542x DTS. But unfortunately it missed to enable it for the >> Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC variant. >> >> Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x Mixer nodes") >> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> >> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> >> >> --- >> >> Changes in v2: >> - Remove RFT tag. > > Thanks guys! However I still would like to see a tested-by for this on > Peach Pi (AFAIU, Marek's only acked the code/solution). On the other hand I could just apply it for my for-next branch and we'll see if it fixes kernel-ci boot tests... Not a nice way of testing but apparently no one has Peach Pi. Best regards, Krzysztof
Hi Krzysztof, On 2017-12-12 11:09, Krzysztof Kozlowski wrote: > On Tue, Dec 12, 2017 at 10:55 AM, Krzysztof Kozlowski <krzk@kernel.org> wrote: >> On Tue, Dec 12, 2017 at 8:42 AM, Javier Martinez Canillas >> <javierm@redhat.com> wrote: >>> Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x >>> Mixer nodes") disabled the Mixer node by default in the DTSI and enabled >>> for each Exynos 542x DTS. But unfortunately it missed to enable it for the >>> Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC variant. >>> >>> Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x Mixer nodes") >>> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> >>> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> >>> >>> --- >>> >>> Changes in v2: >>> - Remove RFT tag. >> Thanks guys! However I still would like to see a tested-by for this on >> Peach Pi (AFAIU, Marek's only acked the code/solution). > On the other hand I could just apply it for my for-next branch and > we'll see if it fixes kernel-ci boot tests... Not a nice way of > testing but apparently no one has Peach Pi. Frankly, I don't expect that this will solve the boot hang issue on PeachPi. However it should at least hide the unbalanced regulator issue. Best regards
Hello Krzysztof, On 12/12/2017 11:09 AM, Krzysztof Kozlowski wrote: > On Tue, Dec 12, 2017 at 10:55 AM, Krzysztof Kozlowski <krzk@kernel.org> wrote: >> On Tue, Dec 12, 2017 at 8:42 AM, Javier Martinez Canillas >> <javierm@redhat.com> wrote: >>> Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x >>> Mixer nodes") disabled the Mixer node by default in the DTSI and enabled >>> for each Exynos 542x DTS. But unfortunately it missed to enable it for the >>> Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC variant. >>> >>> Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x Mixer nodes") >>> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> >>> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> >>> >>> --- >>> >>> Changes in v2: >>> - Remove RFT tag. I removed the tag because Marek acked it and the patch has merits on its own regardless of the boot issue (it just introduces a change that was missed in the mentioned commit). >> >> Thanks guys! However I still would like to see a tested-by for this on >> Peach Pi (AFAIU, Marek's only acked the code/solution). > > On the other hand I could just apply it for my for-next branch and > we'll see if it fixes kernel-ci boot tests... Not a nice way of As Marek said this probably won't solve the issue, or at best it would just mask it. Seems the problem is on the probe deferral path in exynos-drm driver. > testing but apparently no one has Peach Pi. > I do have one, but I haven't used it for months and don't have time to setup a test environment now. I will probably do that this weekend to dig deeper. > Best regards, > Krzysztof > Best regards,
On 12/12/17 10:17, Marek Szyprowski wrote: > Hi Krzysztof, > > On 2017-12-12 11:09, Krzysztof Kozlowski wrote: >> On Tue, Dec 12, 2017 at 10:55 AM, Krzysztof Kozlowski <krzk@kernel.org> wrote: >>> On Tue, Dec 12, 2017 at 8:42 AM, Javier Martinez Canillas >>> <javierm@redhat.com> wrote: >>>> Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x >>>> Mixer nodes") disabled the Mixer node by default in the DTSI and enabled >>>> for each Exynos 542x DTS. But unfortunately it missed to enable it for the >>>> Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC variant. >>>> >>>> Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x Mixer nodes") >>>> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> >>>> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> >>>> >>>> --- >>>> >>>> Changes in v2: >>>> - Remove RFT tag. >>> Thanks guys! However I still would like to see a tested-by for this on >>> Peach Pi (AFAIU, Marek's only acked the code/solution). >> On the other hand I could just apply it for my for-next branch and >> we'll see if it fixes kernel-ci boot tests... Not a nice way of >> testing but apparently no one has Peach Pi. > > Frankly, I don't expect that this will solve the boot hang issue on PeachPi. > However it should at least hide the unbalanced regulator issue. We have a peach-pi in our LAVA lab so I've tested it and actually, it does fix the hang on v4.15-rc3: https://lava.collabora.co.uk/scheduler/job/1019877 https://lava.collabora.co.uk/scheduler/job/1019878 I ran it twice and it booted both times. I also ran the same boot tests with the same kernel but the dtb from v4.15-rc3 without the fix to double check and these failed: https://lava.collabora.co.uk/scheduler/job/1019879 https://lava.collabora.co.uk/scheduler/job/1019880 Tested-by: Guillaume Tucker <guillaume.tucker@collabora.com> Thanks for the fix! Guillaume
[adding Inki to cc list] Hello Guillaume, On 12/12/2017 11:43 AM, Guillaume Tucker wrote: > On 12/12/17 10:17, Marek Szyprowski wrote: >> Hi Krzysztof, >> >> On 2017-12-12 11:09, Krzysztof Kozlowski wrote: >>> On Tue, Dec 12, 2017 at 10:55 AM, Krzysztof Kozlowski <krzk@kernel.org> wrote: >>>> On Tue, Dec 12, 2017 at 8:42 AM, Javier Martinez Canillas >>>> <javierm@redhat.com> wrote: >>>>> Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x >>>>> Mixer nodes") disabled the Mixer node by default in the DTSI and enabled >>>>> for each Exynos 542x DTS. But unfortunately it missed to enable it for the >>>>> Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC variant. >>>>> >>>>> Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x Mixer nodes") >>>>> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> >>>>> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> >>>>> >>>>> --- >>>>> >>>>> Changes in v2: >>>>> - Remove RFT tag. >>>> Thanks guys! However I still would like to see a tested-by for this on >>>> Peach Pi (AFAIU, Marek's only acked the code/solution). >>> On the other hand I could just apply it for my for-next branch and >>> we'll see if it fixes kernel-ci boot tests... Not a nice way of >>> testing but apparently no one has Peach Pi. >> >> Frankly, I don't expect that this will solve the boot hang issue on PeachPi. >> However it should at least hide the unbalanced regulator issue. > > We have a peach-pi in our LAVA lab so I've tested it and > actually, it does fix the hang on v4.15-rc3: > > https://lava.collabora.co.uk/scheduler/job/1019877 > https://lava.collabora.co.uk/scheduler/job/1019878 > > I ran it twice and it booted both times. I also ran the same > boot tests with the same kernel but the dtb from v4.15-rc3 > without the fix to double check and these failed: > > https://lava.collabora.co.uk/scheduler/job/1019879 > https://lava.collabora.co.uk/scheduler/job/1019880 > > > Tested-by: Guillaume Tucker <guillaume.tucker@collabora.com> > > > Thanks for the fix! > Thanks for testing! Now, I think the exynos-drm driver should cope with an incorrect DTB and don't crash the machine, the driver should only fail to probe and lead to a working machine with no display. But as mentioned, that's a different issue and orthogonal to the DTS fix. > Guillaume > Best regards,
On Tue, Dec 12, 2017 at 11:43 AM, Guillaume Tucker <guillaume.tucker@collabora.com> wrote: > On 12/12/17 10:17, Marek Szyprowski wrote: >> >> Hi Krzysztof, >> >> On 2017-12-12 11:09, Krzysztof Kozlowski wrote: >>> >>> On Tue, Dec 12, 2017 at 10:55 AM, Krzysztof Kozlowski <krzk@kernel.org> >>> wrote: >>>> >>>> On Tue, Dec 12, 2017 at 8:42 AM, Javier Martinez Canillas >>>> <javierm@redhat.com> wrote: >>>>> >>>>> Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos >>>>> 542x >>>>> Mixer nodes") disabled the Mixer node by default in the DTSI and >>>>> enabled >>>>> for each Exynos 542x DTS. But unfortunately it missed to enable it for >>>>> the >>>>> Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC >>>>> variant. >>>>> >>>>> Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos >>>>> 542x Mixer nodes") >>>>> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> >>>>> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> >>>>> >>>>> --- >>>>> >>>>> Changes in v2: >>>>> - Remove RFT tag. >>>> >>>> Thanks guys! However I still would like to see a tested-by for this on >>>> Peach Pi (AFAIU, Marek's only acked the code/solution). >>> >>> On the other hand I could just apply it for my for-next branch and >>> we'll see if it fixes kernel-ci boot tests... Not a nice way of >>> testing but apparently no one has Peach Pi. >> >> >> Frankly, I don't expect that this will solve the boot hang issue on >> PeachPi. >> However it should at least hide the unbalanced regulator issue. > > > We have a peach-pi in our LAVA lab so I've tested it and > actually, it does fix the hang on v4.15-rc3: > > https://lava.collabora.co.uk/scheduler/job/1019877 > https://lava.collabora.co.uk/scheduler/job/1019878 > > I ran it twice and it booted both times. I also ran the same > boot tests with the same kernel but the dtb from v4.15-rc3 > without the fix to double check and these failed: > > https://lava.collabora.co.uk/scheduler/job/1019879 > https://lava.collabora.co.uk/scheduler/job/1019880 > > > Tested-by: Guillaume Tucker <guillaume.tucker@collabora.com> Thank you! Best regards, Krzysztof
Hi Guillaume, On 2017-12-12 11:43, Guillaume Tucker wrote: > On 12/12/17 10:17, Marek Szyprowski wrote: >> Hi Krzysztof, >> >> On 2017-12-12 11:09, Krzysztof Kozlowski wrote: >>> On Tue, Dec 12, 2017 at 10:55 AM, Krzysztof Kozlowski >>> <krzk@kernel.org> wrote: >>>> On Tue, Dec 12, 2017 at 8:42 AM, Javier Martinez Canillas >>>> <javierm@redhat.com> wrote: >>>>> Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to >>>>> Exynos 542x >>>>> Mixer nodes") disabled the Mixer node by default in the DTSI and >>>>> enabled >>>>> for each Exynos 542x DTS. But unfortunately it missed to enable it >>>>> for the >>>>> Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC >>>>> variant. >>>>> >>>>> Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to >>>>> Exynos 542x Mixer nodes") >>>>> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> >>>>> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> >>>>> >>>>> --- >>>>> >>>>> Changes in v2: >>>>> - Remove RFT tag. >>>> Thanks guys! However I still would like to see a tested-by for this on >>>> Peach Pi (AFAIU, Marek's only acked the code/solution). >>> On the other hand I could just apply it for my for-next branch and >>> we'll see if it fixes kernel-ci boot tests... Not a nice way of >>> testing but apparently no one has Peach Pi. >> >> Frankly, I don't expect that this will solve the boot hang issue on >> PeachPi. >> However it should at least hide the unbalanced regulator issue. > > We have a peach-pi in our LAVA lab so I've tested it and > actually, it does fix the hang on v4.15-rc3: > > https://lava.collabora.co.uk/scheduler/job/1019877 > https://lava.collabora.co.uk/scheduler/job/1019878 > > I ran it twice and it booted both times. I also ran the same > boot tests with the same kernel but the dtb from v4.15-rc3 > without the fix to double check and these failed: > > https://lava.collabora.co.uk/scheduler/job/1019879 > https://lava.collabora.co.uk/scheduler/job/1019880 > > > Tested-by: Guillaume Tucker <guillaume.tucker@collabora.com> > > > Thanks for the fix! Well, thanks for the test! It proves that there the boot failure is caused by an issue somewhere in the error path of Exynos DRM, Analogix DP, Simple Panel or other drivers. This patch simply hides it by fixing the source issue of the Exynos DRM initialization failure. :-) I hope Javier will be able to investigate the discussed hang issue later, as fixing it is also imho important. Best regards
On 12/12/17 10:55, Marek Szyprowski wrote: > Hi Guillaume, > > On 2017-12-12 11:43, Guillaume Tucker wrote: >> On 12/12/17 10:17, Marek Szyprowski wrote: >>> Hi Krzysztof, >>> >>> On 2017-12-12 11:09, Krzysztof Kozlowski wrote: >>>> On Tue, Dec 12, 2017 at 10:55 AM, Krzysztof Kozlowski <krzk@kernel.org> wrote: >>>>> On Tue, Dec 12, 2017 at 8:42 AM, Javier Martinez Canillas >>>>> <javierm@redhat.com> wrote: >>>>>> Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x >>>>>> Mixer nodes") disabled the Mixer node by default in the DTSI and enabled >>>>>> for each Exynos 542x DTS. But unfortunately it missed to enable it for the >>>>>> Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC variant. >>>>>> >>>>>> Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x Mixer nodes") >>>>>> Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> >>>>>> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> >>>>>> >>>>>> --- >>>>>> >>>>>> Changes in v2: >>>>>> - Remove RFT tag. >>>>> Thanks guys! However I still would like to see a tested-by for this on >>>>> Peach Pi (AFAIU, Marek's only acked the code/solution). >>>> On the other hand I could just apply it for my for-next branch and >>>> we'll see if it fixes kernel-ci boot tests... Not a nice way of >>>> testing but apparently no one has Peach Pi. >>> >>> Frankly, I don't expect that this will solve the boot hang issue on PeachPi. >>> However it should at least hide the unbalanced regulator issue. >> >> We have a peach-pi in our LAVA lab so I've tested it and >> actually, it does fix the hang on v4.15-rc3: >> >> https://lava.collabora.co.uk/scheduler/job/1019877 >> https://lava.collabora.co.uk/scheduler/job/1019878 >> >> I ran it twice and it booted both times. I also ran the same >> boot tests with the same kernel but the dtb from v4.15-rc3 >> without the fix to double check and these failed: >> >> https://lava.collabora.co.uk/scheduler/job/1019879 >> https://lava.collabora.co.uk/scheduler/job/1019880 >> >> >> Tested-by: Guillaume Tucker <guillaume.tucker@collabora.com> >> >> >> Thanks for the fix! > > Well, thanks for the test! It proves that there the boot failure is > caused by an issue somewhere in the error path of Exynos DRM, Analogix > DP, Simple Panel or other drivers. > > This patch simply hides it by fixing the source issue of the Exynos > DRM initialization failure. :-) > > I hope Javier will be able to investigate the discussed hang issue > later, as fixing it is also imho important. Sure. This device tree change is needed to get HDMI to work so it's still a fix for that. Also it's good to know that nothing else breaks when the driver issue is "hidden". Might be worth testing on -next as well as it might help spot any new issues that haven't been merged in mainline yet, or in general give another data point. Guillaume
On Tue, Dec 12, 2017 at 08:42:08AM +0100, Javier Martinez Canillas wrote: > Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x > Mixer nodes") disabled the Mixer node by default in the DTSI and enabled > for each Exynos 542x DTS. But unfortunately it missed to enable it for the > Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC variant. > > Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x Mixer nodes") > Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> > Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> > > --- > > Changes in v2: > - Remove RFT tag. > - Add Marek's Acked-by tag. > - Add fixes tag. > > arch/arm/boot/dts/exynos5800-peach-pi.dts | 4 ++++ > 1 file changed, 4 insertions(+) > Thanks, applied for current cycle (fixes). Best regards, Krzysztof
diff --git a/arch/arm/boot/dts/exynos5800-peach-pi.dts b/arch/arm/boot/dts/exynos5800-peach-pi.dts index b2b95ff205e8..0029ec27819c 100644 --- a/arch/arm/boot/dts/exynos5800-peach-pi.dts +++ b/arch/arm/boot/dts/exynos5800-peach-pi.dts @@ -664,6 +664,10 @@ status = "okay"; }; +&mixer { + status = "okay"; +}; + /* eMMC flash */ &mmc_0 { status = "okay";
Commit 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x Mixer nodes") disabled the Mixer node by default in the DTSI and enabled for each Exynos 542x DTS. But unfortunately it missed to enable it for the Exynos5800 Peach Pi machine, since the 5800 is also an 542x SoC variant. Fixes: 1cb686c08d12 ("ARM: dts: exynos: Add status property to Exynos 542x Mixer nodes") Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> Acked-by: Marek Szyprowski <m.szyprowski@samsung.com> --- Changes in v2: - Remove RFT tag. - Add Marek's Acked-by tag. - Add fixes tag. arch/arm/boot/dts/exynos5800-peach-pi.dts | 4 ++++ 1 file changed, 4 insertions(+)