Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1602669 > unrolled thread

Re: [PATCH v3 3/6] documentation: media: Add documentation for new RGB and YUV bus formats

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v3 3/6] documentation: media: Add documentation for new  RGB and YUV bus formats Archit Taneja <architt@codeaurora.org> - 2017-03-16 18:10 +0100
    Re: [PATCH v3 3/6] documentation: media: Add documentation for new  RGB and YUV bus formats Neil Armstrong <narmstrong@baylibre.com> - 2017-03-17 17:20 +0100
      Re: [PATCH v3 3/6] documentation: media: Add documentation for new  RGB and YUV bus formats Hans Verkuil <hverkuil@xs4all.nl> - 2017-03-20 16:10 +0100

#1602669 — Re: [PATCH v3 3/6] documentation: media: Add documentation for new RGB and YUV bus formats

On 3/7/2017 10:12 PM, Neil Armstrong wrote:
> Add documentation for added Bus Formats to describe RGB and YUS formats used

s/YUS/YUV

> as input to the Synopsys DesignWare HDMI TX Controller.
>
> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> ---
>  Documentation/media/uapi/v4l/subdev-formats.rst | 4992 ++++++++++++++++++-----
>  1 file changed, 3963 insertions(+), 1029 deletions(-)

Do we know if there is a better way to add more columns without
adding so many lines?

If not, one option could be to create a separate tables for
48 bit RGB formats, 48 bit YUV formats etc.

<snip>

Thanks,
Archit

-- 
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
hosted by The Linux Foundation

[toc] | [next] | [standalone]

#1603436

On 03/16/2017 06:01 PM, Archit Taneja wrote:
> 
> 
> On 3/7/2017 10:12 PM, Neil Armstrong wrote:
>> Add documentation for added Bus Formats to describe RGB and YUS formats used
> 
> s/YUS/YUV

Thanks again....

> 
>> as input to the Synopsys DesignWare HDMI TX Controller.
>>
>> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
>> ---
>>  Documentation/media/uapi/v4l/subdev-formats.rst | 4992 ++++++++++++++++++-----
>>  1 file changed, 3963 insertions(+), 1029 deletions(-)
> 
> Do we know if there is a better way to add more columns without
> adding so many lines?

It seems not, the reason is written in the commands.

> If not, one option could be to create a separate tables for
> 48 bit RGB formats, 48 bit YUV formats etc.

It would be simple indeed, any V4L guys for an advice here ?

Thanks,
Neil

> 
> <snip>
> 
> Thanks,
> Archit
> 

[toc] | [prev] | [next] | [standalone]

#1604669

On 03/17/2017 05:11 PM, Neil Armstrong wrote:
> On 03/16/2017 06:01 PM, Archit Taneja wrote:
>>
>>
>> On 3/7/2017 10:12 PM, Neil Armstrong wrote:
>>> Add documentation for added Bus Formats to describe RGB and YUS formats used
>>
>> s/YUS/YUV
> 
> Thanks again....
> 
>>
>>> as input to the Synopsys DesignWare HDMI TX Controller.
>>>
>>> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
>>> ---
>>>  Documentation/media/uapi/v4l/subdev-formats.rst | 4992 ++++++++++++++++++-----
>>>  1 file changed, 3963 insertions(+), 1029 deletions(-)
>>
>> Do we know if there is a better way to add more columns without
>> adding so many lines?
> 
> It seems not, the reason is written in the commands.
> 
>> If not, one option could be to create a separate tables for
>> 48 bit RGB formats, 48 bit YUV formats etc.
> 
> It would be simple indeed, any V4L guys for an advice here ?

I would split up these large tables into separate tables, depending on the
number of bits each pixel uses: so an RGB table for 8 bits, 9-16 bits,
17-24, 25-32, 33-48.

This also avoids a major problem where the horizontal scrollbar is at the bottom
of the table, but the column numbering is at the top and out of sight if the
table is long.

It also prevents these large horizontal widths when they are not needed.

Regards,

	Hans

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web