* [U-Boot] [ARM] Status for 2012.03 @ 2012-03-27 20:18 Albert ARIBAUD 0 siblings, 0 replies; 10+ messages in thread From: Albert ARIBAUD @ 2012-03-27 20:18 UTC (permalink / raw) To: u-boot Hi all, I have been away from the list quite a few days, and will have a very trycky 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. Amicalement, -- Albert. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [ARM] Status for 2012.03 @ 2012-03-27 20:18 Albert ARIBAUD 2012-03-30 6:23 ` Albert ARIBAUD 2012-03-30 7:08 ` Albert ARIBAUD 0 siblings, 2 replies; 10+ messages in thread From: Albert ARIBAUD @ 2012-03-27 20:18 UTC (permalink / raw) To: u-boot 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. Amicalement, -- Albert. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [ARM] Status for 2012.03 2012-03-27 20:18 Albert ARIBAUD @ 2012-03-30 6:23 ` Albert ARIBAUD 2012-03-30 7:08 ` Albert ARIBAUD 1 sibling, 0 replies; 10+ messages in thread From: Albert ARIBAUD @ 2012-03-30 6:23 UTC (permalink / raw) To: u-boot 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. All, I should have fulfilled all pending pull requests and patches. I will be sending out the ARM pull request shortly; let me know if anything is wrong with it. Amicalement, -- Albert. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [ARM] Status for 2012.03 2012-03-27 20:18 Albert ARIBAUD 2012-03-30 6:23 ` Albert ARIBAUD @ 2012-03-30 7:08 ` Albert ARIBAUD 2012-03-30 13:16 ` Simon Glass 2012-03-30 13:31 ` Valentin Longchamp 1 sibling, 2 replies; 10+ messages in thread From: Albert ARIBAUD @ 2012-03-30 7:08 UTC (permalink / raw) To: u-boot 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 (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. Therefore I will issue the ARM pull request right away. Amicalement, -- Albert. ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [ARM] Status for 2012.03 2012-03-30 7:08 ` Albert ARIBAUD @ 2012-03-30 13:16 ` Simon Glass 2012-03-30 13:52 ` Marek Vasut 2012-03-30 13:31 ` Valentin Longchamp 1 sibling, 1 reply; 10+ messages in thread From: Simon Glass @ 2012-03-30 13:16 UTC (permalink / raw) To: u-boot +Marek Hi Albert, On Fri, Mar 30, 2012 at 12:08 AM, Albert ARIBAUD <albert.u.boot@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/ > > (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 at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot Regards, Simon ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [ARM] Status for 2012.03 2012-03-30 13:16 ` Simon Glass @ 2012-03-30 13:52 ` Marek Vasut 2012-03-30 13:56 ` Simon Glass 0 siblings, 1 reply; 10+ messages in thread From: Marek Vasut @ 2012-03-30 13:52 UTC (permalink / raw) To: u-boot Dear Simon Glass, > +Marek > > Hi Albert, > > On Fri, Mar 30, 2012 at 12:08 AM, Albert ARIBAUD > > <albert.u.boot@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? > > > (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 at lists.denx.de > > http://lists.denx.de/mailman/listinfo/u-boot > > Regards, > Simon Best regards, Marek Vasut ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [ARM] Status for 2012.03 2012-03-30 13:52 ` Marek Vasut @ 2012-03-30 13:56 ` Simon Glass 2012-03-30 14:29 ` Marek Vasut 0 siblings, 1 reply; 10+ messages in thread From: Simon Glass @ 2012-03-30 13:56 UTC (permalink / raw) To: u-boot Hi Marek, On Fri, Mar 30, 2012 at 6:52 AM, Marek Vasut <marex@denx.de> wrote: > Dear Simon Glass, > >> +Marek >> >> Hi Albert, >> >> On Fri, Mar 30, 2012 at 12:08 AM, Albert ARIBAUD >> >> <albert.u.boot@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 at lists.denx.de >> > http://lists.denx.de/mailman/listinfo/u-boot >> >> Regards, >> Simon > > Best regards, > Marek Vasut Regards, Simon ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [ARM] Status for 2012.03 2012-03-30 13:56 ` Simon Glass @ 2012-03-30 14:29 ` Marek Vasut 2012-03-31 8:15 ` Simon Glass 0 siblings, 1 reply; 10+ messages in thread From: Marek Vasut @ 2012-03-30 14:29 UTC (permalink / raw) To: u-boot Dear Simon Glass, > Hi Marek, > > On Fri, Mar 30, 2012 at 6:52 AM, Marek Vasut <marex@denx.de> wrote: > > Dear Simon Glass, > > > >> +Marek > >> > >> Hi Albert, > >> > >> On Fri, Mar 30, 2012 at 12:08 AM, Albert ARIBAUD > >> > >> <albert.u.boot@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? I don't see this with ELDK 4.2 or 5.0 or 5.1 ;-) Maybe fundamental rewrite of that code might help instead of adding bandaids ;-) > >> > (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 at lists.denx.de > >> > http://lists.denx.de/mailman/listinfo/u-boot > >> > >> Regards, > >> Simon > > > > Best regards, > > Marek Vasut > > Regards, > Simon ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [ARM] Status for 2012.03 2012-03-30 14:29 ` Marek Vasut @ 2012-03-31 8:15 ` Simon Glass 0 siblings, 0 replies; 10+ messages in thread From: Simon Glass @ 2012-03-31 8:15 UTC (permalink / raw) To: u-boot Hi Marek, On Fri, Mar 30, 2012 at 7:29 AM, Marek Vasut <marex@denx.de> wrote: > Dear Simon Glass, > >> Hi Marek, >> >> On Fri, Mar 30, 2012 at 6:52 AM, Marek Vasut <marex@denx.de> wrote: >> > Dear Simon Glass, >> > >> >> +Marek >> >> >> >> Hi Albert, >> >> >> >> On Fri, Mar 30, 2012 at 12:08 AM, Albert ARIBAUD >> >> >> >> <albert.u.boot@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? > > I don't see this with ELDK 4.2 or 5.0 or 5.1 ;-) Nope. It only happens with some toolchains. That said, it blights my MAKEALL at present. > > Maybe fundamental rewrite of that code might help instead of adding bandaids ;-) Yes I'm sure, or even getting rid of the 3 separate versions of this file in U-Boot. Hope springs eternal. Regards, Simon > >> >> > (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 at lists.denx.de >> >> > http://lists.denx.de/mailman/listinfo/u-boot >> >> >> >> Regards, >> >> Simon >> > >> > Best regards, >> > Marek Vasut >> >> Regards, >> Simon ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [ARM] Status for 2012.03 2012-03-30 7:08 ` Albert ARIBAUD 2012-03-30 13:16 ` Simon Glass @ 2012-03-30 13:31 ` Valentin Longchamp 1 sibling, 0 replies; 10+ messages in thread From: Valentin Longchamp @ 2012-03-30 13:31 UTC (permalink / raw) To: u-boot Hi Albert, On 03/30/2012 09:08 AM, Albert ARIBAUD 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" The 4 warnings come from our boards (Keymile ARM boards). > The first one is obvious and should be dealt with, but I consider it not to be a > blocker, for only 4 boards. > I have just sent a patch that corrects this warning. Regards Valentin ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2012-03-31 8:15 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-03-27 20:18 [U-Boot] [ARM] Status for 2012.03 Albert ARIBAUD -- strict thread matches above, loose matches on Subject: below -- 2012-03-27 20:18 Albert ARIBAUD 2012-03-30 6:23 ` Albert ARIBAUD 2012-03-30 7:08 ` Albert ARIBAUD 2012-03-30 13:16 ` Simon Glass 2012-03-30 13:52 ` Marek Vasut 2012-03-30 13:56 ` Simon Glass 2012-03-30 14:29 ` Marek Vasut 2012-03-31 8:15 ` Simon Glass 2012-03-30 13:31 ` Valentin Longchamp
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.