All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: drivers/media/i2c/ov8865.c:2799 ov8865_get_selection() warn: inconsistent indenting
Date: Tue, 18 Jan 2022 23:48:12 +0800	[thread overview]
Message-ID: <202201182306.qGenOYaU-lkp@intel.com> (raw)

[-- Attachment #1: Type: text/plain, Size: 1924 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   fe81ba137ebcc7f236780996a0b375732c07e85c
commit: acd25e220921de232b027c677668c93aa6ba5d15 media: i2c: Add .get_selection() support to ov8865
date:   7 weeks ago
config: x86_64-randconfig-m001-20220117 (https://download.01.org/0day-ci/archive/20220118/202201182306.qGenOYaU-lkp(a)intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

smatch warnings:
drivers/media/i2c/ov8865.c:2799 ov8865_get_selection() warn: inconsistent indenting

vim +2799 drivers/media/i2c/ov8865.c

  2787	
  2788	static int ov8865_get_selection(struct v4l2_subdev *subdev,
  2789					struct v4l2_subdev_state *state,
  2790					struct v4l2_subdev_selection *sel)
  2791	{
  2792		struct ov8865_sensor *sensor = ov8865_subdev_sensor(subdev);
  2793	
  2794		switch (sel->target) {
  2795		case V4L2_SEL_TGT_CROP:
  2796			mutex_lock(&sensor->mutex);
  2797				__ov8865_get_pad_crop(sensor, state, sel->pad,
  2798						      sel->which, &sel->r);
> 2799			mutex_unlock(&sensor->mutex);
  2800			break;
  2801		case V4L2_SEL_TGT_NATIVE_SIZE:
  2802			sel->r.top = 0;
  2803			sel->r.left = 0;
  2804			sel->r.width = OV8865_NATIVE_WIDTH;
  2805			sel->r.height = OV8865_NATIVE_HEIGHT;
  2806			break;
  2807		case V4L2_SEL_TGT_CROP_BOUNDS:
  2808		case V4L2_SEL_TGT_CROP_DEFAULT:
  2809			sel->r.top = OV8865_ACTIVE_START_TOP;
  2810			sel->r.left = OV8865_ACTIVE_START_LEFT;
  2811			sel->r.width = OV8865_ACTIVE_WIDTH;
  2812			sel->r.height = OV8865_ACTIVE_HEIGHT;
  2813			break;
  2814		default:
  2815			return -EINVAL;
  2816		}
  2817	
  2818		return 0;
  2819	}
  2820	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Daniel Scally <djrscally@gmail.com>
Cc: kbuild-all@lists.01.org, linux-kernel@vger.kernel.org,
	Mauro Carvalho Chehab <mchehab@kernel.org>,
	linux-media@vger.kernel.org,
	Sakari Ailus <sakari.ailus@linux.intel.com>
Subject: drivers/media/i2c/ov8865.c:2799 ov8865_get_selection() warn: inconsistent indenting
Date: Tue, 18 Jan 2022 23:48:12 +0800	[thread overview]
Message-ID: <202201182306.qGenOYaU-lkp@intel.com> (raw)

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   fe81ba137ebcc7f236780996a0b375732c07e85c
commit: acd25e220921de232b027c677668c93aa6ba5d15 media: i2c: Add .get_selection() support to ov8865
date:   7 weeks ago
config: x86_64-randconfig-m001-20220117 (https://download.01.org/0day-ci/archive/20220118/202201182306.qGenOYaU-lkp@intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

smatch warnings:
drivers/media/i2c/ov8865.c:2799 ov8865_get_selection() warn: inconsistent indenting

vim +2799 drivers/media/i2c/ov8865.c

  2787	
  2788	static int ov8865_get_selection(struct v4l2_subdev *subdev,
  2789					struct v4l2_subdev_state *state,
  2790					struct v4l2_subdev_selection *sel)
  2791	{
  2792		struct ov8865_sensor *sensor = ov8865_subdev_sensor(subdev);
  2793	
  2794		switch (sel->target) {
  2795		case V4L2_SEL_TGT_CROP:
  2796			mutex_lock(&sensor->mutex);
  2797				__ov8865_get_pad_crop(sensor, state, sel->pad,
  2798						      sel->which, &sel->r);
> 2799			mutex_unlock(&sensor->mutex);
  2800			break;
  2801		case V4L2_SEL_TGT_NATIVE_SIZE:
  2802			sel->r.top = 0;
  2803			sel->r.left = 0;
  2804			sel->r.width = OV8865_NATIVE_WIDTH;
  2805			sel->r.height = OV8865_NATIVE_HEIGHT;
  2806			break;
  2807		case V4L2_SEL_TGT_CROP_BOUNDS:
  2808		case V4L2_SEL_TGT_CROP_DEFAULT:
  2809			sel->r.top = OV8865_ACTIVE_START_TOP;
  2810			sel->r.left = OV8865_ACTIVE_START_LEFT;
  2811			sel->r.width = OV8865_ACTIVE_WIDTH;
  2812			sel->r.height = OV8865_ACTIVE_HEIGHT;
  2813			break;
  2814		default:
  2815			return -EINVAL;
  2816		}
  2817	
  2818		return 0;
  2819	}
  2820	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

             reply	other threads:[~2022-01-18 15:48 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-01-18 15:48 kernel test robot [this message]
2022-01-18 15:48 ` drivers/media/i2c/ov8865.c:2799 ov8865_get_selection() warn: inconsistent indenting kernel test robot

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=202201182306.qGenOYaU-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    /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.