Hi, On 25 November 2016 at 09:59, Fabio Estevam <feste...@gmail.com> wrote: > On Fri, Nov 25, 2016 at 2:57 PM, Jagan Teki <ja...@openedev.com> wrote: > >> Yeah, ie what if dts has a wrong value and do print that and continue >> with default width, so-that the user will update this for next run. >> Since it's not key a attribute to break or decide functionality better >> to go with it. > > Agreed. This also matches with the kernel behaviour.
So it is correct to print an error, and then continue? This error will almost never occur and thus it wastes code space. SPI is sensitive because it can be used in SPL. Linux doesn't care about code size as much. So how about either: 1. debug() and return an error 2. debug() and skip the error Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
