Commit abbdf22e authored by Rafael J. Wysocki's avatar Rafael J. Wysocki
Browse files

ACPI: battery: Drop redundant check from acpi_battery_notify() 

The battery pointer check against NULL in acpi_battery_notify() is
redundant because the value of that pointer is the one passed to
acpi_dev_install_notify_handler() in acpi_battery_probe() as the
last argument which is not NULL.

Drop the redundant check.

No intentional functional impact.

Closes: https://lore.kernel.org/linux-acpi/aYXvS1h3Bxf_5sCj@stanley.mountain/


Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
Link: https://patch.msgid.link/5981321.DvuYhMxLoT@rafael.j.wysocki
parent d84e1733

drivers/acpi/battery.c

+0 −3
Original line number Diff line number Diff line
@@ -1066,9 +1066,6 @@ static void acpi_battery_notify(acpi_handle handle, u32 event, void *data) 
	struct acpi_device *device = battery->device;

	struct power_supply *old;



	if (!battery)

		return;



	guard(mutex)(&battery->update_lock);



	old = battery->bat;