diff mbox series

ARM: dts: xilinx: Fix I2C and SPI bus warnings

Message ID 20180913181245.25484-15-robh@kernel.org
State Accepted
Commit f5054ceed420b1f38d37920a4c65446fcc5d6b90
Headers show
Series ARM: dts: xilinx: Fix I2C and SPI bus warnings | expand

Commit Message

Rob Herring Sept. 13, 2018, 6:12 p.m. UTC
dtc has new checks for I2C and SPI buses. Fix the warnings in node names
and unit-addresses.

arch/arm/boot/dts/zynq-zc702.dtb: Warning (i2c_bus_reg): /amba/i2c@e0004000/i2c-mux@74/i2c@7/hwmon@52: I2C bus unit address format error, expected "34"
arch/arm/boot/dts/zynq-zc702.dtb: Warning (i2c_bus_reg): /amba/i2c@e0004000/i2c-mux@74/i2c@7/hwmon@53: I2C bus unit address format error, expected "35"
arch/arm/boot/dts/zynq-zc702.dtb: Warning (i2c_bus_reg): /amba/i2c@e0004000/i2c-mux@74/i2c@7/hwmon@54: I2C bus unit address format error, expected "36"
arch/arm/boot/dts/zynq-zc770-xm013.dtb: Warning (spi_bus_reg): /amba/spi@e0006000/eeprom@0: SPI bus unit address format error, expected "2"
arch/arm/boot/dts/zynq-zc770-xm010.dtb: Warning (spi_bus_reg): /amba/spi@e0007000/flash@0: SPI bus unit address format error, expected "1"

Cc: Michal Simek <michal.simek@xilinx.com>
Signed-off-by: Rob Herring <robh@kernel.org>

---
Please apply to the sub-arch tree. The dtc changes haven't landed, but 
will for 4.20.

 arch/arm/boot/dts/zynq-zc702.dts       | 12 ++++++------
 arch/arm/boot/dts/zynq-zc770-xm010.dts |  2 +-
 arch/arm/boot/dts/zynq-zc770-xm013.dts |  2 +-
 3 files changed, 8 insertions(+), 8 deletions(-)

-- 
2.17.1

Comments

Michal Simek Sept. 18, 2018, 10:52 a.m. UTC | #1
On 13.9.2018 20:12, Rob Herring wrote:
> dtc has new checks for I2C and SPI buses. Fix the warnings in node names

> and unit-addresses.

> 

> arch/arm/boot/dts/zynq-zc702.dtb: Warning (i2c_bus_reg): /amba/i2c@e0004000/i2c-mux@74/i2c@7/hwmon@52: I2C bus unit address format error, expected "34"

> arch/arm/boot/dts/zynq-zc702.dtb: Warning (i2c_bus_reg): /amba/i2c@e0004000/i2c-mux@74/i2c@7/hwmon@53: I2C bus unit address format error, expected "35"

> arch/arm/boot/dts/zynq-zc702.dtb: Warning (i2c_bus_reg): /amba/i2c@e0004000/i2c-mux@74/i2c@7/hwmon@54: I2C bus unit address format error, expected "36"

> arch/arm/boot/dts/zynq-zc770-xm013.dtb: Warning (spi_bus_reg): /amba/spi@e0006000/eeprom@0: SPI bus unit address format error, expected "2"

> arch/arm/boot/dts/zynq-zc770-xm010.dtb: Warning (spi_bus_reg): /amba/spi@e0007000/flash@0: SPI bus unit address format error, expected "1"

> 

> Cc: Michal Simek <michal.simek@xilinx.com>

> Signed-off-by: Rob Herring <robh@kernel.org>

> ---

> Please apply to the sub-arch tree. The dtc changes haven't landed, but 

> will for 4.20.

> 

>  arch/arm/boot/dts/zynq-zc702.dts       | 12 ++++++------

>  arch/arm/boot/dts/zynq-zc770-xm010.dts |  2 +-

>  arch/arm/boot/dts/zynq-zc770-xm013.dts |  2 +-

>  3 files changed, 8 insertions(+), 8 deletions(-)

> 

> diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts

> index cc5a3dc2b4a0..27cd6cb52f1b 100644

> --- a/arch/arm/boot/dts/zynq-zc702.dts

> +++ b/arch/arm/boot/dts/zynq-zc702.dts

> @@ -174,17 +174,17 @@

>  			#address-cells = <1>;

>  			#size-cells = <0>;

