linux-kernel - Re: [PATCH] media: v4l: Fix missing tabular column hint for Y14P format

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <bcdce8da-a0de-4138-a821-9ee2a3183e24@gmail.com>
Date: Sun, 9 Jun 2024 08:52:52 +0900
From: Akira Yokosawa <akiyks@...il.com>
To: Jean-Michel Hautbois <jeanmichel.hautbois@...eli.org>
Cc: linux-media@...r.kernel.org, linux-kernel@...r.kernel.org,
 Mauro Carvalho Chehab <mchehab@...nel.org>,
 Laurent Pinchart <laurent.pinchart@...asonboard.com>,
 Sakari Ailus <sakari.ailus@...ux.intel.com>,
 Hans Verkuil <hverkuil-cisco@...all.nl>, Akira Yokosawa <akiyks@...il.com>
Subject: Re: [PATCH] media: v4l: Fix missing tabular column hint for Y14P
 format

Hi,

On Sat, 08 Jun 2024 18:41:27 +0200, Jean-Michel Hautbois wrote:
> The original commit added two columns in the flat-table of Luma-Only
> Image Formats, without updating hints to latex: above it.  This results
> in wrong column count in the output of Sphinx's latex builder.
> 
> Fix it.
> 
> Reported-by: Akira Yokosawa <akiyks@...il.com>
> Link: https://lore.kernel.org/linux-media/bdbc27ba-5098-49fb-aabf-753c81361cc7@gmail.com/
> Fixes: adb1d4655e53 (media: v4l: Add V4L2-PIX-FMT-Y14P format)
> Signed-off-by: Jean-Michel Hautbois <jeanmichel.hautbois@...eli.org>
> ---
>  Documentation/userspace-api/media/v4l/pixfmt-yuv-luma.rst | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/userspace-api/media/v4l/pixfmt-yuv-luma.rst b/Documentation/userspace-api/media/v4l/pixfmt-yuv-luma.rst
> index b3c5779521d8..2e7d0d3151a1 100644
> --- a/Documentation/userspace-api/media/v4l/pixfmt-yuv-luma.rst
> +++ b/Documentation/userspace-api/media/v4l/pixfmt-yuv-luma.rst
> @@ -21,9 +21,9 @@ are often referred to as greyscale formats.
>  
>  .. raw:: latex
>  
> -    \scriptsize
> +    \tiny
>  
> -.. tabularcolumns:: |p{3.6cm}|p{3.0cm}|p{1.3cm}|p{2.6cm}|p{1.3cm}|p{1.3cm}|p{1.3cm}|
> +.. tabularcolumns:: |p{3.6cm}|p{2.4cm}|p{1.3cm}|p{1.3cm}|p{1.3cm}|p{1.3cm}|p{1.3cm}|p{1.3cm}|p{1.3cm}|

With the \tiny font, widths of columns 1 and 2 can be reduced.

Either way, this fixes the build error of pdfdocs.  So,

Reviewed-and-tested-by: Akira Yokosawa <akiyks@...il.com>

        Thanks, Akira

>  
>  .. flat-table:: Luma-Only Image Formats
>      :header-rows: 1
> 
> ---
> base-commit: dc772f8237f9b0c9ea3f34d0dc4a57d1f6a5070d
> change-id: 20240608-jmh-correct-mkdocs-luma-79d348ded7aa
> 
> Best regards,