Re: [PATCH RESEND] media: imagination: Fix value clamping in calculate_qp_tables()

From: Thorsten Blum

Date: Mon Aug 31 2026 - 04:19:32 EST


On Mon, Jul 27, 2026 at 04:04:51PM +0530, Jai Luthra wrote:
> Quoting Thorsten Blum (2026-07-27 15:17:30)
> > The local variable val was never clamped to 1 or 255 because the return
> > value of clamp() was not used. Fix this by assigning the clamped value
> > back to val.
> >
> > Fixes: a1e294045885 ("media: imagination: Add E5010 JPEG Encoder driver")
> > Signed-off-by: Thorsten Blum <thorsten.blum@xxxxxxxxx>
>
> Reviewed-by: Jai Luthra <jai.luthra@xxxxxxxxxxxxxxxx>

Thank you for the review.

Could someone please pick this up?

Thanks,
Thorsten