platform/chrome: chromeos_laptop: drop checks of NULL-safe functions
authorWolfram Sang <wsa+renesas@sang-engineering.com>
Tue, 20 Aug 2019 15:34:49 +0000 (17:34 +0200)
committerEnric Balletbo i Serra <enric.balletbo@collabora.com>
Mon, 26 Aug 2019 13:15:52 +0000 (15:15 +0200)
No need to check the argument of i2c_unregister_device() and
property_entries_free() because the functions do check it.

Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Signed-off-by: Enric Balletbo i Serra <enric.balletbo@collabora.com>
drivers/platform/chrome/chromeos_laptop.c

index 7abbb6167766eec972cbaee3f22e38ba8646ce9a..8723bcf10c9317af6659e421fd70d1c829e86918 100644 (file)
@@ -838,18 +838,14 @@ static void chromeos_laptop_destroy(const struct chromeos_laptop *cros_laptop)
                i2c_dev = &cros_laptop->i2c_peripherals[i];
                info = &i2c_dev->board_info;
 
-               if (i2c_dev->client)
-                       i2c_unregister_device(i2c_dev->client);
-
-               if (info->properties)
-                       property_entries_free(info->properties);
+               i2c_unregister_device(i2c_dev->client);
+               property_entries_free(info->properties);
        }
 
        for (i = 0; i < cros_laptop->num_acpi_peripherals; i++) {
                acpi_dev = &cros_laptop->acpi_peripherals[i];
 
-               if (acpi_dev->properties)
-                       property_entries_free(acpi_dev->properties);
+               property_entries_free(acpi_dev->properties);
        }
 
        kfree(cros_laptop->i2c_peripherals);