Hi Tim, I have no problem with the whole series, I will start to apply. Just a couple of questions:
On 08/04/2015 21:54, Tim Harvey wrote: > Certain older kernels in use by some customers erroneously define a uart3 > for GW54xx with a pinmux that conflicts with NAND. This will remove > that node to avoid such conflicts. > > Signed-off-by: Tim Harvey <thar...@gateworks.com> > --- > board/gateworks/gw_ventana/gw_ventana.c | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/board/gateworks/gw_ventana/gw_ventana.c > b/board/gateworks/gw_ventana/gw_ventana.c > index 068c726..06611b5 100644 > --- a/board/gateworks/gw_ventana/gw_ventana.c > +++ b/board/gateworks/gw_ventana/gw_ventana.c > @@ -1599,6 +1599,17 @@ int ft_board_setup(void *blob, bd_t *bd) > strlen((const char *)info->model) + 1); > > /* > + * disable serial2 node for GW54xx for compatibility with older > + * 3.10.x kernel that improperly had this node enabled in the DT > + */ I understand the issue, but I guess you have a dtb file for your (Freescale) 3.10 kernel and another one for kernel mainline. So why this issue should be fixed here and not in the related DTS file ? > + if (board_type == GW54xx) { > + i = fdt_path_offset(blob, > + "/soc/aips-bus@02100000/serial@021ec000"); > + if (i) > + fdt_del_node(blob, i); > + } > + > + /* > * disable wdog1/wdog2 nodes for GW51xx below revC to work around > * errata causing wdog timer to be unreliable. > */ > Best regards, Stefano Babic -- ===================================================================== DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de ===================================================================== _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot