From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Thu, 25 Aug 2011 05:19:55 +0200 Subject: [U-Boot] [PATCH 8/9] DM9000: change some printf to use debug instead In-Reply-To: <683E266C-CDFB-4310-8145-1926C6FCCBD8@armadeus.org> References: <20110810200828.21204.60050.stgit@shuttle2.etheralp.ch> <20110824202056.71C1211F9E78@gemini.denx.de> <683E266C-CDFB-4310-8145-1926C6FCCBD8@armadeus.org> Message-ID: <201108250519.55535.marek.vasut@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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-A30D5245262A@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 the printf issues as a first priority by using > >> the DM9000_DBG. > > > > NAK. > > > > 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? My wild guess would be to FIRST fix the DM9000_DBG and NEXT convert these printf()s to debug()s. Cheers > > Best regards, > Eric > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot