Skip to content

arm64: dts: qcom: qcs6490-rb3gen2: fix root node overlay for phy-supply#933

Merged
shashim-quic merged 1 commit intoqualcomm-linux:tech/overlay/dtfrom
ayaan-anwar:phy_supply_fix
Apr 13, 2026
Merged

arm64: dts: qcom: qcs6490-rb3gen2: fix root node overlay for phy-supply#933
shashim-quic merged 1 commit intoqualcomm-linux:tech/overlay/dtfrom
ayaan-anwar:phy_supply_fix

Conversation

@ayaan-anwar
Copy link
Copy Markdown
Contributor

The staging overlay used a bare "/ {" root node block to define the qep_vreg and aqr_vreg fixed regulators. In a DT overlay, a bare "/ {" creates a new root fragment that is not merged into the base tree's root node; as a result the regulator nodes are never instantiated and the tc956x driver cannot resolve the phy-supply phandle, leading to a failed MDIO probe:

tc956x_pci-eth 0001:05:00.1: No PHY found

Replace "/ {" with the overlay-correct "&{/} {" syntax so that the fragment is properly applied as an amendment to the base device-tree root node. This ensures the regulator nodes are present at boot and the PHY powers up correctly.

CRs-Fixed: 4499524

… phy-vreg

The staging overlay used a bare "/ {" root node block to define the
qep_vreg and aqr_vreg fixed regulators.  In a DT overlay, a bare "/ {"
creates a new root fragment that is not merged into the base tree's
root node; as a result the regulator nodes are never instantiated and
the tc956x driver cannot resolve the phy-supply phandle, leading to a
failed MDIO probe:

  tc956x_pci-eth 0001:05:00.1: No PHY found

Replace "/ {" with the overlay-correct "&{/} {" syntax so that the
fragment is properly applied as an amendment to the base device-tree
root node.  This ensures the regulator nodes are present at boot and
the PHY powers up correctly.

Fixes: 00ba37e ("PENDING: arm64: dts: qcom: rb3gen2: add overlay for QPS615 ethernet")
Signed-off-by: Mohd Ayaan Anwar <mohd.anwar@oss.qualcomm.com>
@shashim-quic shashim-quic merged commit 7e31633 into qualcomm-linux:tech/overlay/dt Apr 13, 2026
5 of 6 checks passed
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