]> git.baikalelectronics.ru Git - kernel.git/commit
hp-wmi: fix unregister order in hp_wmi_rfkill_setup() once again
authorMaciej S. Szmigiero <mail@maciej.szmigiero.name>
Sun, 6 Mar 2016 22:38:36 +0000 (23:38 +0100)
committerDarren Hart <dvhart@linux.intel.com>
Wed, 23 Mar 2016 17:05:51 +0000 (10:05 -0700)
commit6b4dfe7c41261dc8dab6c774a0709a7fd939db5f
tree0acfce67b2f4e9866445b243b2e35ad34a90a496
parentb716189ef31bf376c6954e78af9261706ecaae7c
hp-wmi: fix unregister order in hp_wmi_rfkill_setup() once again

rfkill registration order in hp_wmi_rfkill_setup() is:
1) WiFi,
2) BT,
3) WWAN,
5) GPS.

Unregistration when cleaning up on error return should happen in reverse
order.

This means that: If BT rfkill fails to be allocated we possibly need to
first unregister WiFi rfkill before destroying it.

The same goes with (WWAN, BT) and (GPS, WWAN) pairs.

Also, if WWAN rfkill fails to register we need to (possibly) unregister BT
not the GPS one.  And if GPS rfkill fails to register we need to unregister
WWAN not the BT one.

We never need to unregister GPS rfkill here since if GPS rfkill
registration succeeds this function returns without error so no cleanup is
necessary.

Signed-off-by: Maciej S. Szmigiero <mail@maciej.szmigiero.name>
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
drivers/platform/x86/hp-wmi.c