Skip to content

Add qref and refgen regulator vote for QCS8300 and SA8775p PHY - #1731

Open
qianyu-qcom wants to merge 2 commits into
qualcomm-linux:tech/bus/pci/phyfrom
qianyu-qcom:tech/bus/pci/phy
Open

Add qref and refgen regulator vote for QCS8300 and SA8775p PHY#1731
qianyu-qcom wants to merge 2 commits into
qualcomm-linux:tech/bus/pci/phyfrom
qianyu-qcom:tech/bus/pci/phy

Conversation

@qianyu-qcom

Copy link
Copy Markdown
Add a new sa8775p_qmp_phy_vreg_l that includes vdda-phy, vdda-pll,
vdda-qref, vdda-refgen and refgen supplies, and use it for QCS8300
and SA8775p PCIe PHY configurations. This avoids modifying
sm8550_qmp_phy_vreg_l and breaking SM8550 and SM8650.

Note that due to a hardware issue, QREF actually depends on refgen3
rather than refgen2 as documented; refgen3 is therefore voted manually
via the refgen supply as a workaround.

@qcomlnxci
qcomlnxci requested review from a team, krishnachaitanya-linux and Matthew Leung (meleung) and removed request for a team August 24, 2026 02:34
Ziyue Zhang added 2 commits August 24, 2026 10:37
…en and refgen supply properties

QCS8300 and SA8775p PCIe PHYs require dedicated vdda-refgen and refgen
LDO supplies for QREF to provide a stable reference clock. Without these
supplies, the PCIe PHYs cannot work properly. Mark them as required for
the affected compatibles even though it breaks ABI.

vdda-refgen-supply is required for all three QCS8300 and SA8775p PHY
compatibles. refgen-supply is required for QCS8300 only, as a workaround
for a hardware issue where QREF actually depends on refgen3 rather than
refgen2 as documented, so the PHY driver votes for refgen3 directly via
this supply.

Link: https://lore.kernel.org/all/20260810112735.1326807-2-ziyue.zhang@oss.qualcomm.com/
Signed-off-by: Ziyue Zhang <ziyue.zhang@oss.qualcomm.com>
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@oss.qualcomm.com>
Signed-off-by: Qiang Yu <qiang.yu@oss.qualcomm.com>
… QCS8300 and SA8775p PHY

Add a new sa8775p_qmp_phy_vreg_l that includes vdda-phy, vdda-pll,
vdda-qref, vdda-refgen and refgen supplies, and use it for QCS8300
and SA8775p PCIe PHY configurations. This avoids modifying
sm8550_qmp_phy_vreg_l and breaking SM8550 and SM8650.

Note that due to a hardware issue, QREF actually depends on refgen3
rather than refgen2 as documented; refgen3 is therefore voted manually
via the refgen supply as a workaround.

Link: https://lore.kernel.org/all/20260810112735.1326807-3-ziyue.zhang@oss.qualcomm.com/
Fixes: a05b6d5 ("phy: qcom-qmp-pcie: add support for sa8775p")
Fixes: ebf198f ("phy: qcom-qmp-pcie: add dual lane PHY support for QCS8300")
Signed-off-by: Ziyue Zhang <ziyue.zhang@oss.qualcomm.com>
Signed-off-by: Qiang Yu <qiang.yu@oss.qualcomm.com>
@qcomlnxci
qcomlnxci requested a review from a team August 24, 2026 02:39
@qlijarvis

Copy link
Copy Markdown

🔨 Build Failure Analysis — PR #1731

PR: #1731
Build run: https://github.com/qualcomm-linux/kernel-config/actions/runs/32683775034

# Error File:Line PR-introduced? Root Cause
1 redefinition of 'sa8775p_qmp_phy_vreg_l' drivers/phy/qualcomm/phy-qcom-qmp-pcie.c:3543 Yes PR adds array sa8775p_qmp_phy_vreg_l[] that already exists in base kernel at line 3531, causing duplicate symbol definition

Verdict

1 of 1 errors is introduced by this PR due to a merge conflict with upstream changes.

📎 Detailed analysis: Full report

@qlijarvis

Copy link
Copy Markdown

🔨 Build Failure Analysis — PR #1731

PR: #1731
Build run: https://github.com/qualcomm-linux/kernel-config/actions/runs/32683775034

# Error File:Line PR-introduced? Root Cause
1 redefinition of 'sa8775p_qmp_phy_vreg_l' drivers/phy/qualcomm/phy-qcom-qmp-pcie.c:3543 Yes PR attempts to add sa8775p_qmp_phy_vreg_l[] array, but it already exists in the target kernel tree at line 3531, causing a duplicate definition error

Verdict

1 of 1 errors is introduced by this PR due to a merge conflict with existing code in the target branch.

📎 Detailed analysis: Full report

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.

2 participants