Re: [RFC PATCH v2 02/16] dt-bindings: media: i2c: max96717: reflow text

From: Rob Herring
Date: Tue Mar 11 2025 - 14:09:21 EST


On Sun, Mar 09, 2025 at 10:47:54AM +0200, Cosmin Tanislav wrote:
> This text is using weird line lengths. Fix it.
>
> Signed-off-by: Cosmin Tanislav <demonsingur@xxxxxxxxx>
> ---
> .../bindings/media/i2c/maxim,max96717.yaml | 19 +++++++++----------
> 1 file changed, 9 insertions(+), 10 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml b/Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml
> index 15ab37702a92..1026678a17a7 100644
> --- a/Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml
> +++ b/Documentation/devicetree/bindings/media/i2c/maxim,max96717.yaml
> @@ -12,16 +12,15 @@ maintainers:
> - Cosmin Tanislav <cosmin.tanislav@xxxxxxxxxx>
>
> description:

Use '>' token for multiple paragraphs.

> - The MAX96717 serializer converts MIPI CSI-2 D-PHY formatted input
> - into GMSL2 serial outputs. The device allows the GMSL2 link to
> - simultaneously transmit bidirectional control-channel data while forward
> - video transmissions are in progress. The MAX96717 can connect to one
> - remotely located deserializer using industry-standard coax or STP
> - interconnects. The device cans operate in pixel or tunnel mode. In pixel mode
> - the MAX96717 can select the MIPI datatype, while the tunnel mode forward all the MIPI
> - data received by the serializer.
> - The MAX96717 supports Reference Over Reverse (channel),
> - to generate a clock output for the sensor from the GMSL reverse channel.
> + The MAX96717 serializer converts MIPI CSI-2 D-PHY formatted input into GMSL2
> + serial outputs. The device allows the GMSL2 link to simultaneously transmit
> + bidirectional control-channel data while forward video transmissions are in
> + progress. The MAX96717 can connect to one remotely located deserializer using
> + industry-standard coax or STP interconnects. The device cans operate in pixel
> + or tunnel mode. In pixel mode the MAX96717 can select the MIPI datatype, while
> + the tunnel mode forward all the MIPI data received by the serializer.

> + The MAX96717 supports Reference Over Reverse (channel), to generate a clock
> + output for the sensor from the GMSL reverse channel.

Is this a separate paragraph? If not, the wrapping is not right. If so,
then needs a blank line in between.

>
> The GMSL2 serial link operates at a fixed rate of 3Gbps or 6Gbps in the
> forward direction and 187.5Mbps in the reverse direction.
> --
> 2.48.1
>