Re: [PATCH RESEND v6 05/11] drm/mediatek: gamma: Enable the Gamma LUT table only after programming

From: Alexandre Mergnat
Date: Fri Jul 28 2023 - 09:01:12 EST


Reviewed-by: Alexandre Mergnat <amergnat@xxxxxxxxxxxx>

On 27/07/2023 11:46, AngeloGioacchino Del Regno wrote:
Move the write to DISP_GAMMA_CFG to enable the Gamma LUT to after
programming the actual table to avoid potential visual glitches during
table modification.

Note:
GAMMA should get enabled in between vblanks, but this requires many
efforts in order to make this happen, as that requires migrating all
of the writes to make use of CMDQ instead of cpu writes and that's
not trivial. For this reason, this patch only moves the LUT enable.
The CMDQ rework will come at a later time.

--
Regards,
Alexandre