From: kernel test robot <lkp@intel.com>
To: Xiaolei Wang <xiaolei.wang@windriver.com>
Cc: oe-kbuild-all@lists.linux.dev, linux-media@vger.kernel.org,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Tarang Raval <tarang.raval@siliconsignals.io>,
Dave Stevenson <dave.stevenson@raspberrypi.com>
Subject: [sailus-media-tree:cleanup 30/33] drivers/media/i2c/ov9282.c:557:2: error: a label can only be part of a statement and a declaration is not a statement
Date: Fri, 06 Mar 2026 19:47:06 +0800 [thread overview]
Message-ID: <202603061953.Dlyggyec-lkp@intel.com> (raw)
tree: git://linuxtv.org/sailus/media_tree.git cleanup
head: 862cacec7d570b9f6c8d60d12b50a210aecd933e
commit: 1f01b1dc2c4922f684a5992e7542cfb03f90223c [30/33] media: i2c: ov9282: Convert to CCI register access helpers
config: microblaze-randconfig-r062-20260306 (https://download.01.org/0day-ci/archive/20260306/202603061953.Dlyggyec-lkp@intel.com/config)
compiler: microblaze-linux-gcc (GCC) 8.5.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260306/202603061953.Dlyggyec-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202603061953.Dlyggyec-lkp@intel.com/
All errors (new ones prefixed by >>):
drivers/media/i2c/ov9282.c: In function 'ov9282_update_exp_gain':
>> drivers/media/i2c/ov9282.c:557:2: error: a label can only be part of a statement and a declaration is not a statement
int ret_hold = cci_write(ov9282->regmap, OV9282_REG_HOLD, 0, NULL);
^~~
vim +557 drivers/media/i2c/ov9282.c
523
524 /**
525 * ov9282_update_exp_gain() - Set updated exposure and gain
526 * @ov9282: pointer to ov9282 device
527 * @exposure: updated exposure value
528 * @gain: updated analog gain value
529 *
530 * Return: 0 if successful, error code otherwise.
531 */
532 static int ov9282_update_exp_gain(struct ov9282 *ov9282, u32 exposure, u32 gain)
533 {
534 int ret;
535 u32 exposure_us = ov9282_exposure_to_us(ov9282, exposure);
536
537 dev_dbg(ov9282->dev, "Set exp %u (~%u us), analog gain %u",
538 exposure, exposure_us, gain);
539
540 ret = cci_write(ov9282->regmap, OV9282_REG_HOLD, 0x01, NULL);
541 if (ret)
542 return ret;
543
544 ret = cci_write(ov9282->regmap, OV9282_REG_EXPOSURE, exposure << 4, NULL);
545 if (ret)
546 goto error_release_group_hold;
547
548 ret = cci_write(ov9282->regmap, OV9282_REG_AGAIN, gain, NULL);
549 if (ret)
550 goto error_release_group_hold;
551
552 ret = __v4l2_ctrl_modify_range(ov9282->flash_duration,
553 0, exposure_us, 1,
554 OV9282_STROBE_FRAME_SPAN_DEFAULT);
555
556 error_release_group_hold:
> 557 int ret_hold = cci_write(ov9282->regmap, OV9282_REG_HOLD, 0, NULL);
558
559 return ret ? ret : ret_hold;
560 }
561
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next reply other threads:[~2026-03-06 11:47 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-06 11:47 kernel test robot [this message]
2026-03-06 12:31 ` [sailus-media-tree:cleanup 30/33] drivers/media/i2c/ov9282.c:557:2: error: a label can only be part of a statement and a declaration is not a statement Sakari Ailus
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=202603061953.Dlyggyec-lkp@intel.com \
--to=lkp@intel.com \
--cc=dave.stevenson@raspberrypi.com \
--cc=linux-media@vger.kernel.org \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=sakari.ailus@linux.intel.com \
--cc=tarang.raval@siliconsignals.io \
--cc=xiaolei.wang@windriver.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox