<div dir="ltr">Hi Laurent,<div><br></div><div>looks like linux/Documentation/userspace-api/media/v4l/subdev-formats.rst doesn't correlate at all to the arrangement and numbering in linux/include/uapi/linux/media-bus-format.h .</div><div><br></div><div>Which sources do you want me to check?</div><div><br></div><div>Looking at <a href="https://github.com/raspberrypi/linux/tree/rpi-6.1.y">https://github.com/raspberrypi/linux/tree/rpi-6.1.y</a> btw.</div><div><br></div><div>Rgds</div><div>Joerg</div><div><br></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Am Di., 29. Nov. 2022 um 13:38 Uhr schrieb Laurent Pinchart <<a href="mailto:laurent.pinchart@ideasonboard.com" target="_blank">laurent.pinchart@ideasonboard.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">Hi Maxime and Joerg,<br>
<br>
Thank you for the patch.<br>
<br>
On Thu, Oct 20, 2022 at 10:30:47AM +0200, Maxime Ripard wrote:<br>
> From: Joerg Quinten <<a href="mailto:aBUGSworstnightmare@gmail.com" target="_blank">aBUGSworstnightmare@gmail.com</a>><br>
> <br>
> Add the BGR666 format MEDIA_BUS_FMT_BGR666_1X24_CPADHI supported by the<br>
> RaspberryPi.<br>
> <br>
> Signed-off-by: Joerg Quinten <<a href="mailto:aBUGSworstnightmare@gmail.com" target="_blank">aBUGSworstnightmare@gmail.com</a>><br>
> Signed-off-by: Maxime Ripard <maxime@cerno.tech><br>
> ---<br>
>  .../userspace-api/media/v4l/subdev-formats.rst     | 37 ++++++++++++++++++++++<br>
>  include/uapi/linux/media-bus-format.h              |  3 +-<br>
>  2 files changed, 39 insertions(+), 1 deletion(-)<br>
> <br>
> diff --git a/Documentation/userspace-api/media/v4l/subdev-formats.rst b/Documentation/userspace-api/media/v4l/subdev-formats.rst<br>
> index 68f8d7d37984..604a30e2f890 100644<br>
> --- a/Documentation/userspace-api/media/v4l/subdev-formats.rst<br>
> +++ b/Documentation/userspace-api/media/v4l/subdev-formats.rst<br>
> @@ -1023,6 +1023,43 @@ The following tables list existing packed RGB formats.<br>
>        - g\ :sub:`2`<br>
>        - g\ :sub:`1`<br>
>        - g\ :sub:`0`<br>
> +    * .. _MEDIA-BUS-FMT-BGR666-1X24_CPADHI:<br>
<br>
I would move this just below MEDIA_BUS_FMT_RGB565_1X24_CPADHI. Actually,<br>
could you check 1/7 and 2/7 to make sure the formats are sorted in the<br>
documentation in the same order as in the header ?<br>
<br>
> +<br>
> +      - MEDIA_BUS_FMT_BGR666_1X24_CPADHI<br>
> +      - 0x1024<br>
> +      -<br>
> +      -<br>
> +      -<br>
> +      -<br>
> +      -<br>
> +      -<br>
> +      -<br>
> +      -<br>
> +      -<br>
> +      - 0<br>
> +      - 0<br>
> +      - b\ :sub:`5`<br>
> +      - b\ :sub:`4`<br>
> +      - b\ :sub:`3`<br>
> +      - b\ :sub:`2`<br>
> +      - b\ :sub:`1`<br>
> +      - b\ :sub:`0`<br>
> +      - 0<br>
> +      - 0<br>
> +      - g\ :sub:`5`<br>
> +      - g\ :sub:`4`<br>
> +      - g\ :sub:`3`<br>
> +      - g\ :sub:`2`<br>
> +      - g\ :sub:`1`<br>
> +      - g\ :sub:`0`<br>
> +      - 0<br>
> +      - 0<br>
> +      - r\ :sub:`5`<br>
> +      - r\ :sub:`4`<br>
> +      - r\ :sub:`3`<br>
> +      - r\ :sub:`2`<br>
> +      - r\ :sub:`1`<br>
> +      - r\ :sub:`0`<br>
>      * .. _MEDIA-BUS-FMT-RGB565-1X24_CPADHI:<br>
>  <br>
>        - MEDIA_BUS_FMT_RGB565_1X24_CPADHI<br>
> diff --git a/include/uapi/linux/media-bus-format.h b/include/uapi/linux/media-bus-format.h<br>
> index 2ee0b38c0a71..d4228d038b54 100644<br>
> --- a/include/uapi/linux/media-bus-format.h<br>
> +++ b/include/uapi/linux/media-bus-format.h<br>
> @@ -34,7 +34,7 @@<br>
>  <br>
>  #define MEDIA_BUS_FMT_FIXED                  0x0001<br>
>  <br>
> -/* RGB - next is     0x1024 */<br>
> +/* RGB - next is     0x1025 */<br>
>  #define MEDIA_BUS_FMT_RGB444_1X12            0x1016<br>
>  #define MEDIA_BUS_FMT_RGB444_2X8_PADHI_BE    0x1001<br>
>  #define MEDIA_BUS_FMT_RGB444_2X8_PADHI_LE    0x1002<br>
> @@ -49,6 +49,7 @@<br>
>  #define MEDIA_BUS_FMT_BGR666_1X18            0x1023<br>
>  #define MEDIA_BUS_FMT_RGB666_1X18            0x1009<br>
>  #define MEDIA_BUS_FMT_RBG888_1X24            0x100e<br>
> +#define MEDIA_BUS_FMT_BGR666_1X24_CPADHI     0x1024<br>
>  #define MEDIA_BUS_FMT_RGB666_1X24_CPADHI     0x1015<br>
>  #define MEDIA_BUS_FMT_RGB666_1X7X3_SPWG              0x1010<br>
>  #define MEDIA_BUS_FMT_BGR888_1X24            0x1013<br>
> <br>
<br>
-- <br>
Regards,<br>
<br>
Laurent Pinchart<br>
</blockquote></div>