-----Original Message----- From: Wolfgang Denk [mailto:wd@denx.de] Sent: Monday, April 25, 2011 7:17 AM To: Zang Roy-R61911 Cc: u-boot@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@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@freescale.com Cc: Wood Scott-B07421 B07421@freescale.com Cc: Kumar Gala galak@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