From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: Rob Clark <robdclark@gmail.com>,
Abhinav Kumar <quic_abhinavk@quicinc.com>,
Dmitry Baryshkov <dmitry.baryshkov@linaro.org>,
Sean Paul <sean@poorly.run>,
Marijn Suijten <marijn.suijten@somainline.org>,
David Airlie <airlied@gmail.com>,
Simona Vetter <simona@ffwll.ch>,
Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
Maxime Ripard <mripard@kernel.org>,
Thomas Zimmermann <tzimmermann@suse.de>,
Rob Herring <robh@kernel.org>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Krishna Manikandan <quic_mkrishn@quicinc.com>,
Jonathan Marek <jonathan@marek.ca>,
Kuogee Hsieh <quic_khsieh@quicinc.com>,
Neil Armstrong <neil.armstrong@linaro.org>
Cc: linux-arm-msm@vger.kernel.org, dri-devel@lists.freedesktop.org,
freedreno@lists.freedesktop.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org,
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
Srini Kandagatla <srinivas.kandagatla@linaro.org>
Subject: [PATCH v2 04/16] dt-bindings: display/msm: dsi-controller-main: Add SM8750
Date: Mon, 17 Feb 2025 17:41:25 +0100 [thread overview]
Message-ID: <20250217-b4-sm8750-display-v2-4-d201dcdda6a4@linaro.org> (raw)
In-Reply-To: <20250217-b4-sm8750-display-v2-0-d201dcdda6a4@linaro.org>
Add DSI controller for Qualcomm SM8750 SoC which is quite different from
previous (SM8650) generation.
It does not allow the display clock controller clocks like "byte" and
"pixel" to be reparented to DSI PHY PLLs while the DSI PHY PLL is not
configured (not prepared, rate not set). Therefore
assigned-clock-parents are not working here and driver is responsible
for reparenting clocks with proper procedure. These clocks are now
inputs to the DSI controller device.
Except that SM8750 DSI comes with several differences, new blocks and
changes in registers, making it incompatible with SM8650.
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
.../bindings/display/msm/dsi-controller-main.yaml | 54 ++++++++++++++++++++--
1 file changed, 49 insertions(+), 5 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 2aab33cd0017cd4a0c915b7297bb3952e62561fa..8ecb2d8e296edf555df7380eac284b41a3f000a5 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -40,6 +40,7 @@ properties:
- qcom,sm8450-dsi-ctrl
- qcom,sm8550-dsi-ctrl
- qcom,sm8650-dsi-ctrl
+ - qcom,sm8750-dsi-ctrl
- const: qcom,mdss-dsi-ctrl
- enum:
- qcom,dsi-ctrl-6g-qcm2290
@@ -68,11 +69,11 @@ properties:
- mnoc:: MNOC clock
- pixel:: Display pixel clock.
minItems: 3
- maxItems: 9
+ maxItems: 12
clock-names:
minItems: 3
- maxItems: 9
+ maxItems: 12
phys:
maxItems: 1
@@ -107,7 +108,8 @@ properties:
minItems: 2
maxItems: 4
description: |
- Parents of "byte" and "pixel" for the given platform.
+ For DSI on SM8650 and older: parents of "byte" and "pixel" for the given
+ platform.
For DSIv2 platforms this should contain "byte", "esc", "src" and
"pixel_src" clocks.
@@ -216,8 +218,6 @@ required:
- clocks
- clock-names
- phys
- - assigned-clocks
- - assigned-clock-parents
- ports
allOf:
@@ -242,6 +242,9 @@ allOf:
- const: byte
- const: pixel
- const: core
+ required:
+ - assigned-clocks
+ - assigned-clock-parents
- if:
properties:
@@ -264,6 +267,9 @@ allOf:
- const: byte
- const: pixel
- const: core
+ required:
+ - assigned-clocks
+ - assigned-clock-parents
- if:
properties:
@@ -286,6 +292,9 @@ allOf:
- const: pixel
- const: core
- const: core_mmss
+ required:
+ - assigned-clocks
+ - assigned-clock-parents
- if:
properties:
@@ -307,6 +316,9 @@ allOf:
- const: core_mmss
- const: pixel
- const: core
+ required:
+ - assigned-clocks
+ - assigned-clock-parents
- if:
properties:
@@ -342,6 +354,35 @@ allOf:
- const: core
- const: iface
- const: bus
+ required:
+ - assigned-clocks
+ - assigned-clock-parents
+
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,sm8750-dsi-ctrl
+ then:
+ properties:
+ clocks:
+ minItems: 12
+ maxItems: 12
+ clock-names:
+ items:
+ - const: byte
+ - const: byte_intf
+ - const: pixel
+ - const: core
+ - const: iface
+ - const: bus
+ - const: dsi_pll_pixel
+ - const: dsi_pll_byte
+ - const: esync
+ - const: osc
+ - const: byte_src
+ - const: pixel_src
- if:
properties:
@@ -365,6 +406,9 @@ allOf:
- const: core_mmss
- const: pixel
- const: core
+ required:
+ - assigned-clocks
+ - assigned-clock-parents
unevaluatedProperties: false
--
2.43.0
next prev parent reply other threads:[~2025-02-17 16:41 UTC|newest]
Thread overview: 54+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-02-17 16:41 [PATCH v2 00/16] drm/msm: Add support for SM8750 Krzysztof Kozlowski
2025-02-17 16:41 ` [PATCH v2 01/16] dt-bindings: display/msm: dsi-controller-main: Combine if:then: entries Krzysztof Kozlowski
2025-02-19 23:19 ` Rob Herring (Arm)
2025-02-17 16:41 ` [PATCH v2 02/16] dt-bindings: display/msm: dsi-controller-main: Add missing minItems Krzysztof Kozlowski
2025-02-19 23:20 ` Rob Herring (Arm)
2025-02-17 16:41 ` [PATCH v2 03/16] dt-bindings: display/msm: dsi-phy-7nm: Add SM8750 Krzysztof Kozlowski
2025-02-19 23:20 ` Rob Herring (Arm)
2025-02-17 16:41 ` Krzysztof Kozlowski [this message]
2025-02-19 23:21 ` [PATCH v2 04/16] dt-bindings: display/msm: dsi-controller-main: " Rob Herring (Arm)
2025-02-17 16:41 ` [PATCH v2 05/16] dt-bindings: display/msm: dp-controller: " Krzysztof Kozlowski
2025-02-17 18:58 ` Dmitry Baryshkov
2025-02-19 17:02 ` Krzysztof Kozlowski
2025-02-19 17:08 ` Dmitry Baryshkov
2025-02-24 18:59 ` Abhinav Kumar
2025-02-25 3:14 ` Dmitry Baryshkov
2025-03-03 21:23 ` Abhinav Kumar
2025-03-04 0:06 ` Dmitry Baryshkov
2025-02-17 16:41 ` [PATCH v2 06/16] dt-bindings: display/msm: qcom,sm8650-dpu: " Krzysztof Kozlowski
2025-02-19 23:23 ` Rob Herring (Arm)
2025-02-17 16:41 ` [PATCH v2 07/16] dt-bindings: display/msm: qcom,sm8750-mdss: " Krzysztof Kozlowski
2025-02-19 23:27 ` Rob Herring
2025-02-17 16:41 ` [PATCH v2 08/16] drm/msm/dpu: Drop useless comments Krzysztof Kozlowski
2025-02-17 19:22 ` Dmitry Baryshkov
2025-02-19 21:19 ` Jessica Zhang
2025-02-17 16:41 ` [PATCH v2 09/16] drm/msm/dpu: Add LM_7, DSC_[67], PP_[67] and MERGE_3D_5 Krzysztof Kozlowski
2025-02-17 19:00 ` Dmitry Baryshkov
2025-02-19 21:22 ` Jessica Zhang
2025-02-19 21:24 ` Jessica Zhang
2025-02-17 16:41 ` [PATCH v2 10/16] drm/msm/dpu: Add handling of LM_6 and LM_7 bits in pending flush mask Krzysztof Kozlowski
2025-02-17 19:01 ` Dmitry Baryshkov
2025-02-19 21:28 ` Jessica Zhang
2025-02-17 16:41 ` [PATCH v2 11/16] drm/msm/dsi/phy: Add support for SM8750 Krzysztof Kozlowski
2025-02-17 19:03 ` Dmitry Baryshkov
2025-02-20 0:50 ` Jessica Zhang
2025-02-21 10:41 ` Krzysztof Kozlowski
2025-02-21 10:43 ` Krzysztof Kozlowski
2025-02-17 16:41 ` [PATCH v2 12/16] drm/msm/dsi: " Krzysztof Kozlowski
2025-02-17 19:05 ` Dmitry Baryshkov
2025-02-21 11:14 ` Krzysztof Kozlowski
2025-02-21 11:14 ` Krzysztof Kozlowski
2025-02-17 16:41 ` [PATCH v2 13/16] drm/msm/dpu: " Krzysztof Kozlowski
2025-02-17 19:07 ` Dmitry Baryshkov
2025-02-17 16:41 ` [PATCH v2 14/16] drm/msm/dpu: Add missing "fetch" name to set_active_pipes() Krzysztof Kozlowski
2025-02-17 19:08 ` Dmitry Baryshkov
2025-02-20 22:52 ` Jessica Zhang
2025-02-17 16:41 ` [PATCH v2 15/16] drm/msm/dpu: Implement new v12.0 DPU differences Krzysztof Kozlowski
2025-02-17 19:18 ` Dmitry Baryshkov
2025-02-19 17:04 ` Krzysztof Kozlowski
2025-02-19 17:24 ` Dmitry Baryshkov
2025-02-21 12:36 ` Krzysztof Kozlowski
2025-02-21 13:33 ` Dmitry Baryshkov
2025-02-17 16:41 ` [PATCH v2 16/16] drm/msm/mdss: Add support for SM8750 Krzysztof Kozlowski
2025-02-19 21:17 ` [PATCH v2 00/16] drm/msm: " Jessica Zhang
2025-02-19 22:02 ` Dmitry Baryshkov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20250217-b4-sm8750-display-v2-4-d201dcdda6a4@linaro.org \
--to=krzysztof.kozlowski@linaro.org \
--cc=airlied@gmail.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dmitry.baryshkov@linaro.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=freedreno@lists.freedesktop.org \
--cc=jonathan@marek.ca \
--cc=krzk+dt@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=marijn.suijten@somainline.org \
--cc=mripard@kernel.org \
--cc=neil.armstrong@linaro.org \
--cc=quic_abhinavk@quicinc.com \
--cc=quic_khsieh@quicinc.com \
--cc=quic_mkrishn@quicinc.com \
--cc=robdclark@gmail.com \
--cc=robh@kernel.org \
--cc=sean@poorly.run \
--cc=simona@ffwll.ch \
--cc=srinivas.kandagatla@linaro.org \
--cc=tzimmermann@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).