Hi Tomi, Thank you for the patch. On Mon, Apr 12, 2021 at 02:34:37PM +0300, Tomi Valkeinen wrote: > cal_ctx's index and cport fields are numbers less than 8. In the > following patches we will get a bunch of new fields, all of which are > similar small numbers, so lets change the type to u8. > > Signed-off-by: Tomi Valkeinen <tomi.valkeinen@xxxxxxxxxxxxxxxx> Reviewed-by: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > --- > drivers/media/platform/ti-vpe/cal.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/platform/ti-vpe/cal.h b/drivers/media/platform/ti-vpe/cal.h > index 09ad20faa9e1..251bb0ba7b3b 100644 > --- a/drivers/media/platform/ti-vpe/cal.h > +++ b/drivers/media/platform/ti-vpe/cal.h > @@ -217,8 +217,8 @@ struct cal_ctx { > > unsigned int sequence; > struct vb2_queue vb_vidq; > - unsigned int index; > - unsigned int cport; > + u8 index; > + u8 cport; > }; > > extern unsigned int cal_debug; -- Regards, Laurent Pinchart