* [RFT PATCH] ARM: dts: exynos: Enable Mixer node for Exynos5800 Peach Pi machine @ 2017-12-11 22:48 ` Javier Martinez Canillas [not found] ` <20171211224856.29264-1-javierm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 0 siblings, 1 reply; 3+ messages in thread From: Javier Martinez Canillas @ 2017-12-11 22:48 UTC (permalink / raw) To: linux-kernel Cc: Marek Szyprowski, Guillaume Tucker, Daniel Vetter, Shuah Khan, Javier Martinez Canillas, devicetree, Kukjin Kim, Russell King, linux-samsung-soc, Rob Herring, Mark Rutland, Krzysztof Kozlowski, linux-arm-kernel 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. Signed-off-by: Javier Martinez Canillas <javierm@redhat.com> --- I believe this may cause the boot issues reported on Exynos5800 Peach Pi from v4.15-rc3, the mentioned commit made to v4.15-rc1 but it seems that didn't cause any harm until commit ("510353a63796 drm/bridge: analogix dp: Fix runtime PM state in get_modes() callback") fixed the runtime PM management in the DP driver. I can't test right now, but I'm posting anyways as a RFT in case others that have access to a Peach Pi can test it. Best regards, Javier 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 ^ permalink raw reply related [flat|nested] 3+ messages in thread
[parent not found: <20171211224856.29264-1-javierm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]
* Re: [RFT PATCH] ARM: dts: exynos: Enable Mixer node for Exynos5800 Peach Pi machine [not found] ` <20171211224856.29264-1-javierm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> @ 2017-12-12 7:01 ` Marek Szyprowski [not found] ` <3e6e4b15-9f8a-bea1-3c8e-2d85fc4c512d-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> 0 siblings, 1 reply; 3+ messages in thread From: Marek Szyprowski @ 2017-12-12 7:01 UTC (permalink / raw) To: Javier Martinez Canillas, linux-kernel-u79uwXL29TY76Z2rM5mHXA Cc: Guillaume Tucker, Daniel Vetter, Shuah Khan, devicetree-u79uwXL29TY76Z2rM5mHXA, Kukjin Kim, Russell King, linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Mark Rutland, Krzysztof Kozlowski, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Hi On 2017-12-11 23:48, 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. > > Signed-off-by: Javier Martinez Canillas <javierm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> Acked-by: Marek Szyprowski <m.szyprowski-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> > --- > > I believe this may cause the boot issues reported on Exynos5800 Peach Pi > from v4.15-rc3, the mentioned commit made to v4.15-rc1 but it seems that > didn't cause any harm until commit ("510353a63796 drm/bridge: analogix > dp: Fix runtime PM state in get_modes() callback") fixed the runtime PM > management in the DP driver. Thanks for analyzing this. Lack of this change was probably responsible for Exynos DRM initialization failure ("exynos-drm exynos-drm: failed to bind 14530000.hdmi (ops hdmi_component_ops): -1" message and probably further error "unbalanced disables for lcd_vdd", which shows that failure path of analogix dp and simple panel causes unbalanced regulator disable. This patch should go to v4.15-rcX (fixes) if possible. > I can't test right now, but I'm posting anyways as a RFT in case others > that have access to a Peach Pi can test it. > > Best regards, > Javier > > 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"; Best regards -- Marek Szyprowski, PhD Samsung R&D Institute Poland -- 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 ^ permalink raw reply [flat|nested] 3+ messages in thread
[parent not found: <3e6e4b15-9f8a-bea1-3c8e-2d85fc4c512d-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>]
* Re: [RFT PATCH] ARM: dts: exynos: Enable Mixer node for Exynos5800 Peach Pi machine [not found] ` <3e6e4b15-9f8a-bea1-3c8e-2d85fc4c512d-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> @ 2017-12-12 7:41 ` Javier Martinez Canillas 0 siblings, 0 replies; 3+ messages in thread From: Javier Martinez Canillas @ 2017-12-12 7:41 UTC (permalink / raw) To: Marek Szyprowski, linux-kernel-u79uwXL29TY76Z2rM5mHXA Cc: Guillaume Tucker, Daniel Vetter, Shuah Khan, devicetree-u79uwXL29TY76Z2rM5mHXA, Kukjin Kim, Russell King, linux-samsung-soc-u79uwXL29TY76Z2rM5mHXA, Rob Herring, Mark Rutland, Krzysztof Kozlowski, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Hello Marek, On 12/12/2017 08:01 AM, Marek Szyprowski wrote: > Hi > > On 2017-12-11 23:48, 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. >> >> Signed-off-by: Javier Martinez Canillas <javierm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> > > Acked-by: Marek Szyprowski <m.szyprowski-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> > Thanks. >> --- >> >> I believe this may cause the boot issues reported on Exynos5800 Peach Pi >> from v4.15-rc3, the mentioned commit made to v4.15-rc1 but it seems that >> didn't cause any harm until commit ("510353a63796 drm/bridge: analogix >> dp: Fix runtime PM state in get_modes() callback") fixed the runtime PM >> management in the DP driver. > > Thanks for analyzing this. Lack of this change was probably responsible for > Exynos DRM initialization failure ("exynos-drm exynos-drm: failed to bind > 14530000.hdmi (ops hdmi_component_ops): -1" message and probably further > error "unbalanced disables for lcd_vdd", which shows that failure path of > analogix dp and simple panel causes unbalanced regulator disable. > Yes, I came to the same conclusion than you in the thread started by Guillaume. I just couldn't test it. > This patch should go to v4.15-rcX (fixes) if possible. > Indeed. I wondered if it also needed a Fixes tag. I didn't include it because I thought that both the culprit and the fix would be in the same kernel release. But I'll include it anyways. >> I can't test right now, but I'm posting anyways as a RFT in case others >> that have access to a Peach Pi can test it. >> >> Best regards, >> Javier >> >> 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"; > > Best regards > Best regards, -- Javier Martinez Canillas Software Engineer - Desktop Hardware Enablement Red Hat -- 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 ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-12-12 7:41 UTC | newest] Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- [not found] <CGME20171211224905epcas1p4e6d66369f60c7dae0678efc95d0bdb6e@epcas1p4.samsung.com> 2017-12-11 22:48 ` [RFT PATCH] ARM: dts: exynos: Enable Mixer node for Exynos5800 Peach Pi machine Javier Martinez Canillas [not found] ` <20171211224856.29264-1-javierm-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> 2017-12-12 7:01 ` Marek Szyprowski [not found] ` <3e6e4b15-9f8a-bea1-3c8e-2d85fc4c512d-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org> 2017-12-12 7:41 ` Javier Martinez Canillas
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).