Re: [PATCH] ARM_DEBUGOVERLAY and ARM_OVERLAYSECTION for readelf

[Daniel Gutson <dgutson at codesourcery dot com>]

Hi Nick,

Nick Clifton wrote:
> Hi Daniel,
>
> >     binutils/
> >     * readelf.c (get_arm_section_type_name): Added support for
> >     new sections headers.
> >
> >     include/elf
> >     * arm.h: (SHT_ARM_DEBUGOVERLAY): New define.
> >     (SHT_ARM_OVERLAYSECTION): New define.
>
> Approved - please apply - but...
>
> ...Did you mean to include the definitions of SHT_ARM_EXIDX, PREEMPTMAP 
> and ATTRIBUTES twice in include/elf/arm.h: ?

No, I didn't. Sorry about that.

Please note that I don't have write access, so I'd need somebody else to 
commit this for me.

Do you want me to re-send the patch with the definitions once?

Thanks!
	Daniel.

>  /* Additional section types.  */
> -#define SHT_ARM_EXIDX       0x70000001    /* Section holds ARM unwind 
> info.  */
> -#define SHT_ARM_PREEMPTMAP 0x70000002    /* Section pre-emption 
> details.  */
> -#define SHT_ARM_ATTRIBUTES 0x70000003    /* Section holds attributes.  */
> +#define SHT_ARM_EXIDX           0x70000001    /* Section holds ARM 
> unwind info.  */
> +#define SHT_ARM_PREEMPTMAP     0x70000002    /* Section pre-emption 
> details.  */
> +#define SHT_ARM_ATTRIBUTES     0x70000003    /* Section holds 
> attributes.  */
> +#define SHT_ARM_EXIDX          0x70000001    /* Section holds ARM 
> unwind info.  */
> +#define SHT_ARM_PREEMPTMAP     0x70000002    /* Section pre-emption 
> details.  */
> +#define SHT_ARM_ATTRIBUTES     0x70000003    /* Section holds 
> attributes.  */
> +#define SHT_ARM_DEBUGOVERLAY   0x70000004    /* Section holds overlay 
> debug info.  */
> +#define SHT_ARM_OVERLAYSECTION 0x70000005    /* Section holds GDB and 
> overlay integration info.  */
>
> Cheers
>   Nick

--
Daniel Gutson
CodeSourcery
www.codesourcery.com