Re: [U-Boot] [PATCH] net: turn name len check into an assert

2011-09-21 Thread Wolfgang Denk
Dear Mike Frysinger, In message <1315802993-12798-1-git-send-email-vap...@gentoo.org> you wrote: > The new sanity check introduces a printf warning for some systems: > eth.c:233: warning: format '%zu' expects type 'size_t', but argument 3 > has type 'int' > > Rather than tweak the format s

Re: [U-Boot] [PATCH] net: turn name len check into an assert

2011-09-12 Thread Mike Frysinger
On Monday, September 12, 2011 13:06:13 Marek Vasut wrote: > On Monday, September 12, 2011 06:29:10 PM Mike Frysinger wrote: > > On Monday, September 12, 2011 01:08:50 Marek Vasut wrote: > > > On Monday, September 12, 2011 06:49:53 AM Mike Frysinger wrote: > > > > --- a/net/eth.c > > > > +++ b/net/e

Re: [U-Boot] [PATCH] net: turn name len check into an assert

2011-09-12 Thread Marek Vasut
On Monday, September 12, 2011 06:29:10 PM Mike Frysinger wrote: > On Monday, September 12, 2011 01:08:50 Marek Vasut wrote: > > On Monday, September 12, 2011 06:49:53 AM Mike Frysinger wrote: > > > --- a/net/eth.c > > > +++ b/net/eth.c > > > > > > - size_t len = strlen(dev->name); > > > - if (len

Re: [U-Boot] [PATCH] net: turn name len check into an assert

2011-09-12 Thread Mike Frysinger
On Monday, September 12, 2011 01:08:50 Marek Vasut wrote: > On Monday, September 12, 2011 06:49:53 AM Mike Frysinger wrote: > > --- a/net/eth.c > > +++ b/net/eth.c > > > > - size_t len = strlen(dev->name); > > - if (len >= NAMESIZE) { > > - printf("Network driver name is too long (%z

Re: [U-Boot] [PATCH] net: turn name len check into an assert

2011-09-11 Thread Marek Vasut
On Monday, September 12, 2011 06:49:53 AM Mike Frysinger wrote: > The new sanity check introduces a printf warning for some systems: > eth.c:233: warning: format '%zu' expects type 'size_t', but argument 3 has > type 'int' > > Rather than tweak the format string, use the new assert() helper

[U-Boot] [PATCH] net: turn name len check into an assert

2011-09-11 Thread Mike Frysinger
The new sanity check introduces a printf warning for some systems: eth.c:233: warning: format '%zu' expects type 'size_t', but argument 3 has type 'int' Rather than tweak the format string, use the new assert() helper instead. Signed-off-by: Mike Frysinger --- net/eth.c |7 +--