On Wed 19 Jan 15:14 PST 2022, Dmitry Baryshkov wrote:

On 28/12/2021 07:59, Bjorn Andersson wrote:

quoted

The Qualcomm SM8350 platform comes with a single DisplayPort controller,
add support for this in the DisplayPort driver.

Signed-off-by: Bjorn Andersson <redacted>

Reviewed-by: Dmitry Baryshkov <redacted>

I don't see this in linux-next, would it be possible to pick it up now
that we're past the merge window etc?

Regards,
Bjorn

quoted

---
  .../devicetree/bindings/display/msm/dp-controller.yaml    | 1 +
  drivers/gpu/drm/msm/dp/dp_display.c                       | 8 ++++++++
  2 files changed, 9 insertions(+)

diff --git a/Documentation/devicetree/bindings/display/msm/dp-controller.yaml b/Documentation/devicetree/bindings/display/msm/dp-controller.yaml
index 5457612ab136..cd05cfd76536 100644
--- a/Documentation/devicetree/bindings/display/msm/dp-controller.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dp-controller.yaml

@@ -21,6 +21,7 @@ properties:
        - qcom,sc7280-edp
        - qcom,sc8180x-dp
        - qcom,sc8180x-edp
+      - qcom,sm8350-dp
    reg:
      items:

diff --git a/drivers/gpu/drm/msm/dp/dp_display.c b/drivers/gpu/drm/msm/dp/dp_display.c
index 8d9c19dbf33e..fd0fd03f8fed 100644
--- a/drivers/gpu/drm/msm/dp/dp_display.c
+++ b/drivers/gpu/drm/msm/dp/dp_display.c

@@ -143,10 +143,18 @@ static const struct msm_dp_config sc7280_dp_cfg = {
  	.num_descs = 2,
  };
+static const struct msm_dp_config sm8350_dp_cfg = {
+	.descs = (const struct msm_dp_desc[]) {
+		[MSM_DP_CONTROLLER_0] = { .io_start = 0x0ae90000, .connector_type = DRM_MODE_CONNECTOR_DisplayPort },
+	},
+	.num_descs = 1,
+};
+
  static const struct of_device_id dp_dt_match[] = {
  	{ .compatible = "qcom,sc7180-dp", .data = &sc7180_dp_cfg },
  	{ .compatible = "qcom,sc7280-dp", .data = &sc7280_dp_cfg },
  	{ .compatible = "qcom,sc7280-edp", .data = &sc7280_dp_cfg },
+	{ .compatible = "qcom,sm8350-dp", .data = &sm8350_dp_cfg },
  	{}
  };

-- 
With best wishes
Dmitry