Hans Verkuil <hverkuil@xxxxxxxxx> wrote on Tue [2019-Jul-23 14:59:47 +0200]: > 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 > @@ -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; > -}; > - Looks fine to me. Acked-by: Benoit Parrot <bparrot@xxxxxx> > struct vpfe_std_info { > int active_pixels; > int active_lines;