Re: [PATCH v3 1/7] media: Add 2-10-10-10 RGB formats

From: Akira Yokosawa <akiyks@gmail.com>
To: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Cc: dri-devel@lists.freedesktop.org, geert@linux-m68k.org,
	kieran.bingham@ideasonboard.com,
	laurent.pinchart@ideasonboard.com, linux-media@vger.kernel.org,
	linux-renesas-soc@vger.kernel.org, nicolas@ndufresne.ca,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	Hans Verkuil <hverkuil-cisco@xs4all.nl>
Subject: Re: [PATCH v3 1/7] media: Add 2-10-10-10 RGB formats
Date: Wed, 8 Feb 2023 17:36:16 +0900	[thread overview]
Message-ID: <6e947a02-ebb7-8a3b-f0f4-87684cb73e6b@gmail.com> (raw)
In-Reply-To: <234cee3f-3902-16d8-8fb7-6d79749d9379@ideasonboard.com>

On Wed, 8 Feb 2023 09:56:24 +0200, Tomi Valkeinen wrote:
> On 08/02/2023 03:53, Akira Yokosawa wrote:
[...]
>> Tomi, can you remove those extra "-"s I mentioned above?
>>
>> Hopefully, this can be fixed before the v6.3 merge window.
> 
> This is all black magics to me... Why does the same format work elsewhere in the file, but not here (I just copy pasted it)?

I think I know your feeling...

Let me explain some background.

The directive "flat-table" is a Linux-kernel-documentation specific
extension to the standard "list-table" directive [1,2].  If you have
inconsistent number of elements in lists, they are not regarded as
proper format list/flat-tables.
Sphinx builders are not strict about them, but in my opinion, latexmk
(or an indexing utility, in this case) can't be blamed for not accepting
resulting .tex files.

This is all I can say.

[1]: https://www.kernel.org/doc/html/latest/doc-guide/sphinx.html#list-tables
[2]: https://docutils.sourceforge.io/docs/ref/rst/directives.html#list-table

> 
> Oh well... I can send a fix, although I don't know why it's broken =).

Thank you for sending out the fix.

        Thanks, Akira

> 
>  Tomi
> 