The function acpi_blacklisted() is defined only when CONFIG_X86 is
enabled and is only used by X86 arch code. To align with its usage and
definition conditions, move its declaration to asm/acpi.h
Signed-off-by: Kuppuswamy Sathyanarayanan <[email protected]>
---
Changes since v1:
- v1 link: https://lore.kernel.org/lkml/[email protected]/T/
- Changed the title to "ACPI: Move acpi_blacklisted() declaration to asm/acpi.h"
- Removed the declaration from include/linux/acpi.h
arch/x86/include/asm/acpi.h | 2 ++
include/linux/acpi.h | 1 -
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/x86/include/asm/acpi.h b/arch/x86/include/asm/acpi.h
index f896eed4516c..5af926c050f0 100644
--- a/arch/x86/include/asm/acpi.h
+++ b/arch/x86/include/asm/acpi.h
@@ -56,6 +56,8 @@ static inline void disable_acpi(void)
extern int acpi_gsi_to_irq(u32 gsi, unsigned int *irq);
+extern int acpi_blacklisted(void);
+
static inline void acpi_noirq_set(void) { acpi_noirq = 1; }
static inline void acpi_disable_pci(void)
{
diff --git a/include/linux/acpi.h b/include/linux/acpi.h
index 34829f2c517a..576e75b4529a 100644
--- a/include/linux/acpi.h
+++ b/include/linux/acpi.h
@@ -421,7 +421,6 @@ extern char *wmi_get_acpi_device_uid(const char *guid);
extern char acpi_video_backlight_string[];
extern long acpi_is_video_device(acpi_handle handle);
-extern int acpi_blacklisted(void);
extern void acpi_osi_setup(char *str);
extern bool acpi_osi_is_win8(void);
--
2.34.1
On Sun, Apr 28, 2024 at 09:04:41PM -0700, Kuppuswamy Sathyanarayanan wrote:
> The function acpi_blacklisted() is defined only when CONFIG_X86 is
> enabled and is only used by X86 arch code. To align with its usage and
> definition conditions, move its declaration to asm/acpi.h
FWIW,
Reviewed-by: Andy Shevchenko <[email protected]>
..
> extern char acpi_video_backlight_string[];
> extern long acpi_is_video_device(acpi_handle handle);
> -extern int acpi_blacklisted(void);
I would replace it with a blank line (to me it seems the above and below are
different groups from semantic point of view, but Rafael may correct me).
> extern void acpi_osi_setup(char *str);
> extern bool acpi_osi_is_win8(void);
--
With Best Regards,
Andy Shevchenko
On Mon, Apr 29, 2024 at 10:58 AM Andy Shevchenko
<[email protected]> wrote:
>
> On Sun, Apr 28, 2024 at 09:04:41PM -0700, Kuppuswamy Sathyanarayanan wrote:
> > The function acpi_blacklisted() is defined only when CONFIG_X86 is
> > enabled and is only used by X86 arch code. To align with its usage and
> > definition conditions, move its declaration to asm/acpi.h
>
> FWIW,
> Reviewed-by: Andy Shevchenko <[email protected]>
I have replaced the v1 that was applied some time ago with this one.
> ...
>
> > extern char acpi_video_backlight_string[];
> > extern long acpi_is_video_device(acpi_handle handle);
> > -extern int acpi_blacklisted(void);
>
> I would replace it with a blank line (to me it seems the above and below are
> different groups from semantic point of view, but Rafael may correct me).
And I have added the blank line as suggested.
> > extern void acpi_osi_setup(char *str);
> > extern bool acpi_osi_is_win8(void);
>
> --
Thanks!