Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1581103 > unrolled thread
| Started by | songxiaowei <songxiaowei@huawei.com> |
|---|---|
| First post | 2017-02-15 09:00 +0100 |
| Last post | 2017-02-15 09:50 +0100 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] PCI-DW: ARM64: check the result of hos_init songxiaowei <songxiaowei@huawei.com> - 2017-02-15 09:00 +0100
Re: [PATCH] PCI-DW: ARM64: check the result of hos_init kbuild test robot <lkp@intel.com> - 2017-02-15 09:50 +0100
| From | songxiaowei <songxiaowei@huawei.com> |
|---|---|
| Date | 2017-02-15 09:00 +0100 |
| Subject | [PATCH] PCI-DW: ARM64: check the result of hos_init |
| Message-ID | <tb2Fk-5kr-7@gated-at.bofh.it> |
Enumeration should be prevented
when Host initialization fails.
Only one device link with Host escpecially,
if host initialization fails or
establishling link-up is not successful,
there may no need for the Enumeration.
Signed-off-by: songxiaowei <songxiaowei@huawei.com>
---
drivers/pci/host/pcie-designware.c | 7 +++++--
drivers/pci/host/pcie-designware.h | 2 +-
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/drivers/pci/host/pcie-designware.c b/drivers/pci/host/pcie-designware.c
index bed19994c1e9..561a2edaf06f 100644
--- a/drivers/pci/host/pcie-designware.c
+++ b/drivers/pci/host/pcie-designware.c
@@ -637,8 +637,11 @@ int dw_pcie_host_init(struct pcie_port *pp)
}
}
- if (pp->ops->host_init)
- pp->ops->host_init(pp);
+ if (pp->ops->host_init) {
+ ret = pp->ops->host_init(pp);
+ if (ret)
+ return ret;
+ }
pp->root_bus_nr = pp->busn->start;
if (IS_ENABLED(CONFIG_PCI_MSI)) {
diff --git a/drivers/pci/host/pcie-designware.h b/drivers/pci/host/pcie-designware.h
index a567ea288ee2..eacf18f9fc79 100644
--- a/drivers/pci/host/pcie-designware.h
+++ b/drivers/pci/host/pcie-designware.h
@@ -63,7 +63,7 @@ struct pcie_host_ops {
int (*wr_other_conf)(struct pcie_port *pp, struct pci_bus *bus,
unsigned int devfn, int where, int size, u32 val);
int (*link_up)(struct pcie_port *pp);
- void (*host_init)(struct pcie_port *pp);
+ int (*host_init)(struct pcie_port *pp);
void (*msi_set_irq)(struct pcie_port *pp, int irq);
void (*msi_clear_irq)(struct pcie_port *pp, int irq);
phys_addr_t (*get_msi_addr)(struct pcie_port *pp);
--
2.11.GIT
[toc] | [next] | [standalone]
| From | kbuild test robot <lkp@intel.com> |
|---|---|
| Date | 2017-02-15 09:50 +0100 |
| Message-ID | <tb3rH-62y-1@gated-at.bofh.it> |
| In reply to | #1581103 |
[Multipart message — attachments visible in raw view] — view raw
Hi songxiaowei,
[auto build test ERROR on pci/next]
[also build test ERROR on v4.10-rc8 next-20170215]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
url: https://github.com/0day-ci/linux/commits/songxiaowei/PCI-DW-ARM64-check-the-result-of-hos_init/20170215-155753
base: https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git next
config: i386-randconfig-a0-02131010 (attached as .config)
compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed by >>):
>> drivers/pci/host/pcie-designware-plat.c:48:15: error: initialization from incompatible pointer type [-Werror=incompatible-pointer-types]
.host_init = dw_plat_pcie_host_init,
^~~~~~~~~~~~~~~~~~~~~~
drivers/pci/host/pcie-designware-plat.c:48:15: note: (near initialization for 'dw_plat_pcie_host_ops.host_init')
cc1: some warnings being treated as errors
vim +48 drivers/pci/host/pcie-designware-plat.c
5a3aa2a8 Joao Pinto 2016-03-10 32 {
5a3aa2a8 Joao Pinto 2016-03-10 33 struct pcie_port *pp = arg;
5a3aa2a8 Joao Pinto 2016-03-10 34
5a3aa2a8 Joao Pinto 2016-03-10 35 return dw_handle_msi_irq(pp);
5a3aa2a8 Joao Pinto 2016-03-10 36 }
5a3aa2a8 Joao Pinto 2016-03-10 37
5a3aa2a8 Joao Pinto 2016-03-10 38 static void dw_plat_pcie_host_init(struct pcie_port *pp)
5a3aa2a8 Joao Pinto 2016-03-10 39 {
5a3aa2a8 Joao Pinto 2016-03-10 40 dw_pcie_setup_rc(pp);
5a3aa2a8 Joao Pinto 2016-03-10 41 dw_pcie_wait_for_link(pp);
5a3aa2a8 Joao Pinto 2016-03-10 42
5a3aa2a8 Joao Pinto 2016-03-10 43 if (IS_ENABLED(CONFIG_PCI_MSI))
5a3aa2a8 Joao Pinto 2016-03-10 44 dw_pcie_msi_init(pp);
5a3aa2a8 Joao Pinto 2016-03-10 45 }
5a3aa2a8 Joao Pinto 2016-03-10 46
5a3aa2a8 Joao Pinto 2016-03-10 47 static struct pcie_host_ops dw_plat_pcie_host_ops = {
5a3aa2a8 Joao Pinto 2016-03-10 @48 .host_init = dw_plat_pcie_host_init,
5a3aa2a8 Joao Pinto 2016-03-10 49 };
5a3aa2a8 Joao Pinto 2016-03-10 50
5a3aa2a8 Joao Pinto 2016-03-10 51 static int dw_plat_add_pcie_port(struct pcie_port *pp,
5a3aa2a8 Joao Pinto 2016-03-10 52 struct platform_device *pdev)
5a3aa2a8 Joao Pinto 2016-03-10 53 {
2d6054b9 Bjorn Helgaas 2016-10-06 54 struct device *dev = pp->dev;
5a3aa2a8 Joao Pinto 2016-03-10 55 int ret;
5a3aa2a8 Joao Pinto 2016-03-10 56
:::::: The code at line 48 was first introduced by commit
:::::: 5a3aa2a8fae4ce1a3ad786d212b8fffca8ee72f5 PCI: designware: Add driver for prototyping kits based on ARC SDP
:::::: TO: Joao Pinto <Joao.Pinto@synopsys.com>
:::::: CC: Bjorn Helgaas <bhelgaas@google.com>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web