On Wed, 25 Mar 2026 11:59:12 +0100, Thierry Reding wrote:

On Tue, Mar 24, 2026 at 10:52:15PM +0530, Akhil R wrote:

quoted

On Tue, 24 Mar 2026 09:43:27 +0100, Alexandre Belloni wrote:

...

quoted

quoted

#include <linux/acpi.h> is added in PATCH 03/12. The functions' prototypes
are present in acpi.h. I think the bot checked this patch individually,
or did I miss something?

#include <acpi/acpi_bus.h> is behind an #ifdef in acpi.h and your code
is not.

Thanks for pointing Alexandre and Guenter. I also noticed that we do not
have stub functions for a few of the acpi_* functions in #else.

Looks like I will have to guard calls to these functions under
#ifdef CONFIG_ACPI.

Alternatively it might make sense to add the stubs in a separate patch.
I don't know if they were purposefully left out or nobody's ever run
into the lack of these before.

I looked into this and it turns out to be more involved than expected,
and requires adding stubs at multiple layers. We may end up either
sneaking in only what we require or a non-trivial change which may
involve too many parameters.

If you would agree, I will guard the ACPI calls used in this patchset with
#ifdef CONFIG_ACPI to keep things self-contained. If you think adding the
stubs is worthwhile, we can take them up as a separate series.

Best Regards,
Akhil