Commit 00981810 authored by Markus Elfring's avatar Markus Elfring Committed by Darren Hart

platform: x86: Deletion of checks before backlight_device_unregister()

The backlight_device_unregister() function tests whether its argument is NULL
and then returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.
Signed-off-by: default avatarMarkus Elfring <elfring@users.sourceforge.net>
For msi-wmi.c:
Acked-by: default avatarAnisse Astier <anisse@astier.eu>
Signed-off-by: default avatarDarren Hart <dvhart@linux.intel.com>
parent a39f46df
...@@ -843,7 +843,6 @@ static int asus_backlight_init(struct asus_laptop *asus) ...@@ -843,7 +843,6 @@ static int asus_backlight_init(struct asus_laptop *asus)
static void asus_backlight_exit(struct asus_laptop *asus) static void asus_backlight_exit(struct asus_laptop *asus)
{ {
if (asus->backlight_device)
backlight_device_unregister(asus->backlight_device); backlight_device_unregister(asus->backlight_device);
asus->backlight_device = NULL; asus->backlight_device = NULL;
} }
......
...@@ -1308,7 +1308,6 @@ static int asus_wmi_backlight_init(struct asus_wmi *asus) ...@@ -1308,7 +1308,6 @@ static int asus_wmi_backlight_init(struct asus_wmi *asus)
static void asus_wmi_backlight_exit(struct asus_wmi *asus) static void asus_wmi_backlight_exit(struct asus_wmi *asus)
{ {
if (asus->backlight_device)
backlight_device_unregister(asus->backlight_device); backlight_device_unregister(asus->backlight_device);
asus->backlight_device = NULL; asus->backlight_device = NULL;
......
...@@ -1174,7 +1174,6 @@ static int eeepc_backlight_init(struct eeepc_laptop *eeepc) ...@@ -1174,7 +1174,6 @@ static int eeepc_backlight_init(struct eeepc_laptop *eeepc)
static void eeepc_backlight_exit(struct eeepc_laptop *eeepc) static void eeepc_backlight_exit(struct eeepc_laptop *eeepc)
{ {
if (eeepc->backlight_device)
backlight_device_unregister(eeepc->backlight_device); backlight_device_unregister(eeepc->backlight_device);
eeepc->backlight_device = NULL; eeepc->backlight_device = NULL;
} }
......
...@@ -1154,7 +1154,6 @@ static int __init fujitsu_init(void) ...@@ -1154,7 +1154,6 @@ static int __init fujitsu_init(void)
fail_hotkey: fail_hotkey:
platform_driver_unregister(&fujitsupf_driver); platform_driver_unregister(&fujitsupf_driver);
fail_backlight: fail_backlight:
if (fujitsu->bl_device)
backlight_device_unregister(fujitsu->bl_device); backlight_device_unregister(fujitsu->bl_device);
fail_sysfs_group: fail_sysfs_group:
sysfs_remove_group(&fujitsu->pf_device->dev.kobj, sysfs_remove_group(&fujitsu->pf_device->dev.kobj,
...@@ -1179,7 +1178,6 @@ static void __exit fujitsu_cleanup(void) ...@@ -1179,7 +1178,6 @@ static void __exit fujitsu_cleanup(void)
platform_driver_unregister(&fujitsupf_driver); platform_driver_unregister(&fujitsupf_driver);
if (fujitsu->bl_device)
backlight_device_unregister(fujitsu->bl_device); backlight_device_unregister(fujitsu->bl_device);
sysfs_remove_group(&fujitsu->pf_device->dev.kobj, sysfs_remove_group(&fujitsu->pf_device->dev.kobj,
......
...@@ -729,7 +729,6 @@ static int ideapad_backlight_init(struct ideapad_private *priv) ...@@ -729,7 +729,6 @@ static int ideapad_backlight_init(struct ideapad_private *priv)
static void ideapad_backlight_exit(struct ideapad_private *priv) static void ideapad_backlight_exit(struct ideapad_private *priv)
{ {
if (priv->blightdev)
backlight_device_unregister(priv->blightdev); backlight_device_unregister(priv->blightdev);
priv->blightdev = NULL; priv->blightdev = NULL;
} }
......
...@@ -271,7 +271,6 @@ static int oaktrail_backlight_init(void) ...@@ -271,7 +271,6 @@ static int oaktrail_backlight_init(void)
static void oaktrail_backlight_exit(void) static void oaktrail_backlight_exit(void)
{ {
if (oaktrail_bl_device)
backlight_device_unregister(oaktrail_bl_device); backlight_device_unregister(oaktrail_bl_device);
} }
......
...@@ -354,7 +354,6 @@ static void __exit msi_wmi_exit(void) ...@@ -354,7 +354,6 @@ static void __exit msi_wmi_exit(void)
sparse_keymap_free(msi_wmi_input_dev); sparse_keymap_free(msi_wmi_input_dev);
input_unregister_device(msi_wmi_input_dev); input_unregister_device(msi_wmi_input_dev);
} }
if (backlight)
backlight_device_unregister(backlight); backlight_device_unregister(backlight);
} }
......
...@@ -3141,7 +3141,6 @@ static void sony_nc_backlight_setup(void) ...@@ -3141,7 +3141,6 @@ static void sony_nc_backlight_setup(void)
static void sony_nc_backlight_cleanup(void) static void sony_nc_backlight_cleanup(void)
{ {
if (sony_bl_props.dev)
backlight_device_unregister(sony_bl_props.dev); backlight_device_unregister(sony_bl_props.dev);
} }
......
...@@ -1797,7 +1797,6 @@ static int toshiba_acpi_remove(struct acpi_device *acpi_dev) ...@@ -1797,7 +1797,6 @@ static int toshiba_acpi_remove(struct acpi_device *acpi_dev)
rfkill_destroy(dev->bt_rfk); rfkill_destroy(dev->bt_rfk);
} }
if (dev->backlight_dev)
backlight_device_unregister(dev->backlight_dev); backlight_device_unregister(dev->backlight_dev);
if (dev->illumination_supported) if (dev->illumination_supported)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment