[U-Boot] [PATCH 8/9] DM9000: change some printf to use debug instead

2011-08-10 Thread Eric Jarrige
Signed-off-by: Eric Jarrige Cc: Ben Warren --- drivers/net/dm9000x.c |8 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/net/dm9000x.c b/drivers/net/dm9000x.c index b5c5573..9cd0195 100644 --- a/drivers/net/dm9000x.c +++ b/drivers/net/dm9000x.c @@ -232,7 +232,

Re: [U-Boot] [PATCH 8/9] DM9000: change some printf to use debug instead

2011-08-11 Thread Simon Schwarz
Dear Eric Jarrige, On 08/10/2011 10:33 PM, Eric Jarrige wrote: > Signed-off-by: Eric Jarrige > Cc: Ben Warren > --- > drivers/net/dm9000x.c |8 > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/drivers/net/dm9000x.c b/drivers/net/dm9000x.c > index b5c5573..9cd0195

Re: [U-Boot] [PATCH 8/9] DM9000: change some printf to use debug instead

2011-08-11 Thread Detlev Zundel
Hi Simon, > Dear Eric Jarrige, > > On 08/10/2011 10:33 PM, Eric Jarrige wrote: >> Signed-off-by: Eric Jarrige >> Cc: Ben Warren >> --- >> drivers/net/dm9000x.c |8 >> 1 files changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/drivers/net/dm9000x.c b/drivers/net/dm9000x.c >

Re: [U-Boot] [PATCH 8/9] DM9000: change some printf to use debug instead

2011-08-11 Thread Stefano Babic
On 08/10/2011 10:33 PM, Eric Jarrige wrote: > Signed-off-by: Eric Jarrige > Cc: Ben Warren > --- > drivers/net/dm9000x.c |8 > 1 files changed, 4 insertions(+), 4 deletions(-) Hi Eric, > - printf("dm9000 i/o: 0x%x, id: 0x%x \n", CONFIG_DM9000_BASE, > + DM90

Re: [U-Boot] [PATCH 8/9] DM9000: change some printf to use debug instead

2011-08-11 Thread Eric Jarrige
Hi Simon, Hi Detlev, > Hi Simon, > >> Dear Eric Jarrige, >> >> On 08/10/2011 10:33 PM, Eric Jarrige wrote: >>> Signed-off-by: Eric Jarrige >>> Cc: Ben Warren >>> --- >>> drivers/net/dm9000x.c |8 >>> 1 files changed, 4 insertions(+), 4 deletions(-) >>> >>> diff --git a/drivers/net

Re: [U-Boot] [PATCH 8/9] DM9000: change some printf to use debug instead

2011-08-24 Thread Wolfgang Denk
Dear Eric Jarrige, In message <2b613989-addb-43d6-a8c3-a30d52452...@armadeus.org> you wrote: > > >> I'am just wondering: I see that DM9000_DBG is used all over dm9000 code > >> - do you know the reason why not just use debug()? > > dm9000 does not use debug() at all. > That's the reason to fix

Re: [U-Boot] [PATCH 8/9] DM9000: change some printf to use debug instead

2011-08-24 Thread Eric Jarrige
Dear Wolfgang, On 24 août 2011, at 22:20, Wolfgang Denk wrote: > Dear Eric Jarrige, > > In message <2b613989-addb-43d6-a8c3-a30d52452...@armadeus.org> you wrote: >> I'am just wondering: I see that DM9000_DBG is used all over dm9000 code - do you know the reason why not just use debug

Re: [U-Boot] [PATCH 8/9] DM9000: change some printf to use debug instead

2011-08-24 Thread Marek Vasut
On Thursday, August 25, 2011 01:04:17 AM Eric Jarrige wrote: > Dear Wolfgang, > > On 24 août 2011, at 22:20, Wolfgang Denk wrote: > > Dear Eric Jarrige, > > > > In message <2b613989-addb-43d6-a8c3-a30d52452...@armadeus.org> you wrote: > I'am just wondering: I see that DM9000_DBG is used all

Re: [U-Boot] [PATCH 8/9] DM9000: change some printf to use debug instead

2011-08-24 Thread Wolfgang Denk
Dear Eric Jarrige, In message <683e266c-cdfb-4310-8145-1926c6fcc...@armadeus.org> you wrote: > > > If you touch that, then please do it right: us debug() instead, and > > remove DM9000_DBG. Thanks. > > Do you mean both changes in one patch? Yes, please. Best regards, Wolfgang Denk -- DENX