Differential D14781

regulator: Disable unused regulator
ClosedPublic
Actions

Authored by manu on Mar 21 2018, 9:16 AM.

Tags

None

Referenced Files

	F82270646: D14781.id.diff
	Sat, Apr 27, 4:08 AM

	F82270187: D14781.id40531.diff
	Sat, Apr 27, 4:02 AM

	F82270176: D14781.id40529.diff
	Sat, Apr 27, 4:02 AM

	F82270174: D14781.id41072.diff
	Sat, Apr 27, 4:02 AM

	F82264396: D14781.diff
	Sat, Apr 27, 2:44 AM

	Unknown Object (File)
	Dec 23 2023, 2:39 AM

	Unknown Object (File)
	Nov 14 2023, 12:22 PM

	Unknown Object (File)
	Nov 10 2023, 8:38 AM

View All 22 Files

Subscribers

Details

Reviewers

Group Reviewers

ARM
arm64

Commits

rS331989: regulator: Disable unused regulator

Summary

bootloaders such as u-boot might enable regulators, or simply regulators could be enabled by default by the PMIC, even if we don't have a driver for the device or subsystem.
Disable unused regulators just before going to userland.
A tunable hw.regulator.disable_unused is added to not disable them in case this causes problems on some board but the default behavior is to disable everything unused. I prefer to break thinks now and fix them rather than never switch to the case were we disable regulators.

Test Plan

Tested on : Pine64-LTS (an idle board goes from ~0.33A to ~0.27A)

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 15678
Build 15707: arc lint + arc unit

Event Timeline

manu created this revision.Mar 21 2018, 9:16 AM

Herald added a subscriber: imp. · View Herald TranscriptMar 21 2018, 9:16 AM

Harbormaster completed remote builds in B15678: Diff 40529.Mar 21 2018, 9:16 AM

It would pay to add a comment to explain what the function is for.

Add comment explaining what the function does.

Harbormaster completed remote builds in B15680: Diff 40531.Mar 21 2018, 12:02 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 4 2018, 6:44 AM

Closed by commit rS331989: regulator: Disable unused regulator (authored by manu). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

dev/

extres/

regulator/

24 lines

Diff 40529

sys/dev/extres/regulator/regulator.c

Loading...