[PATCH 3/3] media: verisilicon: leave room for the JPEG header and one stream burst
Sergei X.
shomenkow at gmail.com
Tue Nov 10 03:27:50 PST 2026
> + 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.
More information about the Linux-rockchip
mailing list