From: Andreas Jaeger <aj@suse.com>
To: David Holsgrove <david.holsgrove@xilinx.com>
Cc: libc-alpha@sourceware.org, libc-ports@sourceware.org,
John Williams <john.williams@xilinx.com>,
"Edgar E. Iglesias" <edgar.iglesias@gmail.com>,
Vinod Kathail <vinod.kathail@xilinx.com>,
Tom Shui <tom.shui@xilinx.com>,
Vidhumouli Hunsigida <vidhumouli.hunsigida@xilinx.com>,
Nagaraju Mekala <nagaraju.mekala@xilinx.com>
Subject: Re: [PATCH 2/3 elf] Add MicroBlaze support to elf.h
Date: Thu, 29 Nov 2012 07:16:00 -0000 [thread overview]
Message-ID: <50B70BC6.5020502@suse.com> (raw)
In-Reply-To: <CAM=EW8Y-tqvbZwtewup6UagcgJiHEtDmapEmoiR7ZW0SW_-pCg@mail.gmail.com>
On 11/29/2012 06:28 AM, David Holsgrove wrote:
> d MicroBlaze relocations to elf/elf.h
>
> 2012-11-29 David Holsgrove<david.holsgrove@xilinx.com>
>
> * elf/elf.h: Add support for MicroBlaze arch
>
>
> 0002-Adding-MicroBlaze-support-to-elf-elf.h.patch
>
David, thanks for your patch. Could you please review the Contribution
checklist at http://sourceware.org/glibc/wiki/Contribution%20checklist?
Your patch misses a valid ChangeLog entry and that's needed for such
changes.
> From 02a0759b648b6aed4538cd0cde88babf5468585a Mon Sep 17 00:00:00 2001
> From: David Holsgrove<david.holsgrove@petalogix.com>
> Date: Wed, 4 Jan 2012 13:56:48 +1000
> Subject: [PATCH] Adding MicroBlaze support to elf/elf.h
>
> Signed-off-by: David Holsgrove<david.holsgrove@petalogix.com>
> ---
> elf/elf.h | 33 +++++++++++++++++++++++++++++++++
> 1 file changed, 33 insertions(+)
>
> diff --git a/elf/elf.h b/elf/elf.h
> index b07e6ad..6a4a4cc 100644
> --- a/elf/elf.h
> +++ b/elf/elf.h
> @@ -259,6 +259,8 @@ typedef struct
> chances of collision with official or non-GNU unofficial values. */
>
> #define EM_ALPHA 0x9026
> +#define EM_NEW_MICROBLAZE 0xbd /* Xilinx MicroBlaze */
> +#define EM_MICROBLAZE 0xbaab
Are any of these official values? The comment above asks for hig numbers
and 0xbd is not height.
>
> /* Legal values for e_version (version). */
>
> @@ -2847,6 +2849,37 @@ typedef Elf32_Addr Elf32_Conflict;
> #define R_M32R_GOTOFF_LO 64 /* Low 16 bit offset to GOT */
> #define R_M32R_NUM 256 /* Keep this the last entry. */
>
> +/* microblaze relocations */
> +#define R_MICROBLAZE_NONE 0
I suggest to be consistent with the writing of the name - so is it
microblaze or MicroBlaze?
> +#define R_MICROBLAZE_32 1
> +#define R_MICROBLAZE_32_PCREL 2
> +#define R_MICROBLAZE_64_PCREL 3
> +#define R_MICROBLAZE_32_PCREL_LO 4
> +#define R_MICROBLAZE_64 5
> +#define R_MICROBLAZE_32_LO 6
> +#define R_MICROBLAZE_SRO32 7
> +#define R_MICROBLAZE_SRW32 8
> [...]
Please add comments for all of them,
Thanks,
Andreas
--
Andreas Jaeger aj@{suse.com,opensuse.org} Twitter/Identica: jaegerandi
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn,Jennifer Guild,Felix Imendörffer,HRB16746 (AG Nürnberg)
GPG fingerprint = 93A3 365E CE47 B889 DF7F FED1 389A 563C C272 A126
next prev parent reply other threads:[~2012-11-29 7:16 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-11-29 5:29 David Holsgrove
2012-11-29 6:04 ` [PATCH] Adding MicroBlaze support to elf/elf.h David Holsgrove
2012-11-29 7:16 ` Andreas Jaeger [this message]
2012-11-29 7:58 ` [PATCH 2/3 elf] Add MicroBlaze support to elf.h Mike Frysinger
2012-11-29 8:04 ` Andreas Jaeger
2012-11-29 8:23 ` David Holsgrove
2012-11-29 16:06 ` Joseph S. Myers
2012-11-29 16:11 ` Carlos O'Donell
2012-11-29 16:20 ` Joseph S. Myers
2012-11-29 8:03 ` David Holsgrove
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=50B70BC6.5020502@suse.com \
--to=aj@suse.com \
--cc=david.holsgrove@xilinx.com \
--cc=edgar.iglesias@gmail.com \
--cc=john.williams@xilinx.com \
--cc=libc-alpha@sourceware.org \
--cc=libc-ports@sourceware.org \
--cc=nagaraju.mekala@xilinx.com \
--cc=tom.shui@xilinx.com \
--cc=vidhumouli.hunsigida@xilinx.com \
--cc=vinod.kathail@xilinx.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).