* [U-Boot] [STATUS] ARM
@ 2011-08-18 22:09 Albert ARIBAUD
2011-08-19 9:38 ` Aneesh V
` (4 more replies)
0 siblings, 5 replies; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-18 22:09 UTC (permalink / raw)
To: u-boot
Hi all,
This week-end I intend to go through the board removal patch set and
apply those patches in it that still fail to compile with the current
u-boot-arm/master tree. So:
1) if there are any pull requests to u-voot-arm/master in the works,
please try to submit them soon or at lease let me know, and
2) if there are any board fix patches posted after merge window closure
and still pending, please also let me know (those before the merge
window closure should all be in u-boot-arm by now).
ATM, a './MAKEALL arm' on u-boot-arm/master using ELDK 4.2 on a 64-bit
Natty Ubuntu yields 200 boards built, with 71 boards having warnings or
errors.
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-18 22:09 [U-Boot] [STATUS] ARM Albert ARIBAUD
@ 2011-08-19 9:38 ` Aneesh V
2011-08-19 18:19 ` Dirk Behme
2011-08-20 11:16 ` Marek Vasut
` (3 subsequent siblings)
4 siblings, 1 reply; 35+ messages in thread
From: Aneesh V @ 2011-08-19 9:38 UTC (permalink / raw)
To: u-boot
Hi Albert,
On Friday 19 August 2011 03:39 AM, Albert ARIBAUD wrote:
> Hi all,
>
> This week-end I intend to go through the board removal patch set and
> apply those patches in it that still fail to compile with the current
> u-boot-arm/master tree. So:
>
> 1) if there are any pull requests to u-voot-arm/master in the works,
> please try to submit them soon or at lease let me know, and
>
> 2) if there are any board fix patches posted after merge window closure
> and still pending, please also let me know (those before the merge
> window closure should all be in u-boot-arm by now).
I have these patches that fix couple of build issues with OMAP
platforms. I am hoping that Sandeep will pull these soon.
1. http://patchwork.ozlabs.org/patch/107678/
2. http://patchwork.ozlabs.org/patch/108865/
best regards,
Aneesh
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-19 9:38 ` Aneesh V
@ 2011-08-19 18:19 ` Dirk Behme
2011-08-20 11:12 ` Albert ARIBAUD
0 siblings, 1 reply; 35+ messages in thread
From: Dirk Behme @ 2011-08-19 18:19 UTC (permalink / raw)
To: u-boot
On 19.08.2011 11:38, Aneesh V wrote:
> Hi Albert,
>
> On Friday 19 August 2011 03:39 AM, Albert ARIBAUD wrote:
>> Hi all,
>>
>> This week-end I intend to go through the board removal patch set and
>> apply those patches in it that still fail to compile with the current
>> u-boot-arm/master tree. So:
>>
>> 1) if there are any pull requests to u-voot-arm/master in the works,
>> please try to submit them soon or at lease let me know, and
Please don't miss
http://lists.denx.de/pipermail/u-boot/2011-August/098795.html
>> 2) if there are any board fix patches posted after merge window closure
>> and still pending, please also let me know (those before the merge
>> window closure should all be in u-boot-arm by now).
>
> I have these patches that fix couple of build issues with OMAP
> platforms. I am hoping that Sandeep will pull these soon.
>
> 1. http://patchwork.ozlabs.org/patch/107678/
> 2. http://patchwork.ozlabs.org/patch/108865/
Yes, please don't miss these two patches.
Additionally,
http://patchwork.ozlabs.org/patch/109264/
is needed to fix an issue introduced with the patch "mkimage: Add OMAP
boot image support".
IMHO with these patches all boards in ./MAKEALL ARMV7 should build
fine, without any warnings. At least the last time I tried it some
days ago ;) So, if you find additional (new?) issues or if you are
about removing any _ARMv7_ board, please let us know before that we
could check.
Many thanks and best regards
Dirk
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-19 18:19 ` Dirk Behme
@ 2011-08-20 11:12 ` Albert ARIBAUD
2011-08-20 15:32 ` Albert ARIBAUD
2011-08-22 8:15 ` Simon Schwarz
0 siblings, 2 replies; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-20 11:12 UTC (permalink / raw)
To: u-boot
Aneesh, Dirk,
Le 19/08/2011 20:19, Dirk Behme a ?crit :
> On 19.08.2011 11:38, Aneesh V wrote:
>> Hi Albert,
>>
>> On Friday 19 August 2011 03:39 AM, Albert ARIBAUD wrote:
>>> Hi all,
>>>
>>> This week-end I intend to go through the board removal patch set and
>>> apply those patches in it that still fail to compile with the current
>>> u-boot-arm/master tree. So:
>>>
>>> 1) if there are any pull requests to u-voot-arm/master in the works,
>>> please try to submit them soon or at lease let me know, and
>
> Please don't miss
>
> http://lists.denx.de/pipermail/u-boot/2011-August/098795.html
>
>>> 2) if there are any board fix patches posted after merge window closure
>>> and still pending, please also let me know (those before the merge
>>> window closure should all be in u-boot-arm by now).
>>
>> I have these patches that fix couple of build issues with OMAP
>> platforms. I am hoping that Sandeep will pull these soon.
>>
>> 1. http://patchwork.ozlabs.org/patch/107678/
>> 2. http://patchwork.ozlabs.org/patch/108865/
Sandeep has requested that I pick these two in u-boot-arm.
> Yes, please don't miss these two patches.
>
> Additionally,
>
> http://patchwork.ozlabs.org/patch/109264/
>
> is needed to fix an issue introduced with the patch "mkimage: Add OMAP
> boot image support".
I'll extend Sandeep's agreement to this one too.
> IMHO with these patches all boards in ./MAKEALL ARMV7 should build fine,
> without any warnings. At least the last time I tried it some days ago ;)
> So, if you find additional (new?) issues or if you are about removing
> any _ARMv7_ board, please let us know before that we could check.
I'll apply the three patches above in the coming hours.
> Many thanks and best regards
>
> Dirk
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-18 22:09 [U-Boot] [STATUS] ARM Albert ARIBAUD
2011-08-19 9:38 ` Aneesh V
@ 2011-08-20 11:16 ` Marek Vasut
2011-08-20 11:44 ` Albert ARIBAUD
2011-08-20 18:50 ` [U-Boot] [STATUS] ARM (Tegra) Mike Frysinger
` (2 subsequent siblings)
4 siblings, 1 reply; 35+ messages in thread
From: Marek Vasut @ 2011-08-20 11:16 UTC (permalink / raw)
To: u-boot
On Friday, August 19, 2011 12:09:56 AM Albert ARIBAUD wrote:
> Hi all,
>
> This week-end I intend to go through the board removal patch set and
> apply those patches in it that still fail to compile with the current
> u-boot-arm/master tree. So:
>
> 1) if there are any pull requests to u-voot-arm/master in the works,
> please try to submit them soon or at lease let me know, and
>
> 2) if there are any board fix patches posted after merge window closure
> and still pending, please also let me know (those before the merge
> window closure should all be in u-boot-arm by now).
>
> ATM, a './MAKEALL arm' on u-boot-arm/master using ELDK 4.2 on a 64-bit
> Natty Ubuntu yields 200 boards built, with 71 boards having warnings or
> errors.
>
> Amicalement,
Hi Albert,
where can I find the board removal patchset?
Cheers
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-20 11:16 ` Marek Vasut
@ 2011-08-20 11:44 ` Albert ARIBAUD
2011-08-20 12:24 ` Marek Vasut
0 siblings, 1 reply; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-20 11:44 UTC (permalink / raw)
To: u-boot
Hi Marek,
Le 20/08/2011 13:16, Marek Vasut a ?crit :
> On Friday, August 19, 2011 12:09:56 AM Albert ARIBAUD wrote:
>> Hi all,
>>
>> This week-end I intend to go through the board removal patch set and
>> apply those patches in it that still fail to compile with the current
>> u-boot-arm/master tree. So:
>>
>> 1) if there are any pull requests to u-voot-arm/master in the works,
>> please try to submit them soon or at lease let me know, and
>>
>> 2) if there are any board fix patches posted after merge window closure
>> and still pending, please also let me know (those before the merge
>> window closure should all be in u-boot-arm by now).
>>
>> ATM, a './MAKEALL arm' on u-boot-arm/master using ELDK 4.2 on a 64-bit
>> Natty Ubuntu yields 200 boards built, with 71 boards having warnings or
>> errors.
>>
>> Amicalement,
>
> Hi Albert,
>
> where can I find the board removal patchset?
Look in patchwork (http://patchwork.ozlabs.org/project/uboot/list/),
filter with "ARM: remove broken". You many need to remove the "action
needed" filter to see them all, as some boards are already marked 'Not
Applicable' or 'Rejected' because they are now building fine.
> Cheers
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-20 11:44 ` Albert ARIBAUD
@ 2011-08-20 12:24 ` Marek Vasut
2011-08-20 12:54 ` Albert ARIBAUD
0 siblings, 1 reply; 35+ messages in thread
From: Marek Vasut @ 2011-08-20 12:24 UTC (permalink / raw)
To: u-boot
On Saturday, August 20, 2011 01:44:23 PM Albert ARIBAUD wrote:
> Hi Marek,
>
> Le 20/08/2011 13:16, Marek Vasut a ?crit :
> > On Friday, August 19, 2011 12:09:56 AM Albert ARIBAUD wrote:
> >> Hi all,
> >>
> >> This week-end I intend to go through the board removal patch set and
> >> apply those patches in it that still fail to compile with the current
> >> u-boot-arm/master tree. So:
> >>
> >> 1) if there are any pull requests to u-voot-arm/master in the works,
> >> please try to submit them soon or at lease let me know, and
> >>
> >> 2) if there are any board fix patches posted after merge window closure
> >> and still pending, please also let me know (those before the merge
> >> window closure should all be in u-boot-arm by now).
> >>
> >> ATM, a './MAKEALL arm' on u-boot-arm/master using ELDK 4.2 on a 64-bit
> >> Natty Ubuntu yields 200 boards built, with 71 boards having warnings or
> >> errors.
> >>
> >> Amicalement,
> >
> > Hi Albert,
> >
> > where can I find the board removal patchset?
>
> Look in patchwork (http://patchwork.ozlabs.org/project/uboot/list/),
> filter with "ARM: remove broken". You many need to remove the "action
> needed" filter to see them all, as some boards are already marked 'Not
> Applicable' or 'Rejected' because they are now building fine.
>
> > Cheers
>
> Amicalement,
Let me send you a few fixes for xscale crap. Do you prefer pull request or just a
bunch of emails ?
Cheers
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-20 12:24 ` Marek Vasut
@ 2011-08-20 12:54 ` Albert ARIBAUD
2011-08-20 13:33 ` Marek Vasut
0 siblings, 1 reply; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-20 12:54 UTC (permalink / raw)
To: u-boot
Hi Marek,
Le 20/08/2011 14:24, Marek Vasut a ?crit :
> On Saturday, August 20, 2011 01:44:23 PM Albert ARIBAUD wrote:
>> Hi Marek,
>>
>> Le 20/08/2011 13:16, Marek Vasut a ?crit :
>>> On Friday, August 19, 2011 12:09:56 AM Albert ARIBAUD wrote:
>>>> Hi all,
>>>>
>>>> This week-end I intend to go through the board removal patch set and
>>>> apply those patches in it that still fail to compile with the current
>>>> u-boot-arm/master tree. So:
>>>>
>>>> 1) if there are any pull requests to u-voot-arm/master in the works,
>>>> please try to submit them soon or at lease let me know, and
>>>>
>>>> 2) if there are any board fix patches posted after merge window closure
>>>> and still pending, please also let me know (those before the merge
>>>> window closure should all be in u-boot-arm by now).
>>>>
>>>> ATM, a './MAKEALL arm' on u-boot-arm/master using ELDK 4.2 on a 64-bit
>>>> Natty Ubuntu yields 200 boards built, with 71 boards having warnings or
>>>> errors.
>>>>
>>>> Amicalement,
>>>
>>> Hi Albert,
>>>
>>> where can I find the board removal patchset?
>>
>> Look in patchwork (http://patchwork.ozlabs.org/project/uboot/list/),
>> filter with "ARM: remove broken". You many need to remove the "action
>> needed" filter to see them all, as some boards are already marked 'Not
>> Applicable' or 'Rejected' because they are now building fine.
>>
>>> Cheers
>>
>> Amicalement,
>
> Let me send you a few fixes for xscale crap. Do you prefer pull request or just a
> bunch of emails ?
I prefer all PXA/Xscale patches to go through your repository and then a
pull request from u-boot-pxa/master to u-boot-arm/master be issued; but
in any case, if the patches have not been posted on the list so far,
please post them first, then pull them in your repo -- and cc: the
respective board maintainers.
> Cheers
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-20 12:54 ` Albert ARIBAUD
@ 2011-08-20 13:33 ` Marek Vasut
0 siblings, 0 replies; 35+ messages in thread
From: Marek Vasut @ 2011-08-20 13:33 UTC (permalink / raw)
To: u-boot
On Saturday, August 20, 2011 02:54:02 PM Albert ARIBAUD wrote:
> Hi Marek,
>
> Le 20/08/2011 14:24, Marek Vasut a ?crit :
> > On Saturday, August 20, 2011 01:44:23 PM Albert ARIBAUD wrote:
> >> Hi Marek,
> >>
> >> Le 20/08/2011 13:16, Marek Vasut a ?crit :
> >>> On Friday, August 19, 2011 12:09:56 AM Albert ARIBAUD wrote:
> >>>> Hi all,
> >>>>
> >>>> This week-end I intend to go through the board removal patch set and
> >>>> apply those patches in it that still fail to compile with the current
> >>>> u-boot-arm/master tree. So:
> >>>>
> >>>> 1) if there are any pull requests to u-voot-arm/master in the works,
> >>>> please try to submit them soon or at lease let me know, and
> >>>>
> >>>> 2) if there are any board fix patches posted after merge window
> >>>> closure and still pending, please also let me know (those before the
> >>>> merge window closure should all be in u-boot-arm by now).
> >>>>
> >>>> ATM, a './MAKEALL arm' on u-boot-arm/master using ELDK 4.2 on a 64-bit
> >>>> Natty Ubuntu yields 200 boards built, with 71 boards having warnings
> >>>> or errors.
> >>>>
> >>>> Amicalement,
> >>>
> >>> Hi Albert,
> >>>
> >>> where can I find the board removal patchset?
> >>
> >> Look in patchwork (http://patchwork.ozlabs.org/project/uboot/list/),
> >> filter with "ARM: remove broken". You many need to remove the "action
> >> needed" filter to see them all, as some boards are already marked 'Not
> >> Applicable' or 'Rejected' because they are now building fine.
> >>
> >>> Cheers
> >>
> >> Amicalement,
> >
> > Let me send you a few fixes for xscale crap. Do you prefer pull request
> > or just a bunch of emails ?
>
> I prefer all PXA/Xscale patches to go through your repository and then a
> pull request from u-boot-pxa/master to u-boot-arm/master be issued; but
> in any case, if the patches have not been posted on the list so far,
> please post them first, then pull them in your repo -- and cc: the
> respective board maintainers.
The board maintainers seem quite dead. There are two easy patches for PXA now,
please just pick them if they're fine (they're posted now and while I ran MAKEALL
on PXA, only zylonite is now FUBAR).
I think Wolfgang will kill Zylonite, but I'll probably (when time permits it)
merge new version back.
Thanks, Albert!
Cheers
>
> > Cheers
>
> Amicalement,
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-20 11:12 ` Albert ARIBAUD
@ 2011-08-20 15:32 ` Albert ARIBAUD
2011-08-21 7:11 ` Dirk Behme
` (2 more replies)
2011-08-22 8:15 ` Simon Schwarz
1 sibling, 3 replies; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-20 15:32 UTC (permalink / raw)
To: u-boot
Hi all,
Le 20/08/2011 13:12, Albert ARIBAUD a ?crit :
>>> 1. http://patchwork.ozlabs.org/patch/107678/
This one lowers bad board builds from 69 to 62, and has been applied.
>>> 2. http://patchwork.ozlabs.org/patch/108865/
This one kind of half-fixes the two boards it touches: they now build
but with warnings, all the same, of which here is one:
clocks.c:274: warning: comparison is always true due to limited range of
data type
I'll put this one on hold unless somebody can tell me what's going on.
>> http://patchwork.ozlabs.org/patch/109264/
This one lowers bad board builds from 62 to 58, and has been applied.
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM (Tegra)
2011-08-18 22:09 [U-Boot] [STATUS] ARM Albert ARIBAUD
2011-08-19 9:38 ` Aneesh V
2011-08-20 11:16 ` Marek Vasut
@ 2011-08-20 18:50 ` Mike Frysinger
2011-08-30 15:20 ` Mike Frysinger
2011-08-21 10:17 ` [U-Boot] [STATUS] ARM Dirk Behme
2011-08-30 15:58 ` [U-Boot] [STATUS] ARM status as of 2011-08-30 Albert ARIBAUD
4 siblings, 1 reply; 35+ messages in thread
From: Mike Frysinger @ 2011-08-20 18:50 UTC (permalink / raw)
To: u-boot
On Thursday, August 18, 2011 18:09:56 Albert ARIBAUD wrote:
> 1) if there are any pull requests to u-voot-arm/master in the works,
> please try to submit them soon or at lease let me know, and
we've got some Tegra stuff that seems to be waiting for merging (or at least
feedback as to anything left that needs to be done so it can be merged):
[RESEND PATCH v3 0/4] Add basic clock and pinmux functions to the Tegra2
to broach the more general tegra question, should these get sent to Tom and
then he sends them to Albert, or should they get sent directly to Albert
(since there is no dedicated tegra or nvidia u-boot tree atm) ? we'd like to
get this process smoothed out so we dont keep annoying people :).
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20110820/d89f4fa2/attachment.pgp
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-20 15:32 ` Albert ARIBAUD
@ 2011-08-21 7:11 ` Dirk Behme
2011-08-21 7:22 ` Albert ARIBAUD
2011-08-21 7:46 ` Dirk Behme
2011-08-21 7:51 ` V, Aneesh
2 siblings, 1 reply; 35+ messages in thread
From: Dirk Behme @ 2011-08-21 7:11 UTC (permalink / raw)
To: u-boot
On 20.08.2011 17:32, Albert ARIBAUD wrote:
> Hi all,
>
> Le 20/08/2011 13:12, Albert ARIBAUD a ?crit :
>
>>>> 1. http://patchwork.ozlabs.org/patch/107678/
>
> This one lowers bad board builds from 69 to 62, and has been applied.
>
>>>> 2. http://patchwork.ozlabs.org/patch/108865/
>
> This one kind of half-fixes the two boards it touches: they now build
> but with warnings, all the same, of which here is one:
>
> clocks.c:274: warning: comparison is always true due to limited range of
> data type
>
> I'll put this one on hold unless somebody can tell me what's going on.
Hmm, I haven't seen this yet. But would like to have a look to it.
But, hmm,
http://git.denx.de/?p=u-boot/u-boot-arm.git;a=summary
doesn't show the applied patches yet? Did I miss something?
Best regards
Dirk
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-21 7:11 ` Dirk Behme
@ 2011-08-21 7:22 ` Albert ARIBAUD
2011-08-23 2:02 ` V, Aneesh
0 siblings, 1 reply; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-21 7:22 UTC (permalink / raw)
To: u-boot
Hi Dirk,
Le 21/08/2011 09:11, Dirk Behme a ?crit :
> But, hmm,
>
> http://git.denx.de/?p=u-boot/u-boot-arm.git;a=summary
>
> doesn't show the applied patches yet? Did I miss something?
You did not; I did -- forgot to push master. Fixed now, thanks for
pointing this out.
> Best regards
>
> Dirk
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-20 15:32 ` Albert ARIBAUD
2011-08-21 7:11 ` Dirk Behme
@ 2011-08-21 7:46 ` Dirk Behme
2011-08-21 7:54 ` V, Aneesh
2011-08-21 7:51 ` V, Aneesh
2 siblings, 1 reply; 35+ messages in thread
From: Dirk Behme @ 2011-08-21 7:46 UTC (permalink / raw)
To: u-boot
On 20.08.2011 17:32, Albert ARIBAUD wrote:
> Hi all,
>
> Le 20/08/2011 13:12, Albert ARIBAUD a ?crit :
...
>>>> 2. http://patchwork.ozlabs.org/patch/108865/
>
> This one kind of half-fixes the two boards it touches: they now build
> but with warnings, all the same, of which here is one:
>
> clocks.c:274: warning: comparison is always true due to limited range of
> data type
>
> I'll put this one on hold unless somebody can tell me what's going on.
First, the patch 108865 above fixes a linking issue with the SPL for
these boards. So I think that this is independent of the clocks.c
warning you report above. That would mean it would be worth to apply
108865 to fix at least the linking issue.
Second, applying 108865 to the recent u-boot-arm.git doesn't show any
clocks.c warning for me [1]. Could this be tool chain dependent? I
tried it with [2] and [3], without any warning.
Btw, about which clocks.c (directory) do we talk?
Best regards
Dirk
[1]
> cat include/configs/omap4_sdp4430.h | grep CONFIG_SPL_MAX_SIZE
#define CONFIG_SPL_MAX_SIZE (38 * 1024)
> ./MAKEALL omap4_sdp4430
Configuring for omap4_sdp4430 board...
text data bss dec hex filename
191043 4468 202816 398327 613f7 ./u-boot
--------------------- SUMMARY ----------------------------
Boards compiled: 1
----------------------------------------------------------
> cat include/configs/omap4_panda.h | grep CONFIG_SPL_MAX_SIZE
#define CONFIG_SPL_MAX_SIZE (38 * 1024)
> ./MAKEALL omap4_panda
Configuring for omap4_panda board...
text data bss dec hex filename
189478 4380 202820 396678 60d86 ./u-boot
--------------------- SUMMARY ----------------------------
Boards compiled: 1
----------------------------------------------------------
[2] gcc version 4.3.3 (Sourcery G++ Lite 2009q1-203)
[3] gcc version 4.5.1 (Sourcery G++ Lite 2010.09-50)
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-20 15:32 ` Albert ARIBAUD
2011-08-21 7:11 ` Dirk Behme
2011-08-21 7:46 ` Dirk Behme
@ 2011-08-21 7:51 ` V, Aneesh
2011-08-21 8:14 ` Albert ARIBAUD
2 siblings, 1 reply; 35+ messages in thread
From: V, Aneesh @ 2011-08-21 7:51 UTC (permalink / raw)
To: u-boot
Hi Albert,
On Sat, Aug 20, 2011 at 9:02 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi all,
>
> Le 20/08/2011 13:12, Albert ARIBAUD a ?crit :
>
>>>> 1. http://patchwork.ozlabs.org/patch/107678/
>
> This one lowers bad board builds from 69 to 62, and has been applied.
>
>>>> 2. http://patchwork.ozlabs.org/patch/108865/
>
> This one kind of half-fixes the two boards it touches: they now build
> but with warnings, all the same, of which here is one:
>
> clocks.c:274: warning: comparison is always true due to limited range of
> data type
My tool-chain (Sourcery G++ Lite 2010q1 - GCC 4.4.1) never reported this issue.
But looks like there is indeed a problem. Hopefully the following patch should
solve it:
http://patchwork.ozlabs.org/patch/110811/
best regards,
Aneesh
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-21 7:46 ` Dirk Behme
@ 2011-08-21 7:54 ` V, Aneesh
0 siblings, 0 replies; 35+ messages in thread
From: V, Aneesh @ 2011-08-21 7:54 UTC (permalink / raw)
To: u-boot
Hi Dirk,
On Sun, Aug 21, 2011 at 1:16 PM, Dirk Behme <dirk.behme@googlemail.com> wrote:
> On 20.08.2011 17:32, Albert ARIBAUD wrote:
>> Hi all,
>>
>> Le 20/08/2011 13:12, Albert ARIBAUD a ?crit :
> ...
>>>>> 2. http://patchwork.ozlabs.org/patch/108865/
>>
>> This one kind of half-fixes the two boards it touches: they now build
>> but with warnings, all the same, of which here is one:
>>
>> clocks.c:274: warning: comparison is always true due to limited range of
>> data type
>>
>> I'll put this one on hold unless somebody can tell me what's going on.
>
> First, the patch 108865 above fixes a linking issue with the SPL for
> these boards. So I think that this is independent of the clocks.c
> warning you report above. That would mean it would be worth to apply
> 108865 to fix at least the linking issue.
>
> Second, applying 108865 to the recent u-boot-arm.git doesn't show any
> clocks.c warning for me [1]. Could this be tool chain dependent? I
> tried it with [2] and [3], without any warning.
>
> Btw, about which clocks.c (directory) do we talk?
I too didn't get this warning. But I see a problem with omap4/clocks.c.
I have submitted a patch just now.
http://patchwork.ozlabs.org/patch/110811/
BTW, thanks for looking into this.
br,
Aneesh
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-21 7:51 ` V, Aneesh
@ 2011-08-21 8:14 ` Albert ARIBAUD
2011-08-21 8:56 ` V, Aneesh
0 siblings, 1 reply; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-21 8:14 UTC (permalink / raw)
To: u-boot
Hi Aneesh,
Le 21/08/2011 09:51, V, Aneesh a ?crit :
> Hi Albert,
>
> On Sat, Aug 20, 2011 at 9:02 PM, Albert ARIBAUD
> <albert.u.boot@aribaud.net> wrote:
>> Hi all,
>>
>> Le 20/08/2011 13:12, Albert ARIBAUD a ?crit :
>>
>>>>> 1. http://patchwork.ozlabs.org/patch/107678/
>>
>> This one lowers bad board builds from 69 to 62, and has been applied.
>>
>>>>> 2. http://patchwork.ozlabs.org/patch/108865/
>>
>> This one kind of half-fixes the two boards it touches: they now build
>> but with warnings, all the same, of which here is one:
>>
>> clocks.c:274: warning: comparison is always true due to limited range of
>> data type
>
> My tool-chain (Sourcery G++ Lite 2010q1 - GCC 4.4.1) never reported this issue.
> But looks like there is indeed a problem. Hopefully the following patch should
> solve it:
>
> http://patchwork.ozlabs.org/patch/110811/
This indeed fixes it. Strange that the warning is not emitted by your
toolchain.
> best regards,
> Aneesh
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-21 8:14 ` Albert ARIBAUD
@ 2011-08-21 8:56 ` V, Aneesh
2011-08-21 10:13 ` Albert ARIBAUD
0 siblings, 1 reply; 35+ messages in thread
From: V, Aneesh @ 2011-08-21 8:56 UTC (permalink / raw)
To: u-boot
Hi Albert,
On Sun, Aug 21, 2011 at 1:44 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Aneesh,
>
> Le 21/08/2011 09:51, V, Aneesh a ?crit :
>>
>> Hi Albert,
>>
>> On Sat, Aug 20, 2011 at 9:02 PM, Albert ARIBAUD
>> <albert.u.boot@aribaud.net> ?wrote:
>>>
>>> Hi all,
>>>
>>> Le 20/08/2011 13:12, Albert ARIBAUD a ?crit :
>>>
>>>>>> 1. http://patchwork.ozlabs.org/patch/107678/
>>>
>>> This one lowers bad board builds from 69 to 62, and has been applied.
>>>
>>>>>> 2. http://patchwork.ozlabs.org/patch/108865/
>>>
>>> This one kind of half-fixes the two boards it touches: they now build
>>> but with warnings, all the same, of which here is one:
>>>
>>> clocks.c:274: warning: comparison is always true due to limited range of
>>> data type
>>
>> My tool-chain (Sourcery G++ Lite 2010q1 - GCC 4.4.1) never reported this
>> issue.
>> But looks like there is indeed a problem. Hopefully the following patch
>> should
>> solve it:
>>
>> http://patchwork.ozlabs.org/patch/110811/
>
> This indeed fixes it. Strange that the warning is not emitted by your
> toolchain.
Yes, that's indeed strange. Even with your tool-chain it should've emitted
more warnings. Many of those unsigned variables were assigned -1 in
elsewhere. That was also never reported.
br,
Aneesh
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-21 8:56 ` V, Aneesh
@ 2011-08-21 10:13 ` Albert ARIBAUD
0 siblings, 0 replies; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-21 10:13 UTC (permalink / raw)
To: u-boot
Hi Aneesh,
Le 21/08/2011 10:56, V, Aneesh a ?crit :
>> This indeed fixes it. Strange that the warning is not emitted by your
>> toolchain.
>
> Yes, that's indeed strange. Even with your tool-chain it should've emitted
> more warnings. Many of those unsigned variables were assigned -1 in
> elsewhere. That was also never reported.
Actually I only gave one of those warnings. For a single build of
omap4_panda, there was the following:
clocks.c:274: warning: comparison is always true due to limited range of
data type
clocks.c:276: warning: comparison is always true due to limited range of
data type
clocks.c:278: warning: comparison is always true due to limited range of
data type
clocks.c:280: warning: comparison is always true due to limited range of
data type
clocks.c:282: warning: comparison is always true due to limited range of
data type
clocks.c:284: warning: comparison is always true due to limited range of
data type
But note that these are comparisons, not assignments. The semantics of
ether operation might explain why one causes a warning and not the other.
> br,
> Aneesh
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-18 22:09 [U-Boot] [STATUS] ARM Albert ARIBAUD
` (2 preceding siblings ...)
2011-08-20 18:50 ` [U-Boot] [STATUS] ARM (Tegra) Mike Frysinger
@ 2011-08-21 10:17 ` Dirk Behme
2011-08-30 15:58 ` [U-Boot] [STATUS] ARM status as of 2011-08-30 Albert ARIBAUD
4 siblings, 0 replies; 35+ messages in thread
From: Dirk Behme @ 2011-08-21 10:17 UTC (permalink / raw)
To: u-boot
On 19.08.2011 00:09, Albert ARIBAUD wrote:
> Hi all,
>
> This week-end I intend to go through the board removal patch set and
> apply those patches in it that still fail to compile with the current
> u-boot-arm/master tree. So:
>
> 1) if there are any pull requests to u-voot-arm/master in the works,
> please try to submit them soon or at lease let me know, and
>
> 2) if there are any board fix patches posted after merge window closure
> and still pending, please also let me know (those before the merge
> window closure should all be in u-boot-arm by now).
For me, recent u-boot-arm.git [1] compiles ./MAKEALL ARMV7 fine now:
> ./MAKEALL ARMV7
...
--------------------- SUMMARY ----------------------------
Boards compiled: 31
----------------------------------------------------------
:)
Many thanks
Dirk
[1]
commit 5557e86bb0793012057d5462976c2a902bc629ac
omap4: increase SRAM budget to fix build error
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-20 11:12 ` Albert ARIBAUD
2011-08-20 15:32 ` Albert ARIBAUD
@ 2011-08-22 8:15 ` Simon Schwarz
2011-08-23 11:27 ` Michael Jones
1 sibling, 1 reply; 35+ messages in thread
From: Simon Schwarz @ 2011-08-22 8:15 UTC (permalink / raw)
To: u-boot
Hi Albert,
On 08/20/2011 01:12 PM, Albert ARIBAUD wrote:
>snip>
>
> Sandeep has requested that I pick these two in u-boot-arm.
>
<snip>
So is right that everything else in the responsibility of Sandeep won't
make it into the next release?
How about the next tree?
Regards
Simon
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-21 7:22 ` Albert ARIBAUD
@ 2011-08-23 2:02 ` V, Aneesh
0 siblings, 0 replies; 35+ messages in thread
From: V, Aneesh @ 2011-08-23 2:02 UTC (permalink / raw)
To: u-boot
Hi Albert,
On Sun, Aug 21, 2011 at 12:52 PM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Hi Dirk,
>
> Le 21/08/2011 09:11, Dirk Behme a ?crit :
>
>> But, hmm,
>>
>> http://git.denx.de/?p=u-boot/u-boot-arm.git;a=summary
>>
>> doesn't show the applied patches yet? Did I miss something?
>
> You did not; I did -- forgot to push master. Fixed now, thanks for
> pointing this out.
How about this series:
http://patchwork.ozlabs.org/bundle/aneeshv/armv7-cache-fixes/
If you pull this, I would prefer that you take all the patches
so that there is no regression for OMAP.
best regards,
Aneesh
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM
2011-08-22 8:15 ` Simon Schwarz
@ 2011-08-23 11:27 ` Michael Jones
0 siblings, 0 replies; 35+ messages in thread
From: Michael Jones @ 2011-08-23 11:27 UTC (permalink / raw)
To: u-boot
Hi Albert,
On 08/22/2011 10:15 AM, Simon Schwarz wrote:
>
> Hi Albert,
>
> On 08/20/2011 01:12 PM, Albert ARIBAUD wrote:
>> snip>
>>
>> Sandeep has requested that I pick these two in u-boot-arm.
>>
> <snip>
>
> So is right that everything else in the responsibility of Sandeep won't
> make it into the next release?
>
> How about the next tree?
I'm also curious about the fate of OMAP patches for the next release. It
seems to me there hasn't been much feedback for OMAP patches this cycle.
The most recent commit at u-boot-ti is a month old now.
For example, I got an ack from Heiko on an OMAP I2C patch [1], and he
asked Sandeep to either ACK/NACK it or pick it up himself, but nothing
has happened with it. The cosmetic patch Heiko refers to there also
hasn't been touched. These were at least reviewed because they
overlapped with I2C, but my OMAP-specific patches [2,3] have gotten
virtually no feedback. I presume, then, that they will not make it in to
v2011.09?
[1] http://article.gmane.org/gmane.comp.boot-loaders.u-boot/104811
[2] http://patchwork.ozlabs.org/patch/105288/
[3] http://patchwork.ozlabs.org/patch/110298/
>
> Regards
> Simon
-Michael
MATRIX VISION GmbH, Talstrasse 16, DE-71570 Oppenweiler
Registergericht: Amtsgericht Stuttgart, HRB 271090
Geschaeftsfuehrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner, Erhard Meier
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM (Tegra)
2011-08-20 18:50 ` [U-Boot] [STATUS] ARM (Tegra) Mike Frysinger
@ 2011-08-30 15:20 ` Mike Frysinger
2011-08-30 15:34 ` Albert ARIBAUD
0 siblings, 1 reply; 35+ messages in thread
From: Mike Frysinger @ 2011-08-30 15:20 UTC (permalink / raw)
To: u-boot
On Saturday, August 20, 2011 14:50:27 Mike Frysinger wrote:
> On Thursday, August 18, 2011 18:09:56 Albert ARIBAUD wrote:
> > 1) if there are any pull requests to u-voot-arm/master in the works,
> > please try to submit them soon or at lease let me know, and
>
> we've got some Tegra stuff that seems to be waiting for merging (or at
> least feedback as to anything left that needs to be done so it can be
> merged): [RESEND PATCH v3 0/4] Add basic clock and pinmux functions to the
> Tegra2
>
> to broach the more general tegra question, should these get sent to Tom and
> then he sends them to Albert, or should they get sent directly to Albert
> (since there is no dedicated tegra or nvidia u-boot tree atm) ? we'd like
> to get this process smoothed out so we dont keep annoying people :).
ping ...
if people dont care either way, we can just set up a u-boot/u-boot-tegra.git
tree for Albert to pull from
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20110830/226fd12e/attachment.pgp
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM (Tegra)
2011-08-30 15:20 ` Mike Frysinger
@ 2011-08-30 15:34 ` Albert ARIBAUD
2011-08-30 15:57 ` Simon Glass
0 siblings, 1 reply; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-30 15:34 UTC (permalink / raw)
To: u-boot
Le 30/08/2011 17:20, Mike Frysinger a ?crit :
> On Saturday, August 20, 2011 14:50:27 Mike Frysinger wrote:
>> On Thursday, August 18, 2011 18:09:56 Albert ARIBAUD wrote:
>>> 1) if there are any pull requests to u-voot-arm/master in the works,
>>> please try to submit them soon or at lease let me know, and
>>
>> we've got some Tegra stuff that seems to be waiting for merging (or at
>> least feedback as to anything left that needs to be done so it can be
>> merged): [RESEND PATCH v3 0/4] Add basic clock and pinmux functions to the
>> Tegra2
>>
>> to broach the more general tegra question, should these get sent to Tom and
>> then he sends them to Albert, or should they get sent directly to Albert
>> (since there is no dedicated tegra or nvidia u-boot tree atm) ? we'd like
>> to get this process smoothed out so we dont keep annoying people :).
>
> ping ...
>
> if people dont care either way, we can just set up a u-boot/u-boot-tegra.git
> tree for Albert to pull from
Seems like people don't care, and I'm fine with applying the patches
myself, but I don't want to pull from a tree: I'll apply the patches as
they appear on patchwork.
Are we talking about these four patches?
<http://patchwork.ozlabs.org/patch/110117/>
<http://patchwork.ozlabs.org/patch/110120/>
<http://patchwork.ozlabs.org/patch/110119/>
<http://patchwork.ozlabs.org/patch/110118/>
> -mike
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM (Tegra)
2011-08-30 15:34 ` Albert ARIBAUD
@ 2011-08-30 15:57 ` Simon Glass
2011-08-30 16:27 ` Simon Glass
2011-08-30 16:31 ` Albert ARIBAUD
0 siblings, 2 replies; 35+ messages in thread
From: Simon Glass @ 2011-08-30 15:57 UTC (permalink / raw)
To: u-boot
On Tue, Aug 30, 2011 at 8:34 AM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Le 30/08/2011 17:20, Mike Frysinger a ?crit :
>>
>> On Saturday, August 20, 2011 14:50:27 Mike Frysinger wrote:
>>>
>>> On Thursday, August 18, 2011 18:09:56 Albert ARIBAUD wrote:
>>>>
>>>> 1) if there are any pull requests to u-voot-arm/master in the works,
>>>> please try to submit them soon or at lease let me know, and
>>>
>>> we've got some Tegra stuff that seems to be waiting for merging (or at
>>> least feedback as to anything left that needs to be done so it can be
>>> merged): [RESEND PATCH v3 0/4] Add basic clock and pinmux functions to
>>> the
>>> Tegra2
>>>
>>> to broach the more general tegra question, should these get sent to Tom
>>> and
>>> then he sends them to Albert, or should they get sent directly to Albert
>>> (since there is no dedicated tegra or nvidia u-boot tree atm) ? ?we'd
>>> like
>>> to get this process smoothed out so we dont keep annoying people :).
>>
>> ping ...
>>
>> if people dont care either way, we can just set up a
>> u-boot/u-boot-tegra.git
>> tree for Albert to pull from
>
> Seems like people don't care, and I'm fine with applying the patches myself,
> but I don't want to pull from a tree: I'll apply the patches as they appear
> on patchwork.
>
> Are we talking about these four patches?
>
> <http://patchwork.ozlabs.org/patch/110117/>
> <http://patchwork.ozlabs.org/patch/110120/>
> <http://patchwork.ozlabs.org/patch/110119/>
> <http://patchwork.ozlabs.org/patch/110118/>
Yes that's right, thanks.
Regards,
Simon
>
>> -mike
>
> Amicalement,
> --
> Albert.
>
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM status as of 2011-08-30
2011-08-18 22:09 [U-Boot] [STATUS] ARM Albert ARIBAUD
` (3 preceding siblings ...)
2011-08-21 10:17 ` [U-Boot] [STATUS] ARM Dirk Behme
@ 2011-08-30 15:58 ` Albert ARIBAUD
2011-08-30 19:08 ` Wolfgang Denk
4 siblings, 1 reply; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-30 15:58 UTC (permalink / raw)
To: u-boot
Hi all,
I have just applied Wolfgang's updated ARM board removal patch set.
Currently, u-boot-arm/master builds 182 boards, of which:
- ELDK4.2 finds 37 boards with warnings or errors ( dnp1110 gcplus lart
shannon ep7312 evb4510 impa7 lpc2292sodimm modnet50 SMN42 jadecpu
smdk2410 VCMA9 versatile omap2420h4 apollon imx31_phycore
imx31_phycore_eet mx31pdk mx31pdk_nand qong smdk6400 csb226 lubbock
zylonite actux1_4_16 actux1_8_16 actux1_4_32 actux1_8_32 actux2 actux3
actux4 dvlhost ixdp425 ixdpg425 pdnb3 scpu )
- CodeSourcery 2010q1 finds 100 (!) boards with warnings or errors (
dnp1110 gcplus lart shannon ep7312 evb4510 impa7 lpc2292sodimm modnet50
SMN42 edminiv2 guruplug jadecpu km_kirkwood mv88f6281gtw_ge openrd_base
openrd_client openrd_ultimate portl2 rd6281a sheevaplug smdk2410 VCMA9
versatile omap2420h4 apollon imx31_phycore imx31_phycore_eet mx31pdk
mx31pdk_nand qong smdk6400 efikamx at91rm9200ek at91rm9200ek_ram
afeb9260 at91sam9260ek_nandflash at91sam9260ek_dataflash_cs0
at91sam9260ek_dataflash_cs1 at91sam9261ek_nandflash
at91sam9261ek_dataflash_cs0 at91sam9261ek_dataflash_cs3
at91sam9263ek_nandflash at91sam9263ek_dataflash_cs0
at91sam9263ek_dataflash at91sam9263ek_norflash
at91sam9263ek_norflash_boot at91sam9g10ek_nandflash
at91sam9g10ek_dataflash_cs0 at91sam9g10ek_dataflash_cs3
at91sam9g20ek_nandflash at91sam9g20ek_dataflash_cs0
at91sam9g20ek_dataflash_cs1 at91sam9m10g45ek_nandflash
at91sam9xeek_nandflash at91sam9xeek_dataflash_cs0
at91sam9xeek_dataflash_cs1 snapper9260 snapper9g20 sbc35_a9g20_nandflash
sbc35_a9g20_eeprom cpu9260 cpu9260_nand cpu9260_128M cpu9260_nand_128M
cpu9G20 cpu9G20_nand cpu9G20_128M cpu9G20_nand_128M top9000eval_xe
top9000su_xe meesc meesc_dataflash otc570 otc570_dataflash pm9261 pm9263
pm9g45 balloon3 colibri_pxa270 csb226 lubbock polaris trizepsiv
vpac270_nor_128 vpac270_nor_256 vpac270_ond_256 zylonite actux1_4_16
actux1_8_16 actux1_4_32 actux1_8_32 actux2 actux3 actux4 dvlhost ixdp425
ixdpg425 pdnb3 scpu )
However, most of the errors in the CodeSourcery case are actually the
same, i.e. "warning: dereferencing pointer 'xxxxx' does break
strict-aliasing rules", mostly in drivers/usb/host/ohci-hcd.c and once
in fs/yaffs2/yaffs_tagscompat.c -- that is, errors seem not to be
ARM-specific but rather compiler-dependent -- seems like each compiler
has its own sensitivity. ELDK4.2 also has some errors in YAFFS code, BTW.
(Cc:ing R?my for USB stuff; for yaffs, I don't know whom I should Cc:)
For both CS and ELDK, there are a lot of "arm-linux-ld: stubs.o:
compiled for a big endian system and target is little endian
arm-linux-ld: failed to merge target specific data of file stubs.o"
which I'll look into.
Meanwhile, I suggest that the current u-boot-arm/master be considered
rc1-ready, with the effort until rc2 focused on finding and fixing
code-wide bugs such as those two above.
If that is ok, I'll send out a pull request today.
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM (Tegra)
2011-08-30 15:57 ` Simon Glass
@ 2011-08-30 16:27 ` Simon Glass
2011-08-30 16:31 ` Albert ARIBAUD
1 sibling, 0 replies; 35+ messages in thread
From: Simon Glass @ 2011-08-30 16:27 UTC (permalink / raw)
To: u-boot
Hi Albert,
On Tue, Aug 30, 2011 at 8:57 AM, Simon Glass <sjg@chromium.org> wrote:
> On Tue, Aug 30, 2011 at 8:34 AM, Albert ARIBAUD
> <albert.u.boot@aribaud.net> wrote:
>> Le 30/08/2011 17:20, Mike Frysinger a ?crit :
>>>
>>> On Saturday, August 20, 2011 14:50:27 Mike Frysinger wrote:
>>>>
>>>> On Thursday, August 18, 2011 18:09:56 Albert ARIBAUD wrote:
>>>>>
>>>>> 1) if there are any pull requests to u-voot-arm/master in the works,
>>>>> please try to submit them soon or at lease let me know, and
>>>>
>>>> we've got some Tegra stuff that seems to be waiting for merging (or at
>>>> least feedback as to anything left that needs to be done so it can be
>>>> merged): [RESEND PATCH v3 0/4] Add basic clock and pinmux functions to
>>>> the
>>>> Tegra2
>>>>
>>>> to broach the more general tegra question, should these get sent to Tom
>>>> and
>>>> then he sends them to Albert, or should they get sent directly to Albert
>>>> (since there is no dedicated tegra or nvidia u-boot tree atm) ? ?we'd
>>>> like
>>>> to get this process smoothed out so we dont keep annoying people :).
>>>
>>> ping ...
>>>
>>> if people dont care either way, we can just set up a
>>> u-boot/u-boot-tegra.git
>>> tree for Albert to pull from
>>
>> Seems like people don't care, and I'm fine with applying the patches myself,
>> but I don't want to pull from a tree: I'll apply the patches as they appear
>> on patchwork.
>>
>> Are we talking about these four patches?
>>
>> <http://patchwork.ozlabs.org/patch/110117/>
>> <http://patchwork.ozlabs.org/patch/110120/>
>> <http://patchwork.ozlabs.org/patch/110119/>
>> <http://patchwork.ozlabs.org/patch/110118/>
I have just sent v4 to the list, which is a rebase against master, and
also removes a function from a header file as noticed in the review.
Please can you please pick up this one? We can then start working on
the peripheral drivers, etc.
Regards,
Simon
>
> Yes that's right, thanks.
>
> Regards,
> Simon
>
>>
>>> -mike
>>
>> Amicalement,
>> --
>> Albert.
>>
>
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM (Tegra)
2011-08-30 15:57 ` Simon Glass
2011-08-30 16:27 ` Simon Glass
@ 2011-08-30 16:31 ` Albert ARIBAUD
2011-08-30 16:36 ` Simon Glass
1 sibling, 1 reply; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-30 16:31 UTC (permalink / raw)
To: u-boot
Le 30/08/2011 17:57, Simon Glass a ?crit :
> On Tue, Aug 30, 2011 at 8:34 AM, Albert ARIBAUD
> <albert.u.boot@aribaud.net> wrote:
>> Le 30/08/2011 17:20, Mike Frysinger a ?crit :
>>>
>>> On Saturday, August 20, 2011 14:50:27 Mike Frysinger wrote:
>>>>
>>>> On Thursday, August 18, 2011 18:09:56 Albert ARIBAUD wrote:
>>>>>
>>>>> 1) if there are any pull requests to u-voot-arm/master in the works,
>>>>> please try to submit them soon or at lease let me know, and
>>>>
>>>> we've got some Tegra stuff that seems to be waiting for merging (or at
>>>> least feedback as to anything left that needs to be done so it can be
>>>> merged): [RESEND PATCH v3 0/4] Add basic clock and pinmux functions to
>>>> the
>>>> Tegra2
>>>>
>>>> to broach the more general tegra question, should these get sent to Tom
>>>> and
>>>> then he sends them to Albert, or should they get sent directly to Albert
>>>> (since there is no dedicated tegra or nvidia u-boot tree atm) ? we'd
>>>> like
>>>> to get this process smoothed out so we dont keep annoying people :).
>>>
>>> ping ...
>>>
>>> if people dont care either way, we can just set up a
>>> u-boot/u-boot-tegra.git
>>> tree for Albert to pull from
>>
>> Seems like people don't care, and I'm fine with applying the patches myself,
>> but I don't want to pull from a tree: I'll apply the patches as they appear
>> on patchwork.
>>
>> Are we talking about these four patches?
>>
>> <http://patchwork.ozlabs.org/patch/110117/>
>> <http://patchwork.ozlabs.org/patch/110120/>
>> <http://patchwork.ozlabs.org/patch/110119/>
>> <http://patchwork.ozlabs.org/patch/110118/>
>
> Yes that's right, thanks.
There was a comment on 110117 (patch 1/4) from Graeme about
timer_get_future_us() still being there despite the history. Can you
post a V4 for patch 1/4 only with this function removed entirely (make
sure the subject and history are up to date for V4) ?
> Regards,
> Simon
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM (Tegra)
2011-08-30 16:31 ` Albert ARIBAUD
@ 2011-08-30 16:36 ` Simon Glass
2011-08-30 16:39 ` Albert ARIBAUD
0 siblings, 1 reply; 35+ messages in thread
From: Simon Glass @ 2011-08-30 16:36 UTC (permalink / raw)
To: u-boot
On Tue, Aug 30, 2011 at 9:31 AM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Le 30/08/2011 17:57, Simon Glass a ?crit :
>>
>> On Tue, Aug 30, 2011 at 8:34 AM, Albert ARIBAUD
>> <albert.u.boot@aribaud.net> ?wrote:
>>>
>>> Le 30/08/2011 17:20, Mike Frysinger a ?crit :
>>>>
>>>> On Saturday, August 20, 2011 14:50:27 Mike Frysinger wrote:
>>>>>
>>>>> On Thursday, August 18, 2011 18:09:56 Albert ARIBAUD wrote:
>>>>>>
>>>>>> 1) if there are any pull requests to u-voot-arm/master in the works,
>>>>>> please try to submit them soon or at lease let me know, and
>>>>>
>>>>> we've got some Tegra stuff that seems to be waiting for merging (or at
>>>>> least feedback as to anything left that needs to be done so it can be
>>>>> merged): [RESEND PATCH v3 0/4] Add basic clock and pinmux functions to
>>>>> the
>>>>> Tegra2
>>>>>
>>>>> to broach the more general tegra question, should these get sent to Tom
>>>>> and
>>>>> then he sends them to Albert, or should they get sent directly to
>>>>> Albert
>>>>> (since there is no dedicated tegra or nvidia u-boot tree atm) ? ?we'd
>>>>> like
>>>>> to get this process smoothed out so we dont keep annoying people :).
>>>>
>>>> ping ...
>>>>
>>>> if people dont care either way, we can just set up a
>>>> u-boot/u-boot-tegra.git
>>>> tree for Albert to pull from
>>>
>>> Seems like people don't care, and I'm fine with applying the patches
>>> myself,
>>> but I don't want to pull from a tree: I'll apply the patches as they
>>> appear
>>> on patchwork.
>>>
>>> Are we talking about these four patches?
>>>
>>> <http://patchwork.ozlabs.org/patch/110117/>
>>> <http://patchwork.ozlabs.org/patch/110120/>
>>> <http://patchwork.ozlabs.org/patch/110119/>
>>> <http://patchwork.ozlabs.org/patch/110118/>
>>
>> Yes that's right, thanks.
>
> There was a comment on 110117 (patch 1/4) from Graeme about
> timer_get_future_us() still being there despite the history. Can you post a
> V4 for patch 1/4 only with this function removed entirely (make sure the
> subject and history are up to date for V4) ?
Hi Albert,
I just resent the series as v4 with this change - is that OK?
Regards,
Simon
> Amicalement,
> --
> Albert.
>
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM (Tegra)
2011-08-30 16:36 ` Simon Glass
@ 2011-08-30 16:39 ` Albert ARIBAUD
2011-08-30 16:45 ` Simon Glass
0 siblings, 1 reply; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-30 16:39 UTC (permalink / raw)
To: u-boot
Le 30/08/2011 18:36, Simon Glass a ?crit :
> Hi Albert,
>
> I just resent the series as v4 with this change - is that OK?
Apologies -- I had not seen the V4 before posting.
V4 is fine to me and since there was only one comment for V3 and V4's
only change is to address this comment, I think I can apply V4 right away.
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM (Tegra)
2011-08-30 16:39 ` Albert ARIBAUD
@ 2011-08-30 16:45 ` Simon Glass
0 siblings, 0 replies; 35+ messages in thread
From: Simon Glass @ 2011-08-30 16:45 UTC (permalink / raw)
To: u-boot
On Tue, Aug 30, 2011 at 9:39 AM, Albert ARIBAUD
<albert.u.boot@aribaud.net> wrote:
> Le 30/08/2011 18:36, Simon Glass a ?crit :
>
>> Hi Albert,
>>
>> I just resent the series as v4 with this change - is that OK?
>
> Apologies -- I had not seen the V4 before posting.
>
> V4 is fine to me and since there was only one comment for V3 and V4's only
> change is to address this comment, I think I can apply V4 right away.
Thank you Albert. Will wait for this, then get to work on the next
series. - Simon
>
> Amicalement,
> --
> Albert.
>
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM status as of 2011-08-30
2011-08-30 15:58 ` [U-Boot] [STATUS] ARM status as of 2011-08-30 Albert ARIBAUD
@ 2011-08-30 19:08 ` Wolfgang Denk
2011-08-31 10:25 ` Albert ARIBAUD
0 siblings, 1 reply; 35+ messages in thread
From: Wolfgang Denk @ 2011-08-30 19:08 UTC (permalink / raw)
To: u-boot
Dear Albert ARIBAUD,
In message <4E5D088B.7090702@aribaud.net> you wrote:
>
> I have just applied Wolfgang's updated ARM board removal patch set.
>
> Currently, u-boot-arm/master builds 182 boards, of which:
>
> - ELDK4.2 finds 37 boards with warnings or errors ( dnp1110 gcplus lart
I think we should try and separate the errors cases from the warnings
only ones - those boards with build errors need to be fixed, or
removed.
We can clean up warnings then, as we go on.
> - CodeSourcery 2010q1 finds 100 (!) boards with warnings or errors (
I guess it's additional warnings. You will see even more when using
ELDK 5.x
> (Cc:ing R?my for USB stuff; for yaffs, I don't know whom I should Cc:)
yeaffs is generic code, i. e. it ends up on my plate.
> For both CS and ELDK, there are a lot of "arm-linux-ld: stubs.o:
> compiled for a big endian system and target is little endian
> arm-linux-ld: failed to merge target specific data of file stubs.o"
> which I'll look into.
You need a big endian ARM compiler for the ixp systems.
> Meanwhile, I suggest that the current u-boot-arm/master be considered
> rc1-ready, with the effort until rc2 focused on finding and fixing
> code-wide bugs such as those two above.
Agreed.
> If that is ok, I'll send out a pull request today.
Thanks a lot!
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
I realize that command does have its fascination, even under circum-
stances such as these, but I neither enjoy the idea of command nor am
I frightened of it. It simply exists, and I will do whatever logi-
cally needs to be done.
-- Spock, "The Galileo Seven", stardate 2812.7
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM status as of 2011-08-30
2011-08-30 19:08 ` Wolfgang Denk
@ 2011-08-31 10:25 ` Albert ARIBAUD
2011-08-31 19:55 ` Wolfgang Denk
0 siblings, 1 reply; 35+ messages in thread
From: Albert ARIBAUD @ 2011-08-31 10:25 UTC (permalink / raw)
To: u-boot
Le 30/08/2011 21:08, Wolfgang Denk a ?crit :
> Dear Albert ARIBAUD,
>
> In message<4E5D088B.7090702@aribaud.net> you wrote:
>>
>> I have just applied Wolfgang's updated ARM board removal patch set.
>>
>> Currently, u-boot-arm/master builds 182 boards, of which:
>>
>> - ELDK4.2 finds 37 boards with warnings or errors ( dnp1110 gcplus lart
>
> I think we should try and separate the errors cases from the warnings
> only ones - those boards with build errors need to be fixed, or
> removed.
Right, but the current MAKEALL output does not make it easy to sort this
out. Maybe we could change MAKEALL to distinguish between error and
warning counts (and produce LOG/*.MAKELOG, LOG/*.ERR and LOG/*.WARN
accordingly).
> We can clean up warnings then, as we go on.
Agreed. However, I'd prefer it if no ARM board produced any warning --
granted, here it *looks like* many of these warnings are inocuous, and
any new toolchain might produce some more, but if we don't keep the
warning count at zero, then a less inocuous warning could creep in
unnoticed.
>> - CodeSourcery 2010q1 finds 100 (!) boards with warnings or errors (
>
> I guess it's additional warnings. You will see even more when using
> ELDK 5.x
>
>> (Cc:ing R?my for USB stuff; for yaffs, I don't know whom I should Cc:)
>
> yeaffs is generic code, i. e. it ends up on my plate.
There you are, then. :)
>> For both CS and ELDK, there are a lot of "arm-linux-ld: stubs.o:
>> compiled for a big endian system and target is little endian
>> arm-linux-ld: failed to merge target specific data of file stubs.o"
>> which I'll look into.
>
> You need a big endian ARM compiler for the ixp systems.
Got it -- actually, it is not a compiler issue, but a library issue: CS
does compile for big-endian as well as little-endian, but does not
provide big-endian libs
(<http://www.codesourcery.com/sgpp/lite/arm/portal/kbentry36>, at least
as I understand it). Apparently ELDK4.2 does not either. Would ELDK5.0
have them? I haven't gotten around to setting it up.
>> Meanwhile, I suggest that the current u-boot-arm/master be considered
>> rc1-ready, with the effort until rc2 focused on finding and fixing
>> code-wide bugs such as those two above.
>
> Agreed.
>
>> If that is ok, I'll send out a pull request today.
>
> Thanks a lot!
Done, with a few hour's delay.
> Best regards,
>
> Wolfgang Denk
Amicalement,
--
Albert.
^ permalink raw reply [flat|nested] 35+ messages in thread
* [U-Boot] [STATUS] ARM status as of 2011-08-30
2011-08-31 10:25 ` Albert ARIBAUD
@ 2011-08-31 19:55 ` Wolfgang Denk
0 siblings, 0 replies; 35+ messages in thread
From: Wolfgang Denk @ 2011-08-31 19:55 UTC (permalink / raw)
To: u-boot
Dear Albert ARIBAUD,
In message <4E5E0C2E.2030902@aribaud.net> you wrote:
>
> Got it -- actually, it is not a compiler issue, but a library issue: CS
> does compile for big-endian as well as little-endian, but does not
> provide big-endian libs
> (<http://www.codesourcery.com/sgpp/lite/arm/portal/kbentry36>, at least
> as I understand it). Apparently ELDK4.2 does not either. Would ELDK5.0
> have them? I haven't gotten around to setting it up.
Sorry, no. The relevance of IXP is not even close to justifying the
efforts to support a configuation for it.
> Done, with a few hour's delay.
At the moment I'm on hold, as you asked for.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"Free markets select for winning solutions." - Eric S. Raymond
^ permalink raw reply [flat|nested] 35+ messages in thread
end of thread, other threads:[~2011-08-31 19:55 UTC | newest]
Thread overview: 35+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-18 22:09 [U-Boot] [STATUS] ARM Albert ARIBAUD
2011-08-19 9:38 ` Aneesh V
2011-08-19 18:19 ` Dirk Behme
2011-08-20 11:12 ` Albert ARIBAUD
2011-08-20 15:32 ` Albert ARIBAUD
2011-08-21 7:11 ` Dirk Behme
2011-08-21 7:22 ` Albert ARIBAUD
2011-08-23 2:02 ` V, Aneesh
2011-08-21 7:46 ` Dirk Behme
2011-08-21 7:54 ` V, Aneesh
2011-08-21 7:51 ` V, Aneesh
2011-08-21 8:14 ` Albert ARIBAUD
2011-08-21 8:56 ` V, Aneesh
2011-08-21 10:13 ` Albert ARIBAUD
2011-08-22 8:15 ` Simon Schwarz
2011-08-23 11:27 ` Michael Jones
2011-08-20 11:16 ` Marek Vasut
2011-08-20 11:44 ` Albert ARIBAUD
2011-08-20 12:24 ` Marek Vasut
2011-08-20 12:54 ` Albert ARIBAUD
2011-08-20 13:33 ` Marek Vasut
2011-08-20 18:50 ` [U-Boot] [STATUS] ARM (Tegra) Mike Frysinger
2011-08-30 15:20 ` Mike Frysinger
2011-08-30 15:34 ` Albert ARIBAUD
2011-08-30 15:57 ` Simon Glass
2011-08-30 16:27 ` Simon Glass
2011-08-30 16:31 ` Albert ARIBAUD
2011-08-30 16:36 ` Simon Glass
2011-08-30 16:39 ` Albert ARIBAUD
2011-08-30 16:45 ` Simon Glass
2011-08-21 10:17 ` [U-Boot] [STATUS] ARM Dirk Behme
2011-08-30 15:58 ` [U-Boot] [STATUS] ARM status as of 2011-08-30 Albert ARIBAUD
2011-08-30 19:08 ` Wolfgang Denk
2011-08-31 10:25 ` Albert ARIBAUD
2011-08-31 19:55 ` Wolfgang Denk
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.