[andi-shyti:i2c/i2c-host-next 51/54] drivers/i2c/busses/i2c-imx-lpi2c.c:207:2: warning: expression result unused

[andi-shyti:i2c/i2c-host-next 51/54] drivers/i2c/busses/i2c-imx-lpi2c.c:207:2: warning: expression result unused

[kernel test robot]

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