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

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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()