-
-
Notifications
You must be signed in to change notification settings - Fork 114
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
Works on ULX3S (Lattice ECP5) #25
Comments
Awesome!!! This is fantastic news 🎉 I would really appreciate it if you could make a pull request for or share those serializer changes. Hoping to support as many of the major platforms as possible. |
Yes, will submit a PR after I cleanup a bit. Looking at serializer.sv, can you tell me what the defines do ? The defines I see are ALTERA_RESERVED_QIS & MODEL_TECH. Is XILINX an implicit define is some way ? For Lattice, I have used a platform specific primitive ODDRX1F. I'll want to add a define for this too. |
Sort of...I read on a Xilinx forum somewhere that the one way to distinguish Vivado is the presence of If there's a similar platform flag for Lattice, it could be another |
Ok. Looks like Lattice Diamond also defines I couldn't find if Lattice Diamond automatically sets up a specific flag to help distinguish it. Documentation is sparse & hard to find. Plus, I am new to FPGAs toolchains. Working through this. Is |
Good find! If there's a Lattice specific flag, we could put that implementation above that one. So it gets picked instead of hitting the Xilinx implementation.
No worries, I am working with some very rough understanding too 🙂 I'll also try to look for a flag. Either that or maybe there is a Xilinx flag and SYNTHESIS can be used for Lattice |
Hi @shreekumar3d , Would you be able to share the changes you made as a PR? We can build on top of it from there to support Lattice out of the box. |
Hi @sameer - apologies for the long delay. I got busy with a few things & lost track of this. I will update this thread with the changes this weekend. |
No worries! |
It's been two years, but it would be possible to share your changes please @shreekumar3d ? Thanks! |
Right. I wanted to cleanup the code and then submit a PR etc so that it merges clean - mostly was getting tripped in the defines . But then you know what happens when you end up in a lot of work. This just slipped off my list of things to do. The important change was in the serializer module, IIRC. Something that works is good enough for now, right ? |
@shreekumar3d anything that works will be very helpful, even if it's not cleaned up. Thank you 😊 |
@gamelaster @sameer serializer.sv
|
Hi, thank you for sharing this! I'll make sure it gets merged in |
Hi !
Filing this to report that I've been able to get this to work on the ULX3S board (based on Lattice ECP5, 12K gates).
Due to compile issues with the open source toolchain of the ULX3S, I ended up using Lattice Diamond...
Tested default settings - 640x480, and 48k stereo audio (externally driven). Had to make some changes to suit the ECP5 (in the serializer module). Also some changes for compilation.
Will be updating with more results & changes soon.
Thanks
-- Shree
The text was updated successfully, but these errors were encountered: