2015-07-29 15:00, Zhang, Helin: > Hi Thomas > > It was just an implicit declaration of function, when set > CONFIG_RTE_PCI_CONFIG=y, as follows. Nobody else cares about it.
Why do we have this config option? Please remove these useless ifdef's. > /home/hzhan75/r22/isg_cid-dpdk_org/lib/librte_eal/common/eal_common_pci.c: In > function ???rte_eal_pci_probe_one_driver???: > /home/hzhan75/r22/isg_cid-dpdk_org/lib/librte_eal/common/eal_common_pci.c:188:4: > error: implicit declaration of function ???pci_config_space_set??? > [-Werror=implicit-function-declaration] > pci_config_space_set(dev); > ^ > /home/hzhan75/r22/isg_cid-dpdk_org/lib/librte_eal/common/eal_common_pci.c:188:4: > error: nested extern declaration of ???pci_config_space_set??? > [-Werror=nested-externs] > cc1: all warnings being treated as errors > /home/hzhan75/r22/isg_cid-dpdk_org/lib/librte_eal/linuxapp/eal/eal_pci.c:561:1: > error: ???pci_config_space_set??? defined but not used > [-Werror=unused-function] > pci_config_space_set(struct rte_pci_device *dev) > ^ > cc1: all warnings being treated as errors So I will change the title to: eal: fix build with pci config enabled and add this into the message: Build log: lib/librte_eal/common/eal_common_pci.c:188:4: error: implicit declaration of function pci_config_space_set > > -----Original Message----- > > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > > Sent: Wednesday, July 29, 2015 1:20 AM > > To: Zhang, Helin > > Cc: dev at dpdk.org > > Subject: Re: [dpdk-dev] [PATCH] eal: fix build > > > > 2015-07-29 06:48, Helin Zhang: > > > It fixes the build error of implicit declaration of function. > > > > What is the error? > > Please show the build log and describe the case when it happens (compiler, > > version).