* [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes
@ 2022-11-07 23:56 Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
` (18 more replies)
0 siblings, 19 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue
V2:
https://www.spinics.net/lists/linux-arm-msm/msg116326.html
- Moves the DSI PHY changes to a different later series.
There are enough dsi-controller-main changes to justify its own
standalone series.
- The original phy-name binding change given discussion with Rob and
Krzysztof became its own standalone series that has since been merged.
https://www.mail-archive.com/dri-devel@lists.freedesktop.org/msg403214.html
- Retains the drop of power-domain from yaml as a required property.
I dug into the available dtsi. The apq8064 doesn't appear to have any
GDSC which can be attached as a power-domain, which means the
power-domain requirement is not universal across the various silicon
versions.
- Adds Dmitry's RB to power-domain drop
- For the clock declarations I've
* I noticed that the simple change I had worked for msm8939 but
subsquently broke other dtsi which drove a bigger change to document
the clocks on a per compatible basis.
* Added compat strings in yaml.
* Moved the allOf down later in the file to acomodate the if/then.
* Number of clocks validated on a per compatible basis
* The driver code which doesn't care about the number of clocks
can still operate on the mdss-dsi-ctrl compat but the dts checks will
validate against the compat string and yaml.
- vdd descriptions
Took the previous text I missed from the .txt file - Krzysztof, Dmitry
Adds vdd, vdda and vddio to the required list. This exposes warnings in
existing dtsi but the previous .txt declared these regulators as
required. - Krzysztof
V1:
This series fixes up a number of dtbs checks which are being flagged adding
in the msm8939 dtsi.
When converting from .txt to .yaml a number of the parameters for the older
msm8916 silicon were not transmitted into the yaml.
Adding in the msm8939 which is a near 1:1 copy of the msm8916 in terms of
dtsi triggers a rake of dtbs checks as a result.
https://www.mail-archive.com/dri-devel@lists.freedesktop.org/msg403211.html
Bryan O'Donoghue (18):
dt-bindings: msm: dsi-controller-main: Fix operating-points-v2
constraint
dt-bindings: msm: dsi-controller-main: Fix power-domain constraint
dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
dt-bindings: msm: dsi-controller-main: Fix clock declarations
dt-bindings: msm: dsi-controller-main: Fix description of core clock
dt-bindings: msm: dsi-controller-main: Alphanumerically sort
compatible enum
dt-bindings: msm: dsi-controller-main: Add compatible strings for
every current SoC
dt-bindings: msm: dsi-controller-main: Document clocks on a per
compatible basis
ARM: dts: qcom: apq8064: add compat qcom,mdss-dsi-ctrl-apq8064
ARM: dts: qcom: msm8974: Add compat qcom,mdss-dsi-ctrl-msm8974
arm64: dts: qcom: msm8916: Add compat qcom,mdss-dsi-ctrl-msm8916
arm64: dts: qcom: msm8996: Add compat qcom,mdss-dsi-ctrl-msm8996
arm64: dts: qcom: sc7180: Add compat qcom,mdss-dsi-ctrl-sc7180
arm64: dts: qcom: sc7280: Add compat qcom,mdss-dsi-ctrl-sc7280
arm64: dts: qcom: sdm630: Add compat qcom,mdss-dsi-ctrl-sdm630
arm64: dts: qcom: sdm660: Add compat qcom,mdss-dsi-ctrl-sdm660
arm64: dts: qcom: sdm845: Add compat qcom,mdss-dsi-ctrl-sdm845
arm64: dts: qcom: sm8250: Add compat qcom,mdss-dsi-ctrl-sm8250
.../display/msm/dsi-controller-main.yaml | 185 ++++++++++++++++--
arch/arm/boot/dts/qcom-apq8064.dtsi | 3 +-
arch/arm/boot/dts/qcom-msm8974.dtsi | 3 +-
arch/arm64/boot/dts/qcom/msm8916.dtsi | 3 +-
arch/arm64/boot/dts/qcom/msm8996.dtsi | 6 +-
arch/arm64/boot/dts/qcom/sc7180.dtsi | 3 +-
arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 +-
arch/arm64/boot/dts/qcom/sdm630.dtsi | 3 +-
arch/arm64/boot/dts/qcom/sdm660.dtsi | 3 +-
arch/arm64/boot/dts/qcom/sdm845.dtsi | 6 +-
arch/arm64/boot/dts/qcom/sm8250.dtsi | 6 +-
11 files changed, 194 insertions(+), 30 deletions(-)
--
2.38.1
^ permalink raw reply [flat|nested] 66+ messages in thread
* [PATCH v2 01/18] dt-bindings: msm: dsi-controller-main: Fix operating-points-v2 constraint
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
` (17 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Krzysztof Kozlowski, Sean Paul,
David Airlie, Daniel Vetter, dri-devel, freedreno, devicetree,
linux-kernel
The existing msm8916.dtsi does not depend on nor require operating points.
Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 1 -
1 file changed, 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 7782bff89afc7..27ebfd5ffb22f 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -135,7 +135,6 @@ required:
- assigned-clocks
- assigned-clock-parents
- power-domains
- - operating-points-v2
- ports
additionalProperties: false
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 01/18] dt-bindings: msm: dsi-controller-main: Fix operating-points-v2 constraint
@ 2022-11-07 23:56 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: freedreno, devicetree, linux-kernel, dri-devel,
Krzysztof Kozlowski, Bryan O'Donoghue, Sean Paul
The existing msm8916.dtsi does not depend on nor require operating points.
Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 1 -
1 file changed, 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 7782bff89afc7..27ebfd5ffb22f 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -135,7 +135,6 @@ required:
- assigned-clocks
- assigned-clock-parents
- power-domains
- - operating-points-v2
- ports
additionalProperties: false
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 02/18] dt-bindings: msm: dsi-controller-main: Fix power-domain constraint
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
` (17 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Sean Paul, David Airlie, Daniel Vetter,
dri-devel, freedreno, devicetree, linux-kernel
power-domain is required for the sc7180 dispcc GDSC but not every qcom SoC
has a similar dependency for example the aqp8064.
Most Qcom SoC's using mdss-dsi-ctrl seem to have the ability to
power-collapse the MDP without collapsing DSI.
For example the qcom vendor kernel commit for apq8084, msm8226, msm8916, msm8974
https://review.carbonrom.org/plugins/gitiles/CarbonROM/android_kernel_oneplus_msm8994/+/7b5c011a770daa2811778937ed646237a28a8694
"ARM: dts: msm: add mdss gdsc supply to dsi controller device
It is possible for the DSI controller to be active when MDP is
power collapsed. DSI controller needs to have it's own vote for
mdss gdsc to ensure that gdsc remains on in such cases."
This however doesn't appear to be the case for the apq8064 so we shouldn't
be marking power-domain as required in yaml checks.
Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 1 -
1 file changed, 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 27ebfd5ffb22f..cf782c5f5bdb0 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -134,7 +134,6 @@ required:
- phys
- assigned-clocks
- assigned-clock-parents
- - power-domains
- ports
additionalProperties: false
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 02/18] dt-bindings: msm: dsi-controller-main: Fix power-domain constraint
@ 2022-11-07 23:56 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel,
Bryan O'Donoghue
power-domain is required for the sc7180 dispcc GDSC but not every qcom SoC
has a similar dependency for example the aqp8064.
Most Qcom SoC's using mdss-dsi-ctrl seem to have the ability to
power-collapse the MDP without collapsing DSI.
For example the qcom vendor kernel commit for apq8084, msm8226, msm8916, msm8974
https://review.carbonrom.org/plugins/gitiles/CarbonROM/android_kernel_oneplus_msm8994/+/7b5c011a770daa2811778937ed646237a28a8694
"ARM: dts: msm: add mdss gdsc supply to dsi controller device
It is possible for the DSI controller to be active when MDP is
power collapsed. DSI controller needs to have it's own vote for
mdss gdsc to ensure that gdsc remains on in such cases."
This however doesn't appear to be the case for the apq8064 so we shouldn't
be marking power-domain as required in yaml checks.
Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 1 -
1 file changed, 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 27ebfd5ffb22f..cf782c5f5bdb0 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -134,7 +134,6 @@ required:
- phys
- assigned-clocks
- assigned-clock-parents
- - power-domains
- ports
additionalProperties: false
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
` (17 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Sean Paul, David Airlie, Daniel Vetter,
dri-devel, freedreno, devicetree, linux-kernel
When converting from .txt to .yaml we didn't include descriptions for the
existing regulator supplies.
- vdd
- vdda
- vddio
Add those descriptions into the yaml now as they were prior to the
conversion. Mark the supplies as required as was previously the case in the
.txt implementation.
Warnings about missing regulators can be resolved by updating the relevant
dtsi files to point to fixed always-on regulators where appropriate.
Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../bindings/display/msm/dsi-controller-main.yaml | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index cf782c5f5bdb0..0f7747e55b9be 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -124,6 +124,18 @@ properties:
- port@0
- port@1
+ vdd-supply:
+ description:
+ Phandle to vdd regulator device node
+
+ vddio-supply:
+ description:
+ Phandle to vdd-io regulator device node
+
+ vdda-supply:
+ description:
+ Phandle to vdda regulator device node
+
required:
- compatible
- reg
@@ -135,6 +147,9 @@ required:
- assigned-clocks
- assigned-clock-parents
- ports
+ - vdd-supply
+ - vddio-supply
+ - vdda-supply
additionalProperties: false
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
@ 2022-11-07 23:56 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel,
Bryan O'Donoghue
When converting from .txt to .yaml we didn't include descriptions for the
existing regulator supplies.
- vdd
- vdda
- vddio
Add those descriptions into the yaml now as they were prior to the
conversion. Mark the supplies as required as was previously the case in the
.txt implementation.
Warnings about missing regulators can be resolved by updating the relevant
dtsi files to point to fixed always-on regulators where appropriate.
Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../bindings/display/msm/dsi-controller-main.yaml | 15 +++++++++++++++
1 file changed, 15 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index cf782c5f5bdb0..0f7747e55b9be 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -124,6 +124,18 @@ properties:
- port@0
- port@1
+ vdd-supply:
+ description:
+ Phandle to vdd regulator device node
+
+ vddio-supply:
+ description:
+ Phandle to vdd-io regulator device node
+
+ vdda-supply:
+ description:
+ Phandle to vdda regulator device node
+
required:
- compatible
- reg
@@ -135,6 +147,9 @@ required:
- assigned-clocks
- assigned-clock-parents
- ports
+ - vdd-supply
+ - vddio-supply
+ - vdda-supply
additionalProperties: false
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 04/18] dt-bindings: msm: dsi-controller-main: Fix clock declarations
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
` (17 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Sean Paul, David Airlie, Daniel Vetter,
dri-devel, freedreno, devicetree, linux-kernel
When converting from .txt to .yaml dt-binding descriptions we appear to
have missed some of the previous detail on the number and names of
permissible clocks.
Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../bindings/display/msm/dsi-controller-main.yaml | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 0f7747e55b9be..cab38a20a54b0 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -35,6 +35,10 @@ properties:
- description: Display escape clock
- description: Display AHB clock
- description: Display AXI clock
+ - description: Core MultiMedia SubSystem clock
+ - description: MDP Core clock
+ - description: MNOC clock
+ minItems: 6
clock-names:
items:
@@ -44,6 +48,10 @@ properties:
- const: core
- const: iface
- const: bus
+ - const: core_mmss
+ - const: mdp_core
+ - const: mnoc
+ minItems: 6
phys:
maxItems: 1
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 04/18] dt-bindings: msm: dsi-controller-main: Fix clock declarations
@ 2022-11-07 23:56 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel,
Bryan O'Donoghue
When converting from .txt to .yaml dt-binding descriptions we appear to
have missed some of the previous detail on the number and names of
permissible clocks.
Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../bindings/display/msm/dsi-controller-main.yaml | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 0f7747e55b9be..cab38a20a54b0 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -35,6 +35,10 @@ properties:
- description: Display escape clock
- description: Display AHB clock
- description: Display AXI clock
+ - description: Core MultiMedia SubSystem clock
+ - description: MDP Core clock
+ - description: MNOC clock
+ minItems: 6
clock-names:
items:
@@ -44,6 +48,10 @@ properties:
- const: core
- const: iface
- const: bus
+ - const: core_mmss
+ - const: mdp_core
+ - const: mnoc
+ minItems: 6
phys:
maxItems: 1
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 05/18] dt-bindings: msm: dsi-controller-main: Fix description of core clock
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
` (17 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Sean Paul, David Airlie, Daniel Vetter,
dri-devel, freedreno, devicetree, linux-kernel
There's a typo in describing the core clock as an 'escape' clock. The
accurate description is 'core'.
Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index cab38a20a54b0..a607ccd4a905a 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -32,7 +32,7 @@ properties:
- description: Display byte clock
- description: Display byte interface clock
- description: Display pixel clock
- - description: Display escape clock
+ - description: Display core clock
- description: Display AHB clock
- description: Display AXI clock
- description: Core MultiMedia SubSystem clock
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 05/18] dt-bindings: msm: dsi-controller-main: Fix description of core clock
@ 2022-11-07 23:56 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel,
Bryan O'Donoghue
There's a typo in describing the core clock as an 'escape' clock. The
accurate description is 'core'.
Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index cab38a20a54b0..a607ccd4a905a 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -32,7 +32,7 @@ properties:
- description: Display byte clock
- description: Display byte interface clock
- description: Display pixel clock
- - description: Display escape clock
+ - description: Display core clock
- description: Display AHB clock
- description: Display AXI clock
- description: Core MultiMedia SubSystem clock
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 06/18] dt-bindings: msm: dsi-controller-main: Alphanumerically sort compatible enum
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
` (17 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Sean Paul, David Airlie, Daniel Vetter,
dri-devel, freedreno, devicetree, linux-kernel
Sort the order of the compatible strings alphanumerically.
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index a607ccd4a905a..b35130a77b43e 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -15,8 +15,8 @@ allOf:
properties:
compatible:
enum:
- - qcom,mdss-dsi-ctrl
- qcom,dsi-ctrl-6g-qcm2290
+ - qcom,mdss-dsi-ctrl
reg:
maxItems: 1
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 06/18] dt-bindings: msm: dsi-controller-main: Alphanumerically sort compatible enum
@ 2022-11-07 23:56 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel,
Bryan O'Donoghue
Sort the order of the compatible strings alphanumerically.
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index a607ccd4a905a..b35130a77b43e 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -15,8 +15,8 @@ allOf:
properties:
compatible:
enum:
- - qcom,mdss-dsi-ctrl
- qcom,dsi-ctrl-6g-qcm2290
+ - qcom,mdss-dsi-ctrl
reg:
maxItems: 1
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
` (17 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Sean Paul, David Airlie, Daniel Vetter,
dri-devel, freedreno, devicetree, linux-kernel
Currently we do not differentiate between the various users of the
qcom,mdss-dsi-ctrl. The driver is flexible enough to operate from one
compatible string but, the hardware does have some significant differences
in the number of clocks.
To facilitate documenting the clocks add the following compatible strings
- qcom,mdss-dsi-ctrl-apq8064
- qcom,mdss-dsi-ctrl-msm8916
- qcom,mdss-dsi-ctrl-msm8974
- qcom,mdss-dsi-ctrl-msm8996
- qcom,mdss-dsi-ctrl-sc7180
- qcom,mdss-dsi-ctrl-sc7280
- qcom,mdss-dsi-ctrl-sdm630
- qcom,mdss-dsi-ctrl-sdm660
- qcom,mdss-dsi-ctrl-sdm845
- qcom,mdss-dsi-ctrl-sm8250
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../bindings/display/msm/dsi-controller-main.yaml | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index b35130a77b43e..9db3e63acda3d 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -17,6 +17,16 @@ properties:
enum:
- qcom,dsi-ctrl-6g-qcm2290
- qcom,mdss-dsi-ctrl
+ - qcom,mdss-dsi-ctrl-apq8064
+ - qcom,mdss-dsi-ctrl-msm8916
+ - qcom,mdss-dsi-ctrl-msm8974
+ - qcom,mdss-dsi-ctrl-msm8996
+ - qcom,mdss-dsi-ctrl-sc7180
+ - qcom,mdss-dsi-ctrl-sc7280
+ - qcom,mdss-dsi-ctrl-sdm630
+ - qcom,mdss-dsi-ctrl-sdm660
+ - qcom,mdss-dsi-ctrl-sdm845
+ - qcom,mdss-dsi-ctrl-sm8250
reg:
maxItems: 1
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC
@ 2022-11-07 23:56 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel,
Bryan O'Donoghue
Currently we do not differentiate between the various users of the
qcom,mdss-dsi-ctrl. The driver is flexible enough to operate from one
compatible string but, the hardware does have some significant differences
in the number of clocks.
To facilitate documenting the clocks add the following compatible strings
- qcom,mdss-dsi-ctrl-apq8064
- qcom,mdss-dsi-ctrl-msm8916
- qcom,mdss-dsi-ctrl-msm8974
- qcom,mdss-dsi-ctrl-msm8996
- qcom,mdss-dsi-ctrl-sc7180
- qcom,mdss-dsi-ctrl-sc7280
- qcom,mdss-dsi-ctrl-sdm630
- qcom,mdss-dsi-ctrl-sdm660
- qcom,mdss-dsi-ctrl-sdm845
- qcom,mdss-dsi-ctrl-sm8250
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../bindings/display/msm/dsi-controller-main.yaml | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index b35130a77b43e..9db3e63acda3d 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -17,6 +17,16 @@ properties:
enum:
- qcom,dsi-ctrl-6g-qcm2290
- qcom,mdss-dsi-ctrl
+ - qcom,mdss-dsi-ctrl-apq8064
+ - qcom,mdss-dsi-ctrl-msm8916
+ - qcom,mdss-dsi-ctrl-msm8974
+ - qcom,mdss-dsi-ctrl-msm8996
+ - qcom,mdss-dsi-ctrl-sc7180
+ - qcom,mdss-dsi-ctrl-sc7280
+ - qcom,mdss-dsi-ctrl-sdm630
+ - qcom,mdss-dsi-ctrl-sdm660
+ - qcom,mdss-dsi-ctrl-sdm845
+ - qcom,mdss-dsi-ctrl-sm8250
reg:
maxItems: 1
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 08/18] dt-bindings: msm: dsi-controller-main: Document clocks on a per compatible basis
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
` (17 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Sean Paul, David Airlie, Daniel Vetter,
dri-devel, freedreno, devicetree, linux-kernel
Each compatible has a different set of clocks which are associated with it.
Add in the list of clocks for each compatible.
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../display/msm/dsi-controller-main.yaml | 177 +++++++++++++++---
1 file changed, 150 insertions(+), 27 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 9db3e63acda3d..c975df0ca22fc 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -9,24 +9,22 @@ title: Qualcomm Display DSI controller
maintainers:
- Krishna Manikandan <quic_mkrishn@quicinc.com>
-allOf:
- - $ref: "../dsi-controller.yaml#"
-
properties:
compatible:
- enum:
- - qcom,dsi-ctrl-6g-qcm2290
- - qcom,mdss-dsi-ctrl
- - qcom,mdss-dsi-ctrl-apq8064
- - qcom,mdss-dsi-ctrl-msm8916
- - qcom,mdss-dsi-ctrl-msm8974
- - qcom,mdss-dsi-ctrl-msm8996
- - qcom,mdss-dsi-ctrl-sc7180
- - qcom,mdss-dsi-ctrl-sc7280
- - qcom,mdss-dsi-ctrl-sdm630
- - qcom,mdss-dsi-ctrl-sdm660
- - qcom,mdss-dsi-ctrl-sdm845
- - qcom,mdss-dsi-ctrl-sm8250
+ items:
+ - enum:
+ - qcom,dsi-ctrl-6g-qcm2290
+ - qcom,mdss-dsi-ctrl-apq8064
+ - qcom,mdss-dsi-ctrl-msm8916
+ - qcom,mdss-dsi-ctrl-msm8974
+ - qcom,mdss-dsi-ctrl-msm8996
+ - qcom,mdss-dsi-ctrl-sc7180
+ - qcom,mdss-dsi-ctrl-sc7280
+ - qcom,mdss-dsi-ctrl-sdm630
+ - qcom,mdss-dsi-ctrl-sdm660
+ - qcom,mdss-dsi-ctrl-sdm845
+ - qcom,mdss-dsi-ctrl-sm8250
+ - const: qcom,mdss-dsi-ctrl
reg:
maxItems: 1
@@ -51,17 +49,8 @@ properties:
minItems: 6
clock-names:
- items:
- - const: byte
- - const: byte_intf
- - const: pixel
- - const: core
- - const: iface
- - const: bus
- - const: core_mmss
- - const: mdp_core
- - const: mnoc
minItems: 6
+ maxItems: 9
phys:
maxItems: 1
@@ -169,6 +158,140 @@ required:
- vddio-supply
- vdda-supply
+allOf:
+ - $ref: "../dsi-controller.yaml#"
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-apq8064
+ then:
+ properties:
+ clocks:
+ maxItems: 7
+ clock-names:
+ items:
+ - const: iface
+ - const: bus
+ - const: core_mmss
+ - const: src
+ - const: byte
+ - const: pixel
+ - const: core
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-msm8916
+ then:
+ properties:
+ clocks:
+ maxItems: 6
+ clock-names:
+ items:
+ - const: mdp_core
+ - const: iface
+ - const: bus
+ - const: byte
+ - const: pixel
+ - const: core
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-msm8974
+ then:
+ properties:
+ clocks:
+ maxItems: 3
+ clock-names:
+ items:
+ - const: iface
+ - const: bus
+ - const: vsync
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-msm8996
+ then:
+ properties:
+ clocks:
+ maxItems: 7
+ clock-names:
+ items:
+ - const: mdp_core
+ - const: byte
+ - const: iface
+ - const: bus
+ - const: core_mmss
+ - const: pixel
+ - const: core
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-sc7180
+ - qcom,mdss-dsi-ctrl-sc7280
+ - qcom,mdss-dsi-ctrl-sm8250
+ then:
+ properties:
+ clocks:
+ maxItems: 6
+ clock-names:
+ items:
+ - const: byte
+ - const: byte_intf
+ - const: pixel
+ - const: core
+ - const: iface
+ - const: bus
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-sdm630
+ - qcom,mdss-dsi-ctrl-sdm660
+ then:
+ properties:
+ clocks:
+ maxItems: 9
+ clock-names:
+ items:
+ - const: mdp_core
+ - const: byte
+ - const: byte_intf
+ - const: mnoc
+ - const: iface
+ - const: bus
+ - const: core_mmss
+ - const: pixel
+ - const: core
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-sdm845
+ then:
+ properties:
+ clocks:
+ maxItems: 6
+ clock-names:
+ items:
+ - const: byte
+ - const: byte_intf
+ - const: pixel
+ - const: core
+ - const: iface
+ - const: bus
+
additionalProperties: false
examples:
@@ -179,7 +302,7 @@ examples:
#include <dt-bindings/power/qcom-rpmpd.h>
dsi@ae94000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-sc7180", "qcom,mdss-dsi-ctrl";
reg = <0x0ae94000 0x400>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 08/18] dt-bindings: msm: dsi-controller-main: Document clocks on a per compatible basis
@ 2022-11-07 23:56 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel,
Bryan O'Donoghue
Each compatible has a different set of clocks which are associated with it.
Add in the list of clocks for each compatible.
Cc: Rob Clark <robdclark@gmail.com>
Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Sean Paul <sean@poorly.run>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: dri-devel@lists.freedesktop.org
Cc: freedreno@lists.freedesktop.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
.../display/msm/dsi-controller-main.yaml | 177 +++++++++++++++---
1 file changed, 150 insertions(+), 27 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
index 9db3e63acda3d..c975df0ca22fc 100644
--- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
+++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
@@ -9,24 +9,22 @@ title: Qualcomm Display DSI controller
maintainers:
- Krishna Manikandan <quic_mkrishn@quicinc.com>
-allOf:
- - $ref: "../dsi-controller.yaml#"
-
properties:
compatible:
- enum:
- - qcom,dsi-ctrl-6g-qcm2290
- - qcom,mdss-dsi-ctrl
- - qcom,mdss-dsi-ctrl-apq8064
- - qcom,mdss-dsi-ctrl-msm8916
- - qcom,mdss-dsi-ctrl-msm8974
- - qcom,mdss-dsi-ctrl-msm8996
- - qcom,mdss-dsi-ctrl-sc7180
- - qcom,mdss-dsi-ctrl-sc7280
- - qcom,mdss-dsi-ctrl-sdm630
- - qcom,mdss-dsi-ctrl-sdm660
- - qcom,mdss-dsi-ctrl-sdm845
- - qcom,mdss-dsi-ctrl-sm8250
+ items:
+ - enum:
+ - qcom,dsi-ctrl-6g-qcm2290
+ - qcom,mdss-dsi-ctrl-apq8064
+ - qcom,mdss-dsi-ctrl-msm8916
+ - qcom,mdss-dsi-ctrl-msm8974
+ - qcom,mdss-dsi-ctrl-msm8996
+ - qcom,mdss-dsi-ctrl-sc7180
+ - qcom,mdss-dsi-ctrl-sc7280
+ - qcom,mdss-dsi-ctrl-sdm630
+ - qcom,mdss-dsi-ctrl-sdm660
+ - qcom,mdss-dsi-ctrl-sdm845
+ - qcom,mdss-dsi-ctrl-sm8250
+ - const: qcom,mdss-dsi-ctrl
reg:
maxItems: 1
@@ -51,17 +49,8 @@ properties:
minItems: 6
clock-names:
- items:
- - const: byte
- - const: byte_intf
- - const: pixel
- - const: core
- - const: iface
- - const: bus
- - const: core_mmss
- - const: mdp_core
- - const: mnoc
minItems: 6
+ maxItems: 9
phys:
maxItems: 1
@@ -169,6 +158,140 @@ required:
- vddio-supply
- vdda-supply
+allOf:
+ - $ref: "../dsi-controller.yaml#"
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-apq8064
+ then:
+ properties:
+ clocks:
+ maxItems: 7
+ clock-names:
+ items:
+ - const: iface
+ - const: bus
+ - const: core_mmss
+ - const: src
+ - const: byte
+ - const: pixel
+ - const: core
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-msm8916
+ then:
+ properties:
+ clocks:
+ maxItems: 6
+ clock-names:
+ items:
+ - const: mdp_core
+ - const: iface
+ - const: bus
+ - const: byte
+ - const: pixel
+ - const: core
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-msm8974
+ then:
+ properties:
+ clocks:
+ maxItems: 3
+ clock-names:
+ items:
+ - const: iface
+ - const: bus
+ - const: vsync
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-msm8996
+ then:
+ properties:
+ clocks:
+ maxItems: 7
+ clock-names:
+ items:
+ - const: mdp_core
+ - const: byte
+ - const: iface
+ - const: bus
+ - const: core_mmss
+ - const: pixel
+ - const: core
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-sc7180
+ - qcom,mdss-dsi-ctrl-sc7280
+ - qcom,mdss-dsi-ctrl-sm8250
+ then:
+ properties:
+ clocks:
+ maxItems: 6
+ clock-names:
+ items:
+ - const: byte
+ - const: byte_intf
+ - const: pixel
+ - const: core
+ - const: iface
+ - const: bus
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-sdm630
+ - qcom,mdss-dsi-ctrl-sdm660
+ then:
+ properties:
+ clocks:
+ maxItems: 9
+ clock-names:
+ items:
+ - const: mdp_core
+ - const: byte
+ - const: byte_intf
+ - const: mnoc
+ - const: iface
+ - const: bus
+ - const: core_mmss
+ - const: pixel
+ - const: core
+ - if:
+ properties:
+ compatible:
+ contains:
+ enum:
+ - qcom,mdss-dsi-ctrl-sdm845
+ then:
+ properties:
+ clocks:
+ maxItems: 6
+ clock-names:
+ items:
+ - const: byte
+ - const: byte_intf
+ - const: pixel
+ - const: core
+ - const: iface
+ - const: bus
+
additionalProperties: false
examples:
@@ -179,7 +302,7 @@ examples:
#include <dt-bindings/power/qcom-rpmpd.h>
dsi@ae94000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-sc7180", "qcom,mdss-dsi-ctrl";
reg = <0x0ae94000 0x400>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 09/18] ARM: dts: qcom: apq8064: add compat qcom,mdss-dsi-ctrl-apq8064
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (7 preceding siblings ...)
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 13:22 ` David Heidelberg
2022-11-07 23:56 ` [PATCH v2 10/18] ARM: dts: qcom: msm8974: Add compat qcom,mdss-dsi-ctrl-msm8974 Bryan O'Donoghue
` (9 subsequent siblings)
18 siblings, 1 reply; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Andy Gross, Bjorn Andersson, Konrad Dybcio,
devicetree, linux-kernel, Srinivas Kandagatla, David Heidelberg
Append silicon specific compatible qcom,mdss-dsi-ctrl-apq8064 to the
mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
for apq8064 against the yaml documentation.
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Cc: David Heidelberg <david@ixit.cz>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm/boot/dts/qcom-apq8064.dtsi | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
index d036dff4b14f7..f3624832e6841 100644
--- a/arch/arm/boot/dts/qcom-apq8064.dtsi
+++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
@@ -1287,7 +1287,8 @@ mmss_sfpb: syscon@5700000 {
};
dsi0: dsi@4700000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-apq8064",
+ "qcom,mdss-dsi-ctrl";
label = "MDSS DSI CTRL->0";
#address-cells = <1>;
#size-cells = <0>;
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 10/18] ARM: dts: qcom: msm8974: Add compat qcom,mdss-dsi-ctrl-msm8974
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (8 preceding siblings ...)
2022-11-07 23:56 ` [PATCH v2 09/18] ARM: dts: qcom: apq8064: add compat qcom,mdss-dsi-ctrl-apq8064 Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 11/18] arm64: dts: qcom: msm8916: Add compat qcom,mdss-dsi-ctrl-msm8916 Bryan O'Donoghue
` (8 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Andy Gross, Bjorn Andersson, Konrad Dybcio,
devicetree, linux-kernel, Krzysztof Kozlowski, Brian Masney,
Stephen Boyd
Add silicon specific compatible qcom,mdss-dsi-ctrl-msm8974 to the
mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
for msm8974 against the yaml documentation.
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Brian Masney <masneyb@onstation.org>
Cc: Stephen Boyd <sboyd@codeaurora.org>
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm/boot/dts/qcom-msm8974.dtsi | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi
index 21d943d4b02d7..ac9e4c194051d 100644
--- a/arch/arm/boot/dts/qcom-msm8974.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8974.dtsi
@@ -1545,7 +1545,8 @@ mdp5_intf1_out: endpoint {
};
dsi0: dsi@fd922800 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-msm8974",
+ "qcom,mdss-dsi-ctrl";
reg = <0xfd922800 0x1f8>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 11/18] arm64: dts: qcom: msm8916: Add compat qcom,mdss-dsi-ctrl-msm8916
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (9 preceding siblings ...)
2022-11-07 23:56 ` [PATCH v2 10/18] ARM: dts: qcom: msm8974: Add compat qcom,mdss-dsi-ctrl-msm8974 Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 12/18] arm64: dts: qcom: msm8996: Add compat qcom,mdss-dsi-ctrl-msm8996 Bryan O'Donoghue
` (7 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Andy Gross, Bjorn Andersson, Konrad Dybcio,
devicetree, linux-kernel, Stephan Gerhold, Amit Kucheria,
Srinivas Kandagatla
Add silicon specific compatible qcom,mdss-dsi-ctrl-msm8916 to the
mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
for msm8916 against the yaml documentation.
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Stephan Gerhold <stephan@gerhold.net>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Amit Kucheria <amit.kucheria@linaro.org>
Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8916.dtsi | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8916.dtsi b/arch/arm64/boot/dts/qcom/msm8916.dtsi
index c1d8a1b388d27..febd151318a23 100644
--- a/arch/arm64/boot/dts/qcom/msm8916.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8916.dtsi
@@ -1021,7 +1021,8 @@ mdp5_intf1_out: endpoint {
};
dsi0: dsi@1a98000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-msm8916",
+ "qcom,mdss-dsi-ctrl";
reg = <0x01a98000 0x25c>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 12/18] arm64: dts: qcom: msm8996: Add compat qcom,mdss-dsi-ctrl-msm8996
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (10 preceding siblings ...)
2022-11-07 23:56 ` [PATCH v2 11/18] arm64: dts: qcom: msm8916: Add compat qcom,mdss-dsi-ctrl-msm8916 Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 13/18] arm64: dts: qcom: sc7180: Add compat qcom,mdss-dsi-ctrl-sc7180 Bryan O'Donoghue
` (6 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Andy Gross, Bjorn Andersson, Konrad Dybcio,
devicetree, linux-kernel, Krzysztof Kozlowski,
Srinivas Kandagatla, Yassine Oudjana
Add silicon specific compatible qcom,mdss-dsi-ctrl-msm8996 to the
mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
for msm8996 against the yaml documentation.
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Cc: Yassine Oudjana <y.oudjana@protonmail.com>
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/msm8996.dtsi | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/msm8996.dtsi b/arch/arm64/boot/dts/qcom/msm8996.dtsi
index 081e20a63c610..c1a99b325269f 100644
--- a/arch/arm64/boot/dts/qcom/msm8996.dtsi
+++ b/arch/arm64/boot/dts/qcom/msm8996.dtsi
@@ -968,7 +968,8 @@ mdp5_intf2_out: endpoint {
};
dsi0: dsi@994000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-msm8996",
+ "qcom,mdss-dsi-ctrl";
reg = <0x00994000 0x400>;
reg-names = "dsi_ctrl";
@@ -1035,7 +1036,8 @@ dsi0_phy: dsi-phy@994400 {
};
dsi1: dsi@996000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-msm8996",
+ "qcom,mdss-dsi-ctrl";
reg = <0x00996000 0x400>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 13/18] arm64: dts: qcom: sc7180: Add compat qcom,mdss-dsi-ctrl-sc7180
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (11 preceding siblings ...)
2022-11-07 23:56 ` [PATCH v2 12/18] arm64: dts: qcom: msm8996: Add compat qcom,mdss-dsi-ctrl-msm8996 Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 0:24 ` Doug Anderson
2022-11-07 23:56 ` [PATCH v2 14/18] arm64: dts: qcom: sc7280: Add compat qcom,mdss-dsi-ctrl-sc7280 Bryan O'Donoghue
` (5 subsequent siblings)
18 siblings, 1 reply; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Andy Gross, Bjorn Andersson, Konrad Dybcio,
devicetree, linux-kernel, Douglas Anderson, Rajendra Nayak,
Sai Prakash Ranjan
Add silicon specific compatible qcom,mdss-dsi-ctrl-sc7180 to the
mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
for sc7180 against the yaml documentation.
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Douglas Anderson <dianders@chromium.org>
Cc: Rajendra Nayak <rnayak@codeaurora.org>
Cc: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sc7180.dtsi | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
index d2c374e9d8c03..07acb7f843d62 100644
--- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
@@ -2985,7 +2985,8 @@ opp-460000000 {
};
dsi0: dsi@ae94000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-sc7180",
+ "qcom,mdss-dsi-ctrl";
reg = <0 0x0ae94000 0 0x400>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 14/18] arm64: dts: qcom: sc7280: Add compat qcom,mdss-dsi-ctrl-sc7280
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (12 preceding siblings ...)
2022-11-07 23:56 ` [PATCH v2 13/18] arm64: dts: qcom: sc7180: Add compat qcom,mdss-dsi-ctrl-sc7180 Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 16:41 ` Doug Anderson
2022-11-07 23:56 ` [PATCH v2 15/18] arm64: dts: qcom: sdm630: Add compat qcom,mdss-dsi-ctrl-sdm630 Bryan O'Donoghue
` (4 subsequent siblings)
18 siblings, 1 reply; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Andy Gross, Bjorn Andersson, Konrad Dybcio,
devicetree, linux-kernel, Douglas Anderson, Sibi Sankar,
Sai Prakash Ranjan
Add silicon specific compatible qcom,mdss-dsi-ctrl-sc7280 to the
mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
for sc7280 against the yaml documentation.
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Douglas Anderson <dianders@chromium.org>
Cc: Sibi Sankar <sibis@codeaurora.org>
Cc: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sc7280.dtsi b/arch/arm64/boot/dts/qcom/sc7280.dtsi
index a4d6e866b5999..7d4df38348e4f 100644
--- a/arch/arm64/boot/dts/qcom/sc7280.dtsi
+++ b/arch/arm64/boot/dts/qcom/sc7280.dtsi
@@ -3903,7 +3903,8 @@ opp-506666667 {
};
mdss_dsi: dsi@ae94000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-sc7280",
+ "qcom,mdss-dsi-ctrl";
reg = <0 0x0ae94000 0 0x400>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 15/18] arm64: dts: qcom: sdm630: Add compat qcom,mdss-dsi-ctrl-sdm630
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (13 preceding siblings ...)
2022-11-07 23:56 ` [PATCH v2 14/18] arm64: dts: qcom: sc7280: Add compat qcom,mdss-dsi-ctrl-sc7280 Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 16/18] arm64: dts: qcom: sdm660: Add compat qcom,mdss-dsi-ctrl-sdm660 Bryan O'Donoghue
` (3 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Andy Gross, Bjorn Andersson, Konrad Dybcio,
devicetree, linux-kernel, AngeloGioacchino Del Regno
Add silicon specific compatible qcom,mdss-dsi-ctrl-sdm630 to the
mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
for sdm630 against the yaml documentation.
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: AngeloGioacchino Del Regno <angelogioacchino.delregno@somainline.org>
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sdm630.dtsi | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm630.dtsi b/arch/arm64/boot/dts/qcom/sdm630.dtsi
index 0e7764f66b104..6a5f84f85bc8f 100644
--- a/arch/arm64/boot/dts/qcom/sdm630.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm630.dtsi
@@ -1581,7 +1581,8 @@ opp-412500000 {
};
dsi0: dsi@c994000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-sdm630",
+ "qcom,mdss-dsi-ctrl";
reg = <0x0c994000 0x400>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 16/18] arm64: dts: qcom: sdm660: Add compat qcom,mdss-dsi-ctrl-sdm660
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (14 preceding siblings ...)
2022-11-07 23:56 ` [PATCH v2 15/18] arm64: dts: qcom: sdm630: Add compat qcom,mdss-dsi-ctrl-sdm630 Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 17/18] arm64: dts: qcom: sdm845: Add compat qcom,mdss-dsi-ctrl-sdm845 Bryan O'Donoghue
` (2 subsequent siblings)
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Andy Gross, Bjorn Andersson, Konrad Dybcio,
devicetree, linux-kernel
Add silicon specific compatible qcom,mdss-dsi-ctrl-sdm660 to the
mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
for sdm660 against the yaml documentation.
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sdm660.dtsi | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm660.dtsi b/arch/arm64/boot/dts/qcom/sdm660.dtsi
index 10bf1c45cf6ec..eeaa3932a5233 100644
--- a/arch/arm64/boot/dts/qcom/sdm660.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm660.dtsi
@@ -154,7 +154,8 @@ mdp5_intf2_out: endpoint {
&mdss {
dsi1: dsi@c996000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-sdm660",
+ "qcom,mdss-dsi-ctrl";
reg = <0x0c996000 0x400>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 17/18] arm64: dts: qcom: sdm845: Add compat qcom,mdss-dsi-ctrl-sdm845
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (15 preceding siblings ...)
2022-11-07 23:56 ` [PATCH v2 16/18] arm64: dts: qcom: sdm660: Add compat qcom,mdss-dsi-ctrl-sdm660 Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 16:42 ` Doug Anderson
2022-11-07 23:56 ` [PATCH v2 18/18] arm64: dts: qcom: sm8250: Add compat qcom,mdss-dsi-ctrl-sm8250 Bryan O'Donoghue
2022-11-08 12:56 ` [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Dmitry Baryshkov
18 siblings, 1 reply; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Andy Gross, Bjorn Andersson, Konrad Dybcio,
devicetree, linux-kernel, Douglas Anderson, Krzysztof Kozlowski,
Rajendra Nayak
Add silicon specific compatible qcom,mdss-dsi-ctrl-sdm845 to the
mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
for sdm845 against the yaml documentation.
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Douglas Anderson <dianders@chromium.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Rajendra Nayak <rnayak@codeaurora.org>
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sdm845.dtsi | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi
index 5420205417c81..8d19611362ffb 100644
--- a/arch/arm64/boot/dts/qcom/sdm845.dtsi
+++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi
@@ -4522,7 +4522,8 @@ opp-430000000 {
};
dsi0: dsi@ae94000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-sdm845",
+ "qcom,mdss-dsi-ctrl";
reg = <0 0x0ae94000 0 0x400>;
reg-names = "dsi_ctrl";
@@ -4593,7 +4594,8 @@ dsi0_phy: dsi-phy@ae94400 {
};
dsi1: dsi@ae96000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-sdm845",
+ "qcom,mdss-dsi-ctrl";
reg = <0 0x0ae96000 0 0x400>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* [PATCH v2 18/18] arm64: dts: qcom: sm8250: Add compat qcom,mdss-dsi-ctrl-sm8250
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (16 preceding siblings ...)
2022-11-07 23:56 ` [PATCH v2 17/18] arm64: dts: qcom: sdm845: Add compat qcom,mdss-dsi-ctrl-sdm845 Bryan O'Donoghue
@ 2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 12:56 ` [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Dmitry Baryshkov
18 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-07 23:56 UTC (permalink / raw)
To: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Bryan O'Donoghue, Andy Gross, Bjorn Andersson, Konrad Dybcio,
devicetree, linux-kernel, Jonathan Marek
Add silicon specific compatible qcom,mdss-dsi-ctrl-sm8250 to the
mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
for sm8250 against the yaml documentation.
Cc: Andy Gross <agross@kernel.org>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Cc: Rob Herring <robh+dt@kernel.org>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc: linux-arm-msm@vger.kernel.org
Cc: devicetree@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Cc: Jonathan Marek <jonathan@marek.ca>
Cc: Bjorn Andersson <andersson@kernel.org>
Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
---
arch/arm64/boot/dts/qcom/sm8250.dtsi | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/qcom/sm8250.dtsi b/arch/arm64/boot/dts/qcom/sm8250.dtsi
index 0d47c5b7148e1..9297816369524 100644
--- a/arch/arm64/boot/dts/qcom/sm8250.dtsi
+++ b/arch/arm64/boot/dts/qcom/sm8250.dtsi
@@ -3528,7 +3528,8 @@ opp-460000000 {
};
dsi0: dsi@ae94000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-sm8250",
+ "qcom,mdss-dsi-ctrl";
reg = <0 0x0ae94000 0 0x400>;
reg-names = "dsi_ctrl";
@@ -3619,7 +3620,8 @@ dsi0_phy: dsi-phy@ae94400 {
};
dsi1: dsi@ae96000 {
- compatible = "qcom,mdss-dsi-ctrl";
+ compatible = "qcom,mdss-dsi-ctrl-sm8250",
+ "qcom,mdss-dsi-ctrl";
reg = <0 0x0ae96000 0 0x400>;
reg-names = "dsi_ctrl";
--
2.38.1
^ permalink raw reply related [flat|nested] 66+ messages in thread
* Re: [PATCH v2 13/18] arm64: dts: qcom: sc7180: Add compat qcom,mdss-dsi-ctrl-sc7180
2022-11-07 23:56 ` [PATCH v2 13/18] arm64: dts: qcom: sc7180: Add compat qcom,mdss-dsi-ctrl-sc7180 Bryan O'Donoghue
@ 2022-11-08 0:24 ` Doug Anderson
2022-11-08 12:16 ` Bryan O'Donoghue
0 siblings, 1 reply; 66+ messages in thread
From: Doug Anderson @ 2022-11-08 0:24 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm,
Andy Gross, Bjorn Andersson, Konrad Dybcio, devicetree,
linux-kernel, Rajendra Nayak, Sai Prakash Ranjan
Hi,
On Mon, Nov 7, 2022 at 3:57 PM Bryan O'Donoghue
<bryan.odonoghue@linaro.org> wrote:
>
> Add silicon specific compatible qcom,mdss-dsi-ctrl-sc7180 to the
> mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
> for sc7180 against the yaml documentation.
>
> Cc: Andy Gross <agross@kernel.org>
> Cc: Bjorn Andersson <andersson@kernel.org>
> Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Cc: Douglas Anderson <dianders@chromium.org>
> Cc: Rajendra Nayak <rnayak@codeaurora.org>
> Cc: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/sc7180.dtsi | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> index d2c374e9d8c03..07acb7f843d62 100644
> --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi
> +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi
> @@ -2985,7 +2985,8 @@ opp-460000000 {
> };
>
> dsi0: dsi@ae94000 {
> - compatible = "qcom,mdss-dsi-ctrl";
> + compatible = "qcom,mdss-dsi-ctrl-sc7180",
> + "qcom,mdss-dsi-ctrl";
This seems fine, but I don't think it matches your bindings. Your
bindings says you can have one compatible string. It could be
"qcom,mdss-dsi-ctrl-sc7180" or it could be "qcom,mdss-dsi-ctrl".
...but your device tree has two compatible strings: the SoC specific
one and the fallback one. You need to change your bindings to make
this work.
-Doug
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 13/18] arm64: dts: qcom: sc7180: Add compat qcom,mdss-dsi-ctrl-sc7180
2022-11-08 0:24 ` Doug Anderson
@ 2022-11-08 12:16 ` Bryan O'Donoghue
2022-11-08 16:41 ` Doug Anderson
0 siblings, 1 reply; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-08 12:16 UTC (permalink / raw)
To: Doug Anderson
Cc: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm,
Andy Gross, Bjorn Andersson, Konrad Dybcio, devicetree,
linux-kernel, Rajendra Nayak, Sai Prakash Ranjan
On 08/11/2022 00:24, Doug Anderson wrote:
> This seems fine, but I don't think it matches your bindings. Your
> bindings says you can have one compatible string. It could be
> "qcom,mdss-dsi-ctrl-sc7180" or it could be "qcom,mdss-dsi-ctrl".
> ...but your device tree has two compatible strings: the SoC specific
> one and the fallback one. You need to change your bindings to make
> this work.
With the update in this series the binding has a required const.
If you drop back to just "qcom,mdss-dsi-ctrl-sc7180" you get a warning
like this.
arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb: dsi@ae94000:
compatible: ['qcom,mdss-dsi-ctrl-sc7180'] is too short
If you just have 'qcom,mdss-dsi-ctrl' you get
arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb: dsi@ae94000:
compatible:0: 'qcom,mdss-dsi-ctrl' is not one of
['qcom,dsi-ctrl-6g-qcm2290', 'qcom,mdss-dsi-ctrl-apq8064',
'qcom,mdss-dsi-ctrl-msm8916', 'qcom,mdss-dsi-ctrl-msm8974',
'qcom,mdss-dsi-ctrl-msm8996', 'qcom,mdss-dsi-ctrl-sc7180',
'qcom,mdss-dsi-ctrl-sc7280', 'qcom,mdss-dsi-ctrl-sdm630',
'qcom,mdss-dsi-ctrl-sdm660', 'qcom,mdss-dsi-ctrl-sdm845',
'qcom,mdss-dsi-ctrl-sm8250']
---
bod
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 08/18] dt-bindings: msm: dsi-controller-main: Document clocks on a per compatible basis
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-08 12:43 ` Dmitry Baryshkov
-1 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 12:43 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> Each compatible has a different set of clocks which are associated with it.
> Add in the list of clocks for each compatible.
>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../display/msm/dsi-controller-main.yaml | 177 +++++++++++++++---
> 1 file changed, 150 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index 9db3e63acda3d..c975df0ca22fc 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -9,24 +9,22 @@ title: Qualcomm Display DSI controller
> maintainers:
> - Krishna Manikandan <quic_mkrishn@quicinc.com>
>
> -allOf:
> - - $ref: "../dsi-controller.yaml#"
> -
> properties:
> compatible:
> - enum:
> - - qcom,dsi-ctrl-6g-qcm2290
> - - qcom,mdss-dsi-ctrl
> - - qcom,mdss-dsi-ctrl-apq8064
> - - qcom,mdss-dsi-ctrl-msm8916
> - - qcom,mdss-dsi-ctrl-msm8974
> - - qcom,mdss-dsi-ctrl-msm8996
> - - qcom,mdss-dsi-ctrl-sc7180
> - - qcom,mdss-dsi-ctrl-sc7280
> - - qcom,mdss-dsi-ctrl-sdm630
> - - qcom,mdss-dsi-ctrl-sdm660
> - - qcom,mdss-dsi-ctrl-sdm845
> - - qcom,mdss-dsi-ctrl-sm8250
> + items:
> + - enum:
> + - qcom,dsi-ctrl-6g-qcm2290
> + - qcom,mdss-dsi-ctrl-apq8064
> + - qcom,mdss-dsi-ctrl-msm8916
> + - qcom,mdss-dsi-ctrl-msm8974
> + - qcom,mdss-dsi-ctrl-msm8996
> + - qcom,mdss-dsi-ctrl-sc7180
> + - qcom,mdss-dsi-ctrl-sc7280
> + - qcom,mdss-dsi-ctrl-sdm630
> + - qcom,mdss-dsi-ctrl-sdm660
> + - qcom,mdss-dsi-ctrl-sdm845
> + - qcom,mdss-dsi-ctrl-sm8250
> + - const: qcom,mdss-dsi-ctrl
I'd prefer to have this part squashed into the previous patch.
>
> reg:
> maxItems: 1
> @@ -51,17 +49,8 @@ properties:
> minItems: 6
>
> clock-names:
> - items:
> - - const: byte
> - - const: byte_intf
> - - const: pixel
> - - const: core
> - - const: iface
> - - const: bus
> - - const: core_mmss
> - - const: mdp_core
> - - const: mnoc
> minItems: 6
> + maxItems: 9
>
> phys:
> maxItems: 1
> @@ -169,6 +158,140 @@ required:
> - vddio-supply
> - vdda-supply
>
> +allOf:
> + - $ref: "../dsi-controller.yaml#"
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-apq8064
> + then:
> + properties:
> + clocks:
> + maxItems: 7
> + clock-names:
> + items:
> + - const: iface
> + - const: bus
> + - const: core_mmss
> + - const: src
> + - const: byte
> + - const: pixel
> + - const: core
Empty lines between clauses please.
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-msm8916
> + then:
> + properties:
> + clocks:
> + maxItems: 6
> + clock-names:
> + items:
> + - const: mdp_core
> + - const: iface
> + - const: bus
> + - const: byte
> + - const: pixel
> + - const: core
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-msm8974
> + then:
> + properties:
> + clocks:
> + maxItems: 3
> + clock-names:
> + items:
> + - const: iface
> + - const: bus
> + - const: vsync
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-msm8996
> + then:
> + properties:
> + clocks:
> + maxItems: 7
> + clock-names:
> + items:
> + - const: mdp_core
> + - const: byte
> + - const: iface
> + - const: bus
> + - const: core_mmss
> + - const: pixel
> + - const: core
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-sc7180
> + - qcom,mdss-dsi-ctrl-sc7280
> + - qcom,mdss-dsi-ctrl-sm8250
> + then:
> + properties:
> + clocks:
> + maxItems: 6
> + clock-names:
> + items:
> + - const: byte
> + - const: byte_intf
> + - const: pixel
> + - const: core
> + - const: iface
> + - const: bus
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-sdm630
> + - qcom,mdss-dsi-ctrl-sdm660
> + then:
> + properties:
> + clocks:
> + maxItems: 9
> + clock-names:
> + items:
> + - const: mdp_core
> + - const: byte
> + - const: byte_intf
> + - const: mnoc
> + - const: iface
> + - const: bus
> + - const: core_mmss
> + - const: pixel
> + - const: core
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-sdm845
> + then:
> + properties:
> + clocks:
> + maxItems: 6
> + clock-names:
> + items:
> + - const: byte
> + - const: byte_intf
> + - const: pixel
> + - const: core
> + - const: iface
> + - const: bus
> +
> additionalProperties: false
>
> examples:
> @@ -179,7 +302,7 @@ examples:
> #include <dt-bindings/power/qcom-rpmpd.h>
>
> dsi@ae94000 {
> - compatible = "qcom,mdss-dsi-ctrl";
> + compatible = "qcom,mdss-dsi-ctrl-sc7180", "qcom,mdss-dsi-ctrl";
> reg = <0x0ae94000 0x400>;
> reg-names = "dsi_ctrl";
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 08/18] dt-bindings: msm: dsi-controller-main: Document clocks on a per compatible basis
@ 2022-11-08 12:43 ` Dmitry Baryshkov
0 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 12:43 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> Each compatible has a different set of clocks which are associated with it.
> Add in the list of clocks for each compatible.
>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../display/msm/dsi-controller-main.yaml | 177 +++++++++++++++---
> 1 file changed, 150 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index 9db3e63acda3d..c975df0ca22fc 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -9,24 +9,22 @@ title: Qualcomm Display DSI controller
> maintainers:
> - Krishna Manikandan <quic_mkrishn@quicinc.com>
>
> -allOf:
> - - $ref: "../dsi-controller.yaml#"
> -
> properties:
> compatible:
> - enum:
> - - qcom,dsi-ctrl-6g-qcm2290
> - - qcom,mdss-dsi-ctrl
> - - qcom,mdss-dsi-ctrl-apq8064
> - - qcom,mdss-dsi-ctrl-msm8916
> - - qcom,mdss-dsi-ctrl-msm8974
> - - qcom,mdss-dsi-ctrl-msm8996
> - - qcom,mdss-dsi-ctrl-sc7180
> - - qcom,mdss-dsi-ctrl-sc7280
> - - qcom,mdss-dsi-ctrl-sdm630
> - - qcom,mdss-dsi-ctrl-sdm660
> - - qcom,mdss-dsi-ctrl-sdm845
> - - qcom,mdss-dsi-ctrl-sm8250
> + items:
> + - enum:
> + - qcom,dsi-ctrl-6g-qcm2290
> + - qcom,mdss-dsi-ctrl-apq8064
> + - qcom,mdss-dsi-ctrl-msm8916
> + - qcom,mdss-dsi-ctrl-msm8974
> + - qcom,mdss-dsi-ctrl-msm8996
> + - qcom,mdss-dsi-ctrl-sc7180
> + - qcom,mdss-dsi-ctrl-sc7280
> + - qcom,mdss-dsi-ctrl-sdm630
> + - qcom,mdss-dsi-ctrl-sdm660
> + - qcom,mdss-dsi-ctrl-sdm845
> + - qcom,mdss-dsi-ctrl-sm8250
> + - const: qcom,mdss-dsi-ctrl
I'd prefer to have this part squashed into the previous patch.
>
> reg:
> maxItems: 1
> @@ -51,17 +49,8 @@ properties:
> minItems: 6
>
> clock-names:
> - items:
> - - const: byte
> - - const: byte_intf
> - - const: pixel
> - - const: core
> - - const: iface
> - - const: bus
> - - const: core_mmss
> - - const: mdp_core
> - - const: mnoc
> minItems: 6
> + maxItems: 9
>
> phys:
> maxItems: 1
> @@ -169,6 +158,140 @@ required:
> - vddio-supply
> - vdda-supply
>
> +allOf:
> + - $ref: "../dsi-controller.yaml#"
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-apq8064
> + then:
> + properties:
> + clocks:
> + maxItems: 7
> + clock-names:
> + items:
> + - const: iface
> + - const: bus
> + - const: core_mmss
> + - const: src
> + - const: byte
> + - const: pixel
> + - const: core
Empty lines between clauses please.
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-msm8916
> + then:
> + properties:
> + clocks:
> + maxItems: 6
> + clock-names:
> + items:
> + - const: mdp_core
> + - const: iface
> + - const: bus
> + - const: byte
> + - const: pixel
> + - const: core
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-msm8974
> + then:
> + properties:
> + clocks:
> + maxItems: 3
> + clock-names:
> + items:
> + - const: iface
> + - const: bus
> + - const: vsync
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-msm8996
> + then:
> + properties:
> + clocks:
> + maxItems: 7
> + clock-names:
> + items:
> + - const: mdp_core
> + - const: byte
> + - const: iface
> + - const: bus
> + - const: core_mmss
> + - const: pixel
> + - const: core
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-sc7180
> + - qcom,mdss-dsi-ctrl-sc7280
> + - qcom,mdss-dsi-ctrl-sm8250
> + then:
> + properties:
> + clocks:
> + maxItems: 6
> + clock-names:
> + items:
> + - const: byte
> + - const: byte_intf
> + - const: pixel
> + - const: core
> + - const: iface
> + - const: bus
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-sdm630
> + - qcom,mdss-dsi-ctrl-sdm660
> + then:
> + properties:
> + clocks:
> + maxItems: 9
> + clock-names:
> + items:
> + - const: mdp_core
> + - const: byte
> + - const: byte_intf
> + - const: mnoc
> + - const: iface
> + - const: bus
> + - const: core_mmss
> + - const: pixel
> + - const: core
> + - if:
> + properties:
> + compatible:
> + contains:
> + enum:
> + - qcom,mdss-dsi-ctrl-sdm845
> + then:
> + properties:
> + clocks:
> + maxItems: 6
> + clock-names:
> + items:
> + - const: byte
> + - const: byte_intf
> + - const: pixel
> + - const: core
> + - const: iface
> + - const: bus
> +
> additionalProperties: false
>
> examples:
> @@ -179,7 +302,7 @@ examples:
> #include <dt-bindings/power/qcom-rpmpd.h>
>
> dsi@ae94000 {
> - compatible = "qcom,mdss-dsi-ctrl";
> + compatible = "qcom,mdss-dsi-ctrl-sc7180", "qcom,mdss-dsi-ctrl";
> reg = <0x0ae94000 0x400>;
> reg-names = "dsi_ctrl";
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 08/18] dt-bindings: msm: dsi-controller-main: Document clocks on a per compatible basis
2022-11-08 12:43 ` Dmitry Baryshkov
@ 2022-11-08 12:45 ` Bryan O'Donoghue
-1 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-08 12:45 UTC (permalink / raw)
To: Dmitry Baryshkov, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 12:43, Dmitry Baryshkov wrote:
> I'd prefer to have this part squashed into the previous patch.
NP.
Actually I'll add you as a suggested by here since you sent me a patch ~
similar to this.
---
bod
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 08/18] dt-bindings: msm: dsi-controller-main: Document clocks on a per compatible basis
@ 2022-11-08 12:45 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-08 12:45 UTC (permalink / raw)
To: Dmitry Baryshkov, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 12:43, Dmitry Baryshkov wrote:
> I'd prefer to have this part squashed into the previous patch.
NP.
Actually I'll add you as a suggested by here since you sent me a patch ~
similar to this.
---
bod
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-08 12:46 ` Dmitry Baryshkov
-1 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 12:46 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> Currently we do not differentiate between the various users of the
> qcom,mdss-dsi-ctrl. The driver is flexible enough to operate from one
> compatible string but, the hardware does have some significant differences
> in the number of clocks.
>
> To facilitate documenting the clocks add the following compatible strings
>
> - qcom,mdss-dsi-ctrl-apq8064
Generic comment: I think we'd better follow the arm/qcom-soc.yaml and
use qcom,soc-something as compat string. This would leave us with
qcom,apq8064-dsi-ctrl
I'm not sure if we want to follow the qcm2290 approach and encode the
DSI ctrl revision here (6g vs v2).
> - qcom,mdss-dsi-ctrl-msm8916
> - qcom,mdss-dsi-ctrl-msm8974
> - qcom,mdss-dsi-ctrl-msm8996
> - qcom,mdss-dsi-ctrl-sc7180
> - qcom,mdss-dsi-ctrl-sc7280
> - qcom,mdss-dsi-ctrl-sdm630
> - qcom,mdss-dsi-ctrl-sdm660
> - qcom,mdss-dsi-ctrl-sdm845
> - qcom,mdss-dsi-ctrl-sm8250
>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../bindings/display/msm/dsi-controller-main.yaml | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index b35130a77b43e..9db3e63acda3d 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -17,6 +17,16 @@ properties:
> enum:
> - qcom,dsi-ctrl-6g-qcm2290
> - qcom,mdss-dsi-ctrl
> + - qcom,mdss-dsi-ctrl-apq8064
> + - qcom,mdss-dsi-ctrl-msm8916
> + - qcom,mdss-dsi-ctrl-msm8974
> + - qcom,mdss-dsi-ctrl-msm8996
> + - qcom,mdss-dsi-ctrl-sc7180
> + - qcom,mdss-dsi-ctrl-sc7280
> + - qcom,mdss-dsi-ctrl-sdm630
> + - qcom,mdss-dsi-ctrl-sdm660
> + - qcom,mdss-dsi-ctrl-sdm845
> + - qcom,mdss-dsi-ctrl-sm8250
>
> reg:
> maxItems: 1
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC
@ 2022-11-08 12:46 ` Dmitry Baryshkov
0 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 12:46 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> Currently we do not differentiate between the various users of the
> qcom,mdss-dsi-ctrl. The driver is flexible enough to operate from one
> compatible string but, the hardware does have some significant differences
> in the number of clocks.
>
> To facilitate documenting the clocks add the following compatible strings
>
> - qcom,mdss-dsi-ctrl-apq8064
Generic comment: I think we'd better follow the arm/qcom-soc.yaml and
use qcom,soc-something as compat string. This would leave us with
qcom,apq8064-dsi-ctrl
I'm not sure if we want to follow the qcm2290 approach and encode the
DSI ctrl revision here (6g vs v2).
> - qcom,mdss-dsi-ctrl-msm8916
> - qcom,mdss-dsi-ctrl-msm8974
> - qcom,mdss-dsi-ctrl-msm8996
> - qcom,mdss-dsi-ctrl-sc7180
> - qcom,mdss-dsi-ctrl-sc7280
> - qcom,mdss-dsi-ctrl-sdm630
> - qcom,mdss-dsi-ctrl-sdm660
> - qcom,mdss-dsi-ctrl-sdm845
> - qcom,mdss-dsi-ctrl-sm8250
>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../bindings/display/msm/dsi-controller-main.yaml | 10 ++++++++++
> 1 file changed, 10 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index b35130a77b43e..9db3e63acda3d 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -17,6 +17,16 @@ properties:
> enum:
> - qcom,dsi-ctrl-6g-qcm2290
> - qcom,mdss-dsi-ctrl
> + - qcom,mdss-dsi-ctrl-apq8064
> + - qcom,mdss-dsi-ctrl-msm8916
> + - qcom,mdss-dsi-ctrl-msm8974
> + - qcom,mdss-dsi-ctrl-msm8996
> + - qcom,mdss-dsi-ctrl-sc7180
> + - qcom,mdss-dsi-ctrl-sc7280
> + - qcom,mdss-dsi-ctrl-sdm630
> + - qcom,mdss-dsi-ctrl-sdm660
> + - qcom,mdss-dsi-ctrl-sdm845
> + - qcom,mdss-dsi-ctrl-sm8250
>
> reg:
> maxItems: 1
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
` (17 preceding siblings ...)
2022-11-07 23:56 ` [PATCH v2 18/18] arm64: dts: qcom: sm8250: Add compat qcom,mdss-dsi-ctrl-sm8250 Bryan O'Donoghue
@ 2022-11-08 12:56 ` Dmitry Baryshkov
18 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 12:56 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Hi,
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> V2:
> https://www.spinics.net/lists/linux-arm-msm/msg116326.html
Please send the whole series to both linux-arm-msm and freedreno MLs.
And to all maintainers (I think it is more useful to get the whole set
rather than being puzzled what happens in the rest of the patches). This
would allow you to drop the Cc lists from the patches and pass the whole
list to git-send-email.
>
> - Moves the DSI PHY changes to a different later series.
> There are enough dsi-controller-main changes to justify its own
> standalone series.
>
> - The original phy-name binding change given discussion with Rob and
> Krzysztof became its own standalone series that has since been merged.
> https://www.mail-archive.com/dri-devel@lists.freedesktop.org/msg403214.html
>
> - Retains the drop of power-domain from yaml as a required property.
> I dug into the available dtsi. The apq8064 doesn't appear to have any
> GDSC which can be attached as a power-domain, which means the
> power-domain requirement is not universal across the various silicon
> versions.
I don't think 8x60/8960/8064 had GDSCs. At least msm-3.4 adds them only
to msm8974.
>
> - Adds Dmitry's RB to power-domain drop
>
> - For the clock declarations I've
> * I noticed that the simple change I had worked for msm8939 but
> subsquently broke other dtsi which drove a bigger change to document
> the clocks on a per compatible basis.
> * Added compat strings in yaml.
> * Moved the allOf down later in the file to acomodate the if/then.
> * Number of clocks validated on a per compatible basis
> * The driver code which doesn't care about the number of clocks
> can still operate on the mdss-dsi-ctrl compat but the dts checks will
> validate against the compat string and yaml.
>
> - vdd descriptions
> Took the previous text I missed from the .txt file - Krzysztof, Dmitry
> Adds vdd, vdda and vddio to the required list. This exposes warnings in
> existing dtsi but the previous .txt declared these regulators as
> required. - Krzysztof
>
> V1:
> This series fixes up a number of dtbs checks which are being flagged adding
> in the msm8939 dtsi.
>
>
> When converting from .txt to .yaml a number of the parameters for the older
> msm8916 silicon were not transmitted into the yaml.
>
> Adding in the msm8939 which is a near 1:1 copy of the msm8916 in terms of
> dtsi triggers a rake of dtbs checks as a result.
>
> https://www.mail-archive.com/dri-devel@lists.freedesktop.org/msg403211.html
>
> Bryan O'Donoghue (18):
> dt-bindings: msm: dsi-controller-main: Fix operating-points-v2
> constraint
> dt-bindings: msm: dsi-controller-main: Fix power-domain constraint
> dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
> dt-bindings: msm: dsi-controller-main: Fix clock declarations
> dt-bindings: msm: dsi-controller-main: Fix description of core clock
> dt-bindings: msm: dsi-controller-main: Alphanumerically sort
> compatible enum
> dt-bindings: msm: dsi-controller-main: Add compatible strings for
> every current SoC
> dt-bindings: msm: dsi-controller-main: Document clocks on a per
> compatible basis
> ARM: dts: qcom: apq8064: add compat qcom,mdss-dsi-ctrl-apq8064
> ARM: dts: qcom: msm8974: Add compat qcom,mdss-dsi-ctrl-msm8974
> arm64: dts: qcom: msm8916: Add compat qcom,mdss-dsi-ctrl-msm8916
> arm64: dts: qcom: msm8996: Add compat qcom,mdss-dsi-ctrl-msm8996
> arm64: dts: qcom: sc7180: Add compat qcom,mdss-dsi-ctrl-sc7180
> arm64: dts: qcom: sc7280: Add compat qcom,mdss-dsi-ctrl-sc7280
> arm64: dts: qcom: sdm630: Add compat qcom,mdss-dsi-ctrl-sdm630
> arm64: dts: qcom: sdm660: Add compat qcom,mdss-dsi-ctrl-sdm660
> arm64: dts: qcom: sdm845: Add compat qcom,mdss-dsi-ctrl-sdm845
> arm64: dts: qcom: sm8250: Add compat qcom,mdss-dsi-ctrl-sm8250
>
> .../display/msm/dsi-controller-main.yaml | 185 ++++++++++++++++--
> arch/arm/boot/dts/qcom-apq8064.dtsi | 3 +-
> arch/arm/boot/dts/qcom-msm8974.dtsi | 3 +-
> arch/arm64/boot/dts/qcom/msm8916.dtsi | 3 +-
> arch/arm64/boot/dts/qcom/msm8996.dtsi | 6 +-
> arch/arm64/boot/dts/qcom/sc7180.dtsi | 3 +-
> arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 +-
> arch/arm64/boot/dts/qcom/sdm630.dtsi | 3 +-
> arch/arm64/boot/dts/qcom/sdm660.dtsi | 3 +-
> arch/arm64/boot/dts/qcom/sdm845.dtsi | 6 +-
> arch/arm64/boot/dts/qcom/sm8250.dtsi | 6 +-
> 11 files changed, 194 insertions(+), 30 deletions(-)
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-08 12:59 ` Dmitry Baryshkov
-1 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 12:59 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> When converting from .txt to .yaml we didn't include descriptions for the
> existing regulator supplies.
>
> - vdd
> - vdda
> - vddio
>
> Add those descriptions into the yaml now as they were prior to the
> conversion. Mark the supplies as required as was previously the case in the
> .txt implementation.
>
> Warnings about missing regulators can be resolved by updating the relevant
> dtsi files to point to fixed always-on regulators where appropriate.
Ugh. Are they missing or are they optional/not used on these platforms?
Can you possibly list all regulator warnings?
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../bindings/display/msm/dsi-controller-main.yaml | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index cf782c5f5bdb0..0f7747e55b9be 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -124,6 +124,18 @@ properties:
> - port@0
> - port@1
>
> + vdd-supply:
> + description:
> + Phandle to vdd regulator device node
> +
> + vddio-supply:
> + description:
> + Phandle to vdd-io regulator device node
> +
> + vdda-supply:
> + description:
> + Phandle to vdda regulator device node
> +
> required:
> - compatible
> - reg
> @@ -135,6 +147,9 @@ required:
> - assigned-clocks
> - assigned-clock-parents
> - ports
> + - vdd-supply
> + - vddio-supply
> + - vdda-supply
>
> additionalProperties: false
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
@ 2022-11-08 12:59 ` Dmitry Baryshkov
0 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 12:59 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> When converting from .txt to .yaml we didn't include descriptions for the
> existing regulator supplies.
>
> - vdd
> - vdda
> - vddio
>
> Add those descriptions into the yaml now as they were prior to the
> conversion. Mark the supplies as required as was previously the case in the
> .txt implementation.
>
> Warnings about missing regulators can be resolved by updating the relevant
> dtsi files to point to fixed always-on regulators where appropriate.
Ugh. Are they missing or are they optional/not used on these platforms?
Can you possibly list all regulator warnings?
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../bindings/display/msm/dsi-controller-main.yaml | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index cf782c5f5bdb0..0f7747e55b9be 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -124,6 +124,18 @@ properties:
> - port@0
> - port@1
>
> + vdd-supply:
> + description:
> + Phandle to vdd regulator device node
> +
> + vddio-supply:
> + description:
> + Phandle to vdd-io regulator device node
> +
> + vdda-supply:
> + description:
> + Phandle to vdda regulator device node
> +
> required:
> - compatible
> - reg
> @@ -135,6 +147,9 @@ required:
> - assigned-clocks
> - assigned-clock-parents
> - ports
> + - vdd-supply
> + - vddio-supply
> + - vdda-supply
>
> additionalProperties: false
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 04/18] dt-bindings: msm: dsi-controller-main: Fix clock declarations
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-08 13:00 ` Dmitry Baryshkov
-1 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 13:00 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> When converting from .txt to .yaml dt-binding descriptions we appear to
> have missed some of the previous detail on the number and names of
> permissible clocks.
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
However it might be easier to move this patch after the patch adding the
per-platform compat strings. Then you can push these items into the
clauses where they are required.
> ---
> .../bindings/display/msm/dsi-controller-main.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index 0f7747e55b9be..cab38a20a54b0 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -35,6 +35,10 @@ properties:
> - description: Display escape clock
> - description: Display AHB clock
> - description: Display AXI clock
> + - description: Core MultiMedia SubSystem clock
> + - description: MDP Core clock
> + - description: MNOC clock
> + minItems: 6
>
> clock-names:
> items:
> @@ -44,6 +48,10 @@ properties:
> - const: core
> - const: iface
> - const: bus
> + - const: core_mmss
> + - const: mdp_core
> + - const: mnoc
> + minItems: 6
>
> phys:
> maxItems: 1
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 04/18] dt-bindings: msm: dsi-controller-main: Fix clock declarations
@ 2022-11-08 13:00 ` Dmitry Baryshkov
0 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 13:00 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> When converting from .txt to .yaml dt-binding descriptions we appear to
> have missed some of the previous detail on the number and names of
> permissible clocks.
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
However it might be easier to move this patch after the patch adding the
per-platform compat strings. Then you can push these items into the
clauses where they are required.
> ---
> .../bindings/display/msm/dsi-controller-main.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index 0f7747e55b9be..cab38a20a54b0 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -35,6 +35,10 @@ properties:
> - description: Display escape clock
> - description: Display AHB clock
> - description: Display AXI clock
> + - description: Core MultiMedia SubSystem clock
> + - description: MDP Core clock
> + - description: MNOC clock
> + minItems: 6
>
> clock-names:
> items:
> @@ -44,6 +48,10 @@ properties:
> - const: core
> - const: iface
> - const: bus
> + - const: core_mmss
> + - const: mdp_core
> + - const: mnoc
> + minItems: 6
>
> phys:
> maxItems: 1
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 05/18] dt-bindings: msm: dsi-controller-main: Fix description of core clock
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-08 13:01 ` Dmitry Baryshkov
-1 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 13:01 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> There's a typo in describing the core clock as an 'escape' clock. The
> accurate description is 'core'.
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index cab38a20a54b0..a607ccd4a905a 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -32,7 +32,7 @@ properties:
> - description: Display byte clock
> - description: Display byte interface clock
> - description: Display pixel clock
> - - description: Display escape clock
> + - description: Display core clock
> - description: Display AHB clock
> - description: Display AXI clock
> - description: Core MultiMedia SubSystem clock
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 05/18] dt-bindings: msm: dsi-controller-main: Fix description of core clock
@ 2022-11-08 13:01 ` Dmitry Baryshkov
0 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 13:01 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 02:56, Bryan O'Donoghue wrote:
> There's a typo in describing the core clock as an 'escape' clock. The
> accurate description is 'core'.
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../devicetree/bindings/display/msm/dsi-controller-main.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index cab38a20a54b0..a607ccd4a905a 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -32,7 +32,7 @@ properties:
> - description: Display byte clock
> - description: Display byte interface clock
> - description: Display pixel clock
> - - description: Display escape clock
> + - description: Display core clock
> - description: Display AHB clock
> - description: Display AXI clock
> - description: Core MultiMedia SubSystem clock
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 09/18] ARM: dts: qcom: apq8064: add compat qcom,mdss-dsi-ctrl-apq8064
2022-11-07 23:56 ` [PATCH v2 09/18] ARM: dts: qcom: apq8064: add compat qcom,mdss-dsi-ctrl-apq8064 Bryan O'Donoghue
@ 2022-11-08 13:22 ` David Heidelberg
0 siblings, 0 replies; 66+ messages in thread
From: David Heidelberg @ 2022-11-08 13:22 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Andy Gross, Bjorn Andersson, Konrad Dybcio, devicetree,
linux-kernel, Srinivas Kandagatla
Reviewed-by: David Heidelberg <david@ixit.cz>
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> Append silicon specific compatible qcom,mdss-dsi-ctrl-apq8064 to the
> mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
> for apq8064 against the yaml documentation.
>
> Cc: Andy Gross <agross@kernel.org>
> Cc: Bjorn Andersson <andersson@kernel.org>
> Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Cc: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> Cc: David Heidelberg <david@ixit.cz>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> arch/arm/boot/dts/qcom-apq8064.dtsi | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/qcom-apq8064.dtsi b/arch/arm/boot/dts/qcom-apq8064.dtsi
> index d036dff4b14f7..f3624832e6841 100644
> --- a/arch/arm/boot/dts/qcom-apq8064.dtsi
> +++ b/arch/arm/boot/dts/qcom-apq8064.dtsi
> @@ -1287,7 +1287,8 @@ mmss_sfpb: syscon@5700000 {
> };
>
> dsi0: dsi@4700000 {
> - compatible = "qcom,mdss-dsi-ctrl";
> + compatible = "qcom,mdss-dsi-ctrl-apq8064",
> + "qcom,mdss-dsi-ctrl";
> label = "MDSS DSI CTRL->0";
> #address-cells = <1>;
> #size-cells = <0>;
--
David Heidelberg
Consultant Software Engineer
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
2022-11-08 12:59 ` Dmitry Baryshkov
@ 2022-11-08 14:42 ` Bryan O'Donoghue
-1 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-08 14:42 UTC (permalink / raw)
To: Dmitry Baryshkov, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 12:59, Dmitry Baryshkov wrote:
>>
>> Warnings about missing regulators can be resolved by updating the
>> relevant
>> dtsi files to point to fixed always-on regulators where appropriate.
>
> Ugh. Are they missing or are they optional/not used on these platforms?
Some platforms either don't implement them or worse possibly do
implement but don't model them when they should.
> Can you possibly list all regulator warnings?
Downstream we have
arch/arm/boot/dts/qcom/msm8916-mdss.dtsi
mdss_dsi0: qcom,mdss_dsi@1a98000 {
vdda-supply = <&pm8916_l2>;
vdd-supply = <&pm8916_l17>;
vddio-supply = <&pm8916_l6>;
};
Looking at something like
arch/arm/boot/dts/qcom/msm8916-mtp.dtsi which references
arch/arm/boot/dts/qcom/dsi-panel-jdi-1080p-video.dtsi it doesn't appear
to delete andy of the vdd*-supply references
apq8016-sbc.dtb: dsi@1a98000: 'vdd-supply' is a required property
msm8916-samsung-a3u-eur.dtb: dsi@1a98000: 'vdd-supply' is a required
property
msm8916-samsung-a5u-eur.dtb: dsi@1a98000: 'vdd-supply' is a required
property
msm8916-samsung-e5.dtb: dsi@1a98000: 'vdd-supply' is a required property
msm8916-samsung-e7.dtb: dsi@1a98000: 'vdd-supply' is a required property
msm8916-samsung-grandmax.dtb: dsi@1a98000: 'vdd-supply' is a required
property
msm8996-xiaomi-natrium.dtb: dsi@994000: 'panel@0', 'vcca-supply' do not
match any of the regexes: 'pinctrl-[0-9]+'
msm8996-xiaomi-scorpio.dtb: dsi@994000: 'vdda-supply' is a required property
qrb5165-rb5.dtb: dsi@ae94000: 'vdd-supply' is a required property
qrb5165-rb5.dtb: dsi@ae94000: 'vddio-supply' is a required property
sc7180-idp.dtb: dsi@ae94000: 'vdd-supply' is a required property
sc7180-idp.dtb: dsi@ae94000: 'vddio-supply' is a required property
sc7180-trogdor-coachz-r1.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-coachz-r1.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-coachz-r1-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-coachz-r1-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-coachz-r3.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-coachz-r3.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-coachz-r3-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-coachz-r3-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-homestar-r2.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-homestar-r2.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-homestar-r3.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-homestar-r3.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-homestar-r4.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-homestar-r4.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-kingoftown-r0.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-kingoftown-r0.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-kingoftown-r1.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-kingoftown-r1.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r0.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r0.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-lazor-r1.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r1.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-lazor-r1-kb.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r1-kb.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r1-lte.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r1-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r3.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r3.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-lazor-r3-kb.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r3-kb.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r3-lte.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r3-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r9.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r9.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-lazor-r9-kb.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r9-kb.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r9-lte.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r9-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-limozeen-r4.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-lazor-limozeen-r4.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-limozeen-r9.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-lazor-limozeen-r9.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-limozeen-nots-r4.dtb: dsi@ae94000: 'vdd-supply' is
a required property
sc7180-trogdor-lazor-limozeen-nots-r4.dtb: dsi@ae94000: 'vddio-supply'
is a required property
sc7180-trogdor-lazor-limozeen-nots-r5.dtb: dsi@ae94000: 'vdd-supply' is
a required property
sc7180-trogdor-lazor-limozeen-nots-r5.dtb: dsi@ae94000: 'vddio-supply'
is a required property
sc7180-trogdor-lazor-limozeen-nots-r9.dtb: dsi@ae94000: 'vdd-supply' is
a required property
sc7180-trogdor-lazor-limozeen-nots-r9.dtb: dsi@ae94000: 'vddio-supply'
is a required property
sc7180-trogdor-mrbland-rev0-auo.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-mrbland-rev0-auo.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-mrbland-rev0-boe.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-mrbland-rev0-boe.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-mrbland-rev1-auo.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-mrbland-rev1-auo.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-mrbland-rev1-boe.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-mrbland-rev1-boe.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pazquel-lte-parade.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pazquel-lte-parade.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pazquel-lte-ti.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pazquel-lte-ti.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pazquel-parade.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pazquel-parade.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pazquel-ti.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-pazquel-ti.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-pompom-r1.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-pompom-r1.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-pompom-r1-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pompom-r1-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pompom-r2.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-pompom-r2.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-pompom-r2-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pompom-r2-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pompom-r3.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-pompom-r3.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-pompom-r3-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pompom-r3-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-quackingstick-r0.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-quackingstick-r0.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-quackingstick-r0-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-quackingstick-r0-lte.dtb: dsi@ae94000: 'vddio-supply' is
a required property
sc7180-trogdor-wormdingler-rev0-boe.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-wormdingler-rev0-boe.dtb: dsi@ae94000: 'vddio-supply' is
a required property
sc7180-trogdor-wormdingler-rev0-inx.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-wormdingler-rev0-inx.dtb: dsi@ae94000: 'vddio-supply' is
a required property
sc7180-trogdor-wormdingler-rev1-boe.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-wormdingler-rev1-boe.dtb: dsi@ae94000: 'vddio-supply' is
a required property
sc7180-trogdor-wormdingler-rev1-inx.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-wormdingler-rev1-inx.dtb: dsi@ae94000: 'vddio-supply' is
a required property
sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb: dsi@ae94000:
'vdd-supply' is a required property
sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb: dsi@ae94000:
'vddio-supply' is a required property
sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb: dsi@ae94000:
'vdd-supply' is a required property
sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb: dsi@ae94000:
'vddio-supply' is a required property
sc7180-trogdor-r1.dtb: dsi@ae94000: 'vdd-supply' is a required property
sc7180-trogdor-r1.dtb: dsi@ae94000: 'vddio-supply' is a required property
sc7180-trogdor-r1-lte.dtb: dsi@ae94000: 'vdd-supply' is a required property
sc7180-trogdor-r1-lte.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sda660-inforce-ifc6560.dtb: dsi@c994000: 'vdd-supply' is a required property
sda660-inforce-ifc6560.dtb: dsi@c994000: 'vddio-supply' is a required
property
sdm845-cheza-r1.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-cheza-r1.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm845-cheza-r2.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-cheza-r2.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm845-cheza-r3.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-cheza-r3.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm845-db845c.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-db845c.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm845-mtp.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-mtp.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm845-mtp.dtb: dsi@ae96000: 'vdd-supply' is a required property
sdm845-mtp.dtb: dsi@ae96000: 'vddio-supply' is a required property
sdm845-oneplus-enchilada.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sdm845-oneplus-enchilada.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sdm845-oneplus-fajita.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-oneplus-fajita.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sdm845-xiaomi-beryllium.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sdm845-xiaomi-beryllium.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sdm845-xiaomi-polaris.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-xiaomi-polaris.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sdm845-shift-axolotl.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-shift-axolotl.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm850-lenovo-yoga-c630.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sdm850-lenovo-yoga-c630.dtb: dsi@ae94000: 'vddio-supply' is a required
property
apq8016-sbc.dtb: dsi@1a98000: 'vdd-supply' is a required property
msm8916-samsung-a5u-eur.dtb: dsi@1a98000: 'vdd-supply' is a required
property
sc7180-trogdor-homestar-r4.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-homestar-r4.dtb: dsi@ae94000: 'vddio-supply' is a
required property
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
@ 2022-11-08 14:42 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-08 14:42 UTC (permalink / raw)
To: Dmitry Baryshkov, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 12:59, Dmitry Baryshkov wrote:
>>
>> Warnings about missing regulators can be resolved by updating the
>> relevant
>> dtsi files to point to fixed always-on regulators where appropriate.
>
> Ugh. Are they missing or are they optional/not used on these platforms?
Some platforms either don't implement them or worse possibly do
implement but don't model them when they should.
> Can you possibly list all regulator warnings?
Downstream we have
arch/arm/boot/dts/qcom/msm8916-mdss.dtsi
mdss_dsi0: qcom,mdss_dsi@1a98000 {
vdda-supply = <&pm8916_l2>;
vdd-supply = <&pm8916_l17>;
vddio-supply = <&pm8916_l6>;
};
Looking at something like
arch/arm/boot/dts/qcom/msm8916-mtp.dtsi which references
arch/arm/boot/dts/qcom/dsi-panel-jdi-1080p-video.dtsi it doesn't appear
to delete andy of the vdd*-supply references
apq8016-sbc.dtb: dsi@1a98000: 'vdd-supply' is a required property
msm8916-samsung-a3u-eur.dtb: dsi@1a98000: 'vdd-supply' is a required
property
msm8916-samsung-a5u-eur.dtb: dsi@1a98000: 'vdd-supply' is a required
property
msm8916-samsung-e5.dtb: dsi@1a98000: 'vdd-supply' is a required property
msm8916-samsung-e7.dtb: dsi@1a98000: 'vdd-supply' is a required property
msm8916-samsung-grandmax.dtb: dsi@1a98000: 'vdd-supply' is a required
property
msm8996-xiaomi-natrium.dtb: dsi@994000: 'panel@0', 'vcca-supply' do not
match any of the regexes: 'pinctrl-[0-9]+'
msm8996-xiaomi-scorpio.dtb: dsi@994000: 'vdda-supply' is a required property
qrb5165-rb5.dtb: dsi@ae94000: 'vdd-supply' is a required property
qrb5165-rb5.dtb: dsi@ae94000: 'vddio-supply' is a required property
sc7180-idp.dtb: dsi@ae94000: 'vdd-supply' is a required property
sc7180-idp.dtb: dsi@ae94000: 'vddio-supply' is a required property
sc7180-trogdor-coachz-r1.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-coachz-r1.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-coachz-r1-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-coachz-r1-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-coachz-r3.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-coachz-r3.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-coachz-r3-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-coachz-r3-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-homestar-r2.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-homestar-r2.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-homestar-r3.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-homestar-r3.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-homestar-r4.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-homestar-r4.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-kingoftown-r0.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-kingoftown-r0.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-kingoftown-r1.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-kingoftown-r1.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r0.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r0.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-lazor-r1.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r1.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-lazor-r1-kb.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r1-kb.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r1-lte.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r1-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r3.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r3.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-lazor-r3-kb.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r3-kb.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r3-lte.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r3-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r9.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r9.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-lazor-r9-kb.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r9-kb.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-r9-lte.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-lazor-r9-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-limozeen-r4.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-lazor-limozeen-r4.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-limozeen-r9.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-lazor-limozeen-r9.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-lazor-limozeen-nots-r4.dtb: dsi@ae94000: 'vdd-supply' is
a required property
sc7180-trogdor-lazor-limozeen-nots-r4.dtb: dsi@ae94000: 'vddio-supply'
is a required property
sc7180-trogdor-lazor-limozeen-nots-r5.dtb: dsi@ae94000: 'vdd-supply' is
a required property
sc7180-trogdor-lazor-limozeen-nots-r5.dtb: dsi@ae94000: 'vddio-supply'
is a required property
sc7180-trogdor-lazor-limozeen-nots-r9.dtb: dsi@ae94000: 'vdd-supply' is
a required property
sc7180-trogdor-lazor-limozeen-nots-r9.dtb: dsi@ae94000: 'vddio-supply'
is a required property
sc7180-trogdor-mrbland-rev0-auo.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-mrbland-rev0-auo.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-mrbland-rev0-boe.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-mrbland-rev0-boe.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-mrbland-rev1-auo.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-mrbland-rev1-auo.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-mrbland-rev1-boe.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-mrbland-rev1-boe.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pazquel-lte-parade.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pazquel-lte-parade.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pazquel-lte-ti.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pazquel-lte-ti.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pazquel-parade.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pazquel-parade.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pazquel-ti.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-pazquel-ti.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-pompom-r1.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-pompom-r1.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-pompom-r1-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pompom-r1-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pompom-r2.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-pompom-r2.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-pompom-r2-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pompom-r2-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-pompom-r3.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-pompom-r3.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sc7180-trogdor-pompom-r3-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-pompom-r3-lte.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-quackingstick-r0.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-quackingstick-r0.dtb: dsi@ae94000: 'vddio-supply' is a
required property
sc7180-trogdor-quackingstick-r0-lte.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-quackingstick-r0-lte.dtb: dsi@ae94000: 'vddio-supply' is
a required property
sc7180-trogdor-wormdingler-rev0-boe.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-wormdingler-rev0-boe.dtb: dsi@ae94000: 'vddio-supply' is
a required property
sc7180-trogdor-wormdingler-rev0-inx.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-wormdingler-rev0-inx.dtb: dsi@ae94000: 'vddio-supply' is
a required property
sc7180-trogdor-wormdingler-rev1-boe.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-wormdingler-rev1-boe.dtb: dsi@ae94000: 'vddio-supply' is
a required property
sc7180-trogdor-wormdingler-rev1-inx.dtb: dsi@ae94000: 'vdd-supply' is a
required property
sc7180-trogdor-wormdingler-rev1-inx.dtb: dsi@ae94000: 'vddio-supply' is
a required property
sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb: dsi@ae94000:
'vdd-supply' is a required property
sc7180-trogdor-wormdingler-rev1-inx-rt5682s.dtb: dsi@ae94000:
'vddio-supply' is a required property
sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb: dsi@ae94000:
'vdd-supply' is a required property
sc7180-trogdor-wormdingler-rev1-boe-rt5682s.dtb: dsi@ae94000:
'vddio-supply' is a required property
sc7180-trogdor-r1.dtb: dsi@ae94000: 'vdd-supply' is a required property
sc7180-trogdor-r1.dtb: dsi@ae94000: 'vddio-supply' is a required property
sc7180-trogdor-r1-lte.dtb: dsi@ae94000: 'vdd-supply' is a required property
sc7180-trogdor-r1-lte.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sda660-inforce-ifc6560.dtb: dsi@c994000: 'vdd-supply' is a required property
sda660-inforce-ifc6560.dtb: dsi@c994000: 'vddio-supply' is a required
property
sdm845-cheza-r1.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-cheza-r1.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm845-cheza-r2.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-cheza-r2.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm845-cheza-r3.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-cheza-r3.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm845-db845c.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-db845c.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm845-mtp.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-mtp.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm845-mtp.dtb: dsi@ae96000: 'vdd-supply' is a required property
sdm845-mtp.dtb: dsi@ae96000: 'vddio-supply' is a required property
sdm845-oneplus-enchilada.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sdm845-oneplus-enchilada.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sdm845-oneplus-fajita.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-oneplus-fajita.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sdm845-xiaomi-beryllium.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sdm845-xiaomi-beryllium.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sdm845-xiaomi-polaris.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-xiaomi-polaris.dtb: dsi@ae94000: 'vddio-supply' is a required
property
sdm845-shift-axolotl.dtb: dsi@ae94000: 'vdd-supply' is a required property
sdm845-shift-axolotl.dtb: dsi@ae94000: 'vddio-supply' is a required property
sdm850-lenovo-yoga-c630.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sdm850-lenovo-yoga-c630.dtb: dsi@ae94000: 'vddio-supply' is a required
property
apq8016-sbc.dtb: dsi@1a98000: 'vdd-supply' is a required property
msm8916-samsung-a5u-eur.dtb: dsi@1a98000: 'vdd-supply' is a required
property
sc7180-trogdor-homestar-r4.dtb: dsi@ae94000: 'vdd-supply' is a required
property
sc7180-trogdor-homestar-r4.dtb: dsi@ae94000: 'vddio-supply' is a
required property
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 13/18] arm64: dts: qcom: sc7180: Add compat qcom,mdss-dsi-ctrl-sc7180
2022-11-08 12:16 ` Bryan O'Donoghue
@ 2022-11-08 16:41 ` Doug Anderson
0 siblings, 0 replies; 66+ messages in thread
From: Doug Anderson @ 2022-11-08 16:41 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm,
Andy Gross, Bjorn Andersson, Konrad Dybcio, devicetree,
linux-kernel, Rajendra Nayak, Sai Prakash Ranjan
Hi,
On Tue, Nov 8, 2022 at 4:16 AM Bryan O'Donoghue
<bryan.odonoghue@linaro.org> wrote:
>
> On 08/11/2022 00:24, Doug Anderson wrote:
> > This seems fine, but I don't think it matches your bindings. Your
> > bindings says you can have one compatible string. It could be
> > "qcom,mdss-dsi-ctrl-sc7180" or it could be "qcom,mdss-dsi-ctrl".
> > ...but your device tree has two compatible strings: the SoC specific
> > one and the fallback one. You need to change your bindings to make
> > this work.
>
> With the update in this series the binding has a required const.
>
> If you drop back to just "qcom,mdss-dsi-ctrl-sc7180" you get a warning
> like this.
>
> arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb: dsi@ae94000:
> compatible: ['qcom,mdss-dsi-ctrl-sc7180'] is too short
>
> If you just have 'qcom,mdss-dsi-ctrl' you get
>
> arch/arm64/boot/dts/qcom/sc7180-trogdor-homestar-r4.dtb: dsi@ae94000:
> compatible:0: 'qcom,mdss-dsi-ctrl' is not one of
> ['qcom,dsi-ctrl-6g-qcm2290', 'qcom,mdss-dsi-ctrl-apq8064',
> 'qcom,mdss-dsi-ctrl-msm8916', 'qcom,mdss-dsi-ctrl-msm8974',
> 'qcom,mdss-dsi-ctrl-msm8996', 'qcom,mdss-dsi-ctrl-sc7180',
> 'qcom,mdss-dsi-ctrl-sc7280', 'qcom,mdss-dsi-ctrl-sdm630',
> 'qcom,mdss-dsi-ctrl-sdm660', 'qcom,mdss-dsi-ctrl-sdm845',
> 'qcom,mdss-dsi-ctrl-sm8250']
Huh. I must have missed something then. Ah, I see the problem. Since
you didn't CC me on the bindings change I searched myself. I found
patch #7 in your series ("dt-bindings: msm: dsi-controller-main: Add
compatible strings for every current SoC"). In that patch everything
is flat. I didn't notice patch #8 in your series which changes this. I
would just get the compatibles right the right time. Looking at patch
#8 I see that Dmitry already suggested this.
So that means this is fine with me. A slight nit that I suspect Bjorn
would prefer you to blow past the 80 column limit and put both
compatibles on one line, but it's not huge in my mind.
Reviewed-by: Douglas Anderson <dianders@chromium.org>
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 14/18] arm64: dts: qcom: sc7280: Add compat qcom,mdss-dsi-ctrl-sc7280
2022-11-07 23:56 ` [PATCH v2 14/18] arm64: dts: qcom: sc7280: Add compat qcom,mdss-dsi-ctrl-sc7280 Bryan O'Donoghue
@ 2022-11-08 16:41 ` Doug Anderson
0 siblings, 0 replies; 66+ messages in thread
From: Doug Anderson @ 2022-11-08 16:41 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm,
Andy Gross, Bjorn Andersson, Konrad Dybcio, devicetree,
linux-kernel, Sibi Sankar, Sai Prakash Ranjan
Hi,
On Mon, Nov 7, 2022 at 3:57 PM Bryan O'Donoghue
<bryan.odonoghue@linaro.org> wrote:
>
> Add silicon specific compatible qcom,mdss-dsi-ctrl-sc7280 to the
> mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
> for sc7280 against the yaml documentation.
>
> Cc: Andy Gross <agross@kernel.org>
> Cc: Bjorn Andersson <andersson@kernel.org>
> Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Cc: Douglas Anderson <dianders@chromium.org>
> Cc: Sibi Sankar <sibis@codeaurora.org>
> Cc: Sai Prakash Ranjan <saiprakash.ranjan@codeaurora.org>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/sc7280.dtsi | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
Reviewed-by: Douglas Anderson <dianders@chromium.org>
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 17/18] arm64: dts: qcom: sdm845: Add compat qcom,mdss-dsi-ctrl-sdm845
2022-11-07 23:56 ` [PATCH v2 17/18] arm64: dts: qcom: sdm845: Add compat qcom,mdss-dsi-ctrl-sdm845 Bryan O'Donoghue
@ 2022-11-08 16:42 ` Doug Anderson
0 siblings, 0 replies; 66+ messages in thread
From: Doug Anderson @ 2022-11-08 16:42 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm,
Andy Gross, Bjorn Andersson, Konrad Dybcio, devicetree,
linux-kernel, Krzysztof Kozlowski, Rajendra Nayak
Hi,
On Mon, Nov 7, 2022 at 3:57 PM Bryan O'Donoghue
<bryan.odonoghue@linaro.org> wrote:
>
> Add silicon specific compatible qcom,mdss-dsi-ctrl-sdm845 to the
> mdss-dsi-ctrl block. This allows us to differentiate the specific bindings
> for sdm845 against the yaml documentation.
>
> Cc: Andy Gross <agross@kernel.org>
> Cc: Bjorn Andersson <andersson@kernel.org>
> Cc: Konrad Dybcio <konrad.dybcio@somainline.org>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Cc: Bjorn Andersson <andersson@kernel.org>
> Cc: Douglas Anderson <dianders@chromium.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Rajendra Nayak <rnayak@codeaurora.org>
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> arch/arm64/boot/dts/qcom/sdm845.dtsi | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
Reviewed-by: Douglas Anderson <dianders@chromium.org>
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
2022-11-08 14:42 ` Bryan O'Donoghue
@ 2022-11-08 16:55 ` Dmitry Baryshkov
-1 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 16:55 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: robdclark, quic_abhinavk, krzysztof.kozlowski+dt, robh+dt,
quic_mkrishn, linux-arm-msm, Sean Paul, David Airlie,
Daniel Vetter, dri-devel, freedreno, devicetree, linux-kernel
On Tue, 8 Nov 2022 at 17:42, Bryan O'Donoghue
<bryan.odonoghue@linaro.org> wrote:
>
> On 08/11/2022 12:59, Dmitry Baryshkov wrote:
> >>
> >> Warnings about missing regulators can be resolved by updating the
> >> relevant
> >> dtsi files to point to fixed always-on regulators where appropriate.
> >
> > Ugh. Are they missing or are they optional/not used on these platforms?
>
> Some platforms either don't implement them or worse possibly do
> implement but don't model them when they should.
>
> > Can you possibly list all regulator warnings?
>
>
> Downstream we have
>
> arch/arm/boot/dts/qcom/msm8916-mdss.dtsi
>
> mdss_dsi0: qcom,mdss_dsi@1a98000 {
> vdda-supply = <&pm8916_l2>;
> vdd-supply = <&pm8916_l17>;
> vddio-supply = <&pm8916_l6>;
> };
>
> Looking at something like
>
> arch/arm/boot/dts/qcom/msm8916-mtp.dtsi which references
> arch/arm/boot/dts/qcom/dsi-panel-jdi-1080p-video.dtsi it doesn't appear
> to delete andy of the vdd*-supply references
Good example. We have vdda-supply and vddio-supply in
msm8916-pm8916.dtsi (correct).
And the mentioned vdd-supply is not applicable to this platform. It
powers up the external DSI-to-HDMI bridge. I'd assume that the same
regulator was destined to power up the panel.
So, I'd suggest defining all supplies at the top level and then
pushing all the required: parts into the per-SoC entries.
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
@ 2022-11-08 16:55 ` Dmitry Baryshkov
0 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-08 16:55 UTC (permalink / raw)
To: Bryan O'Donoghue
Cc: freedreno, devicetree, linux-arm-msm, quic_abhinavk, dri-devel,
linux-kernel, robh+dt, quic_mkrishn, krzysztof.kozlowski+dt,
Sean Paul
On Tue, 8 Nov 2022 at 17:42, Bryan O'Donoghue
<bryan.odonoghue@linaro.org> wrote:
>
> On 08/11/2022 12:59, Dmitry Baryshkov wrote:
> >>
> >> Warnings about missing regulators can be resolved by updating the
> >> relevant
> >> dtsi files to point to fixed always-on regulators where appropriate.
> >
> > Ugh. Are they missing or are they optional/not used on these platforms?
>
> Some platforms either don't implement them or worse possibly do
> implement but don't model them when they should.
>
> > Can you possibly list all regulator warnings?
>
>
> Downstream we have
>
> arch/arm/boot/dts/qcom/msm8916-mdss.dtsi
>
> mdss_dsi0: qcom,mdss_dsi@1a98000 {
> vdda-supply = <&pm8916_l2>;
> vdd-supply = <&pm8916_l17>;
> vddio-supply = <&pm8916_l6>;
> };
>
> Looking at something like
>
> arch/arm/boot/dts/qcom/msm8916-mtp.dtsi which references
> arch/arm/boot/dts/qcom/dsi-panel-jdi-1080p-video.dtsi it doesn't appear
> to delete andy of the vdd*-supply references
Good example. We have vdda-supply and vddio-supply in
msm8916-pm8916.dtsi (correct).
And the mentioned vdd-supply is not applicable to this platform. It
powers up the external DSI-to-HDMI bridge. I'd assume that the same
regulator was destined to power up the panel.
So, I'd suggest defining all supplies at the top level and then
pushing all the required: parts into the per-SoC entries.
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 02/18] dt-bindings: msm: dsi-controller-main: Fix power-domain constraint
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-08 21:10 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:10 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> power-domain is required for the sc7180 dispcc GDSC but not every qcom SoC
> has a similar dependency for example the aqp8064.
>
> Most Qcom SoC's using mdss-dsi-ctrl seem to have the ability to
> power-collapse the MDP without collapsing DSI.
>
> For example the qcom vendor kernel commit for apq8084, msm8226, msm8916, msm8974
>
> https://review.carbonrom.org/plugins/gitiles/CarbonROM/android_kernel_oneplus_msm8994/+/7b5c011a770daa2811778937ed646237a28a8694
>
> "ARM: dts: msm: add mdss gdsc supply to dsi controller device
>
> It is possible for the DSI controller to be active when MDP is
> power collapsed. DSI controller needs to have it's own vote for
> mdss gdsc to ensure that gdsc remains on in such cases."
>
> This however doesn't appear to be the case for the apq8064 so we shouldn't
> be marking power-domain as required in yaml checks.
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
Your Cc list is huge and not necessary to store in git log. For example
I am appearing there twice. Please keep it under '---'.
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 02/18] dt-bindings: msm: dsi-controller-main: Fix power-domain constraint
@ 2022-11-08 21:10 ` Krzysztof Kozlowski
0 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:10 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> power-domain is required for the sc7180 dispcc GDSC but not every qcom SoC
> has a similar dependency for example the aqp8064.
>
> Most Qcom SoC's using mdss-dsi-ctrl seem to have the ability to
> power-collapse the MDP without collapsing DSI.
>
> For example the qcom vendor kernel commit for apq8084, msm8226, msm8916, msm8974
>
> https://review.carbonrom.org/plugins/gitiles/CarbonROM/android_kernel_oneplus_msm8994/+/7b5c011a770daa2811778937ed646237a28a8694
>
> "ARM: dts: msm: add mdss gdsc supply to dsi controller device
>
> It is possible for the DSI controller to be active when MDP is
> power collapsed. DSI controller needs to have it's own vote for
> mdss gdsc to ensure that gdsc remains on in such cases."
>
> This however doesn't appear to be the case for the apq8064 so we shouldn't
> be marking power-domain as required in yaml checks.
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
Your Cc list is huge and not necessary to store in git log. For example
I am appearing there twice. Please keep it under '---'.
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-08 21:12 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> When converting from .txt to .yaml we didn't include descriptions for the
> existing regulator supplies.
>
> - vdd
> - vdda
> - vddio
>
> Add those descriptions into the yaml now as they were prior to the
> conversion. Mark the supplies as required as was previously the case in the
> .txt implementation.
>
> Warnings about missing regulators can be resolved by updating the relevant
> dtsi files to point to fixed always-on regulators where appropriate.
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../bindings/display/msm/dsi-controller-main.yaml | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index cf782c5f5bdb0..0f7747e55b9be 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -124,6 +124,18 @@ properties:
> - port@0
> - port@1
>
> + vdd-supply:
> + description:
> + Phandle to vdd regulator device node
Drop "Phandle to" and "device node", so just "VDD regulator". You
describe hardware rather (when applicable), not Devicetree syntax.
The same in other places and with that:
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in
@ 2022-11-08 21:12 ` Krzysztof Kozlowski
0 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> When converting from .txt to .yaml we didn't include descriptions for the
> existing regulator supplies.
>
> - vdd
> - vdda
> - vddio
>
> Add those descriptions into the yaml now as they were prior to the
> conversion. Mark the supplies as required as was previously the case in the
> .txt implementation.
>
> Warnings about missing regulators can be resolved by updating the relevant
> dtsi files to point to fixed always-on regulators where appropriate.
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../bindings/display/msm/dsi-controller-main.yaml | 15 +++++++++++++++
> 1 file changed, 15 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index cf782c5f5bdb0..0f7747e55b9be 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -124,6 +124,18 @@ properties:
> - port@0
> - port@1
>
> + vdd-supply:
> + description:
> + Phandle to vdd regulator device node
Drop "Phandle to" and "device node", so just "VDD regulator". You
describe hardware rather (when applicable), not Devicetree syntax.
The same in other places and with that:
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 05/18] dt-bindings: msm: dsi-controller-main: Fix description of core clock
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-08 21:12 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> There's a typo in describing the core clock as an 'escape' clock. The
> accurate description is 'core'.
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Cc: Rob Clark <robdclark@gmail.com>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 05/18] dt-bindings: msm: dsi-controller-main: Fix description of core clock
@ 2022-11-08 21:12 ` Krzysztof Kozlowski
0 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:12 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> There's a typo in describing the core clock as an 'escape' clock. The
> accurate description is 'core'.
>
> Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings")
> Cc: Rob Clark <robdclark@gmail.com>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 06/18] dt-bindings: msm: dsi-controller-main: Alphanumerically sort compatible enum
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-08 21:13 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:13 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> Sort the order of the compatible strings alphanumerically.
>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
I think here you can see how the commit msg looks now... One line of
message and 13 lines of CC.
With commit msg fixed:
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 06/18] dt-bindings: msm: dsi-controller-main: Alphanumerically sort compatible enum
@ 2022-11-08 21:13 ` Krzysztof Kozlowski
0 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:13 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> Sort the order of the compatible strings alphanumerically.
>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
I think here you can see how the commit msg looks now... One line of
message and 13 lines of CC.
With commit msg fixed:
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC
2022-11-08 12:46 ` Dmitry Baryshkov
@ 2022-11-08 21:15 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:15 UTC (permalink / raw)
To: Dmitry Baryshkov, Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 13:46, Dmitry Baryshkov wrote:
> On 08/11/2022 02:56, Bryan O'Donoghue wrote:
>> Currently we do not differentiate between the various users of the
>> qcom,mdss-dsi-ctrl. The driver is flexible enough to operate from one
>> compatible string but, the hardware does have some significant differences
>> in the number of clocks.
>>
>> To facilitate documenting the clocks add the following compatible strings
>>
>> - qcom,mdss-dsi-ctrl-apq8064
>
> Generic comment: I think we'd better follow the arm/qcom-soc.yaml and
> use qcom,soc-something as compat string. This would leave us with
> qcom,apq8064-dsi-ctrl
That's a requirement. Only existing patterns are allowed to continue,
but here all there is no such pattern.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC
@ 2022-11-08 21:15 ` Krzysztof Kozlowski
0 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:15 UTC (permalink / raw)
To: Dmitry Baryshkov, Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 13:46, Dmitry Baryshkov wrote:
> On 08/11/2022 02:56, Bryan O'Donoghue wrote:
>> Currently we do not differentiate between the various users of the
>> qcom,mdss-dsi-ctrl. The driver is flexible enough to operate from one
>> compatible string but, the hardware does have some significant differences
>> in the number of clocks.
>>
>> To facilitate documenting the clocks add the following compatible strings
>>
>> - qcom,mdss-dsi-ctrl-apq8064
>
> Generic comment: I think we'd better follow the arm/qcom-soc.yaml and
> use qcom,soc-something as compat string. This would leave us with
> qcom,apq8064-dsi-ctrl
That's a requirement. Only existing patterns are allowed to continue,
but here all there is no such pattern.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 08/18] dt-bindings: msm: dsi-controller-main: Document clocks on a per compatible basis
2022-11-07 23:56 ` Bryan O'Donoghue
@ 2022-11-08 21:19 ` Krzysztof Kozlowski
-1 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:19 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> Each compatible has a different set of clocks which are associated with it.
> Add in the list of clocks for each compatible.
>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../display/msm/dsi-controller-main.yaml | 177 +++++++++++++++---
> 1 file changed, 150 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index 9db3e63acda3d..c975df0ca22fc 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -9,24 +9,22 @@ title: Qualcomm Display DSI controller
> maintainers:
> - Krishna Manikandan <quic_mkrishn@quicinc.com>
>
> -allOf:
> - - $ref: "../dsi-controller.yaml#"
> -
> properties:
> compatible:
> - enum:
> - - qcom,dsi-ctrl-6g-qcm2290
> - - qcom,mdss-dsi-ctrl
> - - qcom,mdss-dsi-ctrl-apq8064
> - - qcom,mdss-dsi-ctrl-msm8916
> - - qcom,mdss-dsi-ctrl-msm8974
> - - qcom,mdss-dsi-ctrl-msm8996
> - - qcom,mdss-dsi-ctrl-sc7180
> - - qcom,mdss-dsi-ctrl-sc7280
> - - qcom,mdss-dsi-ctrl-sdm630
> - - qcom,mdss-dsi-ctrl-sdm660
> - - qcom,mdss-dsi-ctrl-sdm845
> - - qcom,mdss-dsi-ctrl-sm8250
> + items:
> + - enum:
> + - qcom,dsi-ctrl-6g-qcm2290
> + - qcom,mdss-dsi-ctrl-apq8064
> + - qcom,mdss-dsi-ctrl-msm8916
> + - qcom,mdss-dsi-ctrl-msm8974
> + - qcom,mdss-dsi-ctrl-msm8996
> + - qcom,mdss-dsi-ctrl-sc7180
> + - qcom,mdss-dsi-ctrl-sc7280
> + - qcom,mdss-dsi-ctrl-sdm630
> + - qcom,mdss-dsi-ctrl-sdm660
> + - qcom,mdss-dsi-ctrl-sdm845
> + - qcom,mdss-dsi-ctrl-sm8250
> + - const: qcom,mdss-dsi-ctrl
This is not explained in commit msg and looks like part of previous commit.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 08/18] dt-bindings: msm: dsi-controller-main: Document clocks on a per compatible basis
@ 2022-11-08 21:19 ` Krzysztof Kozlowski
0 siblings, 0 replies; 66+ messages in thread
From: Krzysztof Kozlowski @ 2022-11-08 21:19 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk, dmitry.baryshkov,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, freedreno, linux-kernel, dri-devel
On 08/11/2022 00:56, Bryan O'Donoghue wrote:
> Each compatible has a different set of clocks which are associated with it.
> Add in the list of clocks for each compatible.
>
> Cc: Rob Clark <robdclark@gmail.com>
> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com>
> Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> Cc: Sean Paul <sean@poorly.run>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Rob Herring <robh+dt@kernel.org>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> Cc: linux-arm-msm@vger.kernel.org
> Cc: dri-devel@lists.freedesktop.org
> Cc: freedreno@lists.freedesktop.org
> Cc: devicetree@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org
> Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
> ---
> .../display/msm/dsi-controller-main.yaml | 177 +++++++++++++++---
> 1 file changed, 150 insertions(+), 27 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> index 9db3e63acda3d..c975df0ca22fc 100644
> --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml
> @@ -9,24 +9,22 @@ title: Qualcomm Display DSI controller
> maintainers:
> - Krishna Manikandan <quic_mkrishn@quicinc.com>
>
> -allOf:
> - - $ref: "../dsi-controller.yaml#"
> -
> properties:
> compatible:
> - enum:
> - - qcom,dsi-ctrl-6g-qcm2290
> - - qcom,mdss-dsi-ctrl
> - - qcom,mdss-dsi-ctrl-apq8064
> - - qcom,mdss-dsi-ctrl-msm8916
> - - qcom,mdss-dsi-ctrl-msm8974
> - - qcom,mdss-dsi-ctrl-msm8996
> - - qcom,mdss-dsi-ctrl-sc7180
> - - qcom,mdss-dsi-ctrl-sc7280
> - - qcom,mdss-dsi-ctrl-sdm630
> - - qcom,mdss-dsi-ctrl-sdm660
> - - qcom,mdss-dsi-ctrl-sdm845
> - - qcom,mdss-dsi-ctrl-sm8250
> + items:
> + - enum:
> + - qcom,dsi-ctrl-6g-qcm2290
> + - qcom,mdss-dsi-ctrl-apq8064
> + - qcom,mdss-dsi-ctrl-msm8916
> + - qcom,mdss-dsi-ctrl-msm8974
> + - qcom,mdss-dsi-ctrl-msm8996
> + - qcom,mdss-dsi-ctrl-sc7180
> + - qcom,mdss-dsi-ctrl-sc7280
> + - qcom,mdss-dsi-ctrl-sdm630
> + - qcom,mdss-dsi-ctrl-sdm660
> + - qcom,mdss-dsi-ctrl-sdm845
> + - qcom,mdss-dsi-ctrl-sm8250
> + - const: qcom,mdss-dsi-ctrl
This is not explained in commit msg and looks like part of previous commit.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC
2022-11-08 12:46 ` Dmitry Baryshkov
@ 2022-11-18 13:29 ` Bryan O'Donoghue
-1 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-18 13:29 UTC (permalink / raw)
To: Dmitry Baryshkov, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel, Loic Poulain, Shawn Guo
On 08/11/2022 12:46, Dmitry Baryshkov wrote:
> On 08/11/2022 02:56, Bryan O'Donoghue wrote:
>> Currently we do not differentiate between the various users of the
>> qcom,mdss-dsi-ctrl. The driver is flexible enough to operate from one
>> compatible string but, the hardware does have some significant
>> differences
>> in the number of clocks.
>>
>> To facilitate documenting the clocks add the following compatible strings
>>
>> - qcom,mdss-dsi-ctrl-apq8064
>
> Generic comment: I think we'd better follow the arm/qcom-soc.yaml and
> use qcom,soc-something as compat string. This would leave us with
> qcom,apq8064-dsi-ctrl
>
> I'm not sure if we want to follow the qcm2290 approach and encode the
> DSI ctrl revision here (6g vs v2).
For qcm2290 I'm thinking qcm2290-dsi-ctrl - without the 6g piece.
a) Nobody is using the compat at the moment
b) I'm not sure what - if any real information the silicon version
number conveys here.
+ Loic, Shawn
---
bod
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC
@ 2022-11-18 13:29 ` Bryan O'Donoghue
0 siblings, 0 replies; 66+ messages in thread
From: Bryan O'Donoghue @ 2022-11-18 13:29 UTC (permalink / raw)
To: Dmitry Baryshkov, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, Loic Poulain, freedreno, linux-kernel, dri-devel
On 08/11/2022 12:46, Dmitry Baryshkov wrote:
> On 08/11/2022 02:56, Bryan O'Donoghue wrote:
>> Currently we do not differentiate between the various users of the
>> qcom,mdss-dsi-ctrl. The driver is flexible enough to operate from one
>> compatible string but, the hardware does have some significant
>> differences
>> in the number of clocks.
>>
>> To facilitate documenting the clocks add the following compatible strings
>>
>> - qcom,mdss-dsi-ctrl-apq8064
>
> Generic comment: I think we'd better follow the arm/qcom-soc.yaml and
> use qcom,soc-something as compat string. This would leave us with
> qcom,apq8064-dsi-ctrl
>
> I'm not sure if we want to follow the qcm2290 approach and encode the
> DSI ctrl revision here (6g vs v2).
For qcm2290 I'm thinking qcm2290-dsi-ctrl - without the 6g piece.
a) Nobody is using the compat at the moment
b) I'm not sure what - if any real information the silicon version
number conveys here.
+ Loic, Shawn
---
bod
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC
2022-11-18 13:29 ` Bryan O'Donoghue
@ 2022-11-18 13:30 ` Dmitry Baryshkov
-1 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-18 13:30 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, David Airlie, Daniel Vetter, dri-devel, freedreno,
devicetree, linux-kernel, Loic Poulain, Shawn Guo
On 18/11/2022 15:29, Bryan O'Donoghue wrote:
> On 08/11/2022 12:46, Dmitry Baryshkov wrote:
>> On 08/11/2022 02:56, Bryan O'Donoghue wrote:
>>> Currently we do not differentiate between the various users of the
>>> qcom,mdss-dsi-ctrl. The driver is flexible enough to operate from one
>>> compatible string but, the hardware does have some significant
>>> differences
>>> in the number of clocks.
>>>
>>> To facilitate documenting the clocks add the following compatible
>>> strings
>>>
>>> - qcom,mdss-dsi-ctrl-apq8064
>>
>> Generic comment: I think we'd better follow the arm/qcom-soc.yaml and
>> use qcom,soc-something as compat string. This would leave us with
>> qcom,apq8064-dsi-ctrl
>>
>> I'm not sure if we want to follow the qcm2290 approach and encode the
>> DSI ctrl revision here (6g vs v2).
>
> For qcm2290 I'm thinking qcm2290-dsi-ctrl - without the 6g piece.
This sounds good too.
>
> a) Nobody is using the compat at the moment
> b) I'm not sure what - if any real information the silicon version
> number conveys here.
>
> + Loic, Shawn
>
> ---
> bod
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
* Re: [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC
@ 2022-11-18 13:30 ` Dmitry Baryshkov
0 siblings, 0 replies; 66+ messages in thread
From: Dmitry Baryshkov @ 2022-11-18 13:30 UTC (permalink / raw)
To: Bryan O'Donoghue, robdclark, quic_abhinavk,
krzysztof.kozlowski+dt, robh+dt, quic_mkrishn, linux-arm-msm
Cc: Sean Paul, devicetree, Loic Poulain, freedreno, linux-kernel, dri-devel
On 18/11/2022 15:29, Bryan O'Donoghue wrote:
> On 08/11/2022 12:46, Dmitry Baryshkov wrote:
>> On 08/11/2022 02:56, Bryan O'Donoghue wrote:
>>> Currently we do not differentiate between the various users of the
>>> qcom,mdss-dsi-ctrl. The driver is flexible enough to operate from one
>>> compatible string but, the hardware does have some significant
>>> differences
>>> in the number of clocks.
>>>
>>> To facilitate documenting the clocks add the following compatible
>>> strings
>>>
>>> - qcom,mdss-dsi-ctrl-apq8064
>>
>> Generic comment: I think we'd better follow the arm/qcom-soc.yaml and
>> use qcom,soc-something as compat string. This would leave us with
>> qcom,apq8064-dsi-ctrl
>>
>> I'm not sure if we want to follow the qcm2290 approach and encode the
>> DSI ctrl revision here (6g vs v2).
>
> For qcm2290 I'm thinking qcm2290-dsi-ctrl - without the 6g piece.
This sounds good too.
>
> a) Nobody is using the compat at the moment
> b) I'm not sure what - if any real information the silicon version
> number conveys here.
>
> + Loic, Shawn
>
> ---
> bod
>
--
With best wishes
Dmitry
^ permalink raw reply [flat|nested] 66+ messages in thread
end of thread, other threads:[~2022-11-18 13:30 UTC | newest]
Thread overview: 66+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-07 23:56 [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 01/18] dt-bindings: msm: dsi-controller-main: Fix operating-points-v2 constraint Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 02/18] dt-bindings: msm: dsi-controller-main: Fix power-domain constraint Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 21:10 ` Krzysztof Kozlowski
2022-11-08 21:10 ` Krzysztof Kozlowski
2022-11-07 23:56 ` [PATCH v2 03/18] dt-bindings: msm: dsi-controller-main: Add vdd* descriptions back in Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 12:59 ` Dmitry Baryshkov
2022-11-08 12:59 ` Dmitry Baryshkov
2022-11-08 14:42 ` Bryan O'Donoghue
2022-11-08 14:42 ` Bryan O'Donoghue
2022-11-08 16:55 ` Dmitry Baryshkov
2022-11-08 16:55 ` Dmitry Baryshkov
2022-11-08 21:12 ` Krzysztof Kozlowski
2022-11-08 21:12 ` Krzysztof Kozlowski
2022-11-07 23:56 ` [PATCH v2 04/18] dt-bindings: msm: dsi-controller-main: Fix clock declarations Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 13:00 ` Dmitry Baryshkov
2022-11-08 13:00 ` Dmitry Baryshkov
2022-11-07 23:56 ` [PATCH v2 05/18] dt-bindings: msm: dsi-controller-main: Fix description of core clock Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 13:01 ` Dmitry Baryshkov
2022-11-08 13:01 ` Dmitry Baryshkov
2022-11-08 21:12 ` Krzysztof Kozlowski
2022-11-08 21:12 ` Krzysztof Kozlowski
2022-11-07 23:56 ` [PATCH v2 06/18] dt-bindings: msm: dsi-controller-main: Alphanumerically sort compatible enum Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 21:13 ` Krzysztof Kozlowski
2022-11-08 21:13 ` Krzysztof Kozlowski
2022-11-07 23:56 ` [PATCH v2 07/18] dt-bindings: msm: dsi-controller-main: Add compatible strings for every current SoC Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 12:46 ` Dmitry Baryshkov
2022-11-08 12:46 ` Dmitry Baryshkov
2022-11-08 21:15 ` Krzysztof Kozlowski
2022-11-08 21:15 ` Krzysztof Kozlowski
2022-11-18 13:29 ` Bryan O'Donoghue
2022-11-18 13:29 ` Bryan O'Donoghue
2022-11-18 13:30 ` Dmitry Baryshkov
2022-11-18 13:30 ` Dmitry Baryshkov
2022-11-07 23:56 ` [PATCH v2 08/18] dt-bindings: msm: dsi-controller-main: Document clocks on a per compatible basis Bryan O'Donoghue
2022-11-07 23:56 ` Bryan O'Donoghue
2022-11-08 12:43 ` Dmitry Baryshkov
2022-11-08 12:43 ` Dmitry Baryshkov
2022-11-08 12:45 ` Bryan O'Donoghue
2022-11-08 12:45 ` Bryan O'Donoghue
2022-11-08 21:19 ` Krzysztof Kozlowski
2022-11-08 21:19 ` Krzysztof Kozlowski
2022-11-07 23:56 ` [PATCH v2 09/18] ARM: dts: qcom: apq8064: add compat qcom,mdss-dsi-ctrl-apq8064 Bryan O'Donoghue
2022-11-08 13:22 ` David Heidelberg
2022-11-07 23:56 ` [PATCH v2 10/18] ARM: dts: qcom: msm8974: Add compat qcom,mdss-dsi-ctrl-msm8974 Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 11/18] arm64: dts: qcom: msm8916: Add compat qcom,mdss-dsi-ctrl-msm8916 Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 12/18] arm64: dts: qcom: msm8996: Add compat qcom,mdss-dsi-ctrl-msm8996 Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 13/18] arm64: dts: qcom: sc7180: Add compat qcom,mdss-dsi-ctrl-sc7180 Bryan O'Donoghue
2022-11-08 0:24 ` Doug Anderson
2022-11-08 12:16 ` Bryan O'Donoghue
2022-11-08 16:41 ` Doug Anderson
2022-11-07 23:56 ` [PATCH v2 14/18] arm64: dts: qcom: sc7280: Add compat qcom,mdss-dsi-ctrl-sc7280 Bryan O'Donoghue
2022-11-08 16:41 ` Doug Anderson
2022-11-07 23:56 ` [PATCH v2 15/18] arm64: dts: qcom: sdm630: Add compat qcom,mdss-dsi-ctrl-sdm630 Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 16/18] arm64: dts: qcom: sdm660: Add compat qcom,mdss-dsi-ctrl-sdm660 Bryan O'Donoghue
2022-11-07 23:56 ` [PATCH v2 17/18] arm64: dts: qcom: sdm845: Add compat qcom,mdss-dsi-ctrl-sdm845 Bryan O'Donoghue
2022-11-08 16:42 ` Doug Anderson
2022-11-07 23:56 ` [PATCH v2 18/18] arm64: dts: qcom: sm8250: Add compat qcom,mdss-dsi-ctrl-sm8250 Bryan O'Donoghue
2022-11-08 12:56 ` [PATCH v2 00/18] mdss-dsi-ctrl binding and dts fixes Dmitry Baryshkov
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.