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.
Best regards,
Wolfgang Denk