Re: [Xen-devel] [PATCH] xen/arm: pass a struct pending_irq* as parameter to gic helper functions

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Tue, 2014-01-07 at 12:35 +0000, Stefano Stabellini wrote:
> On Tue, 7 Jan 2014, Ian Campbell wrote:
> > On Thu, 2013-12-19 at 13:50 +0000, Ian Campbell wrote:
> > > On Tue, 2013-12-17 at 16:16 +0000, Stefano Stabellini wrote:
> > > > gic_add_to_lr_pending and gic_set_lr should take a struct pending_irq*
> > > > as parameter instead of the virtual_irq number and the priority
> > > > separately and doing yet another irq_to_pending lookup.
> > > > 
> > > > Signed-off-by: Stefano Stabellini <stefano.stabellini@xxxxxxxxxxxxx>
> > > 
> > > Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> > > 
> > > What do you think about this one for 4.4? It whiffs a bit of a cleanup
> > > rather than a bug fix (unless e.g. there is some subtle incorrectness in
> > > the priority which was getting passed around before, rather than just
> > > the potential for it)
> > 
> > Unless there is an actual bug fixed by this I think I'll leave it for
> > 4.5
> 
> That's fine.

Now applied.

Thanks.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel