Page MenuHomeFreeBSD

extres: regulator: Fix regulator_status for already enable regulators
ClosedPublic

Authored by manu on May 16 2021, 3:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 13, 2:45 PM
Unknown Object (File)
Nov 13 2024, 8:25 PM
Unknown Object (File)
Nov 12 2024, 1:12 AM
Unknown Object (File)
Oct 21 2024, 11:51 AM
Unknown Object (File)
Oct 4 2024, 9:43 PM
Unknown Object (File)
Oct 4 2024, 4:11 PM
Unknown Object (File)
Oct 2 2024, 4:15 AM
Unknown Object (File)
Sep 23 2024, 6:36 PM
Subscribers
None

Details

Summary

If a regulator hasn't been enable by a driver but is enabled in hardware
(most likely enabled by U-Boot), regulator_status will returns that it
is enabled and so any call to regulator_disable will panic as it wasn't
enabled by one of our drivers.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

manu requested review of this revision.May 16 2021, 3:16 PM
manu created this revision.
This revision was not accepted when it landed; it landed in state Needs Review.May 21 2021, 4:06 PM
This revision was automatically updated to reflect the committed changes.

IMHO, this is wrong change. regulator_status() must report physical state of regulator - if given deice is powered or not. the mmc power sequencer should use similar technique as backlight. But seems like this kind of usage is common - so we can expand regulator framework with uncounted function -> something like regulator_on()/regulator_off()