* [PATCH] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property
@ 2023-02-07 10:04 Neil Armstrong
2023-02-07 15:28 ` Krzysztof Kozlowski
` (2 more replies)
0 siblings, 3 replies; 6+ messages in thread
From: Neil Armstrong @ 2023-02-07 10:04 UTC (permalink / raw)
To: Thierry Reding, Sam Ravnborg, David Airlie, Daniel Vetter,
Rob Herring, Krzysztof Kozlowski
Cc: devicetree, Krzysztof Kozlowski, linux-kernel, dri-devel, Neil Armstrong
Add missing reg property and update example to add dsi top node.
Fixes: ef85db911134 ("dt-bindings: display: panel: document the Visionox VTDR6130 AMOLED DSI Panel")
Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
---
.../bindings/display/panel/visionox,vtdr6130.yaml | 34 ++++++++++++++--------
1 file changed, 22 insertions(+), 12 deletions(-)
diff --git a/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml b/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
index 49e2fd4b4e99..84562a5b710a 100644
--- a/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
+++ b/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
@@ -16,6 +16,10 @@ properties:
compatible:
const: visionox,vtdr6130
+ reg:
+ maxItems: 1
+ description: DSI virtual channel
+
vddio-supply: true
vci-supply: true
vdd-supply: true
@@ -26,6 +30,7 @@ additionalProperties: false
required:
- compatible
+ - reg
- vddio-supply
- vci-supply
- vdd-supply
@@ -35,18 +40,23 @@ required:
examples:
- |
#include <dt-bindings/gpio/gpio.h>
- panel {
- compatible = "visionox,vtdr6130";
-
- vddio-supply = <&vreg_l12b_1p8>;
- vci-supply = <&vreg_l13b_3p0>;
- vdd-supply = <&vreg_l11b_1p2>;
-
- reset-gpios = <&tlmm 133 GPIO_ACTIVE_LOW>;
-
- port {
- panel0_in: endpoint {
- remote-endpoint = <&dsi0_out>;
+ dsi {
+ #address-cells = <1>;
+ #size-cells = <0>;
+ panel@0 {
+ compatible = "visionox,vtdr6130";
+ reg = <0>;
+
+ vddio-supply = <&vreg_l12b_1p8>;
+ vci-supply = <&vreg_l13b_3p0>;
+ vdd-supply = <&vreg_l11b_1p2>;
+
+ reset-gpios = <&tlmm 133 GPIO_ACTIVE_LOW>;
+
+ port {
+ panel0_in: endpoint {
+ remote-endpoint = <&dsi0_out>;
+ };
};
};
};
---
base-commit: 49a8133221c71b935f36a7c340c0271c2a9ee2db
change-id: 20230207-topic-sm8550-upstream-vtdr6130-bindings-fix-a1d2630efc06
Best regards,
--
Neil Armstrong <neil.armstrong@linaro.org>
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property
2023-02-07 10:04 [PATCH] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property Neil Armstrong
@ 2023-02-07 15:28 ` Krzysztof Kozlowski
2023-02-07 18:36 ` Sam Ravnborg
2023-02-09 8:34 ` Neil Armstrong
2 siblings, 0 replies; 6+ messages in thread
From: Krzysztof Kozlowski @ 2023-02-07 15:28 UTC (permalink / raw)
To: Neil Armstrong, Thierry Reding, Sam Ravnborg, David Airlie,
Daniel Vetter, Rob Herring, Krzysztof Kozlowski
Cc: devicetree, linux-kernel, dri-devel
On 07/02/2023 11:04, Neil Armstrong wrote:
> Add missing reg property and update example to add dsi top node.
>
> Fixes: ef85db911134 ("dt-bindings: display: panel: document the Visionox VTDR6130 AMOLED DSI Panel")
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property
2023-02-07 10:04 [PATCH] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property Neil Armstrong
2023-02-07 15:28 ` Krzysztof Kozlowski
@ 2023-02-07 18:36 ` Sam Ravnborg
2023-02-07 20:37 ` Rob Herring
2023-02-09 8:34 ` Neil Armstrong
2 siblings, 1 reply; 6+ messages in thread
From: Sam Ravnborg @ 2023-02-07 18:36 UTC (permalink / raw)
To: Neil Armstrong
Cc: devicetree, linux-kernel, dri-devel, Krzysztof Kozlowski,
Rob Herring, Thierry Reding, Krzysztof Kozlowski
Hi Neil.
On Tue, Feb 07, 2023 at 11:04:36AM +0100, Neil Armstrong wrote:
> Add missing reg property and update example to add dsi top node.
>
> Fixes: ef85db911134 ("dt-bindings: display: panel: document the Visionox VTDR6130 AMOLED DSI Panel")
> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> ---
> .../bindings/display/panel/visionox,vtdr6130.yaml | 34 ++++++++++++++--------
> 1 file changed, 22 insertions(+), 12 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml b/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
> index 49e2fd4b4e99..84562a5b710a 100644
> --- a/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
> +++ b/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
> @@ -16,6 +16,10 @@ properties:
> compatible:
> const: visionox,vtdr6130
>
> + reg:
> + maxItems: 1
> + description: DSI virtual channel
> +
Other panels using dsi uses a simple
" reg: true"
I think that would suffice here too.
Sam
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property
2023-02-07 18:36 ` Sam Ravnborg
@ 2023-02-07 20:37 ` Rob Herring
2023-02-07 21:22 ` Sam Ravnborg
0 siblings, 1 reply; 6+ messages in thread
From: Rob Herring @ 2023-02-07 20:37 UTC (permalink / raw)
To: Sam Ravnborg
Cc: Neil Armstrong, devicetree, linux-kernel, dri-devel,
Krzysztof Kozlowski, Thierry Reding, Krzysztof Kozlowski
On Tue, Feb 07, 2023 at 07:36:00PM +0100, Sam Ravnborg wrote:
> Hi Neil.
>
>
> On Tue, Feb 07, 2023 at 11:04:36AM +0100, Neil Armstrong wrote:
> > Add missing reg property and update example to add dsi top node.
> >
> > Fixes: ef85db911134 ("dt-bindings: display: panel: document the Visionox VTDR6130 AMOLED DSI Panel")
> > Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> > ---
> > .../bindings/display/panel/visionox,vtdr6130.yaml | 34 ++++++++++++++--------
> > 1 file changed, 22 insertions(+), 12 deletions(-)
> >
> > diff --git a/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml b/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
> > index 49e2fd4b4e99..84562a5b710a 100644
> > --- a/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
> > +++ b/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
> > @@ -16,6 +16,10 @@ properties:
> > compatible:
> > const: visionox,vtdr6130
> >
> > + reg:
> > + maxItems: 1
> > + description: DSI virtual channel
> > +
> Other panels using dsi uses a simple
> " reg: true"
>
> I think that would suffice here too.
Yes, because dsi-controller.yaml restricts it to a single entry.
However, it's description says there can be more than 1 virtual channel
for a device. In that case, it shouldn't be restricted in
dsi-controller.yaml and the above with 'maxItems: 1' would be correct.
So I'd keep this as-is.
Acked-by: Rob Herring <robh@kernel.org>
Rob
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property
2023-02-07 20:37 ` Rob Herring
@ 2023-02-07 21:22 ` Sam Ravnborg
0 siblings, 0 replies; 6+ messages in thread
From: Sam Ravnborg @ 2023-02-07 21:22 UTC (permalink / raw)
To: Rob Herring
Cc: Neil Armstrong, devicetree, linux-kernel, dri-devel,
Krzysztof Kozlowski, Thierry Reding, Krzysztof Kozlowski
On Tue, Feb 07, 2023 at 02:37:16PM -0600, Rob Herring wrote:
> On Tue, Feb 07, 2023 at 07:36:00PM +0100, Sam Ravnborg wrote:
> > Hi Neil.
> >
> >
> > On Tue, Feb 07, 2023 at 11:04:36AM +0100, Neil Armstrong wrote:
> > > Add missing reg property and update example to add dsi top node.
> > >
> > > Fixes: ef85db911134 ("dt-bindings: display: panel: document the Visionox VTDR6130 AMOLED DSI Panel")
> > > Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org>
> > > ---
> > > .../bindings/display/panel/visionox,vtdr6130.yaml | 34 ++++++++++++++--------
> > > 1 file changed, 22 insertions(+), 12 deletions(-)
> > >
> > > diff --git a/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml b/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
> > > index 49e2fd4b4e99..84562a5b710a 100644
> > > --- a/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
> > > +++ b/Documentation/devicetree/bindings/display/panel/visionox,vtdr6130.yaml
> > > @@ -16,6 +16,10 @@ properties:
> > > compatible:
> > > const: visionox,vtdr6130
> > >
> > > + reg:
> > > + maxItems: 1
> > > + description: DSI virtual channel
> > > +
> > Other panels using dsi uses a simple
> > " reg: true"
> >
> > I think that would suffice here too.
>
> Yes, because dsi-controller.yaml restricts it to a single entry.
> However, it's description says there can be more than 1 virtual channel
> for a device. In that case, it shouldn't be restricted in
> dsi-controller.yaml and the above with 'maxItems: 1' would be correct.
> So I'd keep this as-is.
Thanks for the explanation.
>
> Acked-by: Rob Herring <robh@kernel.org>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property
2023-02-07 10:04 [PATCH] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property Neil Armstrong
2023-02-07 15:28 ` Krzysztof Kozlowski
2023-02-07 18:36 ` Sam Ravnborg
@ 2023-02-09 8:34 ` Neil Armstrong
2 siblings, 0 replies; 6+ messages in thread
From: Neil Armstrong @ 2023-02-09 8:34 UTC (permalink / raw)
To: Thierry Reding, Sam Ravnborg, David Airlie, Daniel Vetter,
Rob Herring, Krzysztof Kozlowski, Neil Armstrong
Cc: devicetree, Krzysztof Kozlowski, linux-kernel, dri-devel
Hi,
On Tue, 07 Feb 2023 11:04:36 +0100, Neil Armstrong wrote:
> Add missing reg property and update example to add dsi top node.
>
>
Thanks, Applied to https://anongit.freedesktop.org/git/drm/drm-misc.git (drm-misc-next-fixes)
[1/1] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property
https://cgit.freedesktop.org/drm/drm-misc/commit/?id=467fbc77f673ecc9dcf4e58ffc1fa426a22df7fd
--
Neil
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-02-09 8:34 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-02-07 10:04 [PATCH] dt-bindings: display: panel: visionox,vtdr6130: add missing reg property Neil Armstrong
2023-02-07 15:28 ` Krzysztof Kozlowski
2023-02-07 18:36 ` Sam Ravnborg
2023-02-07 20:37 ` Rob Herring
2023-02-07 21:22 ` Sam Ravnborg
2023-02-09 8:34 ` Neil Armstrong
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).