platform/x86: intel-hid: make intel_hid_set_enable() take a boolean argument
authorMichał Kępień <kernel@kempniu.pl>
Fri, 24 Feb 2017 10:33:07 +0000 (11:33 +0100)
committerDarren Hart <dvhart@infradead.org>
Wed, 15 Mar 2017 05:57:10 +0000 (22:57 -0700)
As the integer value passed to intel_hid_set_enable() is always
explicitly passed and is used solely as a boolean value, make it a bool.

Signed-off-by: Michał Kępień <kernel@kempniu.pl>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-and-tested-by: Alex Hung <alex.hung@canonical.com>
drivers/platform/x86/intel-hid.c

index 4d1c5eb3a96d63369f3df06c4b915ddaacf43277..89d1d98e32143c1c5e8d6a8250dccab99104012e 100644 (file)
@@ -77,7 +77,7 @@ struct intel_hid_priv {
        struct input_dev *array;
 };
 
-static int intel_hid_set_enable(struct device *device, int enable)
+static int intel_hid_set_enable(struct device *device, bool enable)
 {
        acpi_status status;
 
@@ -118,7 +118,7 @@ static void intel_button_array_enable(struct device *device, bool enable)
 
 static int intel_hid_pl_suspend_handler(struct device *device)
 {
-       intel_hid_set_enable(device, 0);
+       intel_hid_set_enable(device, false);
        intel_button_array_enable(device, false);
 
        return 0;
@@ -126,7 +126,7 @@ static int intel_hid_pl_suspend_handler(struct device *device)
 
 static int intel_hid_pl_resume_handler(struct device *device)
 {
-       intel_hid_set_enable(device, 1);
+       intel_hid_set_enable(device, true);
        intel_button_array_enable(device, true);
 
        return 0;
@@ -275,7 +275,7 @@ static int intel_hid_probe(struct platform_device *device)
                goto err_remove_input;
        }
 
-       err = intel_hid_set_enable(&device->dev, 1);
+       err = intel_hid_set_enable(&device->dev, true);
        if (err)
                goto err_remove_notify;
 
@@ -306,7 +306,7 @@ 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);
+       intel_hid_set_enable(&device->dev, false);
        intel_button_array_enable(&device->dev, false);
 
        /*