[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v6 01/31] xen/dt: Handle correctly node with interrupt-map in dt_for_each_irq_map



On Wed, 2015-09-02 at 16:45 +0100, Julien Grall wrote:
> On 02/09/15 16:28, Ian Campbell wrote:
> > On Mon, 2015-08-31 at 15:20 +0100, Julien Grall wrote:
> > > Hi Vijay,
> > > 
> > > On 31/08/2015 12:06, vijay.kilari@xxxxxxxxx wrote:
> > > > From: Vijaya Kumar K <Vijaya.Kumar@xxxxxxxxxxxxxxxxxx>
> > > > 
> > > > dt_for_each_irq_map() returns error if no irq mapping is found.
> > > > With this patch, Ignore error and return success
> > > 
> > > NIT: s/Ignore/ignore/
> > > 
> > > > 
> > > > Signed-off-by: Vijaya Kumar K <Vijaya.Kumar@xxxxxxxxxxxxxxxxxx>
> > > 
> > > I think this could go in Xen 4.6 as it's a bug fix:
> > > 
> > > Reviewed-by: Julien Grall <julien.grall@xxxxxxxxxx>
> > 
> > I replied to v5, forgetting I had a v6 in my hand too. I also forgot to 
> > CC
> > Wei. Lets try again here...
> > 
> > Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> > 
> > Wei -- I think this fix would be good to have for 4.6. It is not an 
> > error
> > to process a zero-length (==non-existent) array I think.
> > 
> > I think the title should be s/with/without/, Julien was that what you 
> > meant
> > when you suggested it?
> 
> Yes.

I will adjust on commit (tomorrow since I'm about to go home).


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.