Re: [PATCH RFT v3 0/2] arm64: dts: qcom: glymur: Enable SoC-wise display and eDP panel on CRD

[Bjorn Andersson]

On Tue, Mar 03, 2026 at 12:44:07PM +0200, Abel Vesa wrote:
> Start by describing the MDSS (Mobile Display SubSystem), the MDP
> (Mobile Display Processor) and the 4 DisplayPort controllers it brings,
> then describe the PHY used for eDP and tie up the PHY provided clocks
> to the Display clock controller.
> 
> Do all this in order to enable the eDP panel the CRD comes with.
> 
> Sent as an RFT since it was only boot-tested on a remote-only accessible
> device.
> 

This doesn't apply, am I doing something wrong?

> Signed-off-by: Abel Vesa <abel.vesa@xxxxxxxxxxxxxxxx>
> ---
> Changes in v3:
> - Fixed opp table according to Konrad's suggestion.
> - Added missing reg regions for all DP controllers, as Konrad suggested.
> - Fixed all sizes of the reg ranges.
> - Replaced all 0s with 0x0 in all reg ranges.
> - Added missing clock name entry reported by Dmitry. 
> - Link to v2: https://patch.msgid.link/20260113-dts-qcom-glymur-crd-add-edp-v2-0-8026af65ecbb@xxxxxxxxxxxxxxxx
> 
> Changes in v2:
> - Add missing PIXEL1 clock to DPs [0-2]
> - Use the same opp table for all DPs and drop the dedicated ones.
> - Drop the extra compatible from DP1.
> - Changed compatible for the panel to samsung,atna60cl08, as that is the
>   actual model.
> - Link to v1: https://patch.msgid.link/20250925-dts-qcom-glymur-crd-add-edp-v1-0-20233de3c1e2@xxxxxxxxxx
> 
> ---
> Abel Vesa (2):
>       arm64: dts: qcom: glymur: Describe display related nodes
>       arm64: dts: qcom: glymur-crd: Enable eDP display support
> 
>  arch/arm64/boot/dts/qcom/glymur-crd.dts |  71 +++++
>  arch/arm64/boot/dts/qcom/glymur.dtsi    | 471 +++++++++++++++++++++++++++++++-
>  2 files changed, 534 insertions(+), 8 deletions(-)
> ---
> base-commit: 767cea52f08277557c8ba0e37638f2e7da271677
> change-id: 20260109-dts-qcom-glymur-crd-add-edp-03f0adde9750
> prerequisite-change-id: 20260227-phy-qcom-m31-eusb2-make-repeater-optional-621c8c1c0354:v2
> prerequisite-patch-id: df42484b224c01014637ec5a8f56bab459890557
> prerequisite-change-id: 20260109-dts-qcom-glymur-add-usb-support-617b6d9d032c:v3
> prerequisite-patch-id: 66643de4d7142692ceee6ab78d4c1fb446182123
> prerequisite-patch-id: 056da4cc346b633ccf7e12536839eeefa9469b78
> prerequisite-patch-id: 3d3c5004e30407229b8f6612ee2c56dd6171447c
> prerequisite-patch-id: a4ed5f8f3d10b47b0d1daee2e0dc44090c13c01c
> prerequisite-patch-id: 144090e55a19a1479f4b35b75f9e5b80a9b919f0
> prerequisite-patch-id: a9ccb06216435308c295e2de9adffb79060439cf
> prerequisite-patch-id: 1e8d403675640d7db68a4c0caf28d1b1be895e8a
> prerequisite-patch-id: 09bf515a2cd6bec5b21f15b18bebdb172f4b4a57
> prerequisite-patch-id: e88699eb550ada640f5f2f5e4f432d6a3ae2552f
> prerequisite-patch-id: df42484b224c01014637ec5a8f56bab459890557
> prerequisite-patch-id: d986d8d948eaf7b80028b2244750dc7aff7de307
> prerequisite-patch-id: 7ec5f802a334d96421d8f95d4d9e9773655cc947
> prerequisite-patch-id: 460edb2664f266b4f33fb213e88316ba9402b3d5
> prerequisite-change-id: 20260227-glymur-fix-dp-bindings-reg-clocks-704d0ccbeef9:v4
> prerequisite-patch-id: 64ec868b066c682f08ff9845e4507cbf7f8f671d
> 

Do you really have all these dependencies? Do I have all these
dependencies?

Regards,
Bjorn

> Best regards,
> --  
> Abel Vesa <abel.vesa@xxxxxxxxxxxxxxxx>
>