On Fri, 19 Jan 2024 11:58:38 +0100 Thomas Zimmermann <tzimmermann@xxxxxxx> wrote: > Hi > > Am 17.01.24 um 17:40 schrieb Jocelyn Falempe: ... > > The last thing, is if I plan to add YUV support, with this > > implementation, I only need to write one function that convert one > > pixel. Otherwise I would need to add the drm_fb_r1_to_yuvxxx_line() and > > drm_fb_r1_to_yuvxxxx() boilerplate. > > 8) YUVs are multi-plane formats IIRC. So it's likely a bit more > complicated.And I'm not aware of any current use case for YUV. If the > framebuffer console doesn't support it, the panic helper probably won't > either. Kernel panic during a fullscreen video playback, maybe? That use case is likely to have an YUV FB as the only visible KMS plane FB, either primary or overlay plane depending on which is capable of displaying it. Sub-titles might not exist, or might be in a fairly small RGB overlay. I don't know if such case is important enough to handle. Thanks, pq
Attachment:
pgp93idi5DVlR.pgp
Description: OpenPGP digital signature
