Thread (1 message) 1 message, 1 author, 2017-03-22

[PATCH 10/15] ARM: dts: bcm: fix msi-controller name and unit address

From: Ray Jui <hidden>
Date: 2017-03-22 06:07:34
Also in: linux-devicetree

Hi Rob,


On 3/21/2017 7:03 PM, Rob Herring wrote:
quoted hunk
The unit address for the msi controller is not valid as there is no reg
property, so remove it. Also, msi-controller is the preferred node name.

Signed-off-by: Rob Herring <robh@kernel.org>
Cc: Ray Jui <rjui@broadcom.com>
Cc: Scott Branden <sbranden@broadcom.com>
Cc: Jon Mason <redacted>
Cc: bcm-kernel-feedback-list at broadcom.com
---
Sub-arch maintainers, please apply to your trees unless arm-soc wants
to take the whole lot.

  arch/arm/boot/dts/bcm-cygnus.dtsi | 4 ++--
  arch/arm/boot/dts/bcm-nsp.dtsi    | 6 +++---
  2 files changed, 5 insertions(+), 5 deletions(-)
diff --git a/arch/arm/boot/dts/bcm-cygnus.dtsi b/arch/arm/boot/dts/bcm-cygnus.dtsi
index 8833a4c3cd96..9644fddb5e3c 100644
--- a/arch/arm/boot/dts/bcm-cygnus.dtsi
+++ b/arch/arm/boot/dts/bcm-cygnus.dtsi
@@ -205,7 +205,7 @@
  			status = "disabled";
  
  			msi-parent = <&msi0>;
-			msi0: msi at 18012000 {
+			msi0: msi-controller {
  				compatible = "brcm,iproc-msi";
  				msi-controller;
  				interrupt-parent = <&gic>;
@@ -240,7 +240,7 @@
  			status = "disabled";
  
  			msi-parent = <&msi1>;
-			msi1: msi at 18013000 {
+			msi1: msi-controller {
  				compatible = "brcm,iproc-msi";
  				msi-controller;
  				interrupt-parent = <&gic>;
diff --git a/arch/arm/boot/dts/bcm-nsp.dtsi b/arch/arm/boot/dts/bcm-nsp.dtsi
index 832795b0fd0f..42c017df490f 100644
--- a/arch/arm/boot/dts/bcm-nsp.dtsi
+++ b/arch/arm/boot/dts/bcm-nsp.dtsi
@@ -450,7 +450,7 @@
  		status = "disabled";
  
  		msi-parent = <&msi0>;
-		msi0: msi at 18012000 {
+		msi0: msi-controller {
  			compatible = "brcm,iproc-msi";
  			msi-controller;
  			interrupt-parent = <&gic>;
@@ -486,7 +486,7 @@
  		status = "disabled";
  
  		msi-parent = <&msi1>;
-		msi1: msi at 18013000 {
+		msi1: msi-controller {
  			compatible = "brcm,iproc-msi";
  			msi-controller;
  			interrupt-parent = <&gic>;
@@ -522,7 +522,7 @@
  		status = "disabled";
  
  		msi-parent = <&msi2>;
-		msi2: msi at 18014000 {
+		msi2: msi-controller {
  			compatible = "brcm,iproc-msi";
  			msi-controller;
  			interrupt-parent = <&gic>;
Looks good and thanks!

Acked-by: Ray Jui <redacted>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help