Re: [U-Boot] drivers: pci: imx: add imx_pcie_remove function

[Stefano Babic]

On 26/06/2017 16:07, Soeren Moch wrote:
> 
> 
> On 12.05.2017 21:58, Tim Harvey wrote:
>> There is no dedicated reset signal wired up for the MX6QDL thus if the
>> bootloader enables the link we need some special handling to get the core
>> back into a state where it is safe to touch it for configuration.
>>
>> While there has been some special handling in the Linux kernel to do this,
>> it was removed in 4.11 thus we need to do it properly in the bootloader
>> and therefore without this if you enable PCI in the bootloader you will hang
>> while booting the 4.11 kernel.
>>
>> This puts the PCIe controller back into a safe state for the kernel driver
>> before launching the kernel.
>>
>> Signed-off-by: Tim Harvey 
>> Reviewed-by: Fabio Estevam 
>> Tested-by: Peter Senna Tschudin 
> 
> Tested-by: Soeren Moch 
> 
> 
> 
> Stefano,
> 
> can we get this fix merged for v2017.07!?

Yes - this is merged in u-boot-imx, I am checking now which pending
patches are bug fixes that can flow in, and I send later my PR to Tom.

Regards,
Stefano


-- 
=
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] drivers: pci: imx: add imx_pcie_remove function

[Soeren Moch]

On 12.05.2017 21:58, Tim Harvey wrote:
> There is no dedicated reset signal wired up for the MX6QDL thus if the
> bootloader enables the link we need some special handling to get the core
> back into a state where it is safe to touch it for configuration.
>
> While there has been some special handling in the Linux kernel to do this,
> it was removed in 4.11 thus we need to do it properly in the bootloader
> and therefore without this if you enable PCI in the bootloader you will hang
> while booting the 4.11 kernel.
>
> This puts the PCIe controller back into a safe state for the kernel driver
> before launching the kernel.
>
> Signed-off-by: Tim Harvey 
> Reviewed-by: Fabio Estevam 
> Tested-by: Peter Senna Tschudin 

Tested-by: Soeren Moch 



Stefano,

can we get this fix merged for v2017.07!?

Thanks,
Soeren

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] drivers: pci: imx: add imx_pcie_remove function

[Soeren Moch]

> There is no dedicated reset signal wired up for the MX6QDL thus if the
> bootloader enables the link we need some special handling to get the core
> back into a state where it is safe to touch it for configuration.
>
> While there has been some special handling in the Linux kernel to do this,
> it was removed in 4.11 thus we need to do it properly in the bootloader
> and therefore without this if you enable PCI in the bootloader you
will hang
> while booting the 4.11 kernel.
>
> This puts the PCIe controller back into a safe state for the kernel driver
> before launching the kernel.
>
> Signed-off-by: Tim Harvey 

Successfully tested on a tbs2910 board.

Tested-by: Soeren Moch 

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot