On Mon, Apr 29, 2024 at 10:58 AM Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> 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 <andriy.shevchenko@xxxxxxxxxxxxxxx> 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!
