[U-Boot] [PATCH] config.mk:Update DBGFLAGS with dwarf information

Built u-boot elf file does not contain any dwarf informations by default. This information is required for debugging.
Add dwarf ver 2 flag in DBGFLAGS.
Signed-off-by: Radu Lazarescu radu.lazarescu@freescale.com Signed-off-by: Prabhakar Kushwaha prabhakar@freescale.com --- Applies on http://git.denx.de/u-boot.git branch master
config.mk | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/config.mk b/config.mk index ddaa477..11bc20d 100644 --- a/config.mk +++ b/config.mk @@ -178,7 +178,7 @@ endif # who are porting old code to latest mainline but not updating $(AR). ARFLAGS = $(error update your Makefile to use cmd_link_o_target and not AR) RELFLAGS= $(PLATFORM_RELFLAGS) -DBGFLAGS= -g # -DDEBUG +DBGFLAGS= -g -gdwarf-2 # -DDEBUG OPTFLAGS= -Os #-fomit-frame-pointer
OBJCFLAGS += --gap-fill=0xff

Hi,
This patch has review-pending from a very long time.
I request the "mailing-list" to review this patch.
Regards, Prabhakar
On Wednesday 15 February 2012 02:25 PM, Prabhakar Kushwaha wrote:
Built u-boot elf file does not contain any dwarf informations by default. This information is required for debugging.
Add dwarf ver 2 flag in DBGFLAGS.
Signed-off-by: Radu Lazarescuradu.lazarescu@freescale.com Signed-off-by: Prabhakar Kushwahaprabhakar@freescale.com
Applies on http://git.denx.de/u-boot.git branch master
config.mk | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/config.mk b/config.mk index ddaa477..11bc20d 100644 --- a/config.mk +++ b/config.mk @@ -178,7 +178,7 @@ endif # who are porting old code to latest mainline but not updating $(AR). ARFLAGS = $(error update your Makefile to use cmd_link_o_target and not AR) RELFLAGS= $(PLATFORM_RELFLAGS) -DBGFLAGS= -g # -DDEBUG +DBGFLAGS= -g -gdwarf-2 # -DDEBUG OPTFLAGS= -Os #-fomit-frame-pointer
OBJCFLAGS += --gap-fill=0xff

Hi Wolfgang,
This patch has been pending for a very long time. No review comments has been provided.
Can you please review/accept this patch.
Regards, Prabhakar
On 04/25/2012 09:41 AM, Prabhakar Kushwaha wrote:
Hi,
This patch has review-pending from a very long time.
I request the "mailing-list" to review this patch.
Regards, Prabhakar
On Wednesday 15 February 2012 02:25 PM, Prabhakar Kushwaha wrote:
Built u-boot elf file does not contain any dwarf informations by default. This information is required for debugging.
Add dwarf ver 2 flag in DBGFLAGS.
Signed-off-by: Radu Lazarescuradu.lazarescu@freescale.com Signed-off-by: Prabhakar Kushwahaprabhakar@freescale.com
Applies on http://git.denx.de/u-boot.git branch master
config.mk | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/config.mk b/config.mk index ddaa477..11bc20d 100644 --- a/config.mk +++ b/config.mk @@ -178,7 +178,7 @@ endif # who are porting old code to latest mainline but not updating $(AR). ARFLAGS = $(error update your Makefile to use cmd_link_o_target and not AR) RELFLAGS= $(PLATFORM_RELFLAGS) -DBGFLAGS= -g # -DDEBUG +DBGFLAGS= -g -gdwarf-2 # -DDEBUG OPTFLAGS= -Os #-fomit-frame-pointer
OBJCFLAGS += --gap-fill=0xff
U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Hi Prabhakar,
On Thu, May 24, 2012 at 2:43 PM, Prabhakar Kushwaha prabhakar@freescale.com wrote:
Hi Wolfgang,
This patch has been pending for a very long time. No review comments has been provided.
Mike Frysinger posted a comment on May 14, 2012:
[quote] no real information here as to what is going wrong. the -g flag provides debug information and many people (myself included) have found this sufficient.
please describe your actual problem in detail and how this actually fixes things for you. -mike
[/quote]
Regards,
Graeme

On Wednesday 15 February 2012 03:55:31 Prabhakar Kushwaha wrote:
Built u-boot elf file does not contain any dwarf informations by default. This information is required for debugging.
no real information here as to what is going wrong. the -g flag provides debug information and many people (myself included) have found this sufficient.
please describe your actual problem in detail and how this actually fixes things for you. -mike

On Tue, Feb 14, 2012 at 10:55:31PM -0000, Prabhakar Kushwaha wrote:
Built u-boot elf file does not contain any dwarf informations by default. This information is required for debugging.
Add dwarf ver 2 flag in DBGFLAGS.
Signed-off-by: Radu Lazarescu radu.lazarescu@freescale.com Signed-off-by: Prabhakar Kushwaha prabhakar@freescale.com
For posterity, with any recent enough toolchain, this shouldn't be needed. -g picks the best available debug format and that will be DWARF v2 symbols (which gdb happily uses).
participants (4)
-
Graeme Russ
-
Mike Frysinger
-
Prabhakar Kushwaha
-
Tom Rini