[RFC,02/11] fixup! arm64: dts: renesas: salvator-x: Add MAX9286 expansion board
diff mbox series

Message ID 20191216171620.372683-3-jacopo+renesas@jmondi.org
State Under Review
Delegated to: Kieran Bingham
Headers show
Series
  • GMSL: Initial RDACM21 support
Related show

Commit Message

Jacopo Mondi Dec. 16, 2019, 5:16 p.m. UTC
Fixes DTC warning:
r8a7795-es1-salvator-x.dt.yaml: camera@31: reg: [[49, 65, 81]] is too short
---
 .../boot/dts/renesas/salvator-x-max9286.dtsi     | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

Comments

Kieran Bingham Dec. 16, 2019, 10:37 p.m. UTC | #1
Hi Jacopo,

On 16/12/2019 17:16, Jacopo Mondi wrote:
> Fixes DTC warning:
> r8a7795-es1-salvator-x.dt.yaml: camera@31: reg: [[49, 65, 81]] is too short

Agreed, these should be separated.

Reviewed-by: Kieran Bingham <kieran.bingham+renesas@ideasonboard.com>

Thanks - I'll squash this onto my gmsl/platform branch.

--
Kieran


> ---
>  .../boot/dts/renesas/salvator-x-max9286.dtsi     | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/renesas/salvator-x-max9286.dtsi b/arch/arm64/boot/dts/renesas/salvator-x-max9286.dtsi
> index cfa3ecb6b2ae..0cc4c82b512f 100644
> --- a/arch/arm64/boot/dts/renesas/salvator-x-max9286.dtsi
> +++ b/arch/arm64/boot/dts/renesas/salvator-x-max9286.dtsi
> @@ -191,7 +191,7 @@ i2c@0 {
>  #ifdef MAXIM_CAMERA0
>  				camera@31 {
>  					compatible = MAXIM_CAMERA0;
> -					reg = <0x31 0x41 0x51>;
> +					reg = <0x31>, <0x41>, <0x51>;
>  
>  					port {
>  						rdacm20_out0: endpoint {
> @@ -211,7 +211,7 @@ i2c@1 {
>  #ifdef MAXIM_CAMERA1
>  				camera@32 {
>  					compatible = MAXIM_CAMERA1;
> -					reg = <0x32 0x42 0x52>;
> +					reg = <0x32>, <0x42>, <0x52>;
>  					port {
>  						rdacm20_out1: endpoint {
>  							remote-endpoint = <&max9286_in1>;
> @@ -229,7 +229,7 @@ i2c@2 {
>  #ifdef MAXIM_CAMERA2
>  				camera@33 {
>  					compatible = MAXIM_CAMERA2;
> -					reg = <0x33 0x43 0x53>;
> +					reg = <0x33>, <0x43>, <0x53>;
>  					port {
>  						rdacm20_out2: endpoint {
>  							remote-endpoint = <&max9286_in2>;
> @@ -247,7 +247,7 @@ i2c@3 {
>  #ifdef MAXIM_CAMERA3
>  				camera@34 {
>  					compatible = MAXIM_CAMERA3;
> -					reg = <0x34 0x44 0x54>;
> +					reg = <0x34>, <0x44>, <0x54>;
>  					port {
>  						rdacm20_out3: endpoint {
>  							remote-endpoint = <&max9286_in3>;
> @@ -326,7 +326,7 @@ i2c@0 {
>  #ifdef MAXIM_CAMERA4
>  				camera@35 {
>  					compatible = MAXIM_CAMERA4;
> -					reg = <0x35 0x45 0x55>;
> +					reg = <0x35>, <0x45>, <0x55>;
>  					port {
>  						rdacm20_out4: endpoint {
>  							remote-endpoint = <&max9286_in4>;
> @@ -344,7 +344,7 @@ i2c@1 {
>  #ifdef MAXIM_CAMERA5
>  				camera@36 {
>  					compatible = MAXIM_CAMERA5;
> -					reg = <0x36 0x46 0x56>;
> +					reg = <0x36>, <0x46>, <0x56>;
>  					port {
>  						rdacm20_out5: endpoint {
>  							remote-endpoint = <&max9286_in5>;
> @@ -362,7 +362,7 @@ i2c@2 {
>  #ifdef MAXIM_CAMERA6
>  				camera@37 {
>  					compatible = MAXIM_CAMERA6;
> -					reg = <0x37 0x47 0x57>;
> +					reg = <0x37>, <0x47>, <0x57>;
>  					port {
>  						rdacm20_out6: endpoint {
>  							remote-endpoint = <&max9286_in6>;
> @@ -380,7 +380,7 @@ i2c@3 {
>  #ifdef MAXIM_CAMERA7
>  				camera@38 {
>  					compatible = MAXIM_CAMERA7;
> -					reg = <0x38 0x48 0x58>;
> +					reg = <0x38>, <0x48>, <0x58>;
>  					port {
>  						rdacm20_out7: endpoint {
>  							remote-endpoint = <&max9286_in7>;
>

Patch
diff mbox series

diff --git a/arch/arm64/boot/dts/renesas/salvator-x-max9286.dtsi b/arch/arm64/boot/dts/renesas/salvator-x-max9286.dtsi
index cfa3ecb6b2ae..0cc4c82b512f 100644
--- a/arch/arm64/boot/dts/renesas/salvator-x-max9286.dtsi
+++ b/arch/arm64/boot/dts/renesas/salvator-x-max9286.dtsi
@@ -191,7 +191,7 @@  i2c@0 {
 #ifdef MAXIM_CAMERA0
 				camera@31 {
 					compatible = MAXIM_CAMERA0;
-					reg = <0x31 0x41 0x51>;
+					reg = <0x31>, <0x41>, <0x51>;
 
 					port {
 						rdacm20_out0: endpoint {
@@ -211,7 +211,7 @@  i2c@1 {
 #ifdef MAXIM_CAMERA1
 				camera@32 {
 					compatible = MAXIM_CAMERA1;
-					reg = <0x32 0x42 0x52>;
+					reg = <0x32>, <0x42>, <0x52>;
 					port {
 						rdacm20_out1: endpoint {
 							remote-endpoint = <&max9286_in1>;
@@ -229,7 +229,7 @@  i2c@2 {
 #ifdef MAXIM_CAMERA2
 				camera@33 {
 					compatible = MAXIM_CAMERA2;
-					reg = <0x33 0x43 0x53>;
+					reg = <0x33>, <0x43>, <0x53>;
 					port {
 						rdacm20_out2: endpoint {
 							remote-endpoint = <&max9286_in2>;
@@ -247,7 +247,7 @@  i2c@3 {
 #ifdef MAXIM_CAMERA3
 				camera@34 {
 					compatible = MAXIM_CAMERA3;
-					reg = <0x34 0x44 0x54>;
+					reg = <0x34>, <0x44>, <0x54>;
 					port {
 						rdacm20_out3: endpoint {
 							remote-endpoint = <&max9286_in3>;
@@ -326,7 +326,7 @@  i2c@0 {
 #ifdef MAXIM_CAMERA4
 				camera@35 {
 					compatible = MAXIM_CAMERA4;
-					reg = <0x35 0x45 0x55>;
+					reg = <0x35>, <0x45>, <0x55>;
 					port {
 						rdacm20_out4: endpoint {
 							remote-endpoint = <&max9286_in4>;
@@ -344,7 +344,7 @@  i2c@1 {
 #ifdef MAXIM_CAMERA5
 				camera@36 {
 					compatible = MAXIM_CAMERA5;
-					reg = <0x36 0x46 0x56>;
+					reg = <0x36>, <0x46>, <0x56>;
 					port {
 						rdacm20_out5: endpoint {
 							remote-endpoint = <&max9286_in5>;
@@ -362,7 +362,7 @@  i2c@2 {
 #ifdef MAXIM_CAMERA6
 				camera@37 {
 					compatible = MAXIM_CAMERA6;
-					reg = <0x37 0x47 0x57>;
+					reg = <0x37>, <0x47>, <0x57>;
 					port {
 						rdacm20_out6: endpoint {
 							remote-endpoint = <&max9286_in6>;
@@ -380,7 +380,7 @@  i2c@3 {
 #ifdef MAXIM_CAMERA7
 				camera@38 {
 					compatible = MAXIM_CAMERA7;
-					reg = <0x38 0x48 0x58>;
+					reg = <0x38>, <0x48>, <0x58>;
 					port {
 						rdacm20_out7: endpoint {
 							remote-endpoint = <&max9286_in7>;