On Mon, Apr 08, 2024 at 08:42:48AM -0700, Kuppuswamy Sathyanarayanan wrote: > On Mon, Apr 8, 2024 at 7:53 AM Andy Shevchenko > <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote: > > On Sat, Apr 06, 2024 at 11:33:41PM -0700, Kuppuswamy Sathyanarayanan wrote: ... > > > extern char acpi_video_backlight_string[]; > > > extern long acpi_is_video_device(acpi_handle handle); > > > +#ifdef CONFIG_X86 > > > extern int acpi_blacklisted(void); > > > +#endif > > > extern void acpi_osi_setup(char *str); > > > extern bool acpi_osi_is_win8(void); > > > > IIRC there is already similar ifdeffery, can we just move the declaration > > there? > > There is none for CONFIG_X86. We only have some combinations or > derived config checks like below: > > #if defined(CONFIG_X86) || defined(CONFIG_LOONGARCH) > #ifdef CONFIG_X86_IO_APIC Okay, it seems I mixed this with acpi_bus.h. -- With Best Regards, Andy Shevchenko