<div dir="ltr"><div dir="ltr"><img class="gmail-ajT" src="https://ssl.gstatic.com/ui/v1/icons/mail/images/cleardot.gif"><img class="gmail-ajT" src="https://ssl.gstatic.com/ui/v1/icons/mail/images/cleardot.gif">Hi Nicolas,<br></div><div><br></div><div>Thanks for your comment.</div><div>I have sent v2 patch to add Hextile document and locate with vendor formats.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Nicolas Dufresne <<a href="mailto:nicolas@ndufresne.ca">nicolas@ndufresne.ca</a>> 於 2022年5月12日 週四 下午9:18寫道：<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hello Marvin,<br>
<br>
Le jeudi 12 mai 2022 à 15:27 +0800, Marvin Lin a écrit :<br>
> Add HEXTILE compressed format. This format is defined in Remote<br>
> Framebuffer Protocol (RFC 6143) and is used by VNC features.<br>
> <br>
> Signed-off-by: Marvin Lin <<a href="mailto:kflin@nuvoton.com" target="_blank">kflin@nuvoton.com</a>><br>
> ---<br>
>  drivers/media/v4l2-core/v4l2-ioctl.c | 1 +<br>
>  include/uapi/linux/videodev2.h       | 1 +<br>
>  2 files changed, 2 insertions(+)<br>
> <br>
> diff --git a/drivers/media/v4l2-core/v4l2-ioctl.c b/drivers/media/v4l2-core/v4l2-ioctl.c<br>
> index 96e307fe3aab..9365519d65bb 100644<br>
> --- a/drivers/media/v4l2-core/v4l2-ioctl.c<br>
> +++ b/drivers/media/v4l2-core/v4l2-ioctl.c<br>
> @@ -1441,6 +1441,7 @@ static void v4l_fill_fmtdesc(struct v4l2_fmtdesc *fmt)<br>
>               case V4L2_PIX_FMT_SE401:        descr = "GSPCA SE401"; break;<br>
>               case V4L2_PIX_FMT_S5C_UYVY_JPG: descr = "S5C73MX interleaved UYVY/JPEG"; break;<br>
>               case V4L2_PIX_FMT_MT21C:        descr = "Mediatek Compressed Format"; break;<br>
> +             case V4L2_PIX_FMT_HEXTILE:      descr = "Hextile Compressed Format"; break;<br>
>               default:<br>
>                       if (fmt->description[0])<br>
>                               return;<br>
> diff --git a/include/uapi/linux/videodev2.h b/include/uapi/linux/videodev2.h<br>
> index 3768a0a80830..e0e69f71d700 100644<br>
> --- a/include/uapi/linux/videodev2.h<br>
> +++ b/include/uapi/linux/videodev2.h<br>
> @@ -710,6 +710,7 @@ struct v4l2_pix_format {<br>
>  #define V4L2_PIX_FMT_FWHT     v4l2_fourcc('F', 'W', 'H', 'T') /* Fast Walsh Hadamard Transform (vicodec) */<br>
>  #define V4L2_PIX_FMT_FWHT_STATELESS     v4l2_fourcc('S', 'F', 'W', 'H') /* Stateless FWHT (vicodec) */<br>
>  #define V4L2_PIX_FMT_H264_SLICE v4l2_fourcc('S', '2', '6', '4') /* H264 parsed slices */<br>
> +#define V4L2_PIX_FMT_HEXTILE  v4l2_fourcc('H', 'X', 'T', 'L') /* Hextile */<br>
<br>
New formats also need to be documented in one of <br>
Documentation/userspace-api/media/v4l/pixfmt*.rst file. Consider that formats,<br>
specially vendor formats disappears over time, so documentation should be<br>
written as if you are to read that in 10+ years.<br>
<br>
I'm also wondering if that should be better located with the "vendor" formats. <br>
Specially that this does not look like a open standard like the other in this<br>
section. A quick search of that format simply gives no results for me. It seems<br>
pretty clear that this is unique to this HW vendor and unlikely to grow withing<br>
other HW or even ever get a software implementation.<br>
>  <br>
>  /*  Vendor-specific formats   */<br>
>  #define V4L2_PIX_FMT_CPIA1    v4l2_fourcc('C', 'P', 'I', 'A') /* cpia1 YUV */<br>
<br>
</blockquote></div></div>