Commit 8446f9d1 authored by Dan Carpenter's avatar Dan Carpenter Committed by Hans de Goede

platform/x86: wmi: Fix wmi_dev_probe()

This has a reversed if statement so it accidentally disables the wmi
method before returning.

Fixes: 704af3a4 ("platform/x86: wmi: Remove chardev interface")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: default avatarArmin Wolf <W_Armin@gmx.de>
Link: https://lore.kernel.org/r/9c81251b-bc87-4ca3-bb86-843dc85e5145@moroto.mountainReviewed-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
parent 29e473f4
...@@ -896,7 +896,7 @@ static int wmi_dev_probe(struct device *dev) ...@@ -896,7 +896,7 @@ static int wmi_dev_probe(struct device *dev)
if (wdriver->probe) { if (wdriver->probe) {
ret = wdriver->probe(dev_to_wdev(dev), ret = wdriver->probe(dev_to_wdev(dev),
find_guid_context(wblock, wdriver)); find_guid_context(wblock, wdriver));
if (!ret) { if (ret) {
if (ACPI_FAILURE(wmi_method_enable(wblock, false))) if (ACPI_FAILURE(wmi_method_enable(wblock, false)))
dev_warn(dev, "Failed to disable device\n"); dev_warn(dev, "Failed to disable device\n");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment