On Thu, May 11, 2017 at 6:48 PM, Stefano Babic sbabic@denx.de wrote:
Hi Fabio,
On 11/05/2017 13:06, Fabio Estevam wrote:
On Thu, May 11, 2017 at 6:01 AM, Peter Senna Tschudin peter.senna@collabora.com wrote:
Thank you for working on this! Your patch worked for me, but I needed to add an "extern void imx_pcie_remove(void);" before calling the function, which probably means I'm missing something.
The extern should be added into a header file.
After applying this patch to u-boot, u-boot can initialize PCI and 4.11.0-next-20170510 will boot. Without your patch 4.11.0-next-20170510 do not boot if u-boot initialize the PCI bus.
Thanks a lot!
Here is the patch I'm using for our board:
Stefano,
The patches that Tim/Peter posted solve a kernel hang with kernel 4.11 on mx6q when U-Boot has PCI support.
Could you please take a look and let us know if there is a more generic way to fix this issue, like fixing it for all mx6qdl boards that have PCI support?
I understand that ideally the imx6 pci driver should be converted to driver model and then we add the .remove hook, like Lucas did for Barebox: https://git.pengutronix.de/cgit/barebox/commit/?id=f1da98da2760c21487bbba8f7...
While the imx pci driver is not converted to device model, would you be willing to accept such per board patches for the time being?
Please advise.
Ok - let wait for Tim's answer, if he has enough time to work on this to move to DM. If not, it will be ok for now to fix it in this way, hoping to move soon to DM.
If none, have time to work, I'll work for dm conversion and have board to test the same.
Jagan.