Re: [PATCH 3/3] media: verisilicon: leave room for the JPEG header and one stream burst

From: Sergei X .

Date: Tue Nov 10 2026 - 06:35:06 EST


> + max_t(u32, pix_mp->plane_fmt[0].sizeimage,
> + fmt->header_size + 64);
>
> Did you mean to align to 64 instead ?

No, the max() is meant as a floor: the 624-byte header plus one 64-byte
stream burst. Patch 2 already rounds the stream limit down to whole
64-byte bursts. ALIGN(sizeimage, 64) would neither keep that floor nor
align the stream part, which starts at offset 624.