Re: [PATCH 1/1] usb: xhci: clear EINT bit in status correctly

[kbuild test robot]

Hi Lu,

[auto build test WARNING on usb/usb-testing]
[also build test WARNING on v4.10-rc1 next-20161224]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Lu-Baolu/usb-xhci-clear-EINT-bit-in-status-correctly/20161230-133444
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
config: i386-randconfig-x003-201652 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/usb/host/xhci.c: In function 'xhci_stop':
>> drivers/usb/host/xhci.c:724:14: warning: suggest parentheses around arithmetic in operand of '|' [-Wparentheses]
     writel(temp & ~0x1fff | STS_EINT, &xhci->op_regs->status);
            ~~~~~^~~~~~~~~
   drivers/usb/host/xhci.c: In function 'xhci_resume':
   drivers/usb/host/xhci.c:1057:15: warning: suggest parentheses around arithmetic in operand of '|' [-Wparentheses]
      writel(temp & ~0x1fff | STS_EINT, &xhci->op_regs->status);
             ~~~~~^~~~~~~~~

vim +724 drivers/usb/host/xhci.c

   708	
   709		/* Deleting Compliance Mode Recovery Timer */
   710		if ((xhci->quirks & XHCI_COMP_MODE_QUIRK) &&
   711				(!(xhci_all_ports_seen_u0(xhci)))) {
   712			del_timer_sync(&xhci->comp_mode_recovery_timer);
   713			xhci_dbg_trace(xhci, trace_xhci_dbg_quirks,
   714					"%s: compliance mode recovery timer deleted",
   715					__func__);
   716		}
   717	
   718		if (xhci->quirks & XHCI_AMD_PLL_FIX)
   719			usb_amd_dev_put();
   720	
   721		xhci_dbg_trace(xhci, trace_xhci_dbg_init,
   722				"// Disabling event ring interrupts");
   723		temp = readl(&xhci->op_regs->status);
 > 724		writel(temp & ~0x1fff | STS_EINT, &xhci->op_regs->status);
   725		temp = readl(&xhci->ir_set->irq_pending);
   726		writel(ER_IRQ_DISABLE(temp), &xhci->ir_set->irq_pending);
   727		xhci_print_ir_set(xhci, 0);
   728	
   729		xhci_dbg_trace(xhci, trace_xhci_dbg_init, "cleaning up memory");
   730		xhci_mem_cleanup(xhci);
   731		xhci_dbg_trace(xhci, trace_xhci_dbg_init,
   732				"xhci_stop completed - status = %x",

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip