Re: [PATCH 2/2] dt-bindings: display: bridge: sil,sii9022: Add data-lines

From: Wadim Egorov
Date: Thu Oct 03 2024 - 07:56:47 EST




Am 03.10.24 um 12:03 schrieb Krzysztof Kozlowski:
On 03/10/2024 10:20, Wadim Egorov wrote:
The SI9022 HDMI transmitter can be configured with 16, 18, or 24 input
data lines. This commit introduces the data-lines property to the input

lines? lanes? What are lines? like pins?

Yes, "lines" in this context refers to the number of pins used for the input pixel data bus, which can support 16, 18, or 24-bit wide data buses. These are parallel data lines (or pins) that carry uncompressed digital video to the HDMI transmitter.


endpoint, specifying the number of parallel RGB input pins connected
to the transmitter.

Signed-off-by: Wadim Egorov <w.egorov@xxxxxxxxx>
---
.../bindings/display/bridge/sil,sii9022.yaml | 13 ++++++++++++-
1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/display/bridge/sil,sii9022.yaml b/Documentation/devicetree/bindings/display/bridge/sil,sii9022.yaml
index 5a69547ad3d7..24306f8eb107 100644
--- a/Documentation/devicetree/bindings/display/bridge/sil,sii9022.yaml
+++ b/Documentation/devicetree/bindings/display/bridge/sil,sii9022.yaml
@@ -81,9 +81,20 @@ properties:
properties:
port@0:
- $ref: /schemas/graph.yaml#/properties/port
+ unevaluatedProperties: false
+ $ref: /schemas/graph.yaml#/$defs/port-base
description: Parallel RGB input port
+ properties:
+ endpoint:
+ $ref: /schemas/graph.yaml#/$defs/endpoint-base
+ unevaluatedProperties: false
+
+ properties:
+ data-lines:

No, this will confuse everyone. Considering lack of description how
anyone would figure out what this means?

I guess from working with the hardware/reference manual and using this chip?

I don't think it is overly confusing, especially since the port is already described as the "Parallel RGB input port" which clearly implies the use of pins for data transmission.

I am open to other suggestions if you believe a different name would improve clarity.

Btw, bridge/toshiba,tc358768.yaml, which performs a similar function, also uses the term data-lines.

Regards,
Wadim


I don't know much about RGB parallel port to advice how this should be
called. Can you describe the hardware more?

+ $ref: /schemas/types.yaml#/definitions/uint32
+ enum: [ 16, 18, 24 ]
+
port@1:
$ref: /schemas/graph.yaml#/properties/port
description: HDMI output port

Best regards,
Krzysztof