On 07/09/16 23:25, Laurent Pinchart wrote: > The structure is not used, remove it. Ahh yes, looks like a left over from my first attempt at serialising input fields. Reviewed-by: Kieran Bingham <kieran@xxxxxxxxxxx> > Signed-off-by: Laurent Pinchart <laurent.pinchart+renesas@xxxxxxxxxxxxxxxx> > --- > drivers/media/platform/rcar_fdp1.c | 13 ------------- > 1 file changed, 13 deletions(-) > > diff --git a/drivers/media/platform/rcar_fdp1.c b/drivers/media/platform/rcar_fdp1.c > index bbeacf1527b5..fdab41165f5a 100644 > --- a/drivers/media/platform/rcar_fdp1.c > +++ b/drivers/media/platform/rcar_fdp1.c > @@ -514,19 +514,6 @@ enum fdp1_deint_mode { > mode == FDP1_PREVFIELD) > > /* > - * fdp1_v4l2_buffer: Track v4l2_buffers with a reference count > - * > - * As buffers come in, they may be used for more than one field. > - * It then becomes necessary to track the usage of these buffers, > - * and only release when the last job has completed using this > - * vb buffer. > - */ > -struct fdp1_v4l2_buffer { > - struct vb2_v4l2_buffer vb; > - struct list_head list; > -}; > - > -/* > * FDP1 operates on potentially 3 fields, which are tracked > * from the VB buffers using this context structure. > * Will always be a field or a full frame, never two fields. > -- Regards Kieran Bingham