platform/x86: dell-laptop: Use bool in struct quirk_entry for true/false fields
authorPali Rohár <pali.rohar@gmail.com>
Sat, 11 Nov 2017 22:20:23 +0000 (23:20 +0100)
committerDarren Hart (VMware) <dvhart@infradead.org>
Fri, 8 Dec 2017 21:40:21 +0000 (13:40 -0800)
In struct quirk_entry some boolean fields used int, some u8 type. Change
them all to bool type.

Signed-off-by: Pali Rohár <pali.rohar@gmail.com>
Signed-off-by: Darren Hart (VMware) <dvhart@infradead.org>
drivers/platform/x86/dell-laptop.c

index cd4725e..629d960 100644 (file)
 #include "dell-smbios.h"
 
 struct quirk_entry {
-       u8 touchpad_led;
-       u8 kbd_led_levels_off_1;
+       bool touchpad_led;
+       bool kbd_led_levels_off_1;
 
-       int needs_kbd_timeouts;
+       bool needs_kbd_timeouts;
        /*
         * Ordered list of timeouts expressed in seconds.
         * The list must end with -1
@@ -50,7 +50,7 @@ struct quirk_entry {
 static struct quirk_entry *quirks;
 
 static struct quirk_entry quirk_dell_vostro_v130 = {
-       .touchpad_led = 1,
+       .touchpad_led = true,
 };
 
 static int __init dmi_matched(const struct dmi_system_id *dmi)
@@ -64,12 +64,12 @@ static int __init dmi_matched(const struct dmi_system_id *dmi)
  * is used then BIOS silently set timeout to 0 without any error message.
  */
 static struct quirk_entry quirk_dell_xps13_9333 = {
-       .needs_kbd_timeouts = 1,
+       .needs_kbd_timeouts = true,
        .kbd_timeouts = { 0, 5, 15, 60, 5 * 60, 15 * 60, -1 },
 };
 
 static struct quirk_entry quirk_dell_latitude_e6410 = {
-       .kbd_led_levels_off_1 = 1,
+       .kbd_led_levels_off_1 = true,
 };
 
 static struct platform_driver platform_driver = {