diff options
author | Lawrence Yiu <lawyiu.dev@gmail.com> | 2016-03-21 01:44:22 -0700 |
---|---|---|
committer | Darren Hart <dvhart@linux.intel.com> | 2016-05-05 16:32:32 -0700 |
commit | ddd9357f6017a8c61a20685feab5ce9eee635f6c (patch) | |
tree | b98c4344cf99bc598e33f218f6f066a417507a34 | |
parent | 775d054aba90a2c787d4c081d6369f1fddaae0f4 (diff) | |
download | op-kernel-dev-ddd9357f6017a8c61a20685feab5ce9eee635f6c.zip op-kernel-dev-ddd9357f6017a8c61a20685feab5ce9eee635f6c.tar.gz |
sony-laptop: Avoid oops on module unload for older laptops
Older VAIO laptops without the SN00 ACPI method will have the "handles"
variable unset. Return early from sony_nc_function_cleanup when "handles"
is null.
Signed-off-by: Lawrence Yiu <lawyiu.dev@gmail.com>
Acked-by: Mattia Dongili <malattia@linux.it>
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
-rw-r--r-- | drivers/platform/x86/sony-laptop.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/platform/x86/sony-laptop.c b/drivers/platform/x86/sony-laptop.c index e9caa34..1dba359 100644 --- a/drivers/platform/x86/sony-laptop.c +++ b/drivers/platform/x86/sony-laptop.c @@ -1446,6 +1446,9 @@ static void sony_nc_function_cleanup(struct platform_device *pd) { unsigned int i, result, bitmask, handle; + if (!handles) + return; + /* get enabled events and disable them */ sony_nc_int_call(sony_nc_acpi_handle, "SN01", NULL, &bitmask); sony_nc_int_call(sony_nc_acpi_handle, "SN03", &bitmask, &result); |