Hi Simon, One minor comment from me.
On Mon, 22 Sep 2014 09:48:52 -0600 Simon Glass <[email protected]> wrote: > + > +U_BOOT_DRIVER(serial_omap_ns16550) = { > + .name = "serial_omap", > + .id = UCLASS_SERIAL, > +#ifdef CONFIG_OF_CONTROL > + .of_match = omap_serial_ids, > + .ofdata_to_platdata = omap_serial_ofdata_to_platdata, > +#endif > + .platdata_auto_alloc_size = sizeof(struct ns16550_platdata), > + .priv_auto_alloc_size = sizeof(struct NS16550), > + .probe = ns16550_serial_probe, > + .ops = &ns16550_serial_ops, > + .flags = DM_FLAG_PRE_RELOC, > +}; U_BOOT_DRIVER(serial_omap_ns16550) = { .name = "serial_omap", .id = UCLASS_SERIAL, .of_match = of_match_ptr(omap_serial_ids), .ofdata_to_platdata = of_match_ptr(omap_serial_ofdata_to_platdata), .platdata_auto_alloc_size = sizeof(struct ns16550_platdata), .priv_auto_alloc_size = sizeof(struct NS16550), .probe = ns16550_serial_probe, .ops = &ns16550_serial_ops, .flags = DM_FLAG_PRE_RELOC, }; is cleaner though you need to apply the following first: http://patchwork.ozlabs.org/patch/397088/ Best Regards Masahiro Yamada _______________________________________________ U-Boot mailing list [email protected] http://lists.denx.de/mailman/listinfo/u-boot

