diff options
| author | Alex Hung <alex.hung@canonical.com> | 2016-06-29 11:41:15 +0800 |
|---|---|---|
| committer | Darren Hart <dvhart@linux.intel.com> | 2016-06-28 22:18:35 -0700 |
| commit | 1d6de071cb0c321279373634d81eb8e176d887c4 (patch) | |
| tree | 90cd86fbc550f26106b09d22bdee95cececa898e | |
| parent | 9ee27487127461b5cf71670b708ed5b2b8da568c (diff) | |
| download | op-kernel-dev-1d6de071cb0c321279373634d81eb8e176d887c4.zip op-kernel-dev-1d6de071cb0c321279373634d81eb8e176d887c4.tar.gz | |
intel-hid: Remove duplicated acpi_remove_notify_handler
The second call to acpi_remove_notify_handler does not result in panic
or generate error messages, but it is unnecessary and the function
returns with an error. Remove the duplicate call. Correct two improperly
indented lines.
Signed-off-by: Alex Hung <alex.hung@canonical.com>
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
| -rw-r--r-- | drivers/platform/x86/intel-hid.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/platform/x86/intel-hid.c b/drivers/platform/x86/intel-hid.c index a818db6..ed58742 100644 --- a/drivers/platform/x86/intel-hid.c +++ b/drivers/platform/x86/intel-hid.c @@ -122,8 +122,8 @@ static int intel_hid_input_setup(struct platform_device *device) return 0; err_free_device: - input_free_device(priv->input_dev); - return ret; + input_free_device(priv->input_dev); + return ret; } static void intel_hid_input_destroy(struct platform_device *device) @@ -224,7 +224,6 @@ static int intel_hid_remove(struct platform_device *device) acpi_remove_notify_handler(handle, ACPI_DEVICE_NOTIFY, notify_handler); intel_hid_input_destroy(device); intel_hid_set_enable(&device->dev, 0); - acpi_remove_notify_handler(handle, ACPI_DEVICE_NOTIFY, notify_handler); /* * Even if we failed to shut off the event stream, we can still |
