On Tue, 8 Apr 2025 17:44:32 +0200
Maxime Ripard <mrip...@kernel.org> wrote:
> hi,
>
> On Tue, Apr 08, 2025 at 05:27:00PM +0200, Kory Maincent wrote:
> > Add the dsi-lanes property to specify the number of DSI lanes used by the
> > panel. This allows configuring the panel for either two, three or four
> > lanes.
> >
> > Signed-off-by: Kory Maincent <kory.mainc...@bootlin.com>
> > ---
> > Documentation/devicetree/bindings/display/panel/ilitek,ili9881c.yaml | 5
> > +++++ 1 file changed, 5 insertions(+)
> >
> > diff --git
> > a/Documentation/devicetree/bindings/display/panel/ilitek,ili9881c.yaml
> > b/Documentation/devicetree/bindings/display/panel/ilitek,ili9881c.yaml
> > index
> > baf5dfe5f5ebdd92f460a78d0e56e1b45e7dd323..e36550616f6aac86c79832a48132ce8c11ebcf7a
> > 100644 ---
> > a/Documentation/devicetree/bindings/display/panel/ilitek,ili9881c.yaml +++
> > b/Documentation/devicetree/bindings/display/panel/ilitek,ili9881c.yaml @@
> > -27,6 +27,11 @@ properties: reg: maxItems: 1
> > + dsi-lanes:
> > + description: Number of DSI lanes to be used must be <2>, <3> or <4>
> > + $ref: /schemas/types.yaml#/definitions/uint32
> > + enum: [2, 3, 4]
> > +
>
> We have the data-lanes property for that already
Indeed but there is no such usage in panel bindings, only in bridge bindings.
You are saying that I should add something like that:
port:
$ref: /schemas/graph.yaml#/properties/port
properties:
endpoint:
$ref: /schemas/media/video-interfaces.yaml#
unevaluatedProperties: false
data-lanes:
minItems: 2
maxItems: 4
And use drm_of_get_data_lanes_count in the drivers.
If we do so, maybe this binding should land in panel-common.yaml instead?
Regards,
--
Köry Maincent, Bootlin
Embedded Linux and kernel engineering
https://bootlin.com
