Re: [PATCH v2 2/3] media: Documentation: BT.601 is not a bus

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sakari,

Thank you for the patch.

On Tue, Oct 17, 2023 at 01:56:29PM +0300, Sakari Ailus wrote:
> BT.601 is not actually a bus specification, leaving parallel bus without a
> specification to refer to. Fix this.

I'm really annoyed there's no standard name for parallel buses :-(

> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
> ---
>  Documentation/driver-api/media/camera-sensor.rst | 4 ++--
>  Documentation/driver-api/media/tx-rx.rst         | 3 +--
>  2 files changed, 3 insertions(+), 4 deletions(-)
> 
> diff --git a/Documentation/driver-api/media/camera-sensor.rst b/Documentation/driver-api/media/camera-sensor.rst
> index 0de5c86cbd1f..19f2feeecc91 100644
> --- a/Documentation/driver-api/media/camera-sensor.rst
> +++ b/Documentation/driver-api/media/camera-sensor.rst
> @@ -9,8 +9,8 @@ This document covers the in-kernel APIs only. For the best practices on
>  userspace API implementation in camera sensor drivers, please see
>  :ref:`media_using_camera_sensor_drivers`.
>  
> -CSI-2 and parallel (BT.601 and BT.656) busses
> ----------------------------------------------
> +CSI-2 and parallel and BT.656 buses

CSI-2, parallel and BT.656 buses

> +-----------------------------------
>  
>  Please see :ref:`transmitter-receiver`.
>  
> diff --git a/Documentation/driver-api/media/tx-rx.rst b/Documentation/driver-api/media/tx-rx.rst
> index e1e9258dd862..7e115e3c4735 100644
> --- a/Documentation/driver-api/media/tx-rx.rst
> +++ b/Documentation/driver-api/media/tx-rx.rst
> @@ -25,9 +25,8 @@ the host SoC. It is defined by the `MIPI alliance`_.
>  Parallel
>  ^^^^^^^^
>  
> -`BT.601`_ and `BT.656`_ are the most common parallel busses.
> +The parallel bus and its `BT.656`_ variant are the most common parallel busses.

We use "parallel" to mean explicit sync signals in many places
(including APIs), and here it covers BT.656 too :-( This sentence is
fairly bad.

>  
> -.. _`BT.601`: https://en.wikipedia.org/wiki/Rec._601
>  .. _`BT.656`: https://en.wikipedia.org/wiki/ITU-R_BT.656
>  
>  Transmitter drivers

-- 
Regards,

Laurent Pinchart



[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux