Hi Harry, On Mon, Feb 07, 2022 at 01:59:38PM -0500, Harry Wentland wrote: > On 2022-02-07 13:57, Harry Wentland wrote: > > On 2022-02-07 11:34, Maxime Ripard wrote: > >> The amdgpu KMS driver calls drm_plane_create_color_properties() with a > >> default encoding set to BT709. > >> > >> However, the core will ignore it and the driver doesn't force it in its > >> plane state reset hook, so the initial value will be 0, which represents > >> BT601. > >> > > > > Isn't this a core issue? Should __drm_atomic_helper_plane_state_reset > > reset all plane_state members to their properties' default values? > > > > Ah, looks like that's exactly what you do in the later patches, which is > perfect. With that, I don't think you'll need this patch anymore. Ok, I'll squash it into the patch that removes the reset code. Thanks! Maxime
Attachment:
signature.asc
Description: PGP signature
