[U-Boot] [PATCH] powerpc:Enable compiler debug option for powerpc platforms

[Zang Roy-R61911]

> -----Original Message-----
> From: Wolfgang Denk [mailto:wd at denx.de]
> Sent: Monday, April 25, 2011 7:17 AM
> To: Zang Roy-R61911
> Cc: u-boot at lists.denx.de; Wood Scott-B07421; Kumar Gala
> Subject: Re: [PATCH] powerpc:Enable compiler debug option for powerpc
> platforms
> 
> Dear Roy Zang,
> 
> In message <1300161222-19050-1-git-send-email-tie-fei.zang at freescale.com> you
> wrote:
> > This patch enables compiler debug option for powerpc platforms
> > by default.  Patch is tested on 85xx/QorIQ platforms in u-boot
> > with CodeWarrior.
> >
> > Signed-off-by: Roy Zang <tie-fei.zang at freescale.com>
> > Cc: Wood Scott-B07421 <B07421 at freescale.com>
> > Cc: Kumar Gala <galak at kernel.crashing.org>
> > ---
> > Consider the comments
> > http://lists.denx.de/pipermail/u-boot/2010-October/080689.html
> > http://lists.denx.de/pipermail/u-boot/2011-January/086485.html
> 
> Indeed.  In
> http://lists.denx.de/pipermail/u-boot/2011-January/086485.html
> I wrote:
> 
> | I don't see where this is specific to the Power architecture either -
> | expect that you only tested it there.
> |
> | My understanding is that you enable debug settings for the GNU
> | debugger.
> |
> | Do these need to be architecture specific? Probably not.
> |
> | Do these need to be enabled by a new #define?  What would be the
> | disadvantages of having these options always enabled, for all
> | architectures?
> 
> You did not comment to these questions, and your current code is
> still powerpc-centric.
Yes. because there is other voice:
http://lists.denx.de/pipermail/u-boot/2011-January/086489.html
http://lists.denx.de/pipermail/u-boot/2011-January/086512.html
any  comment?
Roy