2024-06-08 20:57:21

by Jean-Michel Hautbois

[permalink] [raw]
Subject: [PATCH] media: v4l: Fix missing tabular column hint for Y14P format

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 <[email protected]>
Link: https://lore.kernel.org/linux-media/[email protected]/
Fixes: adb1d4655e53 (media: v4l: Add V4L2-PIX-FMT-Y14P format)
Signed-off-by: Jean-Michel Hautbois <[email protected]>
---
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}|

.. flat-table:: Luma-Only Image Formats
:header-rows: 1

---
base-commit: dc772f8237f9b0c9ea3f34d0dc4a57d1f6a5070d
change-id: 20240608-jmh-correct-mkdocs-luma-79d348ded7aa

Best regards,
--
Jean-Michel Hautbois <[email protected]>



2024-06-08 23:53:40

by Akira Yokosawa

[permalink] [raw]
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 <[email protected]>
> Link: https://lore.kernel.org/linux-media/[email protected]/
> Fixes: adb1d4655e53 (media: v4l: Add V4L2-PIX-FMT-Y14P format)
> Signed-off-by: Jean-Michel Hautbois <[email protected]>
> ---
> 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 <[email protected]>

Thanks, Akira

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