Re: [PATCH v7 3/4] gicv2m: Refactor to prepare for ACPI support

2015-12-22 Thread Suravee Suthikulanit
On 12/17/2015 10:57 AM, Bjorn Helgaas wrote: On Wed, Dec 16, 2015 at 06:23:49PM -0600, Suravee Suthikulanit wrote: Hi Bjorn, Thanks for your review. Please see my comments below. On 12/16/2015 4:12 PM, Bjorn Helgaas wrote: On Thu, Dec 10, 2015 at 08:55:29AM -0800, Suravee Suthikulpanit

Re: [PATCH v7 3/4] gicv2m: Refactor to prepare for ACPI support

2015-12-22 Thread Suravee Suthikulanit
On 12/17/2015 10:57 AM, Bjorn Helgaas wrote: On Wed, Dec 16, 2015 at 06:23:49PM -0600, Suravee Suthikulanit wrote: Hi Bjorn, Thanks for your review. Please see my comments below. On 12/16/2015 4:12 PM, Bjorn Helgaas wrote: On Thu, Dec 10, 2015 at 08:55:29AM -0800, Suravee Suthikulpanit

Re: [PATCH v7 3/4] gicv2m: Refactor to prepare for ACPI support

2015-12-17 Thread Bjorn Helgaas
On Wed, Dec 16, 2015 at 06:23:49PM -0600, Suravee Suthikulanit wrote: > Hi Bjorn, > > Thanks for your review. Please see my comments below. > > On 12/16/2015 4:12 PM, Bjorn Helgaas wrote: > >On Thu, Dec 10, 2015 at 08:55:29AM -0800, Suravee Suthikulpanit wrote: > >>This patch replaces the struct

Re: [PATCH v7 3/4] gicv2m: Refactor to prepare for ACPI support

2015-12-17 Thread Bjorn Helgaas
On Wed, Dec 16, 2015 at 06:23:49PM -0600, Suravee Suthikulanit wrote: > Hi Bjorn, > > Thanks for your review. Please see my comments below. > > On 12/16/2015 4:12 PM, Bjorn Helgaas wrote: > >On Thu, Dec 10, 2015 at 08:55:29AM -0800, Suravee Suthikulpanit wrote: > >>This patch replaces the struct

Re: [PATCH v7 3/4] gicv2m: Refactor to prepare for ACPI support

2015-12-16 Thread Suravee Suthikulanit
Hi Bjorn, Thanks for your review. Please see my comments below. On 12/16/2015 4:12 PM, Bjorn Helgaas wrote: On Thu, Dec 10, 2015 at 08:55:29AM -0800, Suravee Suthikulpanit wrote: This patch replaces the struct device_node with struct fwnode_handle since this structure is common between DT and

Re: [PATCH v7 3/4] gicv2m: Refactor to prepare for ACPI support

2015-12-16 Thread Bjorn Helgaas
On Thu, Dec 10, 2015 at 08:55:29AM -0800, Suravee Suthikulpanit wrote: > This patch replaces the struct device_node with struct fwnode_handle > since this structure is common between DT and ACPI. > > It also refactors gicv2m_init_one() to prepare for ACPI support. > The only functional change is

Re: [PATCH v7 3/4] gicv2m: Refactor to prepare for ACPI support

2015-12-16 Thread Bjorn Helgaas
On Thu, Dec 10, 2015 at 08:55:29AM -0800, Suravee Suthikulpanit wrote: > This patch replaces the struct device_node with struct fwnode_handle > since this structure is common between DT and ACPI. > > It also refactors gicv2m_init_one() to prepare for ACPI support. > The only functional change is

Re: [PATCH v7 3/4] gicv2m: Refactor to prepare for ACPI support

2015-12-16 Thread Suravee Suthikulanit
Hi Bjorn, Thanks for your review. Please see my comments below. On 12/16/2015 4:12 PM, Bjorn Helgaas wrote: On Thu, Dec 10, 2015 at 08:55:29AM -0800, Suravee Suthikulpanit wrote: This patch replaces the struct device_node with struct fwnode_handle since this structure is common between DT and

[PATCH v7 3/4] gicv2m: Refactor to prepare for ACPI support

2015-12-10 Thread Suravee Suthikulpanit
This patch replaces the struct device_node with struct fwnode_handle since this structure is common between DT and ACPI. It also refactors gicv2m_init_one() to prepare for ACPI support. The only functional change is removing the node name from pr_info. Reviewed-by: Marc Zyngier Signed-off-by:

[PATCH v7 3/4] gicv2m: Refactor to prepare for ACPI support

2015-12-10 Thread Suravee Suthikulpanit
This patch replaces the struct device_node with struct fwnode_handle since this structure is common between DT and ACPI. It also refactors gicv2m_init_one() to prepare for ACPI support. The only functional change is removing the node name from pr_info. Reviewed-by: Marc Zyngier