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

From: Prabhakar Kushwaha <prabhakar@freescale.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] config.mk:Update DBGFLAGS with dwarf information
Date: Mon, 25 Jun 2012 10:27:05 +0530	[thread overview]
Message-ID: <4FE7EFA1.2050208@freescale.com> (raw)
In-Reply-To: <4FE00969.5070405@freescale.com>

Hi Mike/Graeme.

Please help me in closing review related comments on this patch.

Thanks,
Prabhakar

On 06/19/2012 10:38 AM, Prabhakar Kushwaha wrote:
>
> Hi,
>
> Sorry for the delayed response. I somehow missed this  :(
>
> On 06/19/2012 10:29 AM, Prabhakar Kushwaha wrote:
>> 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
>>
>>
>
> I agree with Mike's reply i.e. -g flag provides sufficient debug 
> information.
> But some debugger requires debug information in specific format i.e. 
> -gdwarf-2
> For example Freescale CodeWarrior Debugger
>
>
> 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 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
>>>>
>>>>
>>>>
>>>> _______________________________________________
>>>> U-Boot mailing list
>>>> U-Boot at lists.denx.de
>>>> http://lists.denx.de/mailman/listinfo/u-boot
>>>>
>>>
>>>
>>>
>>> _______________________________________________
>>> U-Boot mailing list
>>> U-Boot at lists.denx.de
>>> http://lists.denx.de/mailman/listinfo/u-boot
>>>
>>
>>
>
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>