Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm

2011-06-28 Thread Andreas Bießmann
Dear Graeme Russ,

Am 28.06.2011 13:40, schrieb Graeme Russ:
> 
> Signed-off-by: Graeme Russ 
> ---
..
>  board/atmel/at91rm9200dk/flash.c |   10 ++

don't mind at91rm9200dk, it uses deprecated arm920t/at91rm9200 api and
will therefore removed in this merge window.

regards

Andreas Bießmann
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm

2011-07-11 Thread Wolfgang Denk
Dear Graeme Russ,

In message <1309261269-4363-2-git-send-email-graeme.r...@gmail.com> you wrote:
> 
> Signed-off-by: Graeme Russ 

I consider this a bug fix, independent from the other dicussion.  This
should be applied in any case.

Acked-by: Wolfgang Denk 

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: w...@denx.de
"We don't care.  We don't have to.  We're the Phone Company."
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm

2011-07-14 Thread Albert ARIBAUD
Hi Wolfgang,

Le 11/07/2011 23:57, Wolfgang Denk a écrit :
> Dear Graeme Russ,
>
> In message<1309261269-4363-2-git-send-email-graeme.r...@gmail.com>  you wrote:
>>
>> Signed-off-by: Graeme Russ
>
> I consider this a bug fix, independent from the other dicussion.  This
> should be applied in any case.
>
> Acked-by: Wolfgang Denk
>
> Best regards,
>
> Wolfgang Denk

Patch applies on u-boot-arm/master except for 4 files which are not found:

board/ixdp425/flash.c, removed by commit 
973af335e61eede3578371330abd3971805887f5, "update/fix IXDP425 / IXDPG425 
boards"

board/mpl/vcma9/flash.c: removed by commit 
6d754843ff62312999a265162c67588913cab991, "VCMA9: use CFI driver (and 
remove the old one)"

board/trab/cmd_trab.c and board/trab/flash.c: removed by commit 
566e5cf451ae7e33e31bb62ae5b9b258e33f8609, "ARM: drop unsupported 'trab' 
board"

If this is fine with everyone, I can pull this patch in 
u-boot-arm/master despite the four errors.

Amicalement,
-- 
Albert.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm

2011-07-14 Thread Wolfgang Denk
Dear Albert ARIBAUD,

In message <4e1f2100.2030...@aribaud.net> you wrote:
> 
> Patch applies on u-boot-arm/master except for 4 files which are not found:
> 
> board/ixdp425/flash.c, removed by commit 
> 973af335e61eede3578371330abd3971805887f5, "update/fix IXDP425 / IXDPG425 
> boards"
> 
> board/mpl/vcma9/flash.c: removed by commit 
> 6d754843ff62312999a265162c67588913cab991, "VCMA9: use CFI driver (and 
> remove the old one)"
> 
> board/trab/cmd_trab.c and board/trab/flash.c: removed by commit 
> 566e5cf451ae7e33e31bb62ae5b9b258e33f8609, "ARM: drop unsupported 'trab' 
> board"
> 
> If this is fine with everyone, I can pull this patch in 
> u-boot-arm/master despite the four errors.

Please do.  These files were all correctly removed recently.

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: w...@denx.de
In the bathtub of history the truth is harder to hold than the  soap,
and much more difficult to find ... - Terry Pratchett, _Sourcery_
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm

2011-07-14 Thread Graeme Russ
Hi Wolfgang,

On 15/07/11 05:50, Wolfgang Denk wrote:
> Dear Albert ARIBAUD,
> 
> In message <4e1f2100.2030...@aribaud.net> you wrote:
>>
>> Patch applies on u-boot-arm/master except for 4 files which are not found:
>>
>> board/ixdp425/flash.c, removed by commit 
>> 973af335e61eede3578371330abd3971805887f5, "update/fix IXDP425 / IXDPG425 
>> boards"
>>
>> board/mpl/vcma9/flash.c: removed by commit 
>> 6d754843ff62312999a265162c67588913cab991, "VCMA9: use CFI driver (and 
>> remove the old one)"
>>
>> board/trab/cmd_trab.c and board/trab/flash.c: removed by commit 
>> 566e5cf451ae7e33e31bb62ae5b9b258e33f8609, "ARM: drop unsupported 'trab' 
>> board"
>>
>> If this is fine with everyone, I can pull this patch in 
>> u-boot-arm/master despite the four errors.
> 
> Please do.  These files were all correctly removed recently.

I'm OK with this, but I'm about to rebase - After that, I will rename the
series, truncate it to only include the ack'd patches, rev them to V2, set
their respective in-reply-to to the WIP patches and set the remainder to
RFC in patchwork

So I could either:
 - Send a revised version as a separate patch for ARM and remove it from
the series
 - Include it in the truncated series
 - Remove it from the series and let Albert process it as is

Which do you prefer?

Regards,

Graeme
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm

2011-07-14 Thread Wolfgang Denk
Dear Graeme Russ,

In message <4e1f7ac5.2080...@gmail.com> you wrote:
> 
> So I could either:
>  - Send a revised version as a separate patch for ARM and remove it from
> the series
>  - Include it in the truncated series
>  - Remove it from the series and let Albert process it as is
> 
> Which do you prefer?

I'm fine with each of these - I just like to see this go in :-)

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: w...@denx.de
C++ is the best example of second-system effect since OS/360.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH v1 (WIP) 01/16] [Timer]Fix misuse of ARM *timer_masked() functions outside arch/arm

2011-07-15 Thread Graeme Russ
Hi Wolfgang,

On 15/07/11 16:31, Wolfgang Denk wrote:
> Dear Graeme Russ,
> 
> In message <4e1f7ac5.2080...@gmail.com> you wrote:
>>
>> So I could either:
>>  - Send a revised version as a separate patch for ARM and remove it from
>> the series
>>  - Include it in the truncated series
>>  - Remove it from the series and let Albert process it as is
>>
>> Which do you prefer?
> 
> I'm fine with each of these - I just like to see this go in :-)

OK, I'll go with option #2 and rename the series 'Simplify Timer API'. I'll
post this in the next day or two (hopefully tonight)

This will get the existing Timer API into a state where I can proceed with
the outcome of our latest round of discussion.

Regards,

Graeme
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot