| Message ID | 20220129175332.298666-1-krzysztof.kozlowski@canonical.com |
|---|---|
| State | New |
| Headers | show |
| Series | [1/5] arm64: dts: exynos: align pl330 node name with dtschema | expand |
Hi Krzysztof >-----Original Message----- >From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@canonical.com] >Sent: Saturday, January 29, 2022 11:23 PM >To: Lee Jones <lee.jones@linaro.org>; Rob Herring <robh+dt@kernel.org>; >Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>; Alim Akhtar ><alim.akhtar@samsung.com>; Greg Kroah-Hartman ><gregkh@linuxfoundation.org>; Sylwester Nawrocki ><s.nawrocki@samsung.com>; Marek Szyprowski ><m.szyprowski@samsung.com>; devicetree@vger.kernel.org; linux-arm- >kernel@lists.infradead.org; linux-samsung-soc@vger.kernel.org; linux- >kernel@vger.kernel.org; linux-serial@vger.kernel.org >Subject: [PATCH 1/5] arm64: dts: exynos: align pl330 node name with >dtschema > >Fixes dtbs_check warnings like: > > pdma@15610000: $nodename:0: 'pdma@15610000' does not match '^dma- >controller(@.*)?$' > >Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> >--- Reviewed-by: Alim Akhtar <alim.akhtar@smasung.com> > arch/arm64/boot/dts/exynos/exynos5433.dtsi | 6 +++--- > arch/arm64/boot/dts/exynos/exynos7.dtsi | 4 ++-- > 2 files changed, 5 insertions(+), 5 deletions(-) > >diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi >b/arch/arm64/boot/dts/exynos/exynos5433.dtsi >index bfe4ed8a23d6..b4cde77e02d3 100644 >--- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi >+++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi >@@ -1858,7 +1858,7 @@ mshc_2: mshc@15560000 { > status = "disabled"; > }; > >- pdma0: pdma@15610000 { >+ pdma0: dma-controller@15610000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0x15610000 0x1000>; > interrupts = <GIC_SPI 228 IRQ_TYPE_LEVEL_HIGH>; >@@ -1869,7 +1869,7 @@ pdma0: pdma@15610000 { > #dma-requests = <32>; > }; > >- pdma1: pdma@15600000 { >+ pdma1: dma-controller@15600000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0x15600000 0x1000>; > interrupts = <GIC_SPI 246 IRQ_TYPE_LEVEL_HIGH>; >@@ -1891,7 +1891,7 @@ audio-subsystem@11400000 { > #size-cells = <1>; > ranges; > >- adma: adma@11420000 { >+ adma: dma-controller@11420000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0x11420000 0x1000>; > interrupts = <GIC_SPI 73 >IRQ_TYPE_LEVEL_HIGH>; diff --git >a/arch/arm64/boot/dts/exynos/exynos7.dtsi >b/arch/arm64/boot/dts/exynos/exynos7.dtsi >index 960823b8247a..3364b09c3158 100644 >--- a/arch/arm64/boot/dts/exynos/exynos7.dtsi >+++ b/arch/arm64/boot/dts/exynos/exynos7.dtsi >@@ -142,7 +142,7 @@ gic: interrupt-controller@11001000 { > <0x11006000 0x2000>; > }; > >- pdma0: pdma@10e10000 { >+ pdma0: dma-controller@10e10000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0x10E10000 0x1000>; > interrupts = <GIC_SPI 225 IRQ_TYPE_LEVEL_HIGH>; >@@ -153,7 +153,7 @@ pdma0: pdma@10e10000 { > #dma-requests = <32>; > }; > >- pdma1: pdma@10eb0000 { >+ pdma1: dma-controller@10eb0000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0x10EB0000 0x1000>; > interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>; >-- >2.32.0
>-----Original Message----- >From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@canonical.com] >Sent: Saturday, January 29, 2022 11:23 PM >To: Lee Jones <lee.jones@linaro.org>; Rob Herring <robh+dt@kernel.org>; >Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>; Alim Akhtar ><alim.akhtar@samsung.com>; Greg Kroah-Hartman ><gregkh@linuxfoundation.org>; Sylwester Nawrocki ><s.nawrocki@samsung.com>; Marek Szyprowski ><m.szyprowski@samsung.com>; devicetree@vger.kernel.org; linux-arm- >kernel@lists.infradead.org; linux-samsung-soc@vger.kernel.org; linux- >kernel@vger.kernel.org; linux-serial@vger.kernel.org >Subject: [PATCH 1/5] arm64: dts: exynos: align pl330 node name with >dtschema > >Fixes dtbs_check warnings like: > > pdma@15610000: $nodename:0: 'pdma@15610000' does not match '^dma- >controller(@.*)?$' > >Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> >--- Thanks Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com> > arch/arm64/boot/dts/exynos/exynos5433.dtsi | 6 +++--- > arch/arm64/boot/dts/exynos/exynos7.dtsi | 4 ++-- > 2 files changed, 5 insertions(+), 5 deletions(-) > >diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi >b/arch/arm64/boot/dts/exynos/exynos5433.dtsi >index bfe4ed8a23d6..b4cde77e02d3 100644 >--- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi >+++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi >@@ -1858,7 +1858,7 @@ mshc_2: mshc@15560000 { > status = "disabled"; > }; > >- pdma0: pdma@15610000 { >+ pdma0: dma-controller@15610000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0x15610000 0x1000>; > interrupts = <GIC_SPI 228 IRQ_TYPE_LEVEL_HIGH>; >@@ -1869,7 +1869,7 @@ pdma0: pdma@15610000 { > #dma-requests = <32>; > }; > >- pdma1: pdma@15600000 { >+ pdma1: dma-controller@15600000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0x15600000 0x1000>; > interrupts = <GIC_SPI 246 IRQ_TYPE_LEVEL_HIGH>; >@@ -1891,7 +1891,7 @@ audio-subsystem@11400000 { > #size-cells = <1>; > ranges; > >- adma: adma@11420000 { >+ adma: dma-controller@11420000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0x11420000 0x1000>; > interrupts = <GIC_SPI 73 >IRQ_TYPE_LEVEL_HIGH>; diff --git >a/arch/arm64/boot/dts/exynos/exynos7.dtsi >b/arch/arm64/boot/dts/exynos/exynos7.dtsi >index 960823b8247a..3364b09c3158 100644 >--- a/arch/arm64/boot/dts/exynos/exynos7.dtsi >+++ b/arch/arm64/boot/dts/exynos/exynos7.dtsi >@@ -142,7 +142,7 @@ gic: interrupt-controller@11001000 { > <0x11006000 0x2000>; > }; > >- pdma0: pdma@10e10000 { >+ pdma0: dma-controller@10e10000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0x10E10000 0x1000>; > interrupts = <GIC_SPI 225 IRQ_TYPE_LEVEL_HIGH>; >@@ -153,7 +153,7 @@ pdma0: pdma@10e10000 { > #dma-requests = <32>; > }; > >- pdma1: pdma@10eb0000 { >+ pdma1: dma-controller@10eb0000 { > compatible = "arm,pl330", "arm,primecell"; > reg = <0x10EB0000 0x1000>; > interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>; >-- >2.32.0
>-----Original Message----- >From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@canonical.com] >Sent: Saturday, January 29, 2022 11:23 PM >To: Lee Jones <lee.jones@linaro.org>; Rob Herring <robh+dt@kernel.org>; >Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>; Alim Akhtar ><alim.akhtar@samsung.com>; Greg Kroah-Hartman ><gregkh@linuxfoundation.org>; Sylwester Nawrocki ><s.nawrocki@samsung.com>; Marek Szyprowski ><m.szyprowski@samsung.com>; devicetree@vger.kernel.org; linux-arm- >kernel@lists.infradead.org; linux-samsung-soc@vger.kernel.org; linux- >kernel@vger.kernel.org; linux-serial@vger.kernel.org >Subject: [PATCH 2/5] arm64: dts: exynos: drop unneeded syscon phandle in >Exynos5433 LPASS > >Exynos5433 LPASS audio node does not use syscon phandle since commit >addebf1588ab ("mfd: exynos-lpass: Remove pad retention control"). It was >also dropped from bindings. > >Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> >--- Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com> > arch/arm64/boot/dts/exynos/exynos5433.dtsi | 1 - > 1 file changed, 1 deletion(-) > >diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi >b/arch/arm64/boot/dts/exynos/exynos5433.dtsi >index b4cde77e02d3..661567d2dd7a 100644 >--- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi >+++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi >@@ -1885,7 +1885,6 @@ audio-subsystem@11400000 { > reg = <0x11400000 0x100>, <0x11500000 0x08>; > clocks = <&cmu_aud CLK_PCLK_SFR0_CTRL>; > clock-names = "sfr0_ctrl"; >- samsung,pmu-syscon = <&pmu_system_controller>; > power-domains = <&pd_aud>; > #address-cells = <1>; > #size-cells = <1>; >-- >2.32.0
>-----Original Message----- >From: Krzysztof Kozlowski [mailto:krzysztof.kozlowski@canonical.com] >Sent: Saturday, January 29, 2022 11:24 PM >To: Lee Jones <lee.jones@linaro.org>; Rob Herring <robh+dt@kernel.org>; >Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>; Alim Akhtar ><alim.akhtar@samsung.com>; Greg Kroah-Hartman ><gregkh@linuxfoundation.org>; Sylwester Nawrocki ><s.nawrocki@samsung.com>; Marek Szyprowski ><m.szyprowski@samsung.com>; devicetree@vger.kernel.org; linux-arm- >kernel@lists.infradead.org; linux-samsung-soc@vger.kernel.org; linux- >kernel@vger.kernel.org; linux-serial@vger.kernel.org >Subject: [PATCH 3/5] mfd: exynos-lpass: Drop unneeded syscon.h include > >syscon regmap is not used since commit addebf1588ab ("mfd: exynos-lpass: >Remove pad retention control"). > >Fixes: addebf1588ab ("mfd: exynos-lpass: Remove pad retention control") >Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> >--- Reviewed-by: Alim Akhtar <alim.akhtar@samsung.com> > drivers/mfd/exynos-lpass.c | 1 - > 1 file changed, 1 deletion(-) > >diff --git a/drivers/mfd/exynos-lpass.c b/drivers/mfd/exynos-lpass.c index >99bd0e73c19c..166cd21088cd 100644 >--- a/drivers/mfd/exynos-lpass.c >+++ b/drivers/mfd/exynos-lpass.c >@@ -15,7 +15,6 @@ > #include <linux/delay.h> > #include <linux/io.h> > #include <linux/module.h> >-#include <linux/mfd/syscon.h> > #include <linux/of.h> > #include <linux/of_platform.h> > #include <linux/platform_device.h> >-- >2.32.0
On Sat, 29 Jan 2022 18:53:28 +0100, Krzysztof Kozlowski wrote: > Fixes dtbs_check warnings like: > > pdma@15610000: $nodename:0: 'pdma@15610000' does not match '^dma-controller(@.*)?$' > > Applied, thanks! [1/5] arm64: dts: exynos: align pl330 node name with dtschema commit: 706d8df83c8ec9cbe948ff8c8bd082b7819f4196 Best regards,
diff --git a/arch/arm64/boot/dts/exynos/exynos5433.dtsi b/arch/arm64/boot/dts/exynos/exynos5433.dtsi index bfe4ed8a23d6..b4cde77e02d3 100644 --- a/arch/arm64/boot/dts/exynos/exynos5433.dtsi +++ b/arch/arm64/boot/dts/exynos/exynos5433.dtsi @@ -1858,7 +1858,7 @@ mshc_2: mshc@15560000 { status = "disabled"; }; - pdma0: pdma@15610000 { + pdma0: dma-controller@15610000 { compatible = "arm,pl330", "arm,primecell"; reg = <0x15610000 0x1000>; interrupts = <GIC_SPI 228 IRQ_TYPE_LEVEL_HIGH>; @@ -1869,7 +1869,7 @@ pdma0: pdma@15610000 { #dma-requests = <32>; }; - pdma1: pdma@15600000 { + pdma1: dma-controller@15600000 { compatible = "arm,pl330", "arm,primecell"; reg = <0x15600000 0x1000>; interrupts = <GIC_SPI 246 IRQ_TYPE_LEVEL_HIGH>; @@ -1891,7 +1891,7 @@ audio-subsystem@11400000 { #size-cells = <1>; ranges; - adma: adma@11420000 { + adma: dma-controller@11420000 { compatible = "arm,pl330", "arm,primecell"; reg = <0x11420000 0x1000>; interrupts = <GIC_SPI 73 IRQ_TYPE_LEVEL_HIGH>; diff --git a/arch/arm64/boot/dts/exynos/exynos7.dtsi b/arch/arm64/boot/dts/exynos/exynos7.dtsi index 960823b8247a..3364b09c3158 100644 --- a/arch/arm64/boot/dts/exynos/exynos7.dtsi +++ b/arch/arm64/boot/dts/exynos/exynos7.dtsi @@ -142,7 +142,7 @@ gic: interrupt-controller@11001000 { <0x11006000 0x2000>; }; - pdma0: pdma@10e10000 { + pdma0: dma-controller@10e10000 { compatible = "arm,pl330", "arm,primecell"; reg = <0x10E10000 0x1000>; interrupts = <GIC_SPI 225 IRQ_TYPE_LEVEL_HIGH>; @@ -153,7 +153,7 @@ pdma0: pdma@10e10000 { #dma-requests = <32>; }; - pdma1: pdma@10eb0000 { + pdma1: dma-controller@10eb0000 { compatible = "arm,pl330", "arm,primecell"; reg = <0x10EB0000 0x1000>; interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>;
Fixes dtbs_check warnings like: pdma@15610000: $nodename:0: 'pdma@15610000' does not match '^dma-controller(@.*)?$' Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> --- arch/arm64/boot/dts/exynos/exynos5433.dtsi | 6 +++--- arch/arm64/boot/dts/exynos/exynos7.dtsi | 4 ++-- 2 files changed, 5 insertions(+), 5 deletions(-)