[U-Boot] arm: comment request

[U-Boot] arm: comment request

[Heiko Schocher]

Hello Albert,

can you comment/apply the following patches? As the new merge window is
open, I think they can go to mainline:

arm, logbuffer: make it compileclean
http://patchwork.ozlabs.org/patch/114740/

post, arm: make post support for arm compile clean
http://patchwork.ozlabs.org/patch/114741/

cosmetic, post: Codingstyle cleanup
http://patchwork.ozlabs.org/patch/114742/

arm, post: add missing post_time_ms for arm
http://patchwork.ozlabs.org/patch/114743/

Thanks!

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

[U-Boot] arm: comment request

[Albert ARIBAUD]

Le 05/10/2011 07:55, Heiko Schocher a ?crit :
> Hello Albert,
>
> can you comment/apply the following patches? As the new merge window is
> open, I think they can go to mainline:
>
> arm, logbuffer: make it compileclean
> http://patchwork.ozlabs.org/patch/114740/
>
> post, arm: make post support for arm compile clean
> http://patchwork.ozlabs.org/patch/114741/
>
> cosmetic, post: Codingstyle cleanup
> http://patchwork.ozlabs.org/patch/114742/
>
> arm, post: add missing post_time_ms for arm
> http://patchwork.ozlabs.org/patch/114743/
>
> Thanks!
>
> bye,
> Heiko

Hi Heiko,

Sorry I was unavailable for some time. I'm going through my backlog and 
will comment on these soon.

Amicalement,
-- 
Albert.

[U-Boot] arm: comment request

[Christian Riesch]

Hello Heiko
I tried to apply your patches to the current master, since they are a
prerequisite for your enbw_cmc patches but

[U-Boot] cosmetic, post: Codingstyle cleanup
http://patchwork.ozlabs.org/patch/114742/

did not cleanly apply.

I got it from patchwork via
$ wget -nv http://patchwork.ozlabs.org/patch/114742/mbox/ -O 114742.patch
2011-10-12 10:20:09 URL:http://patchwork.ozlabs.org/patch/114742/mbox/
[10772/10772] -> "114742.patch" [1]
$ git am 114742.patch
Applying: cosmetic, post: Codingstyle cleanup
error: patch failed: post/post.c:95
error: post/post.c: patch does not apply
Patch failed at 0001 cosmetic, post: Codingstyle cleanup
When you have resolved this problem run "git am --resolved".
If you would prefer to skip this patch, instead run "git am --skip".
To restore the original branch and stop patching run "git am --abort".

Best regards, Christian

On Wed, Oct 5, 2011 at 7:55 AM, Heiko Schocher <hs@denx.de> wrote:
> Hello Albert,
>
> can you comment/apply the following patches? As the new merge window is
> open, I think they can go to mainline:
>
> arm, logbuffer: make it compileclean
> http://patchwork.ozlabs.org/patch/114740/
>
> post, arm: make post support for arm compile clean
> http://patchwork.ozlabs.org/patch/114741/
>
> cosmetic, post: Codingstyle cleanup
> http://patchwork.ozlabs.org/patch/114742/
>
> arm, post: add missing post_time_ms for arm
> http://patchwork.ozlabs.org/patch/114743/
>
> Thanks!
>
> bye,
> Heiko
> --
> DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>

[U-Boot] arm: comment request

[Stefano Babic]

On 10/11/2011 10:17 PM, Albert ARIBAUD wrote:
> Le 05/10/2011 07:55, Heiko Schocher a ?crit :
>> Hello Albert,
>>
>> can you comment/apply the following patches? As the new merge window is
>> open, I think they can go to mainline:
>>
>> arm, logbuffer: make it compileclean
>> http://patchwork.ozlabs.org/patch/114740/
>>
>> post, arm: make post support for arm compile clean
>> http://patchwork.ozlabs.org/patch/114741/
>>
>> cosmetic, post: Codingstyle cleanup
>> http://patchwork.ozlabs.org/patch/114742/
>>
>> arm, post: add missing post_time_ms for arm
>> http://patchwork.ozlabs.org/patch/114743/
>>
>> Thanks!
>>
>> bye,
>> Heiko
> 
> Hi Heiko,
> 
> Sorry I was unavailable for some time. I'm going through my backlog and 
> will comment on these soon.

Hi Albert,

if you can, can you also take a look at this one ?

http://patchwork.ozlabs.org/patch/114909/

This is a simply move, but it blocks other patches for i.MX.

Thanks,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

[U-Boot] arm: comment request

[Heiko Schocher]

Hello Christian,

Christian Riesch wrote:
> Hello Heiko
> I tried to apply your patches to the current master, since they are a
> prerequisite for your enbw_cmc patches but
> 
> [U-Boot] cosmetic, post: Codingstyle cleanup
> http://patchwork.ozlabs.org/patch/114742/
> 
> did not cleanly apply.

Oh! I just did a rebase to current TOT ... maxbe I missed
something, I check this, thanks!

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

[U-Boot] arm: comment request

[Heiko Schocher]

Hello Christian, Albert,

Christian Riesch wrote:
> Hello Heiko
> I tried to apply your patches to the current master, since they are a
> prerequisite for your enbw_cmc patches but
> 
> [U-Boot] cosmetic, post: Codingstyle cleanup
> http://patchwork.ozlabs.org/patch/114742/
> 
> did not cleanly apply.
> 
> I got it from patchwork via
> $ wget -nv http://patchwork.ozlabs.org/patch/114742/mbox/ -O 114742.patch
> 2011-10-12 10:20:09 URL:http://patchwork.ozlabs.org/patch/114742/mbox/
> [10772/10772] -> "114742.patch" [1]
> $ git am 114742.patch
> Applying: cosmetic, post: Codingstyle cleanup
> error: patch failed: post/post.c:95
> error: post/post.c: patch does not apply
> Patch failed at 0001 cosmetic, post: Codingstyle cleanup
> When you have resolved this problem run "git am --resolved".
> If you would prefer to skip this patch, instead run "git am --skip".
> To restore the original branch and stop patching run "git am --abort".

updated patches through rebase:

[U-Boot,v2] arm, logbuffer: make it compileclean
http://patchwork.ozlabs.org/patch/119183/

[U-Boot,v2] cosmetic, post: Codingstyle cleanup
http://patchwork.ozlabs.org/patch/119184/

patches not changed through rebase:

arm, post: add missing post_time_ms for arm
http://patchwork.ozlabs.org/patch/114743/

no longer needed patch:

post, arm: make post support for arm compile clean
http://patchwork.ozlabs.org/patch/114741/

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

[U-Boot] arm: comment request

[Christian Riesch]

Hello Heiko,
thanks a lot, now they apply fine :-)
Christian

On Wed, Oct 12, 2011 at 1:36 PM, Heiko Schocher <hs@denx.de> wrote:
> Hello Christian, Albert,
>
> Christian Riesch wrote:
>> Hello Heiko
>> I tried to apply your patches to the current master, since they are a
>> prerequisite for your enbw_cmc patches but
>>
>> [U-Boot] cosmetic, post: Codingstyle cleanup
>> http://patchwork.ozlabs.org/patch/114742/
>>
>> did not cleanly apply.
>>
>> I got it from patchwork via
>> $ wget -nv http://patchwork.ozlabs.org/patch/114742/mbox/ -O 114742.patch
>> 2011-10-12 10:20:09 URL:http://patchwork.ozlabs.org/patch/114742/mbox/
>> [10772/10772] -> "114742.patch" [1]
>> $ git am 114742.patch
>> Applying: cosmetic, post: Codingstyle cleanup
>> error: patch failed: post/post.c:95
>> error: post/post.c: patch does not apply
>> Patch failed at 0001 cosmetic, post: Codingstyle cleanup
>> When you have resolved this problem run "git am --resolved".
>> If you would prefer to skip this patch, instead run "git am --skip".
>> To restore the original branch and stop patching run "git am --abort".
>
> updated patches through rebase:
>
> [U-Boot,v2] arm, logbuffer: make it compileclean
> http://patchwork.ozlabs.org/patch/119183/
>
> [U-Boot,v2] cosmetic, post: Codingstyle cleanup
> http://patchwork.ozlabs.org/patch/119184/
>
> patches not changed through rebase:
>
> arm, post: add missing post_time_ms for arm
> http://patchwork.ozlabs.org/patch/114743/
>
> no longer needed patch:
>
> post, arm: make post support for arm compile clean
> http://patchwork.ozlabs.org/patch/114741/
>
> bye,
> Heiko
> --
> DENX Software Engineering GmbH, ? ? MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>

[U-Boot] arm: comment request

[Albert ARIBAUD]

Hi Stefano,

Le 12/10/2011 11:39, Stefano Babic a ?crit :
> On 10/11/2011 10:17 PM, Albert ARIBAUD wrote:
>> Le 05/10/2011 07:55, Heiko Schocher a ?crit :
>>> Hello Albert,
>>>
>>> can you comment/apply the following patches? As the new merge window is
>>> open, I think they can go to mainline:
>>>
>>> arm, logbuffer: make it compileclean
>>> http://patchwork.ozlabs.org/patch/114740/
>>>
>>> post, arm: make post support for arm compile clean
>>> http://patchwork.ozlabs.org/patch/114741/
>>>
>>> cosmetic, post: Codingstyle cleanup
>>> http://patchwork.ozlabs.org/patch/114742/
>>>
>>> arm, post: add missing post_time_ms for arm
>>> http://patchwork.ozlabs.org/patch/114743/
>>>
>>> Thanks!
>>>
>>> bye,
>>> Heiko
>>
>> Hi Heiko,
>>
>> Sorry I was unavailable for some time. I'm going through my backlog and
>> will comment on these soon.
>
> Hi Albert,
>
> if you can, can you also take a look at this one ?
>
> http://patchwork.ozlabs.org/patch/114909/
>
> This is a simply move, but it blocks other patches for i.MX.

Apparently this is not a move. Did you use option -C or -M with git 
format-patch?

> Thanks,
> Stefano

Amicalement,
-- 
Albert.

[U-Boot] arm: comment request

[Albert ARIBAUD]

Hi Heiko,

Le 12/10/2011 13:36, Heiko Schocher a ?crit :
> Hello Christian, Albert,
>
> Christian Riesch wrote:
>> Hello Heiko
>> I tried to apply your patches to the current master, since they are a
>> prerequisite for your enbw_cmc patches but
>>
>> [U-Boot] cosmetic, post: Codingstyle cleanup
>> http://patchwork.ozlabs.org/patch/114742/
>>
>> did not cleanly apply.
>>
>> I got it from patchwork via
>> $ wget -nv http://patchwork.ozlabs.org/patch/114742/mbox/ -O 114742.patch
>> 2011-10-12 10:20:09 URL:http://patchwork.ozlabs.org/patch/114742/mbox/
>> [10772/10772] ->  "114742.patch" [1]
>> $ git am 114742.patch
>> Applying: cosmetic, post: Codingstyle cleanup
>> error: patch failed: post/post.c:95
>> error: post/post.c: patch does not apply
>> Patch failed at 0001 cosmetic, post: Codingstyle cleanup
>> When you have resolved this problem run "git am --resolved".
>> If you would prefer to skip this patch, instead run "git am --skip".
>> To restore the original branch and stop patching run "git am --abort".
>
> updated patches through rebase:
>
> [U-Boot,v2] arm, logbuffer: make it compileclean
> http://patchwork.ozlabs.org/patch/119183/
>
> [U-Boot,v2] cosmetic, post: Codingstyle cleanup
> http://patchwork.ozlabs.org/patch/119184/
>
> patches not changed through rebase:
>
> arm, post: add missing post_time_ms for arm
> http://patchwork.ozlabs.org/patch/114743/

All three patches applied, thanks.

> no longer needed patch:
>
> post, arm: make post support for arm compile clean
> http://patchwork.ozlabs.org/patch/114741/

This patch marked superseded.

> bye,
> Heiko

Amicalement,
-- 
Albert.

[U-Boot] arm: comment request

[Stefano Babic]

On 10/14/2011 09:38 PM, Albert ARIBAUD wrote:

>> Hi Albert,
>>
>> if you can, can you also take a look at this one ?
>>
>> http://patchwork.ozlabs.org/patch/114909/
>>
>> This is a simply move, but it blocks other patches for i.MX.
> 
> Apparently this is not a move. Did you use option -C or -M with git
> format-patch?

I see..I was not aware of the -C/-M switch, thanks for this hint. I have
learned something new...

I repost V2.

Thanks,
Stefano

-- 
=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================