Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Hanjun Guo
On 2016/11/30 0:26, Rafael J. Wysocki wrote: > On Tue, Nov 29, 2016 at 4:20 PM, Agustin Vega-Frias > wrote: >> Hi Rafael, >> >> >> On 2016-11-29 07:43, Rafael J. Wysocki wrote: >>> On Tue, Nov 29, 2016 at 1:11 PM, Lorenzo Pieralisi >>> wrote:

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Hanjun Guo
On 2016/11/30 0:26, Rafael J. Wysocki wrote: > On Tue, Nov 29, 2016 at 4:20 PM, Agustin Vega-Frias > wrote: >> Hi Rafael, >> >> >> On 2016-11-29 07:43, Rafael J. Wysocki wrote: >>> On Tue, Nov 29, 2016 at 1:11 PM, Lorenzo Pieralisi >>> wrote: Hi Agustin, On Mon, Nov 28, 2016 at

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Rafael J. Wysocki
On Tue, Nov 29, 2016 at 4:20 PM, Agustin Vega-Frias wrote: > Hi Rafael, > > > On 2016-11-29 07:43, Rafael J. Wysocki wrote: >> >> On Tue, Nov 29, 2016 at 1:11 PM, Lorenzo Pieralisi >> wrote: >>> >>> Hi Agustin, >>> >>> On Mon, Nov 28, 2016 at

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Rafael J. Wysocki
On Tue, Nov 29, 2016 at 4:20 PM, Agustin Vega-Frias wrote: > Hi Rafael, > > > On 2016-11-29 07:43, Rafael J. Wysocki wrote: >> >> On Tue, Nov 29, 2016 at 1:11 PM, Lorenzo Pieralisi >> wrote: >>> >>> Hi Agustin, >>> >>> On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote:

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Agustin Vega-Frias
Hi Rafael, On 2016-11-29 07:43, Rafael J. Wysocki wrote: On Tue, Nov 29, 2016 at 1:11 PM, Lorenzo Pieralisi wrote: Hi Agustin, On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote: Hi Rafael, Can you chime in on Lorenzo's feedback and the discussion

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Agustin Vega-Frias
Hi Rafael, On 2016-11-29 07:43, Rafael J. Wysocki wrote: On Tue, Nov 29, 2016 at 1:11 PM, Lorenzo Pieralisi wrote: Hi Agustin, On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote: Hi Rafael, Can you chime in on Lorenzo's feedback and the discussion below? It would be great

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Agustin Vega-Frias
Hi Lorenzo, On 2016-11-29 07:11, Lorenzo Pieralisi wrote: Hi Agustin, On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote: Hi Rafael, Can you chime in on Lorenzo's feedback and the discussion below? It would be great if you can comment on the reason ACPI does things in a

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Agustin Vega-Frias
Hi Lorenzo, On 2016-11-29 07:11, Lorenzo Pieralisi wrote: Hi Agustin, On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote: Hi Rafael, Can you chime in on Lorenzo's feedback and the discussion below? It would be great if you can comment on the reason ACPI does things in a

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Rafael J. Wysocki
On Tue, Nov 29, 2016 at 1:11 PM, Lorenzo Pieralisi wrote: > Hi Agustin, > > On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote: >> Hi Rafael, >> >> Can you chime in on Lorenzo's feedback and the discussion below? >> It would be great if you can comment

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Rafael J. Wysocki
On Tue, Nov 29, 2016 at 1:11 PM, Lorenzo Pieralisi wrote: > Hi Agustin, > > On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote: >> Hi Rafael, >> >> Can you chime in on Lorenzo's feedback and the discussion below? >> It would be great if you can comment on the reason ACPI does

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Lorenzo Pieralisi
Hi Agustin, On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote: > Hi Rafael, > > Can you chime in on Lorenzo's feedback and the discussion below? > It would be great if you can comment on the reason ACPI does things > in a certain way. > > Hi Lorenzo, > > On 2016-11-25 06:40,

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-29 Thread Lorenzo Pieralisi
Hi Agustin, On Mon, Nov 28, 2016 at 05:40:24PM -0500, Agustin Vega-Frias wrote: > Hi Rafael, > > Can you chime in on Lorenzo's feedback and the discussion below? > It would be great if you can comment on the reason ACPI does things > in a certain way. > > Hi Lorenzo, > > On 2016-11-25 06:40,

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-28 Thread Agustin Vega-Frias
Hi Rafael, Can you chime in on Lorenzo's feedback and the discussion below? It would be great if you can comment on the reason ACPI does things in a certain way. Hi Lorenzo, On 2016-11-25 06:40, Lorenzo Pieralisi wrote: Hi Agustin, On Thu, Nov 24, 2016 at 04:15:48PM +, Lorenzo Pieralisi

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-28 Thread Agustin Vega-Frias
Hi Rafael, Can you chime in on Lorenzo's feedback and the discussion below? It would be great if you can comment on the reason ACPI does things in a certain way. Hi Lorenzo, On 2016-11-25 06:40, Lorenzo Pieralisi wrote: Hi Agustin, On Thu, Nov 24, 2016 at 04:15:48PM +, Lorenzo Pieralisi

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-25 Thread Lorenzo Pieralisi
Hi Agustin, On Thu, Nov 24, 2016 at 04:15:48PM +, Lorenzo Pieralisi wrote: [...] > > @@ -448,6 +449,7 @@ bool acpi_dev_resource_interrupt(struct acpi_resource > > *ares, int index, > > { > > struct acpi_resource_irq *irq; > > struct acpi_resource_extended_irq *ext_irq; > > +

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-25 Thread Lorenzo Pieralisi
Hi Agustin, On Thu, Nov 24, 2016 at 04:15:48PM +, Lorenzo Pieralisi wrote: [...] > > @@ -448,6 +449,7 @@ bool acpi_dev_resource_interrupt(struct acpi_resource > > *ares, int index, > > { > > struct acpi_resource_irq *irq; > > struct acpi_resource_extended_irq *ext_irq; > > +

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-24 Thread Lorenzo Pieralisi
Hi Agustin, On Sun, Nov 13, 2016 at 04:59:34PM -0500, Agustin Vega-Frias wrote: > When an Extended IRQ Resource contains a valid ResourceSource > use it to map the IRQ on the domain associated with the ACPI > device referenced. > > With this in place an irqchip driver can create its domain using

Re: [PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-24 Thread Lorenzo Pieralisi
Hi Agustin, On Sun, Nov 13, 2016 at 04:59:34PM -0500, Agustin Vega-Frias wrote: > When an Extended IRQ Resource contains a valid ResourceSource > use it to map the IRQ on the domain associated with the ACPI > device referenced. > > With this in place an irqchip driver can create its domain using

[PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-13 Thread Agustin Vega-Frias
When an Extended IRQ Resource contains a valid ResourceSource use it to map the IRQ on the domain associated with the ACPI device referenced. With this in place an irqchip driver can create its domain using irq_domain_create_linear and pass the device fwnode to create the domain mapping. When

[PATCH V7 2/3] ACPI: Add support for ResourceSource/IRQ domain mapping

2016-11-13 Thread Agustin Vega-Frias
When an Extended IRQ Resource contains a valid ResourceSource use it to map the IRQ on the domain associated with the ACPI device referenced. With this in place an irqchip driver can create its domain using irq_domain_create_linear and pass the device fwnode to create the domain mapping. When