Hi Hans, Thank you for the patch. On Tue, Jul 23, 2019 at 1:59 PM Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > > struct vpfe_pixel_format was defined, but never used. Remove it. > > Signed-off-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx> > --- > diff --git a/drivers/media/platform/am437x/am437x-vpfe.h b/drivers/media/platform/am437x/am437x-vpfe.h > index 17d7aa426788..4678285f34c6 100644 > --- a/drivers/media/platform/am437x/am437x-vpfe.h > +++ b/drivers/media/platform/am437x/am437x-vpfe.h Acked-by: Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx> Regards, --Prabhakar Lad > @@ -65,12 +65,6 @@ struct vpfe_hw_if_param { > #define VPFE_MAX_SUBDEV 1 > #define VPFE_MAX_INPUTS 1 > > -struct vpfe_pixel_format { > - struct v4l2_fmtdesc fmtdesc; > - /* bytes per pixel */ > - int bpp; > -}; > - > struct vpfe_std_info { > int active_pixels; > int active_lines;