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

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

 




Hi Rob,


On 3/21/2017 7:03 PM, Rob Herring wrote:
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@xxxxxxxxxx>
Cc: Ray Jui <rjui@xxxxxxxxxxxx>
Cc: Scott Branden <sbranden@xxxxxxxxxxxx>
Cc: Jon Mason <jonmason@xxxxxxxxxxxx>
Cc: bcm-kernel-feedback-list@xxxxxxxxxxxx
---
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@18012000 {
+			msi0: msi-controller {
  				compatible = "brcm,iproc-msi";
  				msi-controller;
  				interrupt-parent = <&gic>;
@@ -240,7 +240,7 @@
  			status = "disabled";
msi-parent = <&msi1>;
-			msi1: msi@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@18012000 {
+		msi0: msi-controller {
  			compatible = "brcm,iproc-msi";
  			msi-controller;
  			interrupt-parent = <&gic>;
@@ -486,7 +486,7 @@
  		status = "disabled";
msi-parent = <&msi1>;
-		msi1: msi@18013000 {
+		msi1: msi-controller {
  			compatible = "brcm,iproc-msi";
  			msi-controller;
  			interrupt-parent = <&gic>;
@@ -522,7 +522,7 @@
  		status = "disabled";
msi-parent = <&msi2>;
-		msi2: msi@18014000 {
+		msi2: msi-controller {
  			compatible = "brcm,iproc-msi";
  			msi-controller;
  			interrupt-parent = <&gic>;
Looks good and thanks!

Acked-by: Ray Jui <ray.jui@xxxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[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