[jpirko-mlxsw:combined_queue 8/9] net/ethtool/cmis_fw_update.c:240:23: warning: converting the enum constant to a boolean

[kernel test robot <lkp@intel.com>]

tree:   https://github.com/jpirko/linux_mlxsw combined_queue
head:   2d94043682e019700926235499d603818f42460a
commit: 828c7605e322a6ae571a754298dfe361741bdcf2 [8/9] ethtool: cmis_fw_update: add a layer for supporting firmware update using CDB
config: i386-randconfig-001-20240122 (https://download.01.org/0day-ci/archive/20240122/202401220520.5K1hLmww-lkp@intel.com/config)
compiler: ClangBuiltLinux clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240122/202401220520.5K1hLmww-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/202401220520.5K1hLmww-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> net/ethtool/cmis_fw_update.c:240:23: warning: converting the enum constant to a boolean [-Wint-in-bool-context]
     240 |                         (CMIS_MODULE_READY || CMIS_MODULE_LOW_PWR));
         |                                            ^
   1 warning generated.


vim +240 net/ethtool/cmis_fw_update.c

   236	
   237	static bool module_is_ready(u8 data)
   238	{
   239		return (((data >> 1) & 7) ==
 > 240				(CMIS_MODULE_READY || CMIS_MODULE_LOW_PWR));
   241	}
   242	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki