On Tue, Jan 5, 2016 at 9:58 PM, Fabio Estevam festevam@gmail.com wrote:
Hi Bin,
On Thu, Dec 31, 2015 at 1:20 PM, Fabio Estevam festevam@gmail.com wrote:
Do you mean like this for imx?
--- a/drivers/pci/pcie_imx.c +++ b/drivers/pci/pcie_imx.c @@ -381,7 +381,7 @@ static int imx_pcie_read_config(struct pci_controller *hose, ret = imx_pcie_addr_valid(d); if (ret) { *val = 0xffffffff;
return ret;
return 0;Thinking a bit more about this: shouldn't we fix imx_pcie_addr_valid() instead?
I am not sure if the above change is the correct fix. At least I am not able to prepare a convincing commit log for it :-)
Ok, looked at the kernel PCI designware driver and will send a patch series that fixes this issue by using the approach done in the kernel.