From mboxrd@z Thu Jan 1 00:00:00 1970 From: Javier Martinez Canillas Subject: Re: [PATCH v2] ARM: dts: exynos: Enable Mixer node for Exynos5800 Peach Pi machine Date: Tue, 12 Dec 2017 11:40:57 +0100 Message-ID: References: <20171212074208.30753-1-javierm@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-samsung-soc-owner@vger.kernel.org To: Krzysztof Kozlowski Cc: linux-kernel@vger.kernel.org, Marek Szyprowski , Guillaume Tucker , Daniel Vetter , Shuah Khan , devicetree@vger.kernel.org, Kukjin Kim , Russell King , linux-samsung-soc@vger.kernel.org, Rob Herring , Mark Rutland , linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org Hello Krzysztof, On 12/12/2017 11:09 AM, Krzysztof Kozlowski wrote: > On Tue, Dec 12, 2017 at 10:55 AM, Krzysztof Kozlowski wrote: >> On Tue, Dec 12, 2017 at 8:42 AM, 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 >>> Acked-by: Marek Szyprowski >>> >>> --- >>> >>> 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, -- Javier Martinez Canillas Software Engineer - Desktop Hardware Enablement Red Hat