>  			reg = <7>;

> -			hwmon@52 {

> +			hwmon@34 {

>  				compatible = "ti,ucd9248";

> -				reg = <52>;

> +				reg = <0x34>;

>  			};

> -			hwmon@53 {

> +			hwmon@35 {

>  				compatible = "ti,ucd9248";

> -				reg = <53>;

> +				reg = <0x35>;

>  			};

> -			hwmon@54 {

> +			hwmon@36 {

>  				compatible = "ti,ucd9248";

> -				reg = <54>;

> +				reg = <0x36>;

>  			};

>  		};

>  	};

> diff --git a/arch/arm/boot/dts/zynq-zc770-xm010.dts b/arch/arm/boot/dts/zynq-zc770-xm010.dts

> index 0e1bfdd3421f..0dd352289a45 100644

> --- a/arch/arm/boot/dts/zynq-zc770-xm010.dts

> +++ b/arch/arm/boot/dts/zynq-zc770-xm010.dts

> @@ -68,7 +68,7 @@

>  	status = "okay";

>  	num-cs = <4>;

>  	is-decoded-cs = <0>;

> -	flash@0 {

> +	flash@1 {

>  		compatible = "sst25wf080", "jedec,spi-nor";

>  		reg = <1>;

>  		spi-max-frequency = <1000000>;

> diff --git a/arch/arm/boot/dts/zynq-zc770-xm013.dts b/arch/arm/boot/dts/zynq-zc770-xm013.dts

> index 651913f1afa2..4ae2c85df3a0 100644

> --- a/arch/arm/boot/dts/zynq-zc770-xm013.dts

> +++ b/arch/arm/boot/dts/zynq-zc770-xm013.dts

> @@ -62,7 +62,7 @@

>  	status = "okay";

>  	num-cs = <4>;

>  	is-decoded-cs = <0>;

> -	eeprom: eeprom@0 {

> +	eeprom: eeprom@2 {

>  		at25,byte-len = <8192>;

>  		at25,addr-mode = <2>;

>  		at25,page-size = <32>;

> 


Applied. All checked.

Thanks,
Michal
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/zynq-zc702.dts b/arch/arm/boot/dts/zynq-zc702.dts
index cc5a3dc2b4a0..27cd6cb52f1b 100644
--- a/arch/arm/boot/dts/zynq-zc702.dts
+++ b/arch/arm/boot/dts/zynq-zc702.dts
@@ -174,17 +174,17 @@ 
 			#address-cells = <1>;
 			#size-cells = <0>;
 			reg = <7>;
-			hwmon@52 {
+			hwmon@34 {
 				compatible = "ti,ucd9248";
-				reg = <52>;
+				reg = <0x34>;
 			};
-			hwmon@53 {
+			hwmon@35 {
 				compatible = "ti,ucd9248";
-				reg = <53>;
+				reg = <0x35>;
 			};
-			hwmon@54 {
+			hwmon@36 {
 				compatible = "ti,ucd9248";
-				reg = <54>;
+				reg = <0x36>;
 			};
 		};
 	};
diff --git a/arch/arm/boot/dts/zynq-zc770-xm010.dts b/arch/arm/boot/dts/zynq-zc770-xm010.dts
index 0e1bfdd3421f..0dd352289a45 100644
--- a/arch/arm/boot/dts/zynq-zc770-xm010.dts
+++ b/arch/arm/boot/dts/zynq-zc770-xm010.dts
@@ -68,7 +68,7 @@ 
 	status = "okay";
 	num-cs = <4>;
 	is-decoded-cs = <0>;
-	flash@0 {
+	flash@1 {
 		compatible = "sst25wf080", "jedec,spi-nor";
 		reg = <1>;
 		spi-max-frequency = <1000000>;
diff --git a/arch/arm/boot/dts/zynq-zc770-xm013.dts b/arch/arm/boot/dts/zynq-zc770-xm013.dts
index 651913f1afa2..4ae2c85df3a0 100644
--- a/arch/arm/boot/dts/zynq-zc770-xm013.dts
+++ b/arch/arm/boot/dts/zynq-zc770-xm013.dts
@@ -62,7 +62,7 @@ 
 	status = "okay";
 	num-cs = <4>;
 	is-decoded-cs = <0>;
-	eeprom: eeprom@0 {
+	eeprom: eeprom@2 {
 		at25,byte-len = <8192>;
 		at25,addr-mode = <2>;
 		at25,page-size = <32>;