Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

drivers: regulator: fix previous commit and implement always-on regulator #6328

Merged
merged 3 commits into from
Oct 4, 2023

Conversation

etienne-lms
Copy link
Contributor

1 commit to fix a bug in the previously merged commit 1a3d327 (my apologies, bad squashing),
followed by the implementation of always-on regulators.

Copy link
Contributor

@jforissier jforissier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One comment below.
Acked-by: Jerome Forissier <[email protected]>

core/include/drivers/regulator.h Outdated Show resolved Hide resolved
@etienne-lms
Copy link
Contributor Author

Review comment addressed and tag applied.

@etienne-lms
Copy link
Contributor Author

"CI / make check (QEMUv8/Xen)" failed with error message "No space left on device".

@GseoC
Copy link
Contributor

GseoC commented Oct 3, 2023

For the whole patchset:
Acked-by: Gatien Chevallier <[email protected]>

Fixes commit introducing regulator voltage that did not implement
always-on flag.

Fixes: 1a3d327 ("drivers: regulator framework")
Acked-by: Jerome Forissier <[email protected]>
Acked-by: Gatien Chevallier <[email protected]>
Signed-off-by: Etienne Carriere <[email protected]>
Implements always-on regulator property for regulators that once enabled
shall never be disabled. This is archived using a regulator_enable()
call to increment its refcount.

Acked-by: Jerome Forissier <[email protected]>
Acked-by: Gatien Chevallier <[email protected]>
Signed-off-by: Etienne Carriere <[email protected]>
Enables regulator framework on platform stm32mp1.

Acked-by: Jerome Forissier <[email protected]>
Acked-by: Gatien Chevallier <[email protected]>
Signed-off-by: Etienne Carriere <[email protected]>
@etienne-lms
Copy link
Contributor Author

Tags applied.

@jforissier jforissier merged commit 5457b0f into OP-TEE:master Oct 4, 2023
8 checks passed
@etienne-lms etienne-lms deleted the regulator-0 branch October 4, 2023 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants