Hi Srirangan, kernel test robot noticed the following build warnings: [auto build test WARNING on next-20250220] [cannot apply to pci/next pci/for-linus linus/master v6.14-rc3 v6.14-rc2 v6.14-rc1 v6.14-rc3] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in https://git-scm.com/docs/git-format-patch#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Srirangan-Madhavan/cxl-de-duplicate-cxl-DVSEC-reg-defines/20250221-124043 base: next-20250220 patch link: https://lore.kernel.org/r/20250221043906.1593189-3-smadhavan%40nvidia.com patch subject: [PATCH v2 2/2] cxl: add support for cxl reset config: arm64-randconfig-003-20250222 (https://download.01.org/0day-ci/archive/20250222/202502221438.j0UgOryU-lkp@xxxxxxxxx/config) compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff) reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250222/202502221438.j0UgOryU-lkp@xxxxxxxxx/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@xxxxxxxxx> | Closes: https://lore.kernel.org/oe-kbuild-all/202502221438.j0UgOryU-lkp@xxxxxxxxx/ All warnings (new ones prefixed by >>): >> drivers/pci/pci.c:5258:10: warning: & has lower precedence than ==; == will be evaluated first [-Wparentheses] 5258 | if (reg & CXL_DVSEC_CXL_RST_CAPABLE == 0) | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/pci/pci.c:5258:10: note: place parentheses around the '==' expression to silence this warning 5258 | if (reg & CXL_DVSEC_CXL_RST_CAPABLE == 0) | ^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ drivers/pci/pci.c:5258:10: note: place parentheses around the & expression to evaluate it first 5258 | if (reg & CXL_DVSEC_CXL_RST_CAPABLE == 0) | ~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. vim +5258 drivers/pci/pci.c 5235 5236 /** 5237 * cxl_reset - initiate a cxl reset 5238 * @dev: device to reset 5239 * @probe: if true, return 0 if device can be reset this way 5240 * 5241 * Initiate a cxl reset on @dev. 5242 */ 5243 static int cxl_reset(struct pci_dev *dev, bool probe) 5244 { 5245 u16 dvsec, reg; 5246 int rc; 5247 5248 dvsec = pci_find_dvsec_capability(dev, PCI_VENDOR_ID_CXL, 5249 CXL_DVSEC_PCIE_DEVICE); 5250 if (!dvsec) 5251 return -ENOTTY; 5252 5253 /* Check if CXL Reset is supported. */ 5254 rc = pci_read_config_word(dev, dvsec + CXL_DVSEC_CAP_OFFSET, ®); 5255 if (rc) 5256 return -ENOTTY; 5257 > 5258 if (reg & CXL_DVSEC_CXL_RST_CAPABLE == 0) 5259 return -ENOTTY; 5260 5261 if (probe) 5262 return 0; 5263 5264 rc = cxl_reset_prepare(dev, dvsec); 5265 if (rc) 5266 return rc; 5267 5268 return cxl_reset_init(dev, dvsec); 5269 } 5270 -- 0-DAY CI Kernel Test Service https://github.com/intel/lkp-tests/wiki
