[kbuild-all] drivers/usb/cdns3/drd.c:332:17: sparse: sparse: too many warnings

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: b9bbe6ed63b2b9f2c9ee5cbd0f2c946a2723f4ce commit: 70d8b9e5e63d212019ba3f6823c8ec3d2df87645 usb: cdns3: make signed 1 bit bitfields unsigned date: 8 weeks ago config: arc-randconfig-s001-20200518 (attached as .config) reproduce: # apt-get install sparse # sparse version: v0.6.1-193-gb8fad4bc-dirty git checkout 70d8b9e5e63d212019ba3f6823c8ec3d2df87645 # save the attached .config to linux build tree make C=1 ARCH=arc CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' If you fix the issue, kindly add following tag as appropriate Reported-by: kbuild test robot <lkp(a)intel.com&gt; sparse warnings: (new ones prefixed by >>) drivers/usb/cdns3/drd.c:182:23: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:182:23: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:182:23: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:182:23: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:195:17: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:195:17: sparse: expected void volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:195:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:199:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:199:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:199:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:219:9: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:219:9: sparse: expected void volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:219:9: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:284:15: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:284:15: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:284:15: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:284:15: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:284:15: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:284:15: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:284:15: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:284:15: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:284:15: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:284:15: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:284:15: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:284:15: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:284:15: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:284:15: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:284:15: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:284:15: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:284:15: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:284:15: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:303:9: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:303:9: sparse: expected void volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:303:9: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:326:27: sparse: sparse: incorrect type in assignment (different address spaces) @@ expected struct cdns3_otg_legacy_regs *otg_v0_regs @@ got void struct cdns3_otg_legacy_regs *otg_v0_regs @@ drivers/usb/cdns3/drd.c:326:27: sparse: expected struct cdns3_otg_legacy_regs *otg_v0_regs drivers/usb/cdns3/drd.c:326:27: sparse: got void [noderef] <asn:2> *[assigned] regs drivers/usb/cdns3/drd.c:327:14: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:327:14: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:327:14: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:327:14: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:327:14: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:327:14: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:327:14: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:327:14: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:327:14: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:327:14: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:327:14: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:327:14: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:327:14: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:327:14: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:327:14: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:327:14: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:327:14: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:327:14: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:330:32: sparse: sparse: incorrect type in assignment (different address spaces) @@ expected struct cdns3_otg_common_regs *otg_regs @@ got void struct cdns3_otg_common_regs *otg_regs @@ drivers/usb/cdns3/drd.c:330:32: sparse: expected struct cdns3_otg_common_regs *otg_regs drivers/usb/cdns3/drd.c:330:32: sparse: got void [noderef] <asn:2> *[assigned] regs drivers/usb/cdns3/drd.c:331:17: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:331:17: sparse: expected void volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:331:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:332:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:332:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:332:17: sparse: got restricted __le32 * drivers/usb/cdns3/drd.c:332:17: sparse: sparse: incorrect type in argument 1 (different address spaces) @@ expected void const volatile [noderef] <asn:2> *addr @@ got [noderef] <asn:2> *addr @@ drivers/usb/cdns3/drd.c:332:17: sparse: expected void const volatile [noderef] <asn:2> *addr drivers/usb/cdns3/drd.c:332:17: sparse: got restricted __le32 * vim +332 drivers/usb/cdns3/drd.c 7733f6c32e36ff Pawel Laszczak 2019-08-26 306 7733f6c32e36ff Pawel Laszczak 2019-08-26 307 int cdns3_drd_init(struct cdns3 *cdns) 7733f6c32e36ff Pawel Laszczak 2019-08-26 308 { 7733f6c32e36ff Pawel Laszczak 2019-08-26 309 void __iomem *regs; 7733f6c32e36ff Pawel Laszczak 2019-08-26 310 int ret = 0; 7733f6c32e36ff Pawel Laszczak 2019-08-26 311 u32 state; 7733f6c32e36ff Pawel Laszczak 2019-08-26 312 7733f6c32e36ff Pawel Laszczak 2019-08-26 313 regs = devm_ioremap_resource(cdns->dev, &cdns->otg_res); 7733f6c32e36ff Pawel Laszczak 2019-08-26 314 if (IS_ERR(regs)) 7733f6c32e36ff Pawel Laszczak 2019-08-26 315 return PTR_ERR(regs); 7733f6c32e36ff Pawel Laszczak 2019-08-26 316 7733f6c32e36ff Pawel Laszczak 2019-08-26 317 /* Detection of DRD version. Controller has been released 7733f6c32e36ff Pawel Laszczak 2019-08-26 318 * in two versions. Both are similar, but they have same changes 7733f6c32e36ff Pawel Laszczak 2019-08-26 319 * in register maps. 7733f6c32e36ff Pawel Laszczak 2019-08-26 320 * The first register in old version is command register and it's read 7733f6c32e36ff Pawel Laszczak 2019-08-26 321 * only, so driver should read 0 from it. On the other hand, in v1 7733f6c32e36ff Pawel Laszczak 2019-08-26 322 * the first register contains device ID number which is not set to 0. 7733f6c32e36ff Pawel Laszczak 2019-08-26 323 * Driver uses this fact to detect the proper version of 7733f6c32e36ff Pawel Laszczak 2019-08-26 324 * controller. 7733f6c32e36ff Pawel Laszczak 2019-08-26 325 */ 7733f6c32e36ff Pawel Laszczak 2019-08-26 326 cdns->otg_v0_regs = regs; 7733f6c32e36ff Pawel Laszczak 2019-08-26 327 if (!readl(&cdns->otg_v0_regs->cmd)) { 7733f6c32e36ff Pawel Laszczak 2019-08-26 328 cdns->version = CDNS3_CONTROLLER_V0; 7733f6c32e36ff Pawel Laszczak 2019-08-26 329 cdns->otg_v1_regs = NULL; 7733f6c32e36ff Pawel Laszczak 2019-08-26 330 cdns->otg_regs = regs; 7733f6c32e36ff Pawel Laszczak 2019-08-26 331 writel(1, &cdns->otg_v0_regs->simulate); 7733f6c32e36ff Pawel Laszczak 2019-08-26 @332 dev_info(cdns->dev, "DRD version v0 (%08x)\n", 7733f6c32e36ff Pawel Laszczak 2019-08-26 333 readl(&cdns->otg_v0_regs->version)); 7733f6c32e36ff Pawel Laszczak 2019-08-26 334 } else { 7733f6c32e36ff Pawel Laszczak 2019-08-26 335 cdns->otg_v0_regs = NULL; 7733f6c32e36ff Pawel Laszczak 2019-08-26 336 cdns->otg_v1_regs = regs; 7733f6c32e36ff Pawel Laszczak 2019-08-26 337 cdns->otg_regs = (void *)&cdns->otg_v1_regs->cmd; 7733f6c32e36ff Pawel Laszczak 2019-08-26 338 cdns->version = CDNS3_CONTROLLER_V1; 7733f6c32e36ff Pawel Laszczak 2019-08-26 339 writel(1, &cdns->otg_v1_regs->simulate); 7733f6c32e36ff Pawel Laszczak 2019-08-26 340 dev_info(cdns->dev, "DRD version v1 (ID: %08x, rev: %08x)\n", 7733f6c32e36ff Pawel Laszczak 2019-08-26 341 readl(&cdns->otg_v1_regs->did), 7733f6c32e36ff Pawel Laszczak 2019-08-26 342 readl(&cdns->otg_v1_regs->rid)); 7733f6c32e36ff Pawel Laszczak 2019-08-26 343 } 7733f6c32e36ff Pawel Laszczak 2019-08-26 344 7733f6c32e36ff Pawel Laszczak 2019-08-26 345 state = OTGSTS_STRAP(readl(&cdns->otg_regs->sts)); 7733f6c32e36ff Pawel Laszczak 2019-08-26 346 7733f6c32e36ff Pawel Laszczak 2019-08-26 347 /* Update dr_mode according to STRAP configuration. */ 7733f6c32e36ff Pawel Laszczak 2019-08-26 348 cdns->dr_mode = USB_DR_MODE_OTG; 7733f6c32e36ff Pawel Laszczak 2019-08-26 349 if (state == OTGSTS_STRAP_HOST) { 7733f6c32e36ff Pawel Laszczak 2019-08-26 350 dev_dbg(cdns->dev, "Controller strapped to HOST\n"); 7733f6c32e36ff Pawel Laszczak 2019-08-26 351 cdns->dr_mode = USB_DR_MODE_HOST; 7733f6c32e36ff Pawel Laszczak 2019-08-26 352 } else if (state == OTGSTS_STRAP_GADGET) { 7733f6c32e36ff Pawel Laszczak 2019-08-26 353 dev_dbg(cdns->dev, "Controller strapped to PERIPHERAL\n"); 7733f6c32e36ff Pawel Laszczak 2019-08-26 354 cdns->dr_mode = USB_DR_MODE_PERIPHERAL; 7733f6c32e36ff Pawel Laszczak 2019-08-26 355 } 7733f6c32e36ff Pawel Laszczak 2019-08-26 356 7733f6c32e36ff Pawel Laszczak 2019-08-26 357 ret = devm_request_threaded_irq(cdns->dev, cdns->otg_irq, 7733f6c32e36ff Pawel Laszczak 2019-08-26 358 cdns3_drd_irq, 7733f6c32e36ff Pawel Laszczak 2019-08-26 359 cdns3_drd_thread_irq, 7733f6c32e36ff Pawel Laszczak 2019-08-26 360 IRQF_SHARED, 7733f6c32e36ff Pawel Laszczak 2019-08-26 361 dev_name(cdns->dev), cdns); 7733f6c32e36ff Pawel Laszczak 2019-08-26 362 7733f6c32e36ff Pawel Laszczak 2019-08-26 363 if (ret) { 7733f6c32e36ff Pawel Laszczak 2019-08-26 364 dev_err(cdns->dev, "couldn't get otg_irq\n"); 7733f6c32e36ff Pawel Laszczak 2019-08-26 365 return ret; 7733f6c32e36ff Pawel Laszczak 2019-08-26 366 } 7733f6c32e36ff Pawel Laszczak 2019-08-26 367 7733f6c32e36ff Pawel Laszczak 2019-08-26 368 state = readl(&cdns->otg_regs->sts); 7733f6c32e36ff Pawel Laszczak 2019-08-26 369 if (OTGSTS_OTG_NRDY(state) != 0) { 7733f6c32e36ff Pawel Laszczak 2019-08-26 370 dev_err(cdns->dev, "Cadence USB3 OTG device not ready\n"); 7733f6c32e36ff Pawel Laszczak 2019-08-26 371 return -ENODEV; 7733f6c32e36ff Pawel Laszczak 2019-08-26 372 } 7733f6c32e36ff Pawel Laszczak 2019-08-26 373 7733f6c32e36ff Pawel Laszczak 2019-08-26 374 return ret; 7733f6c32e36ff Pawel Laszczak 2019-08-26 375 } 7733f6c32e36ff Pawel Laszczak 2019-08-26 376 :::::: The code at line 332 was first introduced by commit :::::: 7733f6c32e36ff9d7adadf40001039bf219b1cbe usb: cdns3: Add Cadence USB3 DRD Driver :::::: TO: Pawel Laszczak <pawell(a)cadence.com&gt; :::::: CC: Felipe Balbi <felipe.balbi(a)linux.intel.com&gt; --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org