m-labs/misoc

merge a7ddrphy and k7ddrphy

sbourdeauducq opened this issue · 5 comments

@enjoy-digital why was that code duplicated?

Artix7 don't have ODELAYE2, but yes we could probably try to avoid duplicating common code.

I don't understand how this can work without output delays. Is it by chance or...?

Look at the clocks.

Also BIOS fails compilation due to missing delay adjustment CSRs.
Either the BIOS needs fixing to work with the phase shifted DQS or the merged PHY needs to handle it.