Re: [PATCH v2 2/2] power: supply: remove unneeded include of linux/leds.h

From: Sebastian Reichel
Date: Mon May 08 2023 - 12:37:38 EST


Hi,

On Tue, May 02, 2023 at 06:28:45AM +0200, Thomas Weißschuh wrote:
> Instead of including the full linux/leds.h header file a single
> forward-declaration is enough.
>
> Signed-off-by: Thomas Weißschuh <linux@xxxxxxxxxxxxxx>
> ---

Acked-by: Sebastian Reichel <sebastian.reichel@xxxxxxxxxxxxx>

-- Sebastian

> include/linux/power_supply.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h
> index a427f13c757f..dc5e17e8c919 100644
> --- a/include/linux/power_supply.h
> +++ b/include/linux/power_supply.h
> @@ -14,10 +14,11 @@
>
> #include <linux/device.h>
> #include <linux/workqueue.h>
> -#include <linux/leds.h>
> #include <linux/spinlock.h>
> #include <linux/notifier.h>
>
> +struct led_trigger;
> +
> /*
> * All voltages, currents, charges, energies, time and temperatures in uV,
> * µA, µAh, µWh, seconds and tenths of degree Celsius unless otherwise
>
> --
> 2.40.1
>

Attachment: signature.asc
Description: PGP signature