[PATCH v2 04/10] drm/fourcc: Add DRM_FORMAT_Y10_LE32

[Tomi Valkeinen]

On 15/01/2025 16:49, Geert Uytterhoeven wrote:
> Hi Tomi,
> 
> On Wed, Jan 15, 2025 at 3:34 PM Tomi Valkeinen
> <tomi.valkeinen at ideasonboard.com> wrote:
>> On 15/01/2025 16:08, Geert Uytterhoeven wrote:
>>> On Wed, Jan 15, 2025 at 2:46 PM Tomi Valkeinen
>>> <tomi.valkeinen at ideasonboard.com> wrote:
>>>> On 15/01/2025 14:52, Geert Uytterhoeven wrote:
>>>>> On Wed, Jan 15, 2025 at 1:42 PM Tomi Valkeinen
>>>>> <tomi.valkeinen at ideasonboard.com> wrote:
>>>>>> On 15/01/2025 14:33, Geert Uytterhoeven wrote:
>>>>>>> On Wed, Jan 15, 2025 at 12:11 PM Tomi Valkeinen
>>>>>>> <tomi.valkeinen at ideasonboard.com> wrote:
>>>>>>>> On 15/01/2025 12:33, Geert Uytterhoeven wrote:
>>>>>>>>> On Wed, Jan 15, 2025 at 10:04 AM Tomi Valkeinen
>>>>>>>>> <tomi.valkeinen at ideasonboard.com> wrote:
>>>>>>>>>> Add Y10_LE32, a 10 bit greyscale format, with 3 pixels packed into
>>>>>>>>>> 32-bit container.
> 
>> Well, this is all somewhat beside the point. So is "_Y10_32" (or
>> "_R10_32" if we use R) fine?
> 
> IAARDD[*], but "10_32" makes me think it stores one 10-bit pixel in a
> 32-bit word (i.e. a 32-bit variant of R10). So [YR]PA4 would be better.

That would be a very silly format. So I think some HW guy is already 
designing chips that have it.

[YR]10_PACKED_32? We can have longer names, we don't need to do encoding 
like [YR]PA4.

> [*] I am a rookie DRM developer...

I haven't heard that before...

  Tomi