diff options
| author | Aditya Garg <gargaditya08@live.com> | 2025-02-09 07:27:52 +0000 |
|---|---|---|
| committer | Jiri Kosina <jkosina@suse.com> | 2025-02-10 08:24:21 +0100 |
| commit | a4fa2e6bfe6be8f8b4cc4cf9bf75a26b7da7c1b5 (patch) | |
| tree | 43f997c5514f45cf8732d158220822ddbc324ad4 | |
| parent | cc9714a5227b242266c6d62007b99e3c4c7f1e4a (diff) | |
HID: hid-appletb-bl: use appletb_bl_brightness_map instead of magic numbers to set default brightness
This commit makes use of appletb_bl_brightness_map instead of using if
statements and magic numbers to set default brightness, and thus simplifies
the code.
Signed-off-by: Aditya Garg <gargaditya08@live.com>
Signed-off-by: Jiri Kosina <jkosina@suse.com>
| -rw-r--r-- | drivers/hid/hid-appletb-bl.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/hid/hid-appletb-bl.c b/drivers/hid/hid-appletb-bl.c index 819157686e59..1a3dcd44bf1a 100644 --- a/drivers/hid/hid-appletb-bl.c +++ b/drivers/hid/hid-appletb-bl.c @@ -141,12 +141,8 @@ static int appletb_bl_probe(struct hid_device *hdev, const struct hid_device_id bl->aux1_field = aux1_field; bl->brightness_field = brightness_field; - if (appletb_bl_def_brightness == 0) - ret = appletb_bl_set_brightness(bl, APPLETB_BL_OFF); - else if (appletb_bl_def_brightness == 1) - ret = appletb_bl_set_brightness(bl, APPLETB_BL_DIM); - else - ret = appletb_bl_set_brightness(bl, APPLETB_BL_ON); + ret = appletb_bl_set_brightness(bl, + appletb_bl_brightness_map[(appletb_bl_def_brightness > 2) ? 2 : appletb_bl_def_brightness]); if (ret) { dev_err_probe(dev, ret, "Failed to set touch bar brightness to off\n"); |