On Mon, Oct 12, 2020 at 9:09 PM Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> wrote: > > Now that the values are unsigned, we can remove explicit cast to u32. the explicit Reviewed-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx> > > Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > --- > drivers/media/pci/intel/ipu3/ipu3-cio2.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/drivers/media/pci/intel/ipu3/ipu3-cio2.c b/drivers/media/pci/intel/ipu3/ipu3-cio2.c > index 87d040e176f7..e42177f8c426 100644 > --- a/drivers/media/pci/intel/ipu3/ipu3-cio2.c > +++ b/drivers/media/pci/intel/ipu3/ipu3-cio2.c > @@ -1282,9 +1282,8 @@ static int cio2_subdev_set_fmt(struct v4l2_subdev *sd, > } > } > > - fmt->format.width = min_t(u32, fmt->format.width, CIO2_IMAGE_MAX_WIDTH); > - fmt->format.height = min_t(u32, fmt->format.height, > - CIO2_IMAGE_MAX_LENGTH); > + fmt->format.width = min(fmt->format.width, CIO2_IMAGE_MAX_WIDTH); > + fmt->format.height = min(fmt->format.height, CIO2_IMAGE_MAX_LENGTH); > fmt->format.field = V4L2_FIELD_NONE; > > mutex_lock(&q->subdev_lock); > -- > 2.27.0 > -- With Best Regards, Andy Shevchenko
