Re: [PATCH 2/3] phy: qcom: edp: Add per-version LDO configuration callback

From: Yongxing Mou

Date: Tue Feb 10 2026 - 05:20:02 EST




On 2/6/2026 6:52 PM, Konrad Dybcio wrote:
On 2/5/26 10:20 AM, Yongxing Mou wrote:
Introduce the com_ldo_config callback to support per‑PHY LDO
configuration.

Signed-off-by: Yongxing Mou <yongxing.mou@xxxxxxxxxxxxxxxx>
---

[...]

+static int qcom_edp_ldo_config_v4(const struct qcom_edp *edp)
+{
+ const struct phy_configure_opts_dp *dp_opts = &edp->dp_opts;
+ u32 ldo_config;
+
+ if (!edp->is_edp)
+ ldo_config = 0x0;
+ else if (dp_opts->link_rate <= 2700)
+ ldo_config = 0xC1;

lowercase hex, please

Got it. will fix next version
+ else
+ ldo_config = 0x81;
+
+ writel(ldo_config, edp->tx0 + TXn_LDO_CONFIG);
+ writel(ldo_config, edp->tx1 + TXn_LDO_CONFIG);

tx1 should be dp_ops->lanes ? 2 : ldo_config : 0x00, in all cases,
I believe

Konrad
i check the HPG, yes , here need to consider dp_ops->lanes.. will fix it next patch.