[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