[U-Boot] [RFC PATCH 09/14] dm: tegra: Add platform data for the SPL uart

Stephen Warren swarren at wwwdotorg.org
Wed Sep 24 17:42:05 CEST 2014


On 09/23/2014 05:30 PM, Simon Glass wrote:
> Since we currently don't have device tree available in SPL, add platform
> data so the uart works.

> diff --git a/drivers/serial/serial_tegra.c b/drivers/serial/serial_tegra.c

> +#ifdef CONFIG_OF_CONTROL
>   static const struct udevice_id tegra_serial_ids[] = {
>   	{ .compatible = "nvidia,tegra20-uart" },
>   	{ }
> @@ -26,13 +27,28 @@ static int tegra_serial_ofdata_to_platdata(struct udevice *dev)
>
>   	return 0;
>   }
> +#else
> +struct ns16550_platdata tegra_serial = {
> +	.base = CONFIG_SYS_NS16550_COM1,
> +	.reg_shift = 2,
> +	.clock = V_NS16550_CLK,
> +};
> +
> +U_BOOT_DEVICE(ns16550_serial) = {
> +	"serial_tegra20", &tegra_serial
> +};
> +#endif

If we're going to have to add "hard-coded" instantiation of devices to 
the code to support SPL, I rather wonder what the point is of having DT 
support in these (or honestly, any) drivers at all. Why not just switch 
directly to always using the hard-coded instantiation. Then, we can drop 
all the DT parsing code from the drivers, simplify the code, and 
completely avoid all the annoying DT ABI and binding consistency issues.

>   U_BOOT_DRIVER(serial_ns16550) = {
>   	.name	= "serial_tegra20",
>   	.id	= UCLASS_SERIAL,
> +#ifdef CONFIG_OF_CONTROL
>   	.of_match = tegra_serial_ids,
>   	.ofdata_to_platdata = tegra_serial_ofdata_to_platdata,
>   	.platdata_auto_alloc_size = sizeof(struct ns16550_platdata),
> +#endif
>   	.priv_auto_alloc_size = sizeof(struct NS16550),
>   	.probe = ns16550_serial_probe,
>   	.ops	= &ns16550_serial_ops,
> +	.flags	= DM_FLAG_PRE_RELOC,
>   };



More information about the U-Boot mailing list