Re: [PATCH 6/7] drm/bridge: tc358768: Separate video format config

[Francesco Dolcini]

On Tue, Oct 21, 2025 at 04:23:02PM +0300, Tomi Valkeinen wrote:
> Sending long commands using the video buffer (to be implemented in
> following patches) requires setting TC358768_DATAFMT and
> TC358768_DSITX_DT registers for command transfer. The same registers
> also need to be configured properly for video transfer.
> 
> The long commands will be sent between the bridge's pre_enable() and
> enable(), and currently we configure the registers for video transfer in
> pre_enable(). Thus, they would be overwritten by the long command
> transfer code.
> 
> To prevent that from happening, set those registers for video transfer
> in enable(), not in pre_enable().
> 
> Based on code from Parth Pancholi <parth.pancholi@xxxxxxxxxxx>
> 
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@xxxxxxxxxxxxxxxx>

Reviewed-by: Francesco Dolcini <francesco.dolcini@xxxxxxxxxxx>