All of lore.kernel.org
 help / color / mirror / Atom feed
* [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.