From: kernel test robot <lkp@intel.com>
To: Neel Bullywon <neelb2403@gmail.com>, jic23@kernel.org
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
dlechner@baylibre.com, nuno.sa@analog.com, andy@kernel.org,
linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,
Neel Bullywon <neelb2403@gmail.com>
Subject: Re: [PATCH v6 1/3] iio: magnetometer: bmc150_magn: use automated cleanup for mutex
Date: Sat, 14 Feb 2026 07:31:03 +0800 [thread overview]
Message-ID: <202602140708.3KGC8ZJJ-lkp@intel.com> (raw)
In-Reply-To: <20260210233945.40975-2-neelb2403@gmail.com>
Hi Neel,
kernel test robot noticed the following build errors:
[auto build test ERROR on e7aa57247700733e52a8e2e4dee6a52c2a76de02]
url: https://github.com/intel-lab-lkp/linux/commits/Neel-Bullywon/iio-magnetometer-bmc150_magn-use-automated-cleanup-for-mutex/20260211-074324
base: e7aa57247700733e52a8e2e4dee6a52c2a76de02
patch link: https://lore.kernel.org/r/20260210233945.40975-2-neelb2403%40gmail.com
patch subject: [PATCH v6 1/3] iio: magnetometer: bmc150_magn: use automated cleanup for mutex
config: loongarch-allmodconfig (https://download.01.org/0day-ci/archive/20260214/202602140708.3KGC8ZJJ-lkp@intel.com/config)
compiler: clang version 19.1.7 (https://github.com/llvm/llvm-project cd708029e0b2869e80abe31ddb175f7c35361f90)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260214/202602140708.3KGC8ZJJ-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/202602140708.3KGC8ZJJ-lkp@intel.com/
All errors (new ones prefixed by >>):
>> drivers/iio/magnetometer/bmc150_magn.c:561:2: error: cannot jump from switch statement to this case label
561 | default:
| ^
drivers/iio/magnetometer/bmc150_magn.c:527:3: note: jump bypasses initialization of variable with __attribute__((cleanup))
527 | guard(mutex)(&data->mutex);
| ^
include/linux/cleanup.h:414:15: note: expanded from macro 'guard'
414 | CLASS(_name, __UNIQUE_ID(guard))
| ^
include/linux/compiler.h:168:2: note: expanded from macro '__UNIQUE_ID'
168 | __PASTE(__UNIQUE_ID_, \
| ^
include/linux/compiler_types.h:16:23: note: expanded from macro '__PASTE'
16 | #define __PASTE(a, b) ___PASTE(a, b)
| ^
include/linux/compiler_types.h:15:24: note: expanded from macro '___PASTE'
15 | #define ___PASTE(a, b) a##b
| ^
<scratch space>:122:1: note: expanded from here
122 | __UNIQUE_ID_guard_677
| ^
drivers/iio/magnetometer/bmc150_magn.c:529:2: error: cannot jump from switch statement to this case label
529 | case IIO_CHAN_INFO_OVERSAMPLING_RATIO:
| ^
drivers/iio/magnetometer/bmc150_magn.c:527:3: note: jump bypasses initialization of variable with __attribute__((cleanup))
527 | guard(mutex)(&data->mutex);
| ^
include/linux/cleanup.h:414:15: note: expanded from macro 'guard'
414 | CLASS(_name, __UNIQUE_ID(guard))
| ^
include/linux/compiler.h:168:2: note: expanded from macro '__UNIQUE_ID'
168 | __PASTE(__UNIQUE_ID_, \
| ^
include/linux/compiler_types.h:16:23: note: expanded from macro '__PASTE'
16 | #define __PASTE(a, b) ___PASTE(a, b)
| ^
include/linux/compiler_types.h:15:24: note: expanded from macro '___PASTE'
15 | #define ___PASTE(a, b) a##b
| ^
<scratch space>:122:1: note: expanded from here
122 | __UNIQUE_ID_guard_677
| ^
2 errors generated.
vim +561 drivers/iio/magnetometer/bmc150_magn.c
c91746a2361d75 Irina Tirdea 2015-04-29 515
c91746a2361d75 Irina Tirdea 2015-04-29 516 static int bmc150_magn_write_raw(struct iio_dev *indio_dev,
c91746a2361d75 Irina Tirdea 2015-04-29 517 struct iio_chan_spec const *chan,
c91746a2361d75 Irina Tirdea 2015-04-29 518 int val, int val2, long mask)
c91746a2361d75 Irina Tirdea 2015-04-29 519 {
c91746a2361d75 Irina Tirdea 2015-04-29 520 struct bmc150_magn_data *data = iio_priv(indio_dev);
c91746a2361d75 Irina Tirdea 2015-04-29 521 int ret;
c91746a2361d75 Irina Tirdea 2015-04-29 522
c91746a2361d75 Irina Tirdea 2015-04-29 523 switch (mask) {
c91746a2361d75 Irina Tirdea 2015-04-29 524 case IIO_CHAN_INFO_SAMP_FREQ:
5990dc9703679a Irina Tirdea 2015-04-29 525 if (val > data->max_odr)
5990dc9703679a Irina Tirdea 2015-04-29 526 return -EINVAL;
d65a37daae6aa8 Neel Bullywon 2026-02-10 527 guard(mutex)(&data->mutex);
d65a37daae6aa8 Neel Bullywon 2026-02-10 528 return bmc150_magn_set_odr(data, val);
5990dc9703679a Irina Tirdea 2015-04-29 529 case IIO_CHAN_INFO_OVERSAMPLING_RATIO:
5990dc9703679a Irina Tirdea 2015-04-29 530 switch (chan->channel2) {
5990dc9703679a Irina Tirdea 2015-04-29 531 case IIO_MOD_X:
5990dc9703679a Irina Tirdea 2015-04-29 532 case IIO_MOD_Y:
5990dc9703679a Irina Tirdea 2015-04-29 533 if (val < 1 || val > 511)
5990dc9703679a Irina Tirdea 2015-04-29 534 return -EINVAL;
d65a37daae6aa8 Neel Bullywon 2026-02-10 535 {
d65a37daae6aa8 Neel Bullywon 2026-02-10 536 guard(mutex)(&data->mutex);
5990dc9703679a Irina Tirdea 2015-04-29 537 ret = bmc150_magn_set_max_odr(data, val, 0, 0);
d65a37daae6aa8 Neel Bullywon 2026-02-10 538 if (ret < 0)
5990dc9703679a Irina Tirdea 2015-04-29 539 return ret;
d65a37daae6aa8 Neel Bullywon 2026-02-10 540 return regmap_update_bits(data->regmap,
5990dc9703679a Irina Tirdea 2015-04-29 541 BMC150_MAGN_REG_REP_XY,
1506f3cd0b8169 Hartmut Knaack 2015-07-17 542 BMC150_MAGN_REG_REP_DATAMASK,
d65a37daae6aa8 Neel Bullywon 2026-02-10 543 BMC150_MAGN_REPXY_TO_REGVAL(val));
d65a37daae6aa8 Neel Bullywon 2026-02-10 544 }
5990dc9703679a Irina Tirdea 2015-04-29 545 case IIO_MOD_Z:
5990dc9703679a Irina Tirdea 2015-04-29 546 if (val < 1 || val > 256)
5990dc9703679a Irina Tirdea 2015-04-29 547 return -EINVAL;
d65a37daae6aa8 Neel Bullywon 2026-02-10 548 {
d65a37daae6aa8 Neel Bullywon 2026-02-10 549 guard(mutex)(&data->mutex);
5990dc9703679a Irina Tirdea 2015-04-29 550 ret = bmc150_magn_set_max_odr(data, 0, val, 0);
d65a37daae6aa8 Neel Bullywon 2026-02-10 551 if (ret < 0)
5990dc9703679a Irina Tirdea 2015-04-29 552 return ret;
d65a37daae6aa8 Neel Bullywon 2026-02-10 553 return regmap_update_bits(data->regmap,
5990dc9703679a Irina Tirdea 2015-04-29 554 BMC150_MAGN_REG_REP_Z,
1506f3cd0b8169 Hartmut Knaack 2015-07-17 555 BMC150_MAGN_REG_REP_DATAMASK,
d65a37daae6aa8 Neel Bullywon 2026-02-10 556 BMC150_MAGN_REPZ_TO_REGVAL(val));
d65a37daae6aa8 Neel Bullywon 2026-02-10 557 }
5990dc9703679a Irina Tirdea 2015-04-29 558 default:
5990dc9703679a Irina Tirdea 2015-04-29 559 return -EINVAL;
5990dc9703679a Irina Tirdea 2015-04-29 560 }
c91746a2361d75 Irina Tirdea 2015-04-29 @561 default:
c91746a2361d75 Irina Tirdea 2015-04-29 562 return -EINVAL;
c91746a2361d75 Irina Tirdea 2015-04-29 563 }
c91746a2361d75 Irina Tirdea 2015-04-29 564 }
c91746a2361d75 Irina Tirdea 2015-04-29 565
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2026-02-13 23:31 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-02-10 23:39 [PATCH v6 0/3] iio: magnetometer: bmc150_magn: cleanup and formatting Neel Bullywon
2026-02-10 23:39 ` [PATCH v6 1/3] iio: magnetometer: bmc150_magn: use automated cleanup for mutex Neel Bullywon
2026-02-11 8:39 ` Andy Shevchenko
2026-02-13 23:31 ` kernel test robot [this message]
2026-02-14 16:34 ` Jonathan Cameron
2026-02-14 16:28 ` David Lechner
2026-02-10 23:39 ` [PATCH v6 2/3] iio: magnetometer: bmc150_magn: replace msleep with fsleep Neel Bullywon
2026-02-11 8:40 ` Andy Shevchenko
2026-02-10 23:39 ` [PATCH v6 3/3] iio: magnetometer: bmc150_magn: minor formatting cleanup Neel Bullywon
2026-02-11 8:41 ` Andy Shevchenko
2026-02-14 16:31 ` David Lechner
2026-02-14 18:00 ` Andy Shevchenko
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=202602140708.3KGC8ZJJ-lkp@intel.com \
--to=lkp@intel.com \
--cc=andy@kernel.org \
--cc=dlechner@baylibre.com \
--cc=jic23@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=neelb2403@gmail.com \
--cc=nuno.sa@analog.com \
--cc=oe-kbuild-all@lists.linux.dev \
/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