platform/x86: toshiba_acpi: Fix ECO LED control on Toshiba Z830

The toshiba_acpi driver checks for TOS_INPUT_DATA_ERROR and tries a
different format. On the Z830 the error returned is TOS_NOT_SUPPORTED
though the different format still works. Allow either error.

Signed-off-by: Arvid Norlander <lkml@vorpal.se>
Link: https://lore.kernel.org/r/20220821200821.1837460-2-lkml@vorpal.se
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>

diff --git a/drivers/platform/x86/toshiba_acpi.c b/drivers/platform/x86/toshiba_acpi.c
index 0fc9e8b8827bdf7307fe3257e04339688c9963bc..6cc617b2940e3acd8d6e3936e27a0240d0df3a43 100644 (file)
--- a/drivers/platform/x86/toshiba_acpi.c
+++ b/drivers/platform/x86/toshiba_acpi.c
@@ -675,12 +675,15 @@ static void toshiba_eco_mode_available(struct toshiba_acpi_dev *dev)
                return;
        }
 
-       if (out[0] == TOS_INPUT_DATA_ERROR) {
+       if (out[0] == TOS_INPUT_DATA_ERROR || out[0] == TOS_NOT_SUPPORTED) {
                /*
                 * If we receive 0x8300 (Input Data Error), it means that the
                 * LED device is present, but that we just screwed the input
                 * parameters.
                 *
+                * On some laptops 0x8000 (Not supported) is also returned in
+                * this case, so we need to allow for that as well.
+                *
                 * Let's query the status of the LED to see if we really have a
                 * success response, indicating the actual presense of the LED,
                 * bail out otherwise.