Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RFC: phy: add support for Lattice ECP5 (SDPHYIOECP5) #10

Merged
merged 1 commit into from
Jun 3, 2020

Conversation

gsomlo
Copy link
Contributor

@gsomlo gsomlo commented Mar 18, 2020

@enjoy-digital: please don't apply (yet) -- this "should" (but isn't) working at the moment, so let's wait until we figure it out :)

@gsomlo
Copy link
Contributor Author

gsomlo commented Mar 25, 2020

per @daveshah1, it might be that the ECP5 ODDRX1F has a 3 clock cycle latency, which might be higher than the 1 or 2 cycle latency on the Xilinx, which might be a breaking change. I wanted to write that down here, to remember when I (or anyone else) end up working on this later...

@gsomlo gsomlo force-pushed the gls-ecp5-phy branch 2 times, most recently from db27535 to 22fd444 Compare April 12, 2020 11:54
@gsomlo gsomlo force-pushed the gls-ecp5-phy branch 2 times, most recently from 3ca4029 to 5ea899a Compare May 14, 2020 10:54
@gsomlo gsomlo force-pushed the gls-ecp5-phy branch 2 times, most recently from 9415035 to 8028393 Compare May 29, 2020 14:51
Signed-off-by: Gabriel Somlo <gsomlo@gmail.com>
@enjoy-digital enjoy-digital merged commit 636e605 into enjoy-digital:master Jun 3, 2020
@enjoy-digital
Copy link
Owner

@gsomlo: i'm merging it since i'm going to work on it and will also do some cleanup on the PHYs so want to avoid merge conflicts.

@gsomlo gsomlo deleted the gls-ecp5-phy branch June 3, 2020 12:31
@enjoy-digital
Copy link
Owner

The latency was indeed too high and was causing de-synchronization between CLK and CMD/DAT. I merged and replaced the ECP5 PHY with a Generic PHY in 27572e7 and tested it successfully on the ULX3S target of LiteX with:
./ulx3s.py --cpu-type=serv --with-sdcard --integrated-main-ram-size=0x100 --build --load
and then with the following commands:

  • sdinit
  • sdtest 10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants