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

Re: [Xen-devel] [PATCH V1 2/2] xen/device-tree: Add ability to handle nodes with interrupts-extended prop



Hi,

On 5/29/19 7:18 PM, Julien Grall wrote:


On 29/05/2019 18:58, Oleksandr wrote:

On 29.05.19 20:44, Julien Grall wrote:
Hi Oleksandr,

Hi, Julien



On 21/05/2019 18:37, Oleksandr Tyshchenko wrote:
From: Oleksandr Tyshchenko <oleksandr_tyshchenko@xxxxxxxx>

The "interrupts-extended" property is a special form for use when
a node needs to reference multiple interrupt parents. >
According to the:

NIT: s/the//

Linux/Documentation/devicetree/bindings/interrupt-controller/interrupts.txt

But, there are cases when "interrupts-extended" property is used for
"outside /soc node" with a single interrupt parent as an equivalent of
pairs ("interrupt-parent" + "interrupts").

The good example here is ARCH timer node for R-Car Gen3/Gen2 family,

NIT: s/The/A/ I think

which is mandatory device for Xen usage on ARM. And without ability
to handle such nodes, Xen fails to operate.

So, this patch adds required support for Xen to be able to handle
nodes with that property.

Signed-off-by: Oleksandr Tyshchenko <oleksandr_tyshchenko@xxxxxxxx>

Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>

Thank you! I assume you meant Reviewed-by: Julien Grall <julien.grall@xxxxxxx> )

Yes sorry, I was adding is reviewed-by on some of my patches and mechanically copied here. Apologies for the inconvenience.


Shall I send new version with proposed changes?

No need, I can do the change while committing (assuming Stefano has no other comments).

Now applied to my staging branch. It will be committed tonight.

Thank you for the patches.

Cheers,

--
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

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