Hi Marek,
On Fri, Mar 30, 2012 at 6:52 AM, Marek Vasut <ma...@denx.de> wrote:
> Dear Simon Glass,
>
>> +Marek
>>
>> Hi Albert,
>>
>> On Fri, Mar 30, 2012 at 12:08 AM, Albert ARIBAUD
>>
>> <albert.u.b...@aribaud.net> wrote:
>> > Le 27/03/2012 22:18, Albert ARIBAUD a écrit :
>> >> Hi all,
>> >>
>> >>
>> >> I have been away from the list quite a few days, and will have a very
>> >> tricky time catching up, so with my apologies, I ask people who need me
>> >> to apply patches or pull requests for the upcoming release to please
>> >> send me a private e-mail and point me to the list message that I should
>> >> be acting upon.
>> >
>> > FWIW, ./MAKEALL arm on current u-boot-arm/master with CS toolchain 2010q1
>> > builds 277 boards, of which none has errors and 167 have warnings.
>> >
>> > The total count of warnings is 248, distributed as:
>> >
>> > 4 post.c:503: warning: #warning "Not implemented yet"
>> >
>> > 79 mmc.c:132: warning: no return statement in function returning
>> > non-void
>> >
>> > 165 ohci-hcd.c:{1307,1310,1313}: warning: dereferencing pointer
>> > 'pretmp.NNN' does break strict-aliasing rules
>>
>> I did create a patch for this - it is marked delegated to Marek.
>>
>> http://patchwork.ozlabs.org/patch/134600/
>
> And stupid maintainer didn't look into it ... bad, bad maintainer. Accept my
> apology. But wasn't it decided with Mike that this was indeed a toolchain
> issue
> and not all toolchains choke on it?
:-)
Yes. It does feel like a broken toolchain (I felt that the original
code should be warning free, even when I wrote my patch) but it seems
that Albert gets this warning as well. Maybe we should fix it?
>
>>
>> > (the last one is actually 3 * 51 warnings)
>> >
>> > The first one is obvious and should be dealt with, but I consider it not
>> > to be a blocker, for only 4 boards.
>> >
>> > The second one seems straightforward to fix today, if not already
>> > submitted.
>> >
>> > The third one may be an issue, but it could also be a zealous warning
>> > from the specific CS toolchain version I am using. Anyway, it does not
>> > seem ARM related as such.
>>
>> Would be nice to get rid of it, but I think it is specific to some
>> toolchains.
>>
>> > Therefore I will issue the ARM pull request right away.
>> >
>> >
>> > Amicalement,
>> > --
>> > Albert.
>> > _______________________________________________
>> > U-Boot mailing list
>> > U-Boot@lists.denx.de
>> > http://lists.denx.de/mailman/listinfo/u-boot
>>
>> Regards,
>> Simon
>
> Best regards,
> Marek Vasut
Regards,
Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
