[pci:pci/host-cleanups-wip 136/152] drivers/pci/host/pci-tegra.c:342:29: error: 'offset' undeclared

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



tree:   https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git pci/host-cleanups-wip
head:   d043f96bad5fa3ffb3896411f2ba2bc5180a7c83
commit: 60b45e73730aea49cc78d088d968098233a26394 [136/152] PCI: tegra: Swap order of pads_writel() reg/val arguments
config: arm-multi_v7_defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 60b45e73730aea49cc78d088d968098233a26394
        # save the attached .config to linux build tree
        make.cross ARCH=arm 

All error/warnings (new ones prefixed by >>):

   In file included from include/linux/io.h:25:0,
                    from include/linux/irq.h:24,
                    from drivers/pci/host/pci-tegra.c:34:
   drivers/pci/host/pci-tegra.c: In function 'pads_writel':
>> drivers/pci/host/pci-tegra.c:342:29: error: 'offset' undeclared (first use in this function)
     writel(value, pcie->pads + offset);
                                ^
   arch/arm/include/asm/io.h:297:71: note: in definition of macro 'writel_relaxed'
    #define writel_relaxed(v,c) __raw_writel((__force u32) cpu_to_le32(v),c)
                                                                          ^
>> drivers/pci/host/pci-tegra.c:342:2: note: in expansion of macro 'writel'
     writel(value, pcie->pads + offset);
     ^~~~~~
   drivers/pci/host/pci-tegra.c:342:29: note: each undeclared identifier is reported only once for each function it appears in
     writel(value, pcie->pads + offset);
                                ^
   arch/arm/include/asm/io.h:297:71: note: in definition of macro 'writel_relaxed'
    #define writel_relaxed(v,c) __raw_writel((__force u32) cpu_to_le32(v),c)
                                                                          ^
>> drivers/pci/host/pci-tegra.c:342:2: note: in expansion of macro 'writel'
     writel(value, pcie->pads + offset);
     ^~~~~~
   drivers/pci/host/pci-tegra.c: In function 'tegra_pcie_phy_disable':
   drivers/pci/host/pci-tegra.c:863:2: error: unknown type name 'XXXX'
     XXXX
     ^~~~
   drivers/pci/host/pci-tegra.c:866:2: error: conflicting types for 'value'
     value = pads_readl(pcie, soc->pads_pll_ctl);
     ^~~~~
   drivers/pci/host/pci-tegra.c:850:6: note: previous declaration of 'value' was here
     u32 value;
         ^~~~~
   drivers/pci/host/pci-tegra.c:863:2: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
     XXXX
     ^~~~

vim +/offset +342 drivers/pci/host/pci-tegra.c

d1523b52b Thierry Reding 2013-08-09  336  {
d1523b52b Thierry Reding 2013-08-09  337  	return readl(pcie->afi + offset);
d1523b52b Thierry Reding 2013-08-09  338  }
d1523b52b Thierry Reding 2013-08-09  339  
60b45e737 Bjorn Helgaas  2016-10-05  340  static void pads_writel(struct tegra_pcie *pcie, unsigned long ofset, u32 value)
d1523b52b Thierry Reding 2013-08-09  341  {
d1523b52b Thierry Reding 2013-08-09 @342  	writel(value, pcie->pads + offset);
d1523b52b Thierry Reding 2013-08-09  343  }
d1523b52b Thierry Reding 2013-08-09  344  
60b45e737 Bjorn Helgaas  2016-10-05  345  static u32 pads_readl(struct tegra_pcie *pcie, unsigned long offset)

:::::: The code at line 342 was first introduced by commit
:::::: d1523b52bff35ea709141abac87dd701559ef290 PCI: tegra: Move PCIe driver to drivers/pci/host

:::::: TO: Thierry Reding <thierry.reding@xxxxxxxxxxxxxxxxx>
:::::: CC: Stephen Warren <swarren@xxxxxxxxxx>

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

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux