All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH 1/2] usb: host: Enable compile testing for some of drivers
Date: Tue, 31 Dec 2019 06:43:33 +0800	[thread overview]
Message-ID: <201912310603.RpRBLrM4%lkp@intel.com> (raw)
In-Reply-To: <20191230172215.17370-1-krzk@kernel.org>

[-- Attachment #1: Type: text/plain, Size: 6101 bytes --]

Hi Krzysztof,

I love your patch! Yet something to improve:

[auto build test ERROR on usb/usb-testing]
[also build test ERROR on char-misc/char-misc-testing v5.5-rc4 next-20191220]
[cannot apply to balbi-usb/next]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]

url:    https://github.com/0day-ci/linux/commits/Krzysztof-Kozlowski/usb-host-Enable-compile-testing-for-some-of-drivers/20191231-021714
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
config: c6x-allyesconfig (attached as .config)
compiler: c6x-elf-gcc (GCC) 7.5.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=7.5.0 make.cross ARCH=c6x 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   In file included from drivers/usb/chipidea/host.c:18:0:
   drivers/usb/chipidea/../host/ehci.h: In function 'ehci_readl':
>> drivers/usb/chipidea/../host/ehci.h:743:3: error: implicit declaration of function 'readl_be'; did you mean 'readsb'? [-Werror=implicit-function-declaration]
      readl_be(regs) :
      ^~~~~~~~
      readsb
   drivers/usb/chipidea/../host/ehci.h: In function 'ehci_writel':
>> drivers/usb/chipidea/../host/ehci.h:767:3: error: implicit declaration of function 'writel_be'; did you mean 'writesb'? [-Werror=implicit-function-declaration]
      writel_be(val, regs) :
      ^~~~~~~~~
      writesb
   cc1: some warnings being treated as errors
--
   In file included from drivers/usb/host/ehci-hcd.c:96:0:
   drivers/usb/host/ehci.h: In function 'ehci_readl':
>> drivers/usb/host/ehci.h:743:3: error: implicit declaration of function 'readl_be'; did you mean 'readsb'? [-Werror=implicit-function-declaration]
      readl_be(regs) :
      ^~~~~~~~
      readsb
   drivers/usb/host/ehci.h: In function 'ehci_writel':
>> drivers/usb/host/ehci.h:767:3: error: implicit declaration of function 'writel_be'; did you mean 'writesb'? [-Werror=implicit-function-declaration]
      writel_be(val, regs) :
      ^~~~~~~~~
      writesb
   cc1: some warnings being treated as errors

vim +743 drivers/usb/chipidea/../host/ehci.h

91bc4d31e81b15 Vladimir Barinov       2007-12-30  737  
083522d76662cd Benjamin Herrenschmidt 2006-12-15  738  static inline unsigned int ehci_readl(const struct ehci_hcd *ehci,
083522d76662cd Benjamin Herrenschmidt 2006-12-15  739  		__u32 __iomem *regs)
083522d76662cd Benjamin Herrenschmidt 2006-12-15  740  {
d728e327d4f86d Benjamin Herrenschmidt 2006-12-28  741  #ifdef CONFIG_USB_EHCI_BIG_ENDIAN_MMIO
083522d76662cd Benjamin Herrenschmidt 2006-12-15  742  	return ehci_big_endian_mmio(ehci) ?
68f50e52554a0a Al Viro                2007-02-09 @743  		readl_be(regs) :
68f50e52554a0a Al Viro                2007-02-09  744  		readl(regs);
d728e327d4f86d Benjamin Herrenschmidt 2006-12-28  745  #else
68f50e52554a0a Al Viro                2007-02-09  746  	return readl(regs);
d728e327d4f86d Benjamin Herrenschmidt 2006-12-28  747  #endif
083522d76662cd Benjamin Herrenschmidt 2006-12-15  748  }
083522d76662cd Benjamin Herrenschmidt 2006-12-15  749  
feffe09f510c47 Peter Chen             2014-01-10  750  #ifdef CONFIG_SOC_IMX28
feffe09f510c47 Peter Chen             2014-01-10  751  static inline void imx28_ehci_writel(const unsigned int val,
feffe09f510c47 Peter Chen             2014-01-10  752  		volatile __u32 __iomem *addr)
feffe09f510c47 Peter Chen             2014-01-10  753  {
feffe09f510c47 Peter Chen             2014-01-10  754  	__asm__ ("swp %0, %0, [%1]" : : "r"(val), "r"(addr));
feffe09f510c47 Peter Chen             2014-01-10  755  }
feffe09f510c47 Peter Chen             2014-01-10  756  #else
feffe09f510c47 Peter Chen             2014-01-10  757  static inline void imx28_ehci_writel(const unsigned int val,
feffe09f510c47 Peter Chen             2014-01-10  758  		volatile __u32 __iomem *addr)
feffe09f510c47 Peter Chen             2014-01-10  759  {
feffe09f510c47 Peter Chen             2014-01-10  760  }
feffe09f510c47 Peter Chen             2014-01-10  761  #endif
083522d76662cd Benjamin Herrenschmidt 2006-12-15  762  static inline void ehci_writel(const struct ehci_hcd *ehci,
083522d76662cd Benjamin Herrenschmidt 2006-12-15  763  		const unsigned int val, __u32 __iomem *regs)
083522d76662cd Benjamin Herrenschmidt 2006-12-15  764  {
d728e327d4f86d Benjamin Herrenschmidt 2006-12-28  765  #ifdef CONFIG_USB_EHCI_BIG_ENDIAN_MMIO
083522d76662cd Benjamin Herrenschmidt 2006-12-15  766  	ehci_big_endian_mmio(ehci) ?
68f50e52554a0a Al Viro                2007-02-09 @767  		writel_be(val, regs) :
68f50e52554a0a Al Viro                2007-02-09  768  		writel(val, regs);
d728e327d4f86d Benjamin Herrenschmidt 2006-12-28  769  #else
feffe09f510c47 Peter Chen             2014-01-10  770  	if (ehci->imx28_write_fix)
feffe09f510c47 Peter Chen             2014-01-10  771  		imx28_ehci_writel(val, regs);
feffe09f510c47 Peter Chen             2014-01-10  772  	else
68f50e52554a0a Al Viro                2007-02-09  773  		writel(val, regs);
d728e327d4f86d Benjamin Herrenschmidt 2006-12-28  774  #endif
083522d76662cd Benjamin Herrenschmidt 2006-12-15  775  }
8cd42e97bf451b Kumar Gala             2006-01-20  776  

:::::: The code at line 743 was first introduced by commit
:::::: 68f50e52554a0a55dfe2e3fdf659ee0569d73c3f [PATCH] hci_{read,write}l() does force casts to wrong type for no reason

:::::: TO: Al Viro <viro@ftp.linux.org.uk>
:::::: CC: Linus Torvalds <torvalds@woody.linux-foundation.org>

---
0-DAY kernel test infrastructure                 Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org Intel Corporation

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 50772 bytes --]

      parent reply	other threads:[~2019-12-30 22:43 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-30 17:22 [PATCH 1/2] usb: host: Enable compile testing for some of drivers Krzysztof Kozlowski
2019-12-30 17:22 ` [PATCH 2/2] usb: phy: " Krzysztof Kozlowski
2019-12-30 19:29 ` [PATCH 1/2] usb: host: " Greg Kroah-Hartman
2019-12-30 19:36   ` Greg Kroah-Hartman
2019-12-30 19:42     ` Krzysztof Kozlowski
2019-12-30 20:06       ` Greg Kroah-Hartman
2019-12-30 22:43 ` kbuild test robot [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201912310603.RpRBLrM4%lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.