[U-Boot] arm: comment request

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

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,

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

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,

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

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@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

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

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

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@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

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,
participants (4)
-
Albert ARIBAUD
-
Christian Riesch
-
Heiko Schocher
-
Stefano Babic