Re: [PATCH v3] drm/connector: hdmi: Fix writing Dynamic Range Mastering infoframes

2024-08-27 Thread Derek Foreman
On 2024-08-27 05:19, Jani Nikula wrote: On Tue, 27 Aug 2024, Maxime Ripard wrote: Hi, On Mon, Aug 26, 2024 at 07:10:11AM GMT, Derek Foreman wrote: The largest infoframe we create is the DRM (Dynamic Range Mastering) infoframe which is 26 bytes + a 4 byte header, for a total of 30 bytes. With

Re: [PATCH v3] drm/connector: hdmi: Fix writing Dynamic Range Mastering infoframes

2024-08-27 Thread Jani Nikula
On Tue, 27 Aug 2024, Maxime Ripard wrote: > Hi, > > On Mon, Aug 26, 2024 at 07:10:11AM GMT, Derek Foreman wrote: >> The largest infoframe we create is the DRM (Dynamic Range Mastering) >> infoframe which is 26 bytes + a 4 byte header, for a total of 30 >> bytes. >> >> With HDMI_MAX_INFOFRAME_SIZE

Re: [PATCH v3] drm/connector: hdmi: Fix writing Dynamic Range Mastering infoframes

2024-08-27 Thread Maxime Ripard
Hi, On Mon, Aug 26, 2024 at 07:10:11AM GMT, Derek Foreman wrote: > The largest infoframe we create is the DRM (Dynamic Range Mastering) > infoframe which is 26 bytes + a 4 byte header, for a total of 30 > bytes. > > With HDMI_MAX_INFOFRAME_SIZE set to 29 bytes, as it is now, we > allocate too lit

Re: [PATCH v3] drm/connector: hdmi: Fix writing Dynamic Range Mastering infoframes

2024-08-26 Thread Jani Nikula
On Mon, 26 Aug 2024, Derek Foreman wrote: > The largest infoframe we create is the DRM (Dynamic Range Mastering) > infoframe which is 26 bytes + a 4 byte header, for a total of 30 > bytes. > > With HDMI_MAX_INFOFRAME_SIZE set to 29 bytes, as it is now, we > allocate too little space to pack a DRM

[PATCH v3] drm/connector: hdmi: Fix writing Dynamic Range Mastering infoframes

2024-08-26 Thread Derek Foreman
The largest infoframe we create is the DRM (Dynamic Range Mastering) infoframe which is 26 bytes + a 4 byte header, for a total of 30 bytes. With HDMI_MAX_INFOFRAME_SIZE set to 29 bytes, as it is now, we allocate too little space to pack a DRM infoframe in write_device_infoframe(), leading to an E