> -----Original Message-----
> From: Rob Herring <[email protected]>
> Sent: Wednesday, March 29, 2023 2:21 PM
> To: Rafael J. Wysocki <[email protected]>; Len Brown <[email protected]>;
> Marcelo Schmitt <[email protected]>; Lars-Peter Clausen
> <[email protected]>; Michael Hennerich <[email protected]>;
> Jonathan Cameron <[email protected]>; Greg Kroah-Hartman
> <[email protected]>; Johannes Berg <[email protected]>;
> David S. Miller <[email protected]>; Eric Dumazet
> <[email protected]>; Jakub Kicinski <[email protected]>; Paolo Abeni
> <[email protected]>; Jiri Slaby <[email protected]>; Thierry Reding
> <[email protected]>; Jonathan Hunter <[email protected]>; Marc
> Zyngier <[email protected]>
> Cc: [email protected]; [email protected]; linux-
> [email protected]; [email protected];
> [email protected]; [email protected]; linux-
> [email protected]; [email protected]; [email protected]
> Subject: [PATCH 0/5] Remove acpi.h implicit include of of.h
>
> In the process of cleaning up DT includes, I found that some drivers
> using DT functions could build without any explicit DT include. I traced
> the include to be coming from acpi.h via irqdomain.h.
>
> I was pleasantly surprised that there were not 100s or even 10s of
> warnings when breaking the include chain. So here's the resulting
> series.
>
> I'd suggest Rafael take the whole series. Alternatively,the fixes can be
> applied in 6.4 and then the last patch either after rc1 or the
> following cycle.
>
Nice that there aren't too many errors. Use of the forward declarations makes sense rather than including a bunch of unrelated definitions. Nice.
Reviewed-by: Jacob Keller <[email protected]>
Thanks,
Jake
> Signed-off-by: Rob Herring <[email protected]>
> ---
> Rob Herring (5):
> iio: adc: ad7292: Add explicit include for of.h
> staging: iio: resolver: ad2s1210: Add explicit include for of.h
> net: rfkill-gpio: Add explicit include for of.h
> serial: 8250_tegra: Add explicit include for of.h
> ACPI: Replace irqdomain.h include with struct declarations
>
> drivers/iio/adc/ad7292.c | 1 +
> drivers/staging/iio/resolver/ad2s1210.c | 1 +
> drivers/tty/serial/8250/8250_tegra.c | 1 +
> include/linux/acpi.h | 4 +++-
> net/rfkill/rfkill-gpio.c | 1 +
> 5 files changed, 7 insertions(+), 1 deletion(-)
> ---
> base-commit: fe15c26ee26efa11741a7b632e9f23b01aca4cc6
> change-id: 20230329-acpi-header-cleanup-665331828436
>
> Best regards,
> --
> Rob Herring <[email protected]>