Skip to content

Commit

Permalink
Merge branches 'acpi-property' and 'acpi-sbs'
Browse files Browse the repository at this point in the history
* acpi-property:
  ACPI / property: Switch to bitmap_zalloc()

* acpi-sbs:
  ACPI / SBS: Fix rare oops when removing modules
  ACPI / SBS: Fix GPE storm on recent MacBookPro's
  • Loading branch information
Rafael J. Wysocki committed Oct 18, 2018
3 parents 1f825f7 + d030fd0 + 757c968 commit 0a1875a
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 4 deletions.
1 change: 1 addition & 0 deletions drivers/acpi/osl.c
Original file line number Diff line number Diff line change
Expand Up @@ -1132,6 +1132,7 @@ void acpi_os_wait_events_complete(void)
flush_workqueue(kacpid_wq);
flush_workqueue(kacpi_notify_wq);
}
EXPORT_SYMBOL(acpi_os_wait_events_complete);

struct acpi_hp_work {
struct work_struct work;
Expand Down
8 changes: 6 additions & 2 deletions drivers/acpi/sbs.c
Original file line number Diff line number Diff line change
Expand Up @@ -441,9 +441,13 @@ static int acpi_ac_get_present(struct acpi_sbs *sbs)

/*
* The spec requires that bit 4 always be 1. If it's not set, assume
* that the implementation doesn't support an SBS charger
* that the implementation doesn't support an SBS charger.
*
* And on some MacBooks a status of 0xffff is always returned, no
* matter whether the charger is plugged in or not, which is also
* wrong, so ignore the SBS charger for those too.
*/
if (!((status >> 4) & 0x1))
if (!((status >> 4) & 0x1) || status == 0xffff)
return -ENODEV;

sbs->charger_present = (status >> 15) & 0x1;
Expand Down
2 changes: 2 additions & 0 deletions drivers/acpi/sbshc.c
Original file line number Diff line number Diff line change
Expand Up @@ -196,6 +196,7 @@ int acpi_smbus_unregister_callback(struct acpi_smb_hc *hc)
hc->callback = NULL;
hc->context = NULL;
mutex_unlock(&hc->lock);
acpi_os_wait_events_complete();
return 0;
}

Expand Down Expand Up @@ -292,6 +293,7 @@ static int acpi_smbus_hc_remove(struct acpi_device *device)

hc = acpi_driver_data(device);
acpi_ec_remove_query_handler(hc->ec, hc->query_bit);
acpi_os_wait_events_complete();
kfree(hc);
device->driver_data = NULL;
return 0;
Expand Down
4 changes: 2 additions & 2 deletions drivers/acpi/x86/apple.c
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ void acpi_extract_apple_properties(struct acpi_device *adev)
if (!numprops)
goto out_free;

valid = kcalloc(BITS_TO_LONGS(numprops), sizeof(long), GFP_KERNEL);
valid = bitmap_zalloc(numprops, GFP_KERNEL);
if (!valid)
goto out_free;

Expand Down Expand Up @@ -137,5 +137,5 @@ void acpi_extract_apple_properties(struct acpi_device *adev)

out_free:
ACPI_FREE(props);
kfree(valid);
bitmap_free(valid);
}

0 comments on commit 0a1875a

Please sign in to comment.