Message ID | 1498228521-2796-1-git-send-email-mario.limonciello@dell.com (mailing list archive) |
---|---|
State | Accepted, archived |
Delegated to: | Darren Hart |
Headers | show |
On Fri, Jun 23, 2017 at 09:35:21AM -0500, Mario Limonciello wrote: > Pali recently noticed that WMI instances are zero indexed. > > The only reason that these calls all worked properly is because the ASL > didn't verify the instance number. > > Signed-off-by: Louis Davis <louis.davis@dell.com> > Signed-off-by: Mario Limonciello <mario.limonciello@dell.com> Thanks, queued to testing.
diff --git a/drivers/platform/x86/dell-wmi-led.c b/drivers/platform/x86/dell-wmi-led.c index a0c7e99..5bedaf7 100644 --- a/drivers/platform/x86/dell-wmi-led.c +++ b/drivers/platform/x86/dell-wmi-led.c @@ -68,7 +68,7 @@ static int dell_led_perform_fn(u8 length, u8 result_code, u8 device_id, input.length = sizeof(struct bios_args); input.pointer = &args; - status = wmi_evaluate_method(DELL_LED_BIOS_GUID, 1, 1, &input, &output); + status = wmi_evaluate_method(DELL_LED_BIOS_GUID, 0, 1, &input, &output); if (ACPI_FAILURE(status)) return status;