Re: [PATCH 6/7] drm/tidss: Fix race condition while handling interrupt registers

[Aradhya Bhatia]

On 10/21/24 19:37, Tomi Valkeinen wrote:
> From: Devarsh Thakkar <devarsht@xxxxxx>
> 
> The driver has a spinlock for protecting the irq_masks field and irq
> enable registers. However, the driver misses protecting the irq status
> registers which can lead to races.
> 
> Take the spinlock when accessing irqstatus too.
> 
> Fixes: 32a1795f57ee ("drm/tidss: New driver for TI Keystone platform Display SubSystem")
> Cc: stable@xxxxxxxxxxxxxxx
> Signed-off-by: Devarsh Thakkar <devarsht@xxxxxx>
> [Tomi: updated the desc]
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@xxxxxxxxxxxxxxxx>

Reviewed-by: Aradhya Bhatia <aradhya.bhatia@xxxxxxxxx>

Regards
Aradhya

[...]