Hi Ricardo, Thank you for the patch.
On Friday, 8 September 2017 18:47:10 EEST Ricardo Ribalda Delgado wrote: > max_width and max_height are swap with step_width and step_height. > > Signed-off-by: Ricardo Ribalda Delgado <[email protected]> > --- > > 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 > > drivers/media/v4l2-core/v4l2-ioctl.c | 9 ++++++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/drivers/media/v4l2-core/v4l2-ioctl.c > b/drivers/media/v4l2-core/v4l2-ioctl.c index b60a6b0841d1..79614992ee21 > 100644 > --- a/drivers/media/v4l2-core/v4l2-ioctl.c > +++ b/drivers/media/v4l2-core/v4l2-ioctl.c > @@ -730,9 +730,12 @@ 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); If you fix the indentation as well (aligning p-> right after the '(') you could keep two arguments per line :-) > break; > case V4L2_FRMSIZE_TYPE_CONTINUOUS: > /* fall through */ -- Regards, Laurent Pinchart
