From: kernel test robot <lkp@intel.com>
To: Oliver O'Halloran <oohall@gmail.com>
Cc: kbuild-all@lists.01.org, clang-built-linux@googlegroups.com,
linux-kernel@vger.kernel.org,
Michael Ellerman <mpe@ellerman.id.au>
Subject: arch/powerpc/platforms/powernv/pci-ioda.c:1889:13: warning: function 'pnv_ioda_setup_bus_dma' is not needed and will not be emitted
Date: Sun, 5 Jul 2020 19:02:17 +0800 [thread overview]
Message-ID: <202007051910.9MTbUbcd%lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 4819 bytes --]
Hi Oliver,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 35e884f89df4c48566d745dc5a97a0d058d04263
commit: dc3d8f85bb571c3640ebba24b82a527cf2cb3f24 powerpc/powernv/pci: Re-work bus PE configuration
date: 5 weeks ago
config: powerpc64-randconfig-r035-20200705 (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project f804bd586ee58199db4cfb2da8e9ef067425900b)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# install powerpc64 cross compiling tool for clang build
# apt-get install binutils-powerpc64-linux-gnu
git checkout dc3d8f85bb571c3640ebba24b82a527cf2cb3f24
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=powerpc64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
arch/powerpc/platforms/powernv/pci-ioda.c:1494:6: warning: no previous prototype for function 'pnv_pci_sriov_disable' [-Wmissing-prototypes]
void pnv_pci_sriov_disable(struct pci_dev *pdev)
^
arch/powerpc/platforms/powernv/pci-ioda.c:1494:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
void pnv_pci_sriov_disable(struct pci_dev *pdev)
^
static
arch/powerpc/platforms/powernv/pci-ioda.c:1604:5: warning: no previous prototype for function 'pnv_pci_sriov_enable' [-Wmissing-prototypes]
int pnv_pci_sriov_enable(struct pci_dev *pdev, u16 num_vfs)
^
arch/powerpc/platforms/powernv/pci-ioda.c:1604:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int pnv_pci_sriov_enable(struct pci_dev *pdev, u16 num_vfs)
^
static
arch/powerpc/platforms/powernv/pci-ioda.c:1719:5: warning: no previous prototype for function 'pnv_pcibios_sriov_disable' [-Wmissing-prototypes]
int pnv_pcibios_sriov_disable(struct pci_dev *pdev)
^
arch/powerpc/platforms/powernv/pci-ioda.c:1719:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int pnv_pcibios_sriov_disable(struct pci_dev *pdev)
^
static
arch/powerpc/platforms/powernv/pci-ioda.c:1728:5: warning: no previous prototype for function 'pnv_pcibios_sriov_enable' [-Wmissing-prototypes]
int pnv_pcibios_sriov_enable(struct pci_dev *pdev, u16 num_vfs)
^
arch/powerpc/platforms/powernv/pci-ioda.c:1728:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
int pnv_pcibios_sriov_enable(struct pci_dev *pdev, u16 num_vfs)
^
static
>> arch/powerpc/platforms/powernv/pci-ioda.c:1889:13: warning: function 'pnv_ioda_setup_bus_dma' is not needed and will not be emitted [-Wunneeded-internal-declaration]
static void pnv_ioda_setup_bus_dma(struct pnv_ioda_pe *pe, struct pci_bus *bus)
^
5 warnings generated.
vim +/pnv_ioda_setup_bus_dma +1889 arch/powerpc/platforms/powernv/pci-ioda.c
fe7e85c6f5ff63 Gavin Shan 2014-09-30 1888
5eada8a3f087df Alexey Kardashevskiy 2018-12-19 @1889 static void pnv_ioda_setup_bus_dma(struct pnv_ioda_pe *pe, struct pci_bus *bus)
74251fe21bfa93 Benjamin Herrenschmidt 2013-07-01 1890 {
74251fe21bfa93 Benjamin Herrenschmidt 2013-07-01 1891 struct pci_dev *dev;
74251fe21bfa93 Benjamin Herrenschmidt 2013-07-01 1892
74251fe21bfa93 Benjamin Herrenschmidt 2013-07-01 1893 list_for_each_entry(dev, &bus->devices, bus_list) {
b348aa65297659 Alexey Kardashevskiy 2015-06-05 1894 set_iommu_table_base(&dev->dev, pe->table_group.tables[0]);
0617fc0ca412b5 Christoph Hellwig 2019-02-13 1895 dev->dev.archdata.dma_offset = pe->tce_bypass_base;
dff4a39e880062 Gavin Shan 2014-07-15 1896
5c89a87d13d168 Alexey Kardashevskiy 2015-06-18 1897 if ((pe->flags & PNV_IODA_PE_BUS_ALL) && dev->subordinate)
5eada8a3f087df Alexey Kardashevskiy 2018-12-19 1898 pnv_ioda_setup_bus_dma(pe, dev->subordinate);
74251fe21bfa93 Benjamin Herrenschmidt 2013-07-01 1899 }
74251fe21bfa93 Benjamin Herrenschmidt 2013-07-01 1900 }
74251fe21bfa93 Benjamin Herrenschmidt 2013-07-01 1901
:::::: The code at line 1889 was first introduced by commit
:::::: 5eada8a3f087df74af1c2797770a3e2249374fe1 powerpc/iommu_api: Move IOMMU groups setup to a single place
:::::: TO: Alexey Kardashevskiy <aik@ozlabs.ru>
:::::: CC: Michael Ellerman <mpe@ellerman.id.au>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 24764 bytes --]
reply other threads:[~2020-07-05 11:12 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=202007051910.9MTbUbcd%lkp@intel.com \
--to=lkp@intel.com \
--cc=clang-built-linux@googlegroups.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mpe@ellerman.id.au \
--cc=oohall@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox