diff mbox

toshiba_acpi: Fix the kbd_backlight_mode_show printed value

Message ID 20160816180755.1845-1-coproscefalo@gmail.com (mailing list archive)
State Accepted, archived
Headers show

Commit Message

Azael Avalos Aug. 16, 2016, 6:07 p.m. UTC 
The mentioned function prints the value in decimal, but the
documentation and its parent function available_kbd_modes_show
are showing the value as hexadecimal.

This patch simply changes the value printed from decimal to hex,
as stated in the documentation and the parent function.

Signed-off-by: Azael Avalos <coproscefalo@gmail.com>
---
 drivers/platform/x86/toshiba_acpi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Darren Hart Aug. 28, 2016, 5:06 p.m. UTC | #1 
On Tue, Aug 16, 2016 at 12:07:55PM -0600, Azael Avalos wrote:
> The mentioned function prints the value in decimal, but the
> documentation and its parent function available_kbd_modes_show
> are showing the value as hexadecimal.
> 
> This patch simply changes the value printed from decimal to hex,
> as stated in the documentation and the parent function.
> 
> Signed-off-by: Azael Avalos <coproscefalo@gmail.com>

Thanks, Queued to for-next.
diff mbox

Patch

diff --git a/drivers/platform/x86/toshiba_acpi.c b/drivers/platform/x86/toshiba_acpi.c
index 254a4d5..444f351 100644
--- a/drivers/platform/x86/toshiba_acpi.c
+++ b/drivers/platform/x86/toshiba_acpi.c
@@ -1873,7 +1873,7 @@  static ssize_t kbd_backlight_mode_show(struct device *dev,
 	if (toshiba_kbd_illum_status_get(toshiba, &time) < 0)
 		return -EIO;
 
-	return sprintf(buf, "%i\n", time & SCI_KBD_MODE_MASK);
+	return sprintf(buf, "0x%x\n", time & SCI_KBD_MODE_MASK);
 }
 static DEVICE_ATTR_RW(kbd_backlight_mode);