[PATCH v4 0/9] phy: qualcomm: qmp-combo: update DP PHY PLL programming on Glymur
From: Nabige Aala
Date: Tue Sep 08 2026 - 10:37:04 EST
The existing DP PHY PLL and AUX configuration for the Glymur platform
does not fully follow the Hardware Programming Guide requirements for
DP over Type-C, which results in DP link bring-up failures.
Update the DP PHY programming sequence and PLL-related register
settings to align with the latest HPG recommendations. With this
change, DP link training completes successfully on Glymur-based
platforms.
Fixes: d10736db98d2 ("phy: qualcomm: qmp-combo: Add DP offsets and
settings for Glymur platforms")
Fixes: d10736db98d25 ("phy: qualcomm: qmp-combo: Add DP offsets and
settings for Glymur platforms")
Signed-off-by: Ritesh Kumar <ritesh.kumar@xxxxxxxxxxxxxxxx>
Signed-off-by: Mahadevan P <mahadevan.p@xxxxxxxxxxxxxxxx>
Signed-off-by: Nabige Aala <nabige.aala@xxxxxxxxxxxxxxxx>
Changes in v2:
- Use the DP-specific v8 COM status register definitions
(DP_QSERDES_V8_COM_C_READY_STATUS and DP_QSERDES_V8_COM_CMN_STATUS)
instead of adding new N3 USB/DP variants.(Konrad)
- Drop the qmp_v8_combo_configure_dp_mode() implementation and
reuse the common qmp_combo_configure_dp_mode() helper, restoring
orientation-specific DP_PHY_MODE programming (0x4c/0x5c) and addressing
review comments regarding DP_PHY_MODE handling.
- Include phy-qcom-qmp-dp-qserdes-com-v8.h for the DP COM register
definitions.
- Update patch author to Ritesh Kumar.(Konrad)
- Link to v1:
https://lore.kernel.org/all/20260419-glymur_dp-v1-1-ad1067a8e8ae@xxxxxxxxxxxxxxxx/
Changes in v3:
- Split the monolithic patch into a logical 4-patch series (Dmitry):
- Patch 1: Add DP COM v8 header and fix register layout
- Patch 2: Update DP PHY common init tables
- Patch 3: Update link rate specific DP PHY tables
- Patch 4: Rework DP PHY runtime configuration
- Improved commit messages with detailed explanations for each change
- Better separation of concerns: header additions, table updates, and
runtime logic changes are now in separate patches for easier review
- Link to v2:
https://lore.kernel.org/all/20260721-glymur-phy-conf-v1-1-7c8909552c5e@xxxxxxxxxxxxxxxx/
Changes in v4:
- Split patch 4 ("Rework DP PHY runtime configuration") into 4 separate
patches to address review feedback (Mani)
- Add named constants QSERDES_DP_PHY_CFG1_V456 (0x0f) and
QSERDES_DP_PHY_CFG1_V8 (0x2f) for the DP_PHY_CFG_1 magic values;
use them via a new dp_phy_cfg1 field in qmp_phy_cfg (Mani)
- Set QSERDES_DP_PHY_CFG_1 to 0x2f on Glymur instead of 0x0f (Konrad)
- Update qmp_combo_configure_dp_mode() to ignore TypeC reverse state
on Glymur via a new dp_mode_ignore_reverse flag in qmp_phy_cfg
(Konrad)
- Fix ignored return value of cfg->configure_dp_phy() in
qmp_combo_dp_power_on() (Konrad)
- Make AUXLESS_SETUP_CYC, AUXLESS_SILENCE_CYC and LFPS_PERIOD writes
in qmp_v8_configure_dp_clocks() rate-dependent (Konrad)
- Remove duplicated LFPS_PERIOD write in qmp_v8_configure_dp_clocks()
(Konrad)
- Make LN0/LN1_DRV_LVL writes swing/pre-emphasis level dependent via
new qmp_dp_v8_ln_drv_lvl_hbr_rbr and qmp_dp_v8_ln_drv_lvl_hbr3_hbr2
tables
- Make 2-lane drvr0_en/drvr1_en values in qmp_v8_configure_dp_phy()
depend on the reverse state (Konrad)
- Patches 1-3 are unchanged from v3
- Link to v3:
https://patch.msgid.link/20260828-glymur-phy-v3-v3-0-8e73ce7c4636@xxxxxxxxxxxxxxxx
---
Nabige Aala (1):
phy: qualcomm: qmp-combo: Fix error propagation in qmp_combo_dp_power_on()
Ritesh Kumar (8):
phy: qualcomm: qmp-combo: Add DP COM v8 header and fix register layout
phy: qualcomm: qmp-combo: Update DP PHY common init tables
phy: qualcomm: qmp-combo: Update link rate specific DP PHY tables
phy: qualcomm: qmp-combo: Add hardware-specific DP PHY config fields
phy: qualcomm: qmp-combo: Add v8 PCS-level drive level tables for Glymur
phy: qualcomm: qmp-combo: Extract common DP PHY init sequence
phy: qualcomm: qmp-combo: Update qmp_v8_configure_dp_clocks() for Glymur
phy: qualcomm: qmp-combo: Fix Glymur v8 DP PHY init and configure sequences
drivers/phy/qualcomm/phy-qcom-qmp-combo.c | 356 +++++++++++++++++++++---------
1 file changed, 250 insertions(+), 106 deletions(-)
---
base-commit: e30626823a406725ce29bc75cb8ec467d3e1e326
change-id: 20260824-glymur-phy-v3-b67c4702dfb2
Best regards,
--
Nabige Aala <nabige.aala@xxxxxxxxxxxxxxxx>