diff options
| author |   Ilpo Järvinen <ilpo.jarvinen@linux.intel.com> | 2024-07-31 12:53:45 +0300 |
|---|---|---|
| committer |   Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2024-08-02 16:22:11 +0200 |
| commit | 3745de7f2d00d44e098a2cae148cd9c1f1534070 (patch) | |
| tree | 9c47621bc097b801fe2c0ff13936d9e04167c3ff | |
| parent | ACPI: battery: Use strscpy() instead of strcpy() (diff) | |
| download | wireguard-linux-3745de7f2d00d44e098a2cae148cd9c1f1534070.tar.xz wireguard-linux-3745de7f2d00d44e098a2cae148cd9c1f1534070.zip | |
ACPI: battery: Remove redundant NULL initalizations
A local 'battery' variable is initialized to NULL on two occassions
where it is unconditionally rewritten later. Remove the unnecessary
initializations.
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Link: https://patch.msgid.link/20240731095345.2878-1-ilpo.jarvinen@linux.intel.com
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
| -rw-r--r-- | drivers/acpi/battery.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/acpi/battery.c b/drivers/acpi/battery.c index 857d4ef38469..8fe27880ea88 100644 --- a/drivers/acpi/battery.c +++ b/drivers/acpi/battery.c @@ -1207,7 +1207,7 @@ static int acpi_battery_update_retry(struct acpi_battery *battery) static int acpi_battery_add(struct acpi_device *device) { int result = 0; - struct acpi_battery *battery = NULL; + struct acpi_battery *battery; if (!device) return -EINVAL; @@ -1260,7 +1260,7 @@ fail: static void acpi_battery_remove(struct acpi_device *device) { - struct acpi_battery *battery = NULL; + struct acpi_battery *battery; if (!device || !acpi_driver_data(device)) return; |