On 09/03/2017 02:27 AM, Mark Thompson wrote: >> + >> + /* memcpy to the v4l2_buffer planes array when needed */ >> + struct v4l2_plane planes[VIDEO_MAX_PLANES]; >> + struct v4l2_buffer buf; >> + >> + int bytesperline[4]; > Why 4? Should this be VIDEO_MAX_PLANES again? > yes