All of lore.kernel.org
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Alim Akhtar <alim.akhtar@samsung.com>,
	Dinh Nguyen <dinguyen@kernel.org>,
	Michal Simek <michal.simek@xilinx.com>,
	Liviu Dudau <liviu.dudau@arm.com>,
	Sudeep Holla <sudeep.holla@arm.com>,
	Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
	Ray Jui <rjui@broadcom.com>,
	Scott Branden <sbranden@broadcom.com>,
	Broadcom Kernel Team <bcm-kernel-feedback-list@broadcom.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Rob Herring <robh@kernel.org>
Subject: [PATCH v2 2/9] ARM: dts: socfpga: drop useless 'dma-channels/requests' properties
Date: Sat, 30 Apr 2022 14:18:55 +0200	[thread overview]
Message-ID: <20220430121902.59895-3-krzysztof.kozlowski@linaro.org> (raw)
In-Reply-To: <20220430121902.59895-1-krzysztof.kozlowski@linaro.org>

The pl330 DMA controller provides number of DMA channels and requests
through its registers, so duplicating this information (with a chance of
mistakes) in DTS is pointless.  Additionally the DTS used always wrong
property names which causes DT schema check failures - the bindings
documented 'dma-channels' and 'dma-requests' properties without leading
hash sign.

Reported-by: Rob Herring <robh@kernel.org>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm/boot/dts/socfpga.dtsi         | 2 --
 arch/arm/boot/dts/socfpga_arria10.dtsi | 2 --
 2 files changed, 4 deletions(-)

diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi
index b8c5dd7860cb..bfaef45bdd04 100644
--- a/arch/arm/boot/dts/socfpga.dtsi
+++ b/arch/arm/boot/dts/socfpga.dtsi
@@ -80,8 +80,6 @@ pdma: pdma@ffe01000 {
 					     <0 110 4>,
 					     <0 111 4>;
 				#dma-cells = <1>;
-				#dma-channels = <8>;
-				#dma-requests = <32>;
 				clocks = <&l4_main_clk>;
 				clock-names = "apb_pclk";
 				resets = <&rst DMA_RESET>;
diff --git a/arch/arm/boot/dts/socfpga_arria10.dtsi b/arch/arm/boot/dts/socfpga_arria10.dtsi
index f1e50d2e623a..26bda2557fe8 100644
--- a/arch/arm/boot/dts/socfpga_arria10.dtsi
+++ b/arch/arm/boot/dts/socfpga_arria10.dtsi
@@ -73,8 +73,6 @@ pdma: pdma@ffda1000 {
 					     <0 90 IRQ_TYPE_LEVEL_HIGH>,
 					     <0 91 IRQ_TYPE_LEVEL_HIGH>;
 				#dma-cells = <1>;
-				#dma-channels = <8>;
-				#dma-requests = <32>;
 				clocks = <&l4_main_clk>;
 				clock-names = "apb_pclk";
 				resets = <&rst DMA_RESET>, <&rst DMA_OCP_RESET>;
-- 
2.32.0


WARNING: multiple messages have this Message-ID (diff)
From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Alim Akhtar <alim.akhtar@samsung.com>,
	Dinh Nguyen <dinguyen@kernel.org>,
	Michal Simek <michal.simek@xilinx.com>,
	Liviu Dudau <liviu.dudau@arm.com>,
	Sudeep Holla <sudeep.holla@arm.com>,
	Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
	Ray Jui <rjui@broadcom.com>,
	Scott Branden <sbranden@broadcom.com>,
	Broadcom Kernel Team <bcm-kernel-feedback-list@broadcom.com>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	Rob Herring <robh@kernel.org>
Subject: [PATCH v2 2/9] ARM: dts: socfpga: drop useless 'dma-channels/requests' properties
Date: Sat, 30 Apr 2022 14:18:55 +0200	[thread overview]
Message-ID: <20220430121902.59895-3-krzysztof.kozlowski@linaro.org> (raw)
In-Reply-To: <20220430121902.59895-1-krzysztof.kozlowski@linaro.org>

The pl330 DMA controller provides number of DMA channels and requests
through its registers, so duplicating this information (with a chance of
mistakes) in DTS is pointless.  Additionally the DTS used always wrong
property names which causes DT schema check failures - the bindings
documented 'dma-channels' and 'dma-requests' properties without leading
hash sign.

Reported-by: Rob Herring <robh@kernel.org>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm/boot/dts/socfpga.dtsi         | 2 --
 arch/arm/boot/dts/socfpga_arria10.dtsi | 2 --
 2 files changed, 4 deletions(-)

diff --git a/arch/arm/boot/dts/socfpga.dtsi b/arch/arm/boot/dts/socfpga.dtsi
index b8c5dd7860cb..bfaef45bdd04 100644
--- a/arch/arm/boot/dts/socfpga.dtsi
+++ b/arch/arm/boot/dts/socfpga.dtsi
@@ -80,8 +80,6 @@ pdma: pdma@ffe01000 {
 					     <0 110 4>,
 					     <0 111 4>;
 				#dma-cells = <1>;
-				#dma-channels = <8>;
-				#dma-requests = <32>;
 				clocks = <&l4_main_clk>;
 				clock-names = "apb_pclk";
 				resets = <&rst DMA_RESET>;
diff --git a/arch/arm/boot/dts/socfpga_arria10.dtsi b/arch/arm/boot/dts/socfpga_arria10.dtsi
index f1e50d2e623a..26bda2557fe8 100644
--- a/arch/arm/boot/dts/socfpga_arria10.dtsi
+++ b/arch/arm/boot/dts/socfpga_arria10.dtsi
@@ -73,8 +73,6 @@ pdma: pdma@ffda1000 {
 					     <0 90 IRQ_TYPE_LEVEL_HIGH>,
 					     <0 91 IRQ_TYPE_LEVEL_HIGH>;
 				#dma-cells = <1>;
-				#dma-channels = <8>;
-				#dma-requests = <32>;
 				clocks = <&l4_main_clk>;
 				clock-names = "apb_pclk";
 				resets = <&rst DMA_RESET>, <&rst DMA_OCP_RESET>;
-- 
2.32.0


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  parent reply	other threads:[~2022-04-30 12:19 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-30 12:18 [PATCH v2 0/9] ARM/arm64: dts: drop useless 'dma-channels/requests' properties Krzysztof Kozlowski
2022-04-30 12:18 ` Krzysztof Kozlowski
2022-04-30 12:18 ` [PATCH v2 1/9] ARM: dts: zynq-7000: " Krzysztof Kozlowski
2022-04-30 12:18   ` Krzysztof Kozlowski
2022-05-02  7:27   ` Michal Simek
2022-05-02  7:27     ` Michal Simek
2022-05-02  7:28     ` Krzysztof Kozlowski
2022-05-02  7:28       ` Krzysztof Kozlowski
2022-05-04  8:30   ` (subset) " Krzysztof Kozlowski
2022-05-04  8:30     ` Krzysztof Kozlowski
2022-04-30 12:18 ` Krzysztof Kozlowski [this message]
2022-04-30 12:18   ` [PATCH v2 2/9] ARM: dts: socfpga: " Krzysztof Kozlowski
2022-05-04  8:30   ` (subset) " Krzysztof Kozlowski
2022-05-04  8:30     ` Krzysztof Kozlowski
2022-04-30 12:18 ` [PATCH v2 3/9] arm64: dts: stratix10/agilex: " Krzysztof Kozlowski
2022-04-30 12:18   ` Krzysztof Kozlowski
2022-05-04  8:30   ` (subset) " Krzysztof Kozlowski
2022-05-04  8:30     ` Krzysztof Kozlowski
2022-04-30 12:18 ` [PATCH v2 4/9] arm64: dts: juno: " Krzysztof Kozlowski
2022-04-30 12:18   ` Krzysztof Kozlowski
2022-05-03  8:04   ` (subset) " Sudeep Holla
2022-05-03  8:04     ` Sudeep Holla
2022-04-30 12:18 ` [PATCH v2 5/9] arm64: dts: broadcom: " Krzysztof Kozlowski
2022-04-30 12:18   ` Krzysztof Kozlowski
2022-05-04  8:30   ` (subset) " Krzysztof Kozlowski
2022-05-04  8:30     ` Krzysztof Kozlowski
2022-04-30 12:18 ` [PATCH v2 6/9] ARM: dts: s5pv210: " Krzysztof Kozlowski
2022-04-30 12:18   ` Krzysztof Kozlowski
2022-05-04  8:30   ` (subset) " Krzysztof Kozlowski
2022-05-04  8:30     ` Krzysztof Kozlowski
2022-04-30 12:19 ` [PATCH v2 7/9] ARM: dts: exynos: " Krzysztof Kozlowski
2022-04-30 12:19   ` Krzysztof Kozlowski
2022-05-04  8:30   ` (subset) " Krzysztof Kozlowski
2022-05-04  8:30     ` Krzysztof Kozlowski
2022-04-30 12:19 ` [PATCH v2 8/9] arm64: " Krzysztof Kozlowski
2022-04-30 12:19   ` Krzysztof Kozlowski
2022-05-04  8:30   ` (subset) " Krzysztof Kozlowski
2022-05-04  8:30     ` Krzysztof Kozlowski
2022-04-30 12:19 ` [PATCH v2 9/9] arm64: dts: fsd: " Krzysztof Kozlowski
2022-04-30 12:19   ` Krzysztof Kozlowski
2022-05-04  8:30   ` (subset) " Krzysztof Kozlowski
2022-05-04  8:30     ` Krzysztof Kozlowski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220430121902.59895-3-krzysztof.kozlowski@linaro.org \
    --to=krzysztof.kozlowski@linaro.org \
    --cc=alim.akhtar@samsung.com \
    --cc=bcm-kernel-feedback-list@broadcom.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dinguyen@kernel.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=liviu.dudau@arm.com \
    --cc=lorenzo.pieralisi@arm.com \
    --cc=michal.simek@xilinx.com \
    --cc=rjui@broadcom.com \
    --cc=robh+dt@kernel.org \
    --cc=robh@kernel.org \
    --cc=sbranden@broadcom.com \
    --cc=sudeep.holla@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.