Re: [U-Boot] [PATCH 1/2] ARM: bcm2835: add missing mbox overscan response field
On Tue, 22 Oct 2013 22:27:20 +0200 Andre Heider a.hei...@gmail.com wrote: Add the missing right field to struct bcm2835_mbox_tag_overscan. Signed-off-by: Andre Heider a.hei...@gmail.com --- arch/arm/include/asm/arch-bcm2835/mbox.h | 1 + 1 file changed, 1 insertion(+) applied to u-boot-video/master, thanks! Anatolij ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/2] ARM: bcm2835: add missing mbox overscan response field
Hi Andre, On Tue, 22 Oct 2013 22:27:20 +0200, Andre Heider a.hei...@gmail.com wrote: Add the missing right field to struct bcm2835_mbox_tag_overscan. Signed-off-by: Andre Heider a.hei...@gmail.com --- arch/arm/include/asm/arch-bcm2835/mbox.h | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/include/asm/arch-bcm2835/mbox.h b/arch/arm/include/asm/arch-bcm2835/mbox.h index 24abe57..6b806ec 100644 --- a/arch/arm/include/asm/arch-bcm2835/mbox.h +++ b/arch/arm/include/asm/arch-bcm2835/mbox.h @@ -350,6 +350,7 @@ struct bcm2835_mbox_tag_overscan { u32 top; u32 bottom; u32 left; + u32 right; } resp; } body; }; Acked-by: Albert ARIBAUD albert.u.b...@aribaud.net Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/2] ARM: bcm2835: add missing mbox overscan response field
Hi Stephen, On Thu, 07 Nov 2013 14:30:54 -0700, Stephen Warren swar...@wwwdotorg.org wrote: So, there are 2 patches and 2 custodians. One of you has to ack the patch for the other to merge them both through the other's subsystem, right? I guess you want Anatolij to take both of these through the video tree based on your comments. If so, can you ack this patch and re-assign them both in patchwork, so he knows that? Done (although I suspect that in this instance, my Ack was not really needed, as the change was quite trivial and did not require any ARM- specific knowledge). Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/2] ARM: bcm2835: add missing mbox overscan response field
Hi Andre, On Wed, 23 Oct 2013 21:46:31 +0200, Andre Heider a.hei...@gmail.com wrote: On Wed, Oct 23, 2013 at 05:54:13PM +0100, Stephen Warren wrote: On 10/22/2013 09:27 PM, Andre Heider wrote: Add the missing right field to struct bcm2835_mbox_tag_overscan. This one patch, Acked-by: Stephen Warren swar...@wwwdotorg.org You need to send/cc this patch to Albert Aribauld, since he applies ARM patches. Or, perhaps just make sure the patch gets assigned to him in patchwork? Okay, I did the patchwork dance :) Thanks, Andre In fact, the series could be applied as a whole by the same custodian -- and actually, I'd prefer it to be applied as a whole, since patch 1/2 alone looks like dead/useless code, whereas it makes sense if applied along with 2/2. Amicalement, -- Albert. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/2] ARM: bcm2835: add missing mbox overscan response field
On 11/07/2013 06:29 AM, Albert ARIBAUD wrote: Hi Andre, On Wed, 23 Oct 2013 21:46:31 +0200, Andre Heider a.hei...@gmail.com wrote: On Wed, Oct 23, 2013 at 05:54:13PM +0100, Stephen Warren wrote: On 10/22/2013 09:27 PM, Andre Heider wrote: Add the missing right field to struct bcm2835_mbox_tag_overscan. This one patch, Acked-by: Stephen Warren swar...@wwwdotorg.org You need to send/cc this patch to Albert Aribauld, since he applies ARM patches. Or, perhaps just make sure the patch gets assigned to him in patchwork? Okay, I did the patchwork dance :) Thanks, Andre In fact, the series could be applied as a whole by the same custodian -- and actually, I'd prefer it to be applied as a whole, since patch 1/2 alone looks like dead/useless code, whereas it makes sense if applied along with 2/2. So, there are 2 patches and 2 custodians. One of you has to ack the patch for the other to merge them both through the other's subsystem, right? I guess you want Anatolij to take both of these through the video tree based on your comments. If so, can you ack this patch and re-assign them both in patchwork, so he knows that? ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/2] ARM: bcm2835: add missing mbox overscan response field
On 10/22/2013 09:27 PM, Andre Heider wrote: Add the missing right field to struct bcm2835_mbox_tag_overscan. This one patch, Acked-by: Stephen Warren swar...@wwwdotorg.org You need to send/cc this patch to Albert Aribauld, since he applies ARM patches. Or, perhaps just make sure the patch gets assigned to him in patchwork? ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH 1/2] ARM: bcm2835: add missing mbox overscan response field
On Wed, Oct 23, 2013 at 05:54:13PM +0100, Stephen Warren wrote: On 10/22/2013 09:27 PM, Andre Heider wrote: Add the missing right field to struct bcm2835_mbox_tag_overscan. This one patch, Acked-by: Stephen Warren swar...@wwwdotorg.org You need to send/cc this patch to Albert Aribauld, since he applies ARM patches. Or, perhaps just make sure the patch gets assigned to him in patchwork? Okay, I did the patchwork dance :) Thanks, Andre ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH 1/2] ARM: bcm2835: add missing mbox overscan response field
Add the missing right field to struct bcm2835_mbox_tag_overscan. Signed-off-by: Andre Heider a.hei...@gmail.com --- arch/arm/include/asm/arch-bcm2835/mbox.h | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/include/asm/arch-bcm2835/mbox.h b/arch/arm/include/asm/arch-bcm2835/mbox.h index 24abe57..6b806ec 100644 --- a/arch/arm/include/asm/arch-bcm2835/mbox.h +++ b/arch/arm/include/asm/arch-bcm2835/mbox.h @@ -350,6 +350,7 @@ struct bcm2835_mbox_tag_overscan { u32 top; u32 bottom; u32 left; + u32 right; } resp; } body; }; -- 1.8.3.2 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot