Re: [PATCH v2 2/2] [media] v4l: Add 10/16-bits per channel YUV pixel formats

From: Ayaka
Date: Wed Feb 08 2017 - 04:18:50 EST




從我的 iPad 傳送

> Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxxxxx> 於 2017年2月3日 下午10:04 寫道:
>
> Em Thu, 5 Jan 2017 20:27:17 +0200
> Sakari Ailus <sakari.ailus@xxxxxx> escreveu:
>
>> Hi Randy,
>>
>>> On Thu, Jan 05, 2017 at 11:22:26PM +0800, ayaka wrote:
>>>
>>>
>>>> On 01/05/2017 06:30 PM, Sakari Ailus wrote:
>>>> Hi Randy,
>>>>
>>>> Thanks for the update.
>>>>
>>>>> On Thu, Jan 05, 2017 at 12:29:11AM +0800, Randy Li wrote:
>>>>> The formats added by this patch are:
>>>>> V4L2_PIX_FMT_P010
>>>>> V4L2_PIX_FMT_P010M
>>>>> V4L2_PIX_FMT_P016
>>>>> V4L2_PIX_FMT_P016M
>>>>> Currently, none of driver uses those format, but some video device
>>>>> has been confirmed with could as those format for video output.
>>>>> The Rockchip's new decoder has supported those 10 bits format for
>>>>> profile_10 HEVC/AVC video.
>>>>>
>>>>> Signed-off-by: Randy Li <ayaka@xxxxxxxxxxx>
>>>>>
>>>>> v4l2
>>>>> ---
>>>>> Documentation/media/uapi/v4l/pixfmt-p010.rst | 86 ++++++++++++++++
>>>>> Documentation/media/uapi/v4l/pixfmt-p010m.rst | 94 ++++++++++++++++++
>>>>> Documentation/media/uapi/v4l/pixfmt-p016.rst | 126 ++++++++++++++++++++++++
>>>>> Documentation/media/uapi/v4l/pixfmt-p016m.rst | 136 ++++++++++++++++++++++++++
>>>> You need to include the formats in pixfmt.rst in order to compile the
>>>> documentation.
>>>>
>>>> $ make htmldocs
>>>>
>>>> And you'll find it in Documentation/output/media/uapi/v4l/v4l2.html .
>>>>
>>>> In Debian you'll need to install sphinx-common and python3-sphinx-rtd-theme
>>>> .
>>> OK, I would fix them in new version.
>>> The view of byte order for P010 serial is left empty, it is a little hard
>>> for me to use flat-table to draw them. Is there possible to use something
>>> like latex to do this job?
>>
>> Hmm. Not as far as I know. We recently switched from DocBook mostly due to
>> ReST being more simple to use AFAIU. I think LaTeX output could be produced
>> ReST, that might not be very helpful here though.
>
> No, you can't use LaTeX, as it won't be properly displayed on all output
> formats. There are a few options to define tables in ReST, but we prefer
> using flat-table because the other formats are harder to maintain at the
> V4L2 uAPI documentation.
>
> Just one note about this series: it won't be merged upstream until
> someone adds a driver needing those pixel formats.
>
I made a mistake, the pixel format I want is not P010, I would post a new patch for that.
> Regards,
> Mauro
>
>
> Thanks,
> Mauro