diff mbox

PCIe regression with DRA7xx in 4.4-rc1

Message ID EE11001F9E5DDD47B7634E2F8A612F2E162C92FB@lhreml503-mbs
State New
Headers show

Commit Message

Gabriele Paoloni Nov. 24, 2015, 12:08 p.m. UTC
Hi Kishon

> -----Original Message-----

> From: Kishon Vijay Abraham I [mailto:kishon@ti.com]

> Sent: 24 November 2015 12:01

> To: linux-pci@vger.kernel.org; linux-kernel@vger.kernel.org; linux-

> omap@vger.kernel.org; james.morse@arm.com; gabriel.fernandez@st.com;

> Minghuan.Lian@freescale.com; Wangzhou (B); Gabriele Paoloni; arnd@arndb.de;

> bhelgaas@google.com; pratyush.anand@gmail.com; Nori, Sekhar;

> jingoohan1@gmail.com; linux-arm-kernel@lists.infradead.org

> Subject: PCIe regression with DRA7xx in 4.4-rc1

> 

> Hi,

> 

> I'm seeing a regression with <d1b9cf31252e905ad563b9e863fd4470f6c3dd1c>("PCI:

> designware: Make driver arch-agnostic").

> 

> Logs using a SATA PCIe card [1]. The PCIe card enumerates fine but after that

> I

> observe "ata3.00: qc timeout (cmd 0xec), ata3.00: failed to IDENTIFY (I/O

> error, err_mask=0x4)"

> 


May this be related to the bug flagged in:

[PATCH] PCI: designware: remove wrong io_base assignment

[...]
-- 
1.7.9.5
[...]

Can you try to see if applying the patch above solves the issue?

Thanks

Gab

> Logs using a Ethenet PCIe card [2]. Again here the PCIe card enumerates fine

> but when I give ifconfig up, it fails.

> 

> If I just revert commit <d1b9cf31252e905ad563b9e863fd4470f6c3dd1c>, the PCIe

> cards starts to work fine again

> 

> Logs using a SATA PCIe card [3]. Here the KINGSTON SSD gets detected fine.

> Logs using a Ethernet PCIe card [4]. I'm able to do ping tests now.

> 

> Actually I'm not able to find any obvious problems with the patch and the irq

> number and the memory resource also looks fine. Any idea what could be the

> problem?

> 

> [1] -> http://pastebin.ubuntu.com/13491456/

> [2] -> http://pastebin.ubuntu.com/13491526/

> 

> [3] -> http://pastebin.ubuntu.com/13491658/

> [4] -> http://pastebin.ubuntu.com/13491593/

> 

> Thanks

> Kishon

Comments

Kishon Vijay Abraham I Nov. 24, 2015, 12:18 p.m. UTC | #1
Hi,

On Tuesday 24 November 2015 05:38 PM, Gabriele Paoloni wrote:
> Hi Kishon

> 

>> -----Original Message-----

>> From: Kishon Vijay Abraham I [mailto:kishon@ti.com]

>> Sent: 24 November 2015 12:01

>> To: linux-pci@vger.kernel.org; linux-kernel@vger.kernel.org; linux-

>> omap@vger.kernel.org; james.morse@arm.com; gabriel.fernandez@st.com;

>> Minghuan.Lian@freescale.com; Wangzhou (B); Gabriele Paoloni; arnd@arndb.de;

>> bhelgaas@google.com; pratyush.anand@gmail.com; Nori, Sekhar;

>> jingoohan1@gmail.com; linux-arm-kernel@lists.infradead.org

>> Subject: PCIe regression with DRA7xx in 4.4-rc1

>>

>> Hi,

>>

>> I'm seeing a regression with <d1b9cf31252e905ad563b9e863fd4470f6c3dd1c>("PCI:

>> designware: Make driver arch-agnostic").

>>

>> Logs using a SATA PCIe card [1]. The PCIe card enumerates fine but after that

>> I

>> observe "ata3.00: qc timeout (cmd 0xec), ata3.00: failed to IDENTIFY (I/O

>> error, err_mask=0x4)"

>>

> 

> May this be related to the bug flagged in:

> 

> [PATCH] PCI: designware: remove wrong io_base assignment

> 

> [...]

> diff --git a/drivers/pci/host/pcie-designware.c b/drivers/pci/host/pcie-designware.c

> index 540f077c37ea..02a7452bdf23 100644

> --- a/drivers/pci/host/pcie-designware.c

> +++ b/drivers/pci/host/pcie-designware.c

> @@ -440,7 +440,6 @@ int dw_pcie_host_init(struct pcie_port *pp)

>  					 ret, pp->io);

>  				continue;

>  			}

> -			pp->io_base = pp->io->start;

>  			break;

>  		case IORESOURCE_MEM:

>  			pp->mem = win->res;

> 



yes, this indeed solved the bug.

Thanks
Kishon
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
diff mbox

Patch

diff --git a/drivers/pci/host/pcie-designware.c b/drivers/pci/host/pcie-designware.c
index 540f077c37ea..02a7452bdf23 100644
--- a/drivers/pci/host/pcie-designware.c
+++ b/drivers/pci/host/pcie-designware.c
@@ -440,7 +440,6 @@  int dw_pcie_host_init(struct pcie_port *pp)
 					 ret, pp->io);
 				continue;
 			}
-			pp->io_base = pp->io->start;
 			break;
 		case IORESOURCE_MEM:
 			pp->mem = win->res;