Messages in this thread | | | Subject | Re: [PATCH] regulator: qcom-smd: Batch up requests for disabled regulators | From | Jeffrey Hugo <> | Date | Tue, 22 Jan 2019 14:57:46 -0700 |
| |
On 1/22/2019 12:01 PM, Bjorn Andersson wrote: > In some scenarios the early stages of the boot chain has configured > regulators to be in a required state, but the later stages has skipped > to inform the RPM about it's requirements. > > But as the SMD RPM regulators are being initialized voltage change > requests will be issued to align the voltage with the valid ranges. The > RPM aggregates all parameters for the specific regulator, the voltage > will be adjusted and the "enabled" state will be "off" - and the > regulator is turned off. > > This patch addresses this problem by caching the requested enable state, > voltage and load and send the parameters in a batch, depending on the > enable state - effectively delaying the voltage request for disabled > regulators. > > Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
This happens to fix an annoyance I've been avoiding on 8998.
Reviewed-by: Jeffrey Hugo <jhugo@codeaurora.org> Tested-by: Jeffrey Hugo <jhugo@codeaurora.org>
-- Jeffrey Hugo Qualcomm Datacenter Technologies as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.
| |