Re: [PATCH v2 2/3] dts: arm64: mediatek: mt8188: Update OVL compatible from MT8183 to MT8195

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Il 13/12/24 11:26, Pin-yen Lin ha scritto:
Hi Jason,

On Fri, Dec 13, 2024 at 5:50 PM Jason-JH.Lin <jason-jh.lin@xxxxxxxxxxxx> wrote:

The OVL hardware capabilities have changed starting from MT8195,
making the MT8183 compatible no longer applicable.
Therefore, it is necessary to update the OVL compatible from MT8183 to
MT8195.

This probably doesn't need an (immediate) respin, but this patch needs:

Fixes: 7075b21d1a8e ("arm64: dts: mediatek: mt8188: Add display nodes
for vdosys0")

No, because the binding must otherwise get a Fixes tag.

This is not a fix anyway, and is technically adding new features to the MT8188
OVL; without the additional features, the MT8188 is still compatible with the
MT8183 OVL, as much as MT8195.

Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>


Signed-off-by: Jason-JH.Lin <jason-jh.lin@xxxxxxxxxxxx>
---
  arch/arm64/boot/dts/mediatek/mt8188.dtsi | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm64/boot/dts/mediatek/mt8188.dtsi b/arch/arm64/boot/dts/mediatek/mt8188.dtsi
index faccc7f16259..23ec3ff6cad9 100644
--- a/arch/arm64/boot/dts/mediatek/mt8188.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8188.dtsi
@@ -2488,7 +2488,7 @@ jpeg_decoder: jpeg-decoder@1a040000 {
                 };

                 ovl0: ovl@1c000000 {
-                       compatible = "mediatek,mt8188-disp-ovl", "mediatek,mt8183-disp-ovl";
+                       compatible = "mediatek,mt8188-disp-ovl", "mediatek,mt8195-disp-ovl";
                         reg = <0 0x1c000000 0 0x1000>;
                         clocks = <&vdosys0 CLK_VDO0_DISP_OVL0>;
                         interrupts = <GIC_SPI 636 IRQ_TYPE_LEVEL_HIGH 0>;
--
2.43.0


Regards,
Pin-yen





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux