History

Revert "hwmon: (pmbus) Add regulator supply into macro"

This reverts commit 54cc3dbfc10dc3db7cb1cf49aee4477a8398fbde. Zev Weiss reports that the reverted patch may cause a regulator undercount. Here is his report: ... having regulator-dummy set as a supply on my PMBus regulators (instead of having them as their own top-level regulators without an upstream supply) leads to enable-count underflow errors when disabling them: # echo 0 > /sys/bus/platform/devices/efuse01/state [ 906.094477] regulator-dummy: Underflow of regulator enable count [ 906.100563] Failed to disable vout: -EINVAL [ 136.992676] reg-userspace-consumer efuse01: Failed to configure state: -22 Zev reports that reverting the patch fixes the problem. So let's do that for now.

Guenter Roeck committed 4 years ago

Commit 1e699e11 parent c4f68371 file · +0 −1Browse files

Unified · Split

drivers/hwmon/pmbus/pmbus.h+0−1
@@ -467,7 +467,6 @@ extern const struct regulator_ops pmbus_regulator_ops; · 466 unmodified lines
467467 #define PMBUS_REGULATOR_STEP(_name, _id, _voltages, _step) \
468468 [_id] = { \
469469 .name = (_name # _id), \
470− .supply_name = "vin", \
471470 .id = (_id), \
472471 .of_match = of_match_ptr(_name # _id), \
473472 .regulators_node = of_match_ptr("regulators"), \

Keyboard shortcuts

General

Show keyboard shortcuts
?
Focus the current search
/

Issues and pull requests

Open filters
f

Issues, pull requests and Inbox

Select next item
j↓
Select previous item
k↑
Open selected item
Enter
Dismiss filters or clear selection
Esc

Repository code

Go to file
t