From: kernel test robot <lkp@intel.com>
To: Emanuele Ghidoli <emanuele.ghidoli@toradex.com>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
Andi Shyti <andi.shyti@kernel.org>,
Francesco Dolcini <francesco.dolcini@toradex.com>,
Carlos Song <carlos.song@nxp.com>
Subject: [andi-shyti:i2c/i2c-host-next 51/54] drivers/i2c/busses/i2c-imx-lpi2c.c:207:2: warning: expression result unused
Date: Tue, 13 May 2025 07:40:55 +0800 [thread overview]
Message-ID: <202505130735.zh3WuTNu-lkp@intel.com> (raw)
tree: https://git.kernel.org/pub/scm/linux/kernel/git/andi.shyti/linux.git i2c/i2c-host-next
head: 6fa92cdd693b8ea056fba482afee567a7c81fc97
commit: c6ab35c95a2ff3dbc87d08115657f9ed943bbbda [51/54] i2c: lpi2c: implement master_xfer_atomic callback
config: arm-randconfig-004-20250513 (https://download.01.org/0day-ci/archive/20250513/202505130735.zh3WuTNu-lkp@intel.com/config)
compiler: clang version 16.0.6 (https://github.com/llvm/llvm-project 7cbf1a2591520c2491aa35339f227775f4d3adf6)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250513/202505130735.zh3WuTNu-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/202505130735.zh3WuTNu-lkp@intel.com/
All warnings (new ones prefixed by >>):
>> drivers/i2c/busses/i2c-imx-lpi2c.c:207:2: warning: expression result unused [-Wunused-value]
lpi2c_imx_read_msr_poll_timeout(atomic, temp, temp & (MSR_ALF | MSR_BBF | MSR_MBF), 1);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/i2c/busses/i2c-imx-lpi2c.c:194:5: note: expanded from macro 'lpi2c_imx_read_msr_poll_timeout'
readl_poll_timeout(lpi2c_imx->base + LPI2C_MSR, val, cond, \
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/iopoll.h:170:2: note: expanded from macro 'readl_poll_timeout'
readx_poll_timeout(readl, addr, val, cond, delay_us, timeout_us)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/iopoll.h:135:2: note: expanded from macro 'readx_poll_timeout'
read_poll_timeout(op, val, cond, sleep_us, timeout_us, false, addr)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/iopoll.h:58:15: note: expanded from macro 'read_poll_timeout'
(cond) ? 0 : -ETIMEDOUT; \
^~~~~~~~~~
drivers/i2c/busses/i2c-imx-lpi2c.c:267:2: warning: expression result unused [-Wunused-value]
lpi2c_imx_read_msr_poll_timeout(atomic, temp, temp & MSR_SDF, 1);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/i2c/busses/i2c-imx-lpi2c.c:194:5: note: expanded from macro 'lpi2c_imx_read_msr_poll_timeout'
readl_poll_timeout(lpi2c_imx->base + LPI2C_MSR, val, cond, \
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/iopoll.h:170:2: note: expanded from macro 'readl_poll_timeout'
readx_poll_timeout(readl, addr, val, cond, delay_us, timeout_us)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/iopoll.h:135:2: note: expanded from macro 'readx_poll_timeout'
read_poll_timeout(op, val, cond, sleep_us, timeout_us, false, addr)
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/iopoll.h:58:15: note: expanded from macro 'read_poll_timeout'
(cond) ? 0 : -ETIMEDOUT; \
^~~~~~~~~~
2 warnings generated.
vim +207 drivers/i2c/busses/i2c-imx-lpi2c.c
202
203 static int lpi2c_imx_bus_busy(struct lpi2c_imx_struct *lpi2c_imx, bool atomic)
204 {
205 unsigned int temp;
206
> 207 lpi2c_imx_read_msr_poll_timeout(atomic, temp, temp & (MSR_ALF | MSR_BBF | MSR_MBF), 1);
208
209 /* check for arbitration lost, clear if set */
210 if (temp & MSR_ALF) {
211 writel(temp, lpi2c_imx->base + LPI2C_MSR);
212 return -EAGAIN;
213 }
214
215 /* check for bus not busy */
216 if (!(temp & (MSR_BBF | MSR_MBF))) {
217 dev_dbg(&lpi2c_imx->adapter.dev, "bus not work\n");
218 if (lpi2c_imx->adapter.bus_recovery_info)
219 i2c_recover_bus(&lpi2c_imx->adapter);
220 return -ETIMEDOUT;
221 }
222
223 return 0;
224 }
225
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
reply other threads:[~2025-05-12 23:41 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=202505130735.zh3WuTNu-lkp@intel.com \
--to=lkp@intel.com \
--cc=andi.shyti@kernel.org \
--cc=carlos.song@nxp.com \
--cc=emanuele.ghidoli@toradex.com \
--cc=francesco.dolcini@toradex.com \
--cc=llvm@lists.linux.dev \
--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