Received-SPF: pass (google.com: domain of linux-kernel+bounces-207219-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <bcdce8da-a0de-4138-a821-9ee2a3183e24@gmail.com>
Date: Sun, 9 Jun 2024 08:52:52 +0900
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] media: v4l: Fix missing tabular column hint for Y14P
 format
To: Jean-Michel Hautbois <jeanmichel.hautbois@yoseli.org>
Cc: linux-media@vger.kernel.org, linux-kernel@vger.kernel.org,
 Mauro Carvalho Chehab <mchehab@kernel.org>,
 Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
 Sakari Ailus <sakari.ailus@linux.intel.com>,
 Hans Verkuil <hverkuil-cisco@xs4all.nl>, Akira Yokosawa <akiyks@gmail.com>
References: <20240608-jmh-correct-mkdocs-luma-v1-1-7e114a2c4bdf@yoseli.org>
Content-Language: en-US
From: Akira Yokosawa <akiyks@gmail.com>
In-Reply-To: <20240608-jmh-correct-mkdocs-luma-v1-1-7e114a2c4bdf@yoseli.org>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

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@gmail.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@yoseli.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@gmail.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,