Re: [PATCH 2/2] ARM: dts: stm32: Auto-detect DHSOM with uSD level translator

2021-11-21 Thread Patrice CHOTARD 
Hi Marek

On 11/13/21 3:29 AM, Marek Vasut wrote:
> The uSD level translator on DHSOM and Avenger96 are optional, however it
> is possible to auto-detect it. This is done by setting SD CMD line high,
> and then testing whether signal level on CK line matches the signal level
> on CKIN line. If so, the uSD level translator is present, otherwise it is
> not populated.
> 
> Signed-off-by: Marek Vasut 
> Cc: Patrice Chotard 
> Cc: Patrick Delaunay 
> ---
>  arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi   | 4 
>  arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi | 4 
>  2 files changed, 8 insertions(+)
> 
> diff --git a/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi 
> b/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi
> index 71b0486f02a..5b2b09bcfb9 100644
> --- a/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi
> +++ b/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi
> @@ -216,6 +216,10 @@
>  
>   {
>   u-boot,dm-spl;
> + st,use-ckin;
> + st,cmd-gpios = < 2 0>;
> + st,ck-gpios = < 12 0>;
> + st,ckin-gpios = < 4 0>;
>  };
>  
>  _b4_pins_a {
> diff --git a/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi 
> b/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi
> index 4b1dbf08387..c96eba99c5e 100644
> --- a/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi
> +++ b/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi
> @@ -32,6 +32,10 @@
>  
>   {
>   u-boot,dm-spl;
> + st,use-ckin;
> + st,cmd-gpios = < 2 0>;
> + st,ck-gpios = < 12 0>;
> + st,ckin-gpios = < 4 0>;
>  };
>  
>  _b4_pins_a {
> 
Reviewed-by: Patrice Chotard 
Thanks
Patrice

[PATCH 2/2] ARM: dts: stm32: Auto-detect DHSOM with uSD level translator

2021-11-12 Thread Marek Vasut 
The uSD level translator on DHSOM and Avenger96 are optional, however it
is possible to auto-detect it. This is done by setting SD CMD line high,
and then testing whether signal level on CK line matches the signal level
on CKIN line. If so, the uSD level translator is present, otherwise it is
not populated.

Signed-off-by: Marek Vasut 
Cc: Patrice Chotard 
Cc: Patrick Delaunay 
---
 arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi   | 4 
 arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi | 4 
 2 files changed, 8 insertions(+)

diff --git a/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi 
b/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi
index 71b0486f02a..5b2b09bcfb9 100644
--- a/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp15xx-dhcom-u-boot.dtsi
@@ -216,6 +216,10 @@
 
  {
u-boot,dm-spl;
+   st,use-ckin;
+   st,cmd-gpios = < 2 0>;
+   st,ck-gpios = < 12 0>;
+   st,ckin-gpios = < 4 0>;
 };
 
 _b4_pins_a {
diff --git a/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi 
b/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi
index 4b1dbf08387..c96eba99c5e 100644
--- a/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi
+++ b/arch/arm/dts/stm32mp15xx-dhcor-avenger96-u-boot.dtsi
@@ -32,6 +32,10 @@
 
  {
u-boot,dm-spl;
+   st,use-ckin;
+   st,cmd-gpios = < 2 0>;
+   st,ck-gpios = < 12 0>;
+   st,ckin-gpios = < 4 0>;
 };
 
 _b4_pins_a {
-- 
2.33.0