diff mbox series

[1/3] arm64: dts: mediatek: mt8188: Fix VPPSYS0/1 node name/compatibles

Message ID 20240619085322.66716-1-angelogioacchino.delregno@collabora.com (mailing list archive)
State New
Headers show
Series [1/3] arm64: dts: mediatek: mt8188: Fix VPPSYS0/1 node name/compatibles | expand

Commit Message

AngeloGioacchino Del Regno June 19, 2024, 8:53 a.m. UTC
Like VDOSYS0/1, the VPPSYS0 and VPPSYS1 are syscon nodes, as
described by the mediatek,mmsys binding: fix the node name to
be syscon@address and add "syscon" to the list of compatible
strings for both VPPSYS0 and VPPSYS1.

Fixes: 9461e0caac9e ("arm64: dts: Add MediaTek MT8188 dts and evaluation board and Makefile")
Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
---
 arch/arm64/boot/dts/mediatek/mt8188.dtsi | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

Comments

Fei Shao June 20, 2024, 8:49 a.m. UTC | #1
On Wed, Jun 19, 2024 at 4:53 PM AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com> wrote:
>
> Like VDOSYS0/1, the VPPSYS0 and VPPSYS1 are syscon nodes, as
> described by the mediatek,mmsys binding: fix the node name to
> be syscon@address and add "syscon" to the list of compatible
> strings for both VPPSYS0 and VPPSYS1.
>
> Fixes: 9461e0caac9e ("arm64: dts: Add MediaTek MT8188 dts and evaluation board and Makefile")
> Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

Reviewed-by: Fei Shao <fshao@chromium.org>

> ---
>  arch/arm64/boot/dts/mediatek/mt8188.dtsi | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8188.dtsi b/arch/arm64/boot/dts/mediatek/mt8188.dtsi
> index 3b02170562ae..77c4f2d104cd 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8188.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8188.dtsi
> @@ -1331,8 +1331,8 @@ mfgcfg: clock-controller@13fbf000 {
>                         #clock-cells = <1>;
>                 };
>
> -               vppsys0: clock-controller@14000000 {
> -                       compatible = "mediatek,mt8188-vppsys0";
> +               vppsys0: syscon@14000000 {
> +                       compatible = "mediatek,mt8188-vppsys0", "syscon";
>                         reg = <0 0x14000000 0 0x1000>;
>                         #clock-cells = <1>;
>                 };
> @@ -1349,8 +1349,8 @@ wpesys_vpp0: clock-controller@14e02000 {
>                         #clock-cells = <1>;
>                 };
>
> -               vppsys1: clock-controller@14f00000 {
> -                       compatible = "mediatek,mt8188-vppsys1";
> +               vppsys1: syscon@14f00000 {
> +                       compatible = "mediatek,mt8188-vppsys1", "syscon";
>                         reg = <0 0x14f00000 0 0x1000>;
>                         #clock-cells = <1>;
>                 };
> --
> 2.45.2
>
>
AngeloGioacchino Del Regno June 24, 2024, 10:46 a.m. UTC | #2
On Wed, 19 Jun 2024 10:53:20 +0200, AngeloGioacchino Del Regno wrote:
> Like VDOSYS0/1, the VPPSYS0 and VPPSYS1 are syscon nodes, as
> described by the mediatek,mmsys binding: fix the node name to
> be syscon@address and add "syscon" to the list of compatible
> strings for both VPPSYS0 and VPPSYS1.
> 
> 

Applied to for-next, thanks!

[1/3] arm64: dts: mediatek: mt8188: Fix VPPSYS0/1 node name/compatibles
      commit: 013b7ab4080af0dccae16c6ea8a1e88ad530114c

Cheers,
Angelo
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/mediatek/mt8188.dtsi b/arch/arm64/boot/dts/mediatek/mt8188.dtsi
index 3b02170562ae..77c4f2d104cd 100644
--- a/arch/arm64/boot/dts/mediatek/mt8188.dtsi
+++ b/arch/arm64/boot/dts/mediatek/mt8188.dtsi
@@ -1331,8 +1331,8 @@  mfgcfg: clock-controller@13fbf000 {
 			#clock-cells = <1>;
 		};
 
-		vppsys0: clock-controller@14000000 {
-			compatible = "mediatek,mt8188-vppsys0";
+		vppsys0: syscon@14000000 {
+			compatible = "mediatek,mt8188-vppsys0", "syscon";
 			reg = <0 0x14000000 0 0x1000>;
 			#clock-cells = <1>;
 		};
@@ -1349,8 +1349,8 @@  wpesys_vpp0: clock-controller@14e02000 {
 			#clock-cells = <1>;
 		};
 
-		vppsys1: clock-controller@14f00000 {
-			compatible = "mediatek,mt8188-vppsys1";
+		vppsys1: syscon@14f00000 {
+			compatible = "mediatek,mt8188-vppsys1", "syscon";
 			reg = <0 0x14f00000 0 0x1000>;
 			#clock-cells = <1>;
 		};