| Message ID | 20230212-include-power_supply-leds-v2-2-e9910a3f5837@weissschuh.net (mailing list archive) |
|---|---|
| State | Handled Elsewhere, archived |
| Headers | show |
| Series | power: supply: remove unneeded include of linux/leds.h | expand |
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@weissschuh.net> > --- Acked-by: Sebastian Reichel <sebastian.reichel@collabora.com> -- 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 >
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
Instead of including the full linux/leds.h header file a single forward-declaration is enough. Signed-off-by: Thomas Weißschuh <linux@weissschuh.net> --- include/linux/power_supply.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)