[PATCH V2] ARM: plt_size functions need to read instructions in right byte order

[Victor Kamensky]

On 22 October 2014 18:12, Alan Modra <amodra@gmail.com> wrote:
> On Wed, Oct 22, 2014 at 06:01:12PM -0700, Victor Kamensky wrote:
>> On 22 October 2014 15:50, Alan Modra <amodra@gmail.com> wrote:
>> > On Tue, Oct 21, 2014 at 10:25:13PM -0700, Victor Kamensky wrote:
>> >> +  if ((elf_elfheader (abfd)->e_flags) & EF_ARM_BE8)
>> >                                        ^ this parenthesis is in the
>> > wrong place.  gcc will warn about "if (x & y)", breaking -Werror
>> > builds.  Writing "if ((x & y))" silences the gcc warning, so you
>> > should write
>> >
>> >   if ((elf_elfheader (abfd)->e_flags & EF_ARM_BE8))
>> >
>> > or as I suggested in the previous email (without explaining why)
>> >
>> >   if ((elf_elfheader (abfd)->e_flags & EF_ARM_BE8) != 0)
>>
>> Thanks, Alan.
>>
>> Sorry, did not catch that in your previous email I will fix that
>> and repost patch shortly.
>>
>> > The patch is OK to commit with that change.
>>
>> Besides reposting updated patch is there any other actions
>> on my side? I am bit a new to this. My assumption is that once
>> patch is reviewed and approved, some of maintainers will pick it
>> and commit. Is it not correct?
>
> Ah, I thought you had git commit privilege.  Don't worry about
> reposting the patch.  I'll commit a fixed version for you.

Emails crossing. It is already out. Thank you. I appreciate your
time and help.

Thanks,
Victor

> --
> Alan Modra
> Australia Development Lab, IBM