Hi Ezequiel, On Fri, Dec 7, 2018 at 4:46 AM Ezequiel Garcia <ezequiel@xxxxxxxxxxxxx> wrote: > > Make the core set the reserved fields to zero in > vv4l2_pix_format_mplane.4l2_plane_pix_format, > for _MPLANE queue types. > > Moving this to the core avoids having to do so in each > and every driver. > > Suggested-by: Tomasz Figa <tfiga@xxxxxxxxxxxx> > Signed-off-by: Ezequiel Garcia <ezequiel@xxxxxxxxxxxxx> > Acked-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > -- > v3: > * s/int/unsigned int, suggested by Sakari > > v2: > * Drop unneeded clear in g_fmt. > The sturct was already being cleared here. > * Only zero plane_fmt reserved fields. > * Use CLEAR_FIELD_MACRO. > > drivers/media/v4l2-core/v4l2-ioctl.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > Thanks for the patch. Reviewed-by: Tomasz Figa <tfiga@xxxxxxxxxxxx> Best regards, Tomasz