[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f11001d4-fac1-490d-988c-01c6b4b588db@gmail.com>
Date: Sat, 24 Aug 2024 16:26:22 +0300
From: Alper Nebi Yasak <alpernebiyasak@...il.com>
To: Nícolas F. R. A. Prado <nfraprado@...labora.com>,
Chen-Yu Tsai <wenst@...omium.org>
Cc: Matthias Brugger <matthias.bgg@...il.com>,
AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-mediatek@...ts.infradead.org,
Stephen Boyd <swboyd@...omium.org>, Pin-yen Lin <treapking@...omium.org>,
stable@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: mediatek: mt8186-corsola: Disable DPI display
interface
Hi,
On 2024-08-22 18:25 +03:00, Nícolas F. R. A. Prado wrote:
> On Wed, Aug 21, 2024 at 12:28:34PM +0800, Chen-Yu Tsai wrote:
>> The DPI display interface feeds the external display pipeline. However
>> the pipeline representation is currently incomplete. Efforts are still
>> under way to come up with a way to represent the "creative" repurposing
>> of the DP bridge chip's internal output mux, which is meant to support
>> USB type-C orientation changes, to output to one of two type-C ports.
>>
>> Until that is finalized, the external display can't be fully described,
>> and thus won't work. Even worse, the half complete graph potentially
>> confuses the OS, breaking the internal display as well.
>>
>> Disable the external display interface across the whole Corsola family
>> until the DP / USB Type-C muxing graph binding is ready.
>>
>> Reported-by: Alper Nebi Yasak <alpernebiyasak@...il.com>
>> Closes: https://lore.kernel.org/linux-mediatek/38a703a9-6efb-456a-a248-1dd3687e526d@gmail.com/
>> Fixes: 8855d01fb81f ("arm64: dts: mediatek: Add MT8186 Krabby platform based Tentacruel / Tentacool")
>> Cc: <stable@...r.kernel.org>
>> Signed-off-by: Chen-Yu Tsai <wenst@...omium.org>
>
> Reviewed-by: Nícolas F. R. A. Prado <nfraprado@...labora.com>
>
> Would be good to have Alper verify that with this change the internal display
> works again in their specific setup, although this change seems reasonable to me
> either way.
Tested-by: Alper Nebi Yasak <alpernebiyasak@...il.com>
Fixes that linked issue I had with internal display on my magneton. And
apparently I don't even need a custom kernel for it, I managed to get
the display working on Debian's v6.11-rc4 build with this dts change
(and `softdep mediatek-drm pre: mtk-iommu mt6358-regulator` iirc).
Thanks!
Powered by blists - more mailing lists