Re: [PATCH 06/12] Add support for microblaze little-endian

[David Holsgrove <david dot holsgrove at xilinx dot com>]

Hi Mike, All,

On 11 October 2012 04:00, Mike Frysinger <vapier@gentoo.org> wrote:
> On Tuesday 09 October 2012 22:13:59 david.holsgrove@xilinx.com wrote:
>> --- a/ChangeLog
>> +++ b/ChangeLog
>> @@ -1,3 +1,7 @@
>> +2012-10-10  Edgar E. Iglesias  <edgar.iglesias@gmail.com>
>> +
>> +     * config.sub: Add microblaze little endian support
>
> changes to this file have to be sent to config-patches@gnu.org first, then you
> copy back the entire file to the gcc and binutils repo.
>

Thanks very much for the comments thus far. The config.sub update was
submitted and included by config-patches yesterday, so I can amend
this patch to exclude config.sub and submit the new upstream version
in a separate patch to binutils and gcc.

In copying the upstream config.sub back into binutils, I noticed that
it will include more than just my change to add microblazeel, as the
config.sub in binutils has not been updated since 2012-04-18. Is this
expected?

>> --- a/bfd/ChangeLog
>> +++ b/bfd/ChangeLog
>> @@ -1,5 +1,11 @@
>>  2012-10-10  Edgar E. Iglesias  <edgar.iglesias@gmail.com>
>>
>> +     * elf32-microblaze.c: Add microblaze little endian support
>> +     * target.c: Likewise
>> +     * config.bfd, configure, configure.in: Likewise
>
> configure is a generated file, so the entry should say:
>         * configure: Regenerated.
> -mike

And I've updated the ChangeLog accordingly.

thanks,
David