Re: [PATCH v10 6/6] ACPI/IORT: Stub out ACS functions when CONFIG_PCI is not set

2018-12-17 Thread Sinan Kaya
On 12/18/2018 3:14 AM, Hanjun Guo wrote: I prefer stub function for iort_enable_acs(), not adding #ifdef/#endif pair inside this function. Yes, I'm reworking this per input from Lorenzo. By the way, there are other pci function called in iort.c, could you explain a little bit why no need

Re: [PATCH v10 6/6] ACPI/IORT: Stub out ACS functions when CONFIG_PCI is not set

2018-12-17 Thread Hanjun Guo
Hi Sinan, On 2018/12/15 9:02, Sinan Kaya wrote: > Remove PCI dependent code out of iort.c when CONFIG_PCI is not defined. > > Signed-off-by: Sinan Kaya > --- > drivers/acpi/arm64/iort.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/acpi/arm64/iort.c

Re: [PATCH v10 6/6] ACPI/IORT: Stub out ACS functions when CONFIG_PCI is not set

2018-12-17 Thread Lorenzo Pieralisi
On Sat, Dec 15, 2018 at 01:02:47AM +, Sinan Kaya wrote: > Remove PCI dependent code out of iort.c when CONFIG_PCI is not defined. > > Signed-off-by: Sinan Kaya > --- > drivers/acpi/arm64/iort.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/acpi/arm64/iort.c

Re: [PATCH v10 6/6] ACPI/IORT: Stub out ACS functions when CONFIG_PCI is not set

2018-12-17 Thread Rafael J. Wysocki
On Saturday, December 15, 2018 2:02:47 AM CET Sinan Kaya wrote: > Remove PCI dependent code out of iort.c when CONFIG_PCI is not defined. > > Signed-off-by: Sinan Kaya > --- > drivers/acpi/arm64/iort.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/acpi/arm64/iort.c

[PATCH v10 6/6] ACPI/IORT: Stub out ACS functions when CONFIG_PCI is not set

2018-12-14 Thread Sinan Kaya
Remove PCI dependent code out of iort.c when CONFIG_PCI is not defined. Signed-off-by: Sinan Kaya --- drivers/acpi/arm64/iort.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/acpi/arm64/iort.c b/drivers/acpi/arm64/iort.c index 70f4e80b9246..d0f68607efe6 100644 ---