On Wed, Sep 13, 2017 at 09:35:52AM +0200, Ricardo Ribalda Delgado wrote: > max_width and max_height are swap with step_width and step_height. > > Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@xxxxxxxxx> Acked-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > --- > > Since that this bug has been here for ever. I do not know if we should > notify stable or not > > I have also cut the lines to respect the 80 char limit > > v2: Suggested by Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> > -Fix indentation > > drivers/media/v4l2-core/v4l2-ioctl.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/media/v4l2-core/v4l2-ioctl.c b/drivers/media/v4l2-core/v4l2-ioctl.c > index b60a6b0841d1..0292f327467d 100644 > --- a/drivers/media/v4l2-core/v4l2-ioctl.c > +++ b/drivers/media/v4l2-core/v4l2-ioctl.c > @@ -730,9 +730,9 @@ static void v4l_print_frmsizeenum(const void *arg, bool write_only) > break; > case V4L2_FRMSIZE_TYPE_STEPWISE: > pr_cont(", min=%ux%u, max=%ux%u, step=%ux%u\n", > - p->stepwise.min_width, p->stepwise.min_height, > - p->stepwise.step_width, p->stepwise.step_height, > - p->stepwise.max_width, p->stepwise.max_height); > + p->stepwise.min_width, p->stepwise.min_height, > + p->stepwise.max_width, p->stepwise.max_height, > + p->stepwise.step_width, p->stepwise.step_height); > break; > case V4L2_FRMSIZE_TYPE_CONTINUOUS: > /* fall through */ -- Sakari Ailus e-mail: sakari.ailus@xxxxxx