diff mbox series

[v2,1/2] arm64: dts: ti: am65: align ti,pindir-d0-out-d1-in property with dt-shema

Message ID 20210608051414.14873-2-a-govindraju@ti.com
State Accepted
Commit 4f76ea7b4da1cce9a9bda1fa678ef8036584c66b
Headers show
Series arm64: am65/am64: Fix ti,pindir-d0-out-d1-in property to boolean | expand

Commit Message

Aswath Govindraju June 8, 2021, 5:14 a.m. UTC
ti,pindir-d0-out-d1-in property is expected to be of type boolean.
Therefore, fix the property accordingly.

Fixes: e180f76d0641 ("arm64: dts: ti: Add support for Siemens IOT2050 boards")
Fixes: 5da94b50475a ("arm64: dts: ti: k3-am654: Enable main domain McSPI0")
Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 2 +-
 arch/arm64/boot/dts/ti/k3-am654-base-board.dts     | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

Comments

Jan Kiszka June 8, 2021, 5:20 a.m. UTC | #1
On 08.06.21 07:14, Aswath Govindraju wrote:
> ti,pindir-d0-out-d1-in property is expected to be of type boolean.
> Therefore, fix the property accordingly.
> 

Is there a patch for Documentation/devicetree/bindings/spi/omap-spi.txt
on the way as well that makes this even clearer?

> Fixes: e180f76d0641 ("arm64: dts: ti: Add support for Siemens IOT2050 boards")
> Fixes: 5da94b50475a ("arm64: dts: ti: k3-am654: Enable main domain McSPI0")
> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>
> ---
>  arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 2 +-
>  arch/arm64/boot/dts/ti/k3-am654-base-board.dts     | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> index f4ec9ed52939..23d51b6a9cf2 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> @@ -575,7 +575,7 @@
>  
>  	#address-cells = <1>;
>  	#size-cells= <0>;
> -	ti,pindir-d0-out-d1-in = <1>;
> +	ti,pindir-d0-out-d1-in;
>  };
>  

Acked-by: Jan Kiszka <jan.kiszka@siemens.com>

Jan

>  &tscadc0 {
> diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> index eddb2ffb93ca..1b947e2c2e74 100644
> --- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> @@ -299,7 +299,7 @@
>  	pinctrl-0 = <&main_spi0_pins_default>;
>  	#address-cells = <1>;
>  	#size-cells= <0>;
> -	ti,pindir-d0-out-d1-in = <1>;
> +	ti,pindir-d0-out-d1-in;
>  
>  	flash@0{
>  		compatible = "jedec,spi-nor";
>
Vignesh Raghavendra June 9, 2021, 1:48 p.m. UTC | #2
On 6/8/21 10:44 AM, Aswath Govindraju wrote:
> ti,pindir-d0-out-d1-in property is expected to be of type boolean.

> Therefore, fix the property accordingly.

> 

> Fixes: e180f76d0641 ("arm64: dts: ti: Add support for Siemens IOT2050 boards")

> Fixes: 5da94b50475a ("arm64: dts: ti: k3-am654: Enable main domain McSPI0")

> Signed-off-by: Aswath Govindraju <a-govindraju@ti.com>

> ---

>  arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 2 +-

>  arch/arm64/boot/dts/ti/k3-am654-base-board.dts     | 2 +-

>  2 files changed, 2 insertions(+), 2 deletions(-)

> 

> diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi

> index f4ec9ed52939..23d51b6a9cf2 100644

> --- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi

> +++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi

> @@ -575,7 +575,7 @@

>  

>  	#address-cells = <1>;

>  	#size-cells= <0>;

> -	ti,pindir-d0-out-d1-in = <1>;

> +	ti,pindir-d0-out-d1-in;

>  };

>  

>  &tscadc0 {

> diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts

> index eddb2ffb93ca..1b947e2c2e74 100644

> --- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts

> +++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts

> @@ -299,7 +299,7 @@

>  	pinctrl-0 = <&main_spi0_pins_default>;

>  	#address-cells = <1>;

>  	#size-cells= <0>;

> -	ti,pindir-d0-out-d1-in = <1>;

> +	ti,pindir-d0-out-d1-in;

>  

>  	flash@0{

>  		compatible = "jedec,spi-nor";



Reviewed-by: Vignesh Raghavendra <vigneshr@ti.com>



Regards
Vignesh
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
index f4ec9ed52939..23d51b6a9cf2 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
@@ -575,7 +575,7 @@ 
 
 	#address-cells = <1>;
 	#size-cells= <0>;
-	ti,pindir-d0-out-d1-in = <1>;
+	ti,pindir-d0-out-d1-in;
 };
 
 &tscadc0 {
diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
index eddb2ffb93ca..1b947e2c2e74 100644
--- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
@@ -299,7 +299,7 @@ 
 	pinctrl-0 = <&main_spi0_pins_default>;
 	#address-cells = <1>;
 	#size-cells= <0>;
-	ti,pindir-d0-out-d1-in = <1>;
+	ti,pindir-d0-out-d1-in;
 
 	flash@0{
 		compatible = "jedec,spi-nor";