All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Baoquan He <bhe@redhat.com>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
	Andrew Morton <akpm@linux-foundation.org>,
	Linux Memory Management List <linux-mm@kvack.org>,
	Niklas Schnelle <schnelle@linux.ibm.com>,
	Christoph Hellwig <hch@lst.de>,
	"Mike Rapoport (IBM)" <rppt@kernel.org>
Subject: [akpm-mm:mm-unstable 93/173] drivers/clk/clk-fixed-mmio.c:33:2: error: call to undeclared function 'iounmap'; ISO C99 and later do not support implicit function declarations
Date: Thu, 13 Jul 2023 10:57:34 +0800	[thread overview]
Message-ID: <202307131033.JgvCSVz4-lkp@intel.com> (raw)

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm.git mm-unstable
head:   e726c62e70a70ae5536caa0aa131c5dce5c6f764
commit: 827057ff3ce7554741d431145c3b93f1a54125fc [93/173] s390: mm: convert to GENERIC_IOREMAP
config: s390-randconfig-r033-20230713 (https://download.01.org/0day-ci/archive/20230713/202307131033.JgvCSVz4-lkp@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project.git 4a5ac14ee968ff0ad5d2cc1ffa0299048db4c88a)
reproduce: (https://download.01.org/0day-ci/archive/20230713/202307131033.JgvCSVz4-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/202307131033.JgvCSVz4-lkp@intel.com/

All errors (new ones prefixed by >>):

   In file included from drivers/clk/clk-fixed-mmio.c:13:
   In file included from include/linux/io.h:13:
   In file included from arch/s390/include/asm/io.h:78:
   include/asm-generic/io.h:547:31: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     547 |         val = __raw_readb(PCI_IOBASE + addr);
         |                           ~~~~~~~~~~ ^
   include/asm-generic/io.h:560:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     560 |         val = __le16_to_cpu((__le16 __force)__raw_readw(PCI_IOBASE + addr));
         |                                                         ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:37:59: note: expanded from macro '__le16_to_cpu'
      37 | #define __le16_to_cpu(x) __swab16((__force __u16)(__le16)(x))
         |                                                           ^
   include/uapi/linux/swab.h:102:54: note: expanded from macro '__swab16'
     102 | #define __swab16(x) (__u16)__builtin_bswap16((__u16)(x))
         |                                                      ^
   In file included from drivers/clk/clk-fixed-mmio.c:13:
   In file included from include/linux/io.h:13:
   In file included from arch/s390/include/asm/io.h:78:
   include/asm-generic/io.h:573:61: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     573 |         val = __le32_to_cpu((__le32 __force)__raw_readl(PCI_IOBASE + addr));
         |                                                         ~~~~~~~~~~ ^
   include/uapi/linux/byteorder/big_endian.h:35:59: note: expanded from macro '__le32_to_cpu'
      35 | #define __le32_to_cpu(x) __swab32((__force __u32)(__le32)(x))
         |                                                           ^
   include/uapi/linux/swab.h:115:54: note: expanded from macro '__swab32'
     115 | #define __swab32(x) (__u32)__builtin_bswap32((__u32)(x))
         |                                                      ^
   In file included from drivers/clk/clk-fixed-mmio.c:13:
   In file included from include/linux/io.h:13:
   In file included from arch/s390/include/asm/io.h:78:
   include/asm-generic/io.h:584:33: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     584 |         __raw_writeb(value, PCI_IOBASE + addr);
         |                             ~~~~~~~~~~ ^
   include/asm-generic/io.h:594:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     594 |         __raw_writew((u16 __force)cpu_to_le16(value), PCI_IOBASE + addr);
         |                                                       ~~~~~~~~~~ ^
   include/asm-generic/io.h:604:59: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     604 |         __raw_writel((u32 __force)cpu_to_le32(value), PCI_IOBASE + addr);
         |                                                       ~~~~~~~~~~ ^
   include/asm-generic/io.h:692:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     692 |         readsb(PCI_IOBASE + addr, buffer, count);
         |                ~~~~~~~~~~ ^
   include/asm-generic/io.h:700:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     700 |         readsw(PCI_IOBASE + addr, buffer, count);
         |                ~~~~~~~~~~ ^
   include/asm-generic/io.h:708:20: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     708 |         readsl(PCI_IOBASE + addr, buffer, count);
         |                ~~~~~~~~~~ ^
   include/asm-generic/io.h:717:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     717 |         writesb(PCI_IOBASE + addr, buffer, count);
         |                 ~~~~~~~~~~ ^
   include/asm-generic/io.h:726:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     726 |         writesw(PCI_IOBASE + addr, buffer, count);
         |                 ~~~~~~~~~~ ^
   include/asm-generic/io.h:735:21: warning: performing pointer arithmetic on a null pointer has undefined behavior [-Wnull-pointer-arithmetic]
     735 |         writesl(PCI_IOBASE + addr, buffer, count);
         |                 ~~~~~~~~~~ ^
>> drivers/clk/clk-fixed-mmio.c:33:2: error: call to undeclared function 'iounmap'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
      33 |         iounmap(base);
         |         ^
   arch/s390/include/asm/io.h:29:17: note: expanded from macro 'iounmap'
      29 | #define iounmap iounmap
         |                 ^
   drivers/clk/clk-fixed-mmio.c:33:2: note: did you mean 'vunmap'?
   arch/s390/include/asm/io.h:29:17: note: expanded from macro 'iounmap'
      29 | #define iounmap iounmap
         |                 ^
   include/linux/vmalloc.h:167:13: note: 'vunmap' declared here
     167 | extern void vunmap(const void *addr);
         |             ^
   12 warnings and 1 error generated.


vim +/iounmap +33 drivers/clk/clk-fixed-mmio.c

50cc4caf531abb2 Jan Kotas 2018-12-13  17  
50cc4caf531abb2 Jan Kotas 2018-12-13  18  static struct clk_hw *fixed_mmio_clk_setup(struct device_node *node)
50cc4caf531abb2 Jan Kotas 2018-12-13  19  {
50cc4caf531abb2 Jan Kotas 2018-12-13  20  	struct clk_hw *clk;
50cc4caf531abb2 Jan Kotas 2018-12-13  21  	const char *clk_name = node->name;
50cc4caf531abb2 Jan Kotas 2018-12-13  22  	void __iomem *base;
50cc4caf531abb2 Jan Kotas 2018-12-13  23  	u32 freq;
50cc4caf531abb2 Jan Kotas 2018-12-13  24  	int ret;
50cc4caf531abb2 Jan Kotas 2018-12-13  25  
50cc4caf531abb2 Jan Kotas 2018-12-13  26  	base = of_iomap(node, 0);
50cc4caf531abb2 Jan Kotas 2018-12-13  27  	if (!base) {
50cc4caf531abb2 Jan Kotas 2018-12-13  28  		pr_err("%pOFn: failed to map address\n", node);
50cc4caf531abb2 Jan Kotas 2018-12-13  29  		return ERR_PTR(-EIO);
50cc4caf531abb2 Jan Kotas 2018-12-13  30  	}
50cc4caf531abb2 Jan Kotas 2018-12-13  31  
50cc4caf531abb2 Jan Kotas 2018-12-13  32  	freq = readl(base);
50cc4caf531abb2 Jan Kotas 2018-12-13 @33  	iounmap(base);
50cc4caf531abb2 Jan Kotas 2018-12-13  34  	of_property_read_string(node, "clock-output-names", &clk_name);
50cc4caf531abb2 Jan Kotas 2018-12-13  35  
50cc4caf531abb2 Jan Kotas 2018-12-13  36  	clk = clk_hw_register_fixed_rate(NULL, clk_name, NULL, 0, freq);
50cc4caf531abb2 Jan Kotas 2018-12-13  37  	if (IS_ERR(clk)) {
50cc4caf531abb2 Jan Kotas 2018-12-13  38  		pr_err("%pOFn: failed to register fixed rate clock\n", node);
50cc4caf531abb2 Jan Kotas 2018-12-13  39  		return clk;
50cc4caf531abb2 Jan Kotas 2018-12-13  40  	}
50cc4caf531abb2 Jan Kotas 2018-12-13  41  
50cc4caf531abb2 Jan Kotas 2018-12-13  42  	ret = of_clk_add_hw_provider(node, of_clk_hw_simple_get, clk);
50cc4caf531abb2 Jan Kotas 2018-12-13  43  	if (ret) {
50cc4caf531abb2 Jan Kotas 2018-12-13  44  		pr_err("%pOFn: failed to add clock provider\n", node);
50cc4caf531abb2 Jan Kotas 2018-12-13  45  		clk_hw_unregister(clk);
50cc4caf531abb2 Jan Kotas 2018-12-13  46  		clk = ERR_PTR(ret);
50cc4caf531abb2 Jan Kotas 2018-12-13  47  	}
50cc4caf531abb2 Jan Kotas 2018-12-13  48  
50cc4caf531abb2 Jan Kotas 2018-12-13  49  	return clk;
50cc4caf531abb2 Jan Kotas 2018-12-13  50  }
50cc4caf531abb2 Jan Kotas 2018-12-13  51  

:::::: The code at line 33 was first introduced by commit
:::::: 50cc4caf531abb2075c6973451fccdaba9f02715 clk: Add Fixed MMIO clock driver

:::::: TO: Jan Kotas <jank@cadence.com>
:::::: CC: Stephen Boyd <sboyd@kernel.org>

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

             reply	other threads:[~2023-07-13  2:58 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-13  2:57 kernel test robot [this message]
2023-07-13 16:26 ` [akpm-mm:mm-unstable 93/173] drivers/clk/clk-fixed-mmio.c:33:2: error: call to undeclared function 'iounmap'; ISO C99 and later do not support implicit function declarations Andrew Morton
2023-07-13 23:59   ` Baoquan He
2023-07-14 18:08     ` Andrew Morton
2023-07-17  7:05     ` Liu, Yujie
2023-07-18  0:06       ` bhe
2023-07-18  6:19         ` Liu, Yujie
2023-07-18  8:47           ` bhe

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=202307131033.JgvCSVz4-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=akpm@linux-foundation.org \
    --cc=bhe@redhat.com \
    --cc=hch@lst.de \
    --cc=linux-mm@kvack.org \
    --cc=llvm@lists.linux.dev \
    --cc=oe-kbuild-all@lists.linux.dev \
    --cc=rppt@kernel.org \
    --cc=schnelle@linux.ibm.com \
    /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.