From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-oi0-f47.google.com ([209.85.218.47]:34154 "EHLO mail-oi0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751962AbbFRWJv (ORCPT ); Thu, 18 Jun 2015 18:09:51 -0400 Received: by oigx81 with SMTP id x81so67827920oig.1 for ; Thu, 18 Jun 2015 15:09:51 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <20150618200053.GG7710@google.com> References: <1434653139-14461-1-git-send-email-dhdang@apm.com> <20150618200053.GG7710@google.com> From: Duc Dang Date: Thu, 18 Jun 2015 15:09:20 -0700 Message-ID: Subject: Re: [PATCH 1/1] PCI: xgene: Allow config access to Root Port even when link is down To: Bjorn Helgaas Cc: linux-pci@vger.kernel.org, linux-arm , linux-kernel@vger.kernel.org, Tanmay Inamdar , patches@apm.com Content-Type: text/plain; charset=UTF-8 Sender: linux-pci-owner@vger.kernel.org List-ID: On Thu, Jun 18, 2015 at 1:00 PM, Bjorn Helgaas wrote: > On Thu, Jun 18, 2015 at 11:45:39AM -0700, Duc Dang wrote: >> Even if a Root Port's link is down, we should still be able to access the >> Port's configuration space. >> >> Suggested-by: Bjorn Helgaas >> Signed-off-by: Duc Dang >> Signed-off-by: Tanmay Inamdar >> --- >> drivers/pci/host/pci-xgene.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/pci/host/pci-xgene.c b/drivers/pci/host/pci-xgene.c >> index 7e3cf28..2e66c87 100644 >> --- a/drivers/pci/host/pci-xgene.c >> +++ b/drivers/pci/host/pci-xgene.c >> @@ -139,7 +139,7 @@ static void __iomem *xgene_pcie_map_bus(struct pci_bus *bus, unsigned int devfn, >> { >> struct xgene_pcie_port *port = bus->sysdata; >> >> - if ((pci_is_root_bus(bus) && devfn != 0) || !port->link_up || >> + if ((pci_is_root_bus(bus) && devfn != 0) || >> xgene_pcie_hide_rc_bars(bus, offset)) >> return NULL; >> > > Applied with changelog as follows to pci/host-xgene for v4.2, thanks! It was my fault that I missed a compilation warning due to struct xgene_pcie_port *port becomes unused variable. I will send another patch that fix this compilation warning right after this email. > > commit 979595bcb8da04381ed85d3b0eb1f471cc5863e4 > Author: Duc Dang > Date: Thu Jun 18 11:45:39 2015 -0700 > > PCI: xgene: Allow config access to Root Port even when link is down > > Previously, when a Root Port's link was down, we didn't allow config access > to the Root Port, which meant that if the Root Port led to an empty slot, > "lspci" didn't even show the Root Port. > > Allow config access to Root Port even when link is down. > > [bhelgaas: changelog] > Suggested-by: Bjorn Helgaas > Signed-off-by: Duc Dang > Signed-off-by: Tanmay Inamdar > Signed-off-by: Bjorn Helgaas > > diff --git a/drivers/pci/host/pci-xgene.c b/drivers/pci/host/pci-xgene.c > index 70af714..1fc8910 100644 > --- a/drivers/pci/host/pci-xgene.c > +++ b/drivers/pci/host/pci-xgene.c > @@ -139,7 +139,7 @@ static void __iomem *xgene_pcie_map_bus(struct pci_bus *bus, unsigned int devfn, > { > struct xgene_pcie_port *port = bus->sysdata; > > - if ((pci_is_root_bus(bus) && devfn != 0) || !port->link_up || > + if ((pci_is_root_bus(bus) && devfn != 0) || > xgene_pcie_hide_rc_bars(bus, offset)) > return NULL; > - Thanks, Duc Dang.