* [PATCH] ACPI / PCI: Stub out pci_acpi_crs_quirks() and make it x86 specific @ 2014-05-06 3:24 Hanjun Guo 2014-05-06 11:28 ` Rafael J. Wysocki 0 siblings, 1 reply; 4+ messages in thread From: Hanjun Guo @ 2014-05-06 3:24 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Arnd Bergmann, Lv Zheng, Tony Luck, Yijing Wang, linux-ia64, linux-acpi, linaro-acpi, Hanjun Guo For pci_acpi_crs_quirks(), ia64 already doesn't use it, and we can not foresee it should be used in ARM64, so stub out pci_acpi_crs_quirks() to avoid introducing platform specific dummy stub function. Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org> --- arch/ia64/include/asm/acpi.h | 1 - include/acpi/acpi_drivers.h | 5 +++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/arch/ia64/include/asm/acpi.h b/arch/ia64/include/asm/acpi.h index d651102..97cb160 100644 --- a/arch/ia64/include/asm/acpi.h +++ b/arch/ia64/include/asm/acpi.h @@ -92,7 +92,6 @@ ia64_acpi_release_global_lock (unsigned int *lock) #endif #define acpi_processor_cstate_check(x) (x) /* no idle limits on IA64 :) */ static inline void disable_acpi(void) { } -static inline void pci_acpi_crs_quirks(void) { } #ifdef CONFIG_IA64_GENERIC const char *acpi_get_sysname (void); diff --git a/include/acpi/acpi_drivers.h b/include/acpi/acpi_drivers.h index d504613..ea6428b 100644 --- a/include/acpi/acpi_drivers.h +++ b/include/acpi/acpi_drivers.h @@ -96,7 +96,12 @@ struct pci_dev *acpi_get_pci_dev(acpi_handle); /* Arch-defined function to add a bus to the system */ struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root); + +#ifdef CONFIG_X86 void pci_acpi_crs_quirks(void); +#else +static inline void pci_acpi_crs_quirks(void) { } +#endif /* -------------------------------------------------------------------------- Processor -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] ACPI / PCI: Stub out pci_acpi_crs_quirks() and make it x86 specific 2014-05-06 3:24 [PATCH] ACPI / PCI: Stub out pci_acpi_crs_quirks() and make it x86 specific Hanjun Guo @ 2014-05-06 11:28 ` Rafael J. Wysocki 2014-05-06 19:24 ` Bjorn Helgaas 0 siblings, 1 reply; 4+ messages in thread From: Rafael J. Wysocki @ 2014-05-06 11:28 UTC (permalink / raw) To: Hanjun Guo, Bjorn Helgaas Cc: Arnd Bergmann, Lv Zheng, Tony Luck, Yijing Wang, linux-ia64, linux-acpi, linaro-acpi On Tuesday, May 06, 2014 11:24:30 AM Hanjun Guo wrote: > For pci_acpi_crs_quirks(), ia64 already doesn't use it, and we can > not foresee it should be used in ARM64, so stub out pci_acpi_crs_quirks() > to avoid introducing platform specific dummy stub function. > > Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org> Bjorn, what do you think? > --- > arch/ia64/include/asm/acpi.h | 1 - > include/acpi/acpi_drivers.h | 5 +++++ > 2 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/arch/ia64/include/asm/acpi.h b/arch/ia64/include/asm/acpi.h > index d651102..97cb160 100644 > --- a/arch/ia64/include/asm/acpi.h > +++ b/arch/ia64/include/asm/acpi.h > @@ -92,7 +92,6 @@ ia64_acpi_release_global_lock (unsigned int *lock) > #endif > #define acpi_processor_cstate_check(x) (x) /* no idle limits on IA64 :) */ > static inline void disable_acpi(void) { } > -static inline void pci_acpi_crs_quirks(void) { } > > #ifdef CONFIG_IA64_GENERIC > const char *acpi_get_sysname (void); > diff --git a/include/acpi/acpi_drivers.h b/include/acpi/acpi_drivers.h > index d504613..ea6428b 100644 > --- a/include/acpi/acpi_drivers.h > +++ b/include/acpi/acpi_drivers.h > @@ -96,7 +96,12 @@ struct pci_dev *acpi_get_pci_dev(acpi_handle); > /* Arch-defined function to add a bus to the system */ > > struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root); > + > +#ifdef CONFIG_X86 > void pci_acpi_crs_quirks(void); > +#else > +static inline void pci_acpi_crs_quirks(void) { } > +#endif > > /* -------------------------------------------------------------------------- > Processor > -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ACPI / PCI: Stub out pci_acpi_crs_quirks() and make it x86 specific 2014-05-06 11:28 ` Rafael J. Wysocki @ 2014-05-06 19:24 ` Bjorn Helgaas 2014-05-06 23:58 ` Rafael J. Wysocki 0 siblings, 1 reply; 4+ messages in thread From: Bjorn Helgaas @ 2014-05-06 19:24 UTC (permalink / raw) To: Rafael J. Wysocki Cc: Hanjun Guo, Arnd Bergmann, Lv Zheng, Tony Luck, Yijing Wang, linux-ia64@vger.kernel.org, linux-acpi@vger.kernel.org, linaro-acpi On Tue, May 6, 2014 at 5:28 AM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote: > On Tuesday, May 06, 2014 11:24:30 AM Hanjun Guo wrote: >> For pci_acpi_crs_quirks(), ia64 already doesn't use it, and we can >> not foresee it should be used in ARM64, so stub out pci_acpi_crs_quirks() >> to avoid introducing platform specific dummy stub function. >> >> Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org> > > Bjorn, what do you think? Looks OK to me. If you want to avoid the ifdef, we could put an empty weak version in pci_root.c. Either way, Reviewed-by: Bjorn Helgaas <bhelgaas@google.com> >> --- >> arch/ia64/include/asm/acpi.h | 1 - >> include/acpi/acpi_drivers.h | 5 +++++ >> 2 files changed, 5 insertions(+), 1 deletion(-) >> >> diff --git a/arch/ia64/include/asm/acpi.h b/arch/ia64/include/asm/acpi.h >> index d651102..97cb160 100644 >> --- a/arch/ia64/include/asm/acpi.h >> +++ b/arch/ia64/include/asm/acpi.h >> @@ -92,7 +92,6 @@ ia64_acpi_release_global_lock (unsigned int *lock) >> #endif >> #define acpi_processor_cstate_check(x) (x) /* no idle limits on IA64 :) */ >> static inline void disable_acpi(void) { } >> -static inline void pci_acpi_crs_quirks(void) { } >> >> #ifdef CONFIG_IA64_GENERIC >> const char *acpi_get_sysname (void); >> diff --git a/include/acpi/acpi_drivers.h b/include/acpi/acpi_drivers.h >> index d504613..ea6428b 100644 >> --- a/include/acpi/acpi_drivers.h >> +++ b/include/acpi/acpi_drivers.h >> @@ -96,7 +96,12 @@ struct pci_dev *acpi_get_pci_dev(acpi_handle); >> /* Arch-defined function to add a bus to the system */ >> >> struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root); >> + >> +#ifdef CONFIG_X86 >> void pci_acpi_crs_quirks(void); >> +#else >> +static inline void pci_acpi_crs_quirks(void) { } >> +#endif >> >> /* -------------------------------------------------------------------------- >> Processor >> > > -- > I speak only for myself. > Rafael J. Wysocki, Intel Open Source Technology Center. ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] ACPI / PCI: Stub out pci_acpi_crs_quirks() and make it x86 specific 2014-05-06 19:24 ` Bjorn Helgaas @ 2014-05-06 23:58 ` Rafael J. Wysocki 0 siblings, 0 replies; 4+ messages in thread From: Rafael J. Wysocki @ 2014-05-06 23:58 UTC (permalink / raw) To: Bjorn Helgaas Cc: Hanjun Guo, Arnd Bergmann, Lv Zheng, Tony Luck, Yijing Wang, linux-ia64@vger.kernel.org, linux-acpi@vger.kernel.org, linaro-acpi On Tuesday, May 06, 2014 01:24:38 PM Bjorn Helgaas wrote: > On Tue, May 6, 2014 at 5:28 AM, Rafael J. Wysocki <rjw@rjwysocki.net> wrote: > > On Tuesday, May 06, 2014 11:24:30 AM Hanjun Guo wrote: > >> For pci_acpi_crs_quirks(), ia64 already doesn't use it, and we can > >> not foresee it should be used in ARM64, so stub out pci_acpi_crs_quirks() > >> to avoid introducing platform specific dummy stub function. > >> > >> Signed-off-by: Hanjun Guo <hanjun.guo@linaro.org> > > > > Bjorn, what do you think? > > Looks OK to me. If you want to avoid the ifdef, we could put an empty > weak version in pci_root.c. Either way, > > Reviewed-by: Bjorn Helgaas <bhelgaas@google.com> OK, I've queued this up for 3.16. Thanks! > >> --- > >> arch/ia64/include/asm/acpi.h | 1 - > >> include/acpi/acpi_drivers.h | 5 +++++ > >> 2 files changed, 5 insertions(+), 1 deletion(-) > >> > >> diff --git a/arch/ia64/include/asm/acpi.h b/arch/ia64/include/asm/acpi.h > >> index d651102..97cb160 100644 > >> --- a/arch/ia64/include/asm/acpi.h > >> +++ b/arch/ia64/include/asm/acpi.h > >> @@ -92,7 +92,6 @@ ia64_acpi_release_global_lock (unsigned int *lock) > >> #endif > >> #define acpi_processor_cstate_check(x) (x) /* no idle limits on IA64 :) */ > >> static inline void disable_acpi(void) { } > >> -static inline void pci_acpi_crs_quirks(void) { } > >> > >> #ifdef CONFIG_IA64_GENERIC > >> const char *acpi_get_sysname (void); > >> diff --git a/include/acpi/acpi_drivers.h b/include/acpi/acpi_drivers.h > >> index d504613..ea6428b 100644 > >> --- a/include/acpi/acpi_drivers.h > >> +++ b/include/acpi/acpi_drivers.h > >> @@ -96,7 +96,12 @@ struct pci_dev *acpi_get_pci_dev(acpi_handle); > >> /* Arch-defined function to add a bus to the system */ > >> > >> struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root); > >> + > >> +#ifdef CONFIG_X86 > >> void pci_acpi_crs_quirks(void); > >> +#else > >> +static inline void pci_acpi_crs_quirks(void) { } > >> +#endif > >> > >> /* -------------------------------------------------------------------------- > >> Processor > >> > > > > -- > > I speak only for myself. > > Rafael J. Wysocki, Intel Open Source Technology Center. -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-05-06 23:58 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-05-06 3:24 [PATCH] ACPI / PCI: Stub out pci_acpi_crs_quirks() and make it x86 specific Hanjun Guo 2014-05-06 11:28 ` Rafael J. Wysocki 2014-05-06 19:24 ` Bjorn Helgaas 2014-05-06 23:58 ` Rafael J. Wysocki
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox