Hi, On 25 November 2016 at 20:28, Jagan Teki <ja...@openedev.com> wrote: > On Sat, Nov 26, 2016 at 1:08 AM, Simon Glass <s...@chromium.org> wrote: >> 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 > > I prefer 2. for SPL and replace debug with printf for U-Boot.
OK I have sent v3. Regards, Simon _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot