]> git.hungrycats.org Git - linux/commitdiff
arm64: dts: mediatek: mt8186-corsola: Disable DPI display interface
authorChen-Yu Tsai <wenst@chromium.org>
Wed, 21 Aug 2024 04:28:34 +0000 (12:28 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 4 Oct 2024 14:33:40 +0000 (16:33 +0200)
commit 3079fb09ddac159bd8bb87f6f15b924e265f8d4d upstream.

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@gmail.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@vger.kernel.org>
Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
Tested-by: Alper Nebi Yasak <alpernebiyasak@gmail.com>
Reviewed-by: NĂ­colas F. R. A. Prado <nfraprado@collabora.com>
Link: https://lore.kernel.org/r/20240821042836.2631815-1-wenst@chromium.org
Signed-off-by: Matthias Brugger <matthias.bgg@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
arch/arm64/boot/dts/mediatek/mt8186-corsola.dtsi

index 1807e9d6cb0e4123329c5b369fc83a61f7677721..0a4838b35eab6e29766c2019b3617f619e5b983d 100644 (file)
        pinctrl-names = "default", "sleep";
        pinctrl-0 = <&dpi_pins_default>;
        pinctrl-1 = <&dpi_pins_sleep>;
-       status = "okay";
+       /* TODO Re-enable after DP to Type-C port muxing can be described */
+       status = "disabled";
 };
 
 &dpi_out {