Re: [U-Boot] [PATCH] ARM: dts: da850-lcdk: Update DTS files for SPL device tree support

2020-01-19 Thread Lokesh Vutla 



On 13/11/19 9:16 PM, Adam Ford wrote:
> Currently, the da850-lcdk uses SPL_OF_PLATDATA and manually loads
> the necessary source code instead of using the auto-generated,
> because the drivers don't properly autogenerate the code.
> 
> This patch simply enables the various device tree options to
> mimic the da850-evm which doesn't need or use OF_PLATDATA for
> device tree support.  It does not disable OF_PLATDATA.
> 
> Signed-off-by: Adam Ford 

Merged into u-boot-ti.

Thanks and regards,
Lokesh

Re: [U-Boot] [PATCH] ARM: dts: da850-lcdk: Update DTS files for SPL device tree support

2019-11-14 Thread Bartosz Golaszewski 
śr., 13 lis 2019 o 16:46 Adam Ford  napisał(a):
>
> Currently, the da850-lcdk uses SPL_OF_PLATDATA and manually loads
> the necessary source code instead of using the auto-generated,
> because the drivers don't properly autogenerate the code.
>
> This patch simply enables the various device tree options to
> mimic the da850-evm which doesn't need or use OF_PLATDATA for
> device tree support.  It does not disable OF_PLATDATA.
>
> Signed-off-by: Adam Ford 
>
> diff --git a/arch/arm/dts/da850-lcdk-u-boot.dtsi 
> b/arch/arm/dts/da850-lcdk-u-boot.dtsi
> index 541f4ca200..b372d06ca9 100644
> --- a/arch/arm/dts/da850-lcdk-u-boot.dtsi
> +++ b/arch/arm/dts/da850-lcdk-u-boot.dtsi
> @@ -8,9 +8,23 @@
>  / {
> aliases {
> i2c0 = 
> +   mmc0 = 
> +   serial2 = 
> +   };
> +
> +   soc@1c0 {
> +   u-boot,dm-spl;
> };
>
> nand {
> compatible = "ti,davinci-nand";
> };
>  };
> +
> + {
> +   u-boot,dm-spl;
> +};
> +
> + {
> +   u-boot,dm-spl;
> +};
> diff --git a/arch/arm/dts/da850-lcdk.dts b/arch/arm/dts/da850-lcdk.dts
> index 0177e3ed20..db8ae566f3 100644
> --- a/arch/arm/dts/da850-lcdk.dts
> +++ b/arch/arm/dts/da850-lcdk.dts
> @@ -18,7 +18,7 @@
> };
>
> chosen {
> -   stdout-path = "serial2:115200n8";
> +   stdout-path = 
> };
>
> memory@c000 {
> --
> 2.20.1
>

Hi Adam,

the generated platdata has wrong driver names. I'm not sure how these
are generated but at first glance it looks like the compatible string
is transformed by converting commas and hyphens into underscores. This
of course won't work as the real driver names are different and the
lookup fails (with your patch - if I remove the serial U_BOOT_DEVICE()
from the board file, the serial stops working). Are you aware of how
other SPL users handle this?

Bart
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

[U-Boot] [PATCH] ARM: dts: da850-lcdk: Update DTS files for SPL device tree support

2019-11-13 Thread Adam Ford 
Currently, the da850-lcdk uses SPL_OF_PLATDATA and manually loads
the necessary source code instead of using the auto-generated,
because the drivers don't properly autogenerate the code.

This patch simply enables the various device tree options to
mimic the da850-evm which doesn't need or use OF_PLATDATA for
device tree support.  It does not disable OF_PLATDATA.

Signed-off-by: Adam Ford 

diff --git a/arch/arm/dts/da850-lcdk-u-boot.dtsi 
b/arch/arm/dts/da850-lcdk-u-boot.dtsi
index 541f4ca200..b372d06ca9 100644
--- a/arch/arm/dts/da850-lcdk-u-boot.dtsi
+++ b/arch/arm/dts/da850-lcdk-u-boot.dtsi
@@ -8,9 +8,23 @@
 / {
aliases {
i2c0 = 
+   mmc0 = 
+   serial2 = 
+   };
+
+   soc@1c0 {
+   u-boot,dm-spl;
};
 
nand {
compatible = "ti,davinci-nand";
};
 };
+
+ {
+   u-boot,dm-spl;
+};
+
+ {
+   u-boot,dm-spl;
+};
diff --git a/arch/arm/dts/da850-lcdk.dts b/arch/arm/dts/da850-lcdk.dts
index 0177e3ed20..db8ae566f3 100644
--- a/arch/arm/dts/da850-lcdk.dts
+++ b/arch/arm/dts/da850-lcdk.dts
@@ -18,7 +18,7 @@
};
 
chosen {
-   stdout-path = "serial2:115200n8";
+   stdout-path = 
};
 
memory@c000 {
-- 
2.20.1

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot