From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Mon, 24 Oct 2011 12:16:43 +0200 Subject: [U-Boot] [RFC] [PATCH 00/39] Rework of the debug() macro In-Reply-To: References: <1319242654-15534-1-git-send-email-marek.vasut@gmail.com> <201110230017.34182.marek.vasut@gmail.com> Message-ID: <201110241216.43763.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 Monday, October 24, 2011 12:10:02 PM Detlev Zundel wrote: > Hi Marek, > > [...] > > > So because noone actually bothered to compile u-boot with DEBUG enabled, > > we (me ?) now have literally piles of places to fix. Hm :-( > > So it seems. This is a very good reminder of the fact that with ifdef > we actually hide source code from the compiler under certain > circumstances. The number of these "certain circumstances" produces the > number of _differing input_ to our compiler. Given the trivial fact > that a simple ifdef results in two different inputs one needs only to > visualize the graph of 2^x to get an idea of how many sources we are > actually using. > > Unrelated to your current efforts (that are greatly appreciated) We > should really try not to use #ifdef at all. Using constants together > with the dead code eliminatino of the compiler can usually reach the > same goal but with syntax checks on every compile. > > Sorry, but I couldn't resist lobbying for this. Perhaps it helps when > it gets repeated approximately every year ;) > > Cheers > Detlev Hey Detlev, I certainly see your point. Btw. do you feel like help reviewing the PPC portion of the patches? That'd be greatly appreciated as I'm not a PPC expert. I wanted to avoid resubmission, so I only pushed the new set to git://git.denx.de/u-boot- marex.git / debug branch. There is over 50 patches now. Cheers