| Message ID | 1382473641-20614-1-git-send-email-a.heider@gmail.com |
|---|---|
| State | Accepted |
| Delegated to: | Anatolij Gustschin |
| Headers | show |
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 <swarren@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?
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 <swarren@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
Hi Andre, On Wed, 23 Oct 2013 21:46:31 +0200, Andre Heider <a.heider@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 <swarren@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,
On 11/07/2013 06:29 AM, Albert ARIBAUD wrote: > Hi Andre, > > On Wed, 23 Oct 2013 21:46:31 +0200, Andre Heider <a.heider@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 <swarren@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?
Hi Andre, On Tue, 22 Oct 2013 22:27:20 +0200, Andre Heider <a.heider@gmail.com> wrote: > Add the missing "right" field to struct bcm2835_mbox_tag_overscan. > > Signed-off-by: Andre Heider <a.heider@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.boot@aribaud.net> Amicalement,
Hi Stephen, On Thu, 07 Nov 2013 14:30:54 -0700, Stephen Warren <swarren@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,
On Tue, 22 Oct 2013 22:27:20 +0200 Andre Heider <a.heider@gmail.com> wrote: > Add the missing "right" field to struct bcm2835_mbox_tag_overscan. > > Signed-off-by: Andre Heider <a.heider@gmail.com> > --- > arch/arm/include/asm/arch-bcm2835/mbox.h | 1 + > 1 file changed, 1 insertion(+) applied to u-boot-video/master, thanks! Anatolij
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; };
Add the missing "right" field to struct bcm2835_mbox_tag_overscan. Signed-off-by: Andre Heider <a.heider@gmail.com> --- arch/arm/include/asm/arch-bcm2835/mbox.h | 1 + 1 file changed, 1 insertion(+)