L4c is the supply for UFS vccq, As per UFS spec range of vccq is
1.14V to 1.26V, There are stability issues when operating at
marginal voltage. Hence configure the min and max vccq voltages
to 1.2V.
Signed-off-by: Naveen Kumar Goud Arepalli <[email protected]>
---
arch/arm64/boot/dts/qcom/sa8775p-ride.dts | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sa8775p-ride.dts b/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
index ab767cfa51ff..26f5a4e0ffed 100644
--- a/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
+++ b/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
@@ -153,8 +153,8 @@
vreg_l4c: ldo4 {
regulator-name = "vreg_l4c";
- regulator-min-microvolt = <1100000>;
- regulator-max-microvolt = <1300000>;
+ regulator-min-microvolt = <1200000>;
+ regulator-max-microvolt = <1200000>;
regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
/*
* FIXME: This should have regulator-allow-set-load but
--
2.17.1
On 11.07.2023 12:59, Naveen Kumar Goud Arepalli wrote:
> L4c is the supply for UFS vccq, As per UFS spec range of vccq is
> 1.14V to 1.26V, There are stability issues when operating at
> marginal voltage. Hence configure the min and max vccq voltages
> to 1.2V.
>
> Signed-off-by: Naveen Kumar Goud Arepalli <[email protected]>
> ---
Couple of things I'd like you to fix in the future:
1. you sent a v2 minutes after v1, with no changelog or reference
(ideally you should include something like:
Changes since v1:
- foo
- bar
- baz
Link to v1: ...)
You can use the b4 tool [1] which makes this really easy for everyone!
2. You overtook the series from your colleague. It's okay, but:
- you should have mentioned this (see point 1)
- it's fine for your colleagues to send your patches, so long as all
of the co-authors AND the person sending add their Signed-off-by tags
Also, check out [2].
Aside from that, this patch looks sane to me.
Acked-by: Konrad Dybcio <[email protected]>
[1] https://b4.docs.kernel.org/en/latest/index.html
[2] https://www.kernel.org/doc/html/latest/process/submitting-patches.html
Konrad
> arch/arm64/boot/dts/qcom/sa8775p-ride.dts | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sa8775p-ride.dts b/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
> index ab767cfa51ff..26f5a4e0ffed 100644
> --- a/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
> +++ b/arch/arm64/boot/dts/qcom/sa8775p-ride.dts
> @@ -153,8 +153,8 @@
>
> vreg_l4c: ldo4 {
> regulator-name = "vreg_l4c";
> - regulator-min-microvolt = <1100000>;
> - regulator-max-microvolt = <1300000>;
> + regulator-min-microvolt = <1200000>;
> + regulator-max-microvolt = <1200000>;
> regulator-initial-mode = <RPMH_REGULATOR_MODE_HPM>;
> /*
> * FIXME: This should have regulator-allow-set-load but
On Tue, 11 Jul 2023 16:29:14 +0530, Naveen Kumar Goud Arepalli wrote:
> L4c is the supply for UFS vccq, As per UFS spec range of vccq is
> 1.14V to 1.26V, There are stability issues when operating at
> marginal voltage. Hence configure the min and max vccq voltages
> to 1.2V.
>
>
Applied, thanks!
[1/1] arm64: dts: qcom: sa8775p-ride: Update L4C parameters
commit: e608d16e01456f82738bd98f6f5ca9f46634369d
Best regards,
--
Bjorn Andersson <[email protected]>