Paul Gortmaker wrote: > In message: Re: [U-Boot-Users] cpu/mpc85xx/fdt.c forcing incorrect UART clock > into fdt. > on 09/07/2008 Kumar Gala wrote: > >> On Jul 8, 2008, at 4:42 PM, Paul Gortmaker wrote: >> >>> I fixed it with this simple patch, but I was wondering whether it >>> would >>> be better to alternatively just check if the dtb value is zero, and >>> then only insert a value; otherwise leave it alone... >> I think that should be left up to a board to decide, but in general it >> seems like doing the setting unconditionally is ok, we just didn't >> handle the case you had and the patch address it. > > OK, so that we only end up changing just one thing at a time, I'll > spin a proper patch which just makes the NS16550_CLK change, and > we can change the create flag as per Jerry's comment at a later date > if required. > > In the interest of consistency, I've make the same change for 83xx and > 86xx as well, since the code is pretty much identical, and a MAKEALL > shows no boards failing due to a non-defined NS16550_CLK. Patch to > follow shortly... > > Thanks, > Paul.
FWIIW, this works for me (with my disclaimer of not being a domain expert). gvb ------------------------------------------------------------------------- Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW! Studies have shown that voting for your favorite open source project, along with a healthy diet, reduces your potential for chronic lameness and boredom. Vote Now at http://www.sourceforge.net/community/cca08 _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users
