Since this new platform supports both DP and eDP, it's the perfect time to drop the dual compatible (eDP and DP) and figure out a different way to specify the mode. After some off-list discussion, one suggested way was to add a 'is-edp' property to the controller node and call phy_set_mode to let the PHY know which mode it should configure itself. The PHY counterpart patchset is here: https://lore.kernel.org/all/20240220-x1e80100-phy-edp-compatible-refactor-v5-0-e8658adf5461@xxxxxxxxxx/ Signed-off-by: Abel Vesa <abel.vesa@xxxxxxxxxx> --- Abel Vesa (3): dt-bindings: display: msm: dp-controller: document X1E80100 compatible drm/msm/dp: Add support for setting the eDP mode from devicetree drm/msm/dp: Add support for the X1E80100 .../bindings/display/msm/dp-controller.yaml | 6 +++++ drivers/gpu/drm/msm/dp/dp_ctrl.c | 11 +++++++++ drivers/gpu/drm/msm/dp/dp_ctrl.h | 1 + drivers/gpu/drm/msm/dp/dp_display.c | 28 +++++++++++++++++++--- 4 files changed, 43 insertions(+), 3 deletions(-) --- base-commit: 2d5c7b7eb345249cb34d42cbc2b97b4c57ea944e change-id: 20231219-x1e80100-display-refactor-connector-e1c66548cae3 Best regards, -- Abel Vesa <abel.vesa@xxxxxxxxxx>
