On 01.03.19 13:10, Soeren Moch wrote:
Disable this support for cm_fx6 to avoid breakage.
Signed-off-by: Soeren Moch smoch@web.de
It would be easy to add weak board_ahci_enable/disable functions which can be overwritten (e.g. in the cm_fx6 board file) to handle power enable/ disable on driver probe/remove. But I have no idea why this board needs these repeated probe/remove calls for the same sata device. In my tests on a tbs2910 board a sata harddisk is always recognized by a 'sata init' command, also without this probe loop. If this can be ironed out, we can remove the DWC_AHSATA_AHCI config option.
Cc: Nikita Kiryanov nikita@compulab.co.il Cc: Stefano Babic sbabic@denx.de Cc: Fabio Estevam festevam@gmail.com Cc: Marcel Ziswiler marcel@ziswiler.com Cc: Stefan Roese sr@denx.de Cc: Simon Glass sjg@chromium.org Cc: u-boot@lists.denx.de
After rebasing this patch series from v2019.04-rc2 to -rc3, I get this:
Matrix U-Boot> sata init
AHCI 0001.0300 32 slots 1 ports 3 Gbps 0x1 impl SATA mode
flags: ncq stag pm led clo only pmp pio slum part
Can not start port 0
data abort
pc : [<8ffab37c>] lr : [<8ffab595>]
reloc pc : [<1781837c>] lr : [<17818595>]
sp : 87f909c8 ip : 87f90cd4 fp : 00000004
r10: 6c41202e r9 : 87f90ea0 r8 : 00000000
r7 : 87f90a80 r6 : 00000200 r5 : 00000000 r4 : 87f90a80
r3 : 00000070 r2 : 87f90cc0 r1 : 87f90cc0 r0 : 00000000
Flags: nzcv IRQs off FIQs off Mode SVC_32
Code: fb03231c f8d3030b 9001a00c 9e0c4611 (3038f8da)
Resetting CPU ...
Any idea what got broken here?
Thanks, Soeren