From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Cc: linux-media@vger.kernel.org,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Hans Verkuil <hverkuil@xs4all.nl>,
Sakari Ailus <sakari.ailus@linux.intel.com>
Subject: Re: [PATCH v3 2/5] media: omap3isp: fix indentation
Date: Thu, 10 Jun 2021 14:23:54 +0300 [thread overview]
Message-ID: <YMH2SqZ9L+6HbK/D@pendragon.ideasonboard.com> (raw)
In-Reply-To: <20210610094903.343183-1-tomi.valkeinen@ideasonboard.com>
Hi Tomi,
Thank you for the patch.
On Thu, Jun 10, 2021 at 12:49:00PM +0300, Tomi Valkeinen wrote:
> Fix a few indentation warnings from checkpatch.
>
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> ---
> drivers/media/platform/omap3isp/ispccp2.c | 14 +++++++-------
> 1 file changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/media/platform/omap3isp/ispccp2.c b/drivers/media/platform/omap3isp/ispccp2.c
> index 366da6fb8b4f..acb58b6ddba1 100644
> --- a/drivers/media/platform/omap3isp/ispccp2.c
> +++ b/drivers/media/platform/omap3isp/ispccp2.c
> @@ -619,8 +619,8 @@ static const unsigned int ccp2_fmts[] = {
> */
> static struct v4l2_mbus_framefmt *
> __ccp2_get_format(struct isp_ccp2_device *ccp2,
> - struct v4l2_subdev_state *sd_state,
> - unsigned int pad, enum v4l2_subdev_format_whence which)
> + struct v4l2_subdev_state *sd_state,
> + unsigned int pad, enum v4l2_subdev_format_whence which)
> {
> if (which == V4L2_SUBDEV_FORMAT_TRY)
> return v4l2_subdev_get_try_format(&ccp2->subdev, sd_state,
> @@ -708,7 +708,7 @@ static int ccp2_enum_mbus_code(struct v4l2_subdev *sd,
> return -EINVAL;
>
> format = __ccp2_get_format(ccp2, sd_state, CCP2_PAD_SINK,
> - code->which);
> + code->which);
> code->code = format->code;
> }
>
> @@ -753,8 +753,8 @@ static int ccp2_enum_frame_size(struct v4l2_subdev *sd,
> * return -EINVAL or zero on success
> */
> static int ccp2_get_format(struct v4l2_subdev *sd,
> - struct v4l2_subdev_state *sd_state,
> - struct v4l2_subdev_format *fmt)
> + struct v4l2_subdev_state *sd_state,
> + struct v4l2_subdev_format *fmt)
> {
> struct isp_ccp2_device *ccp2 = v4l2_get_subdevdata(sd);
> struct v4l2_mbus_framefmt *format;
> @@ -775,8 +775,8 @@ static int ccp2_get_format(struct v4l2_subdev *sd,
> * returns zero
> */
> static int ccp2_set_format(struct v4l2_subdev *sd,
> - struct v4l2_subdev_state *sd_state,
> - struct v4l2_subdev_format *fmt)
> + struct v4l2_subdev_state *sd_state,
> + struct v4l2_subdev_format *fmt)
> {
> struct isp_ccp2_device *ccp2 = v4l2_get_subdevdata(sd);
> struct v4l2_mbus_framefmt *format;
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2021-06-10 11:24 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-09 10:02 [PATCH v3 0/1] media: v4l2-subdev: add subdev-wide state struct Tomi Valkeinen
[not found] ` <20210609100228.278798-2-tomi.valkeinen@ideasonboard.com>
2021-06-10 8:00 ` [PATCH v3 1/1] " Hans Verkuil
2021-06-10 8:24 ` Tomi Valkeinen
2021-06-10 11:11 ` Laurent Pinchart
2021-06-10 9:49 ` [PATCH v3 2/5] media: omap3isp: fix indentation Tomi Valkeinen
2021-06-10 9:49 ` [PATCH v3 3/5] media: fix kernel doc errors for sd_state parameter Tomi Valkeinen
2021-06-10 11:25 ` Laurent Pinchart
2021-06-10 9:49 ` [PATCH v3 4/5] media: v4l2-subdev: v4l2_subdev_free_state() to accept a NULL state Tomi Valkeinen
2021-06-10 11:26 ` Laurent Pinchart
2021-06-10 9:49 ` [PATCH v3 5/5] media: v4l2-subdev: fix and rename check_cfg() Tomi Valkeinen
2021-06-10 11:26 ` Laurent Pinchart
2021-06-10 11:23 ` Laurent Pinchart [this message]
2021-06-10 9:51 ` [PATCH v3 0/1] media: v4l2-subdev: add subdev-wide state struct Tomi Valkeinen
2021-06-10 9:54 ` Hans Verkuil
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YMH2SqZ9L+6HbK/D@pendragon.ideasonboard.com \
--to=laurent.pinchart@ideasonboard.com \
--cc=hverkuil@xs4all.nl \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=tomi.valkeinen@ideasonboard.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.