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] Add support for multiple CS pin for SDRAM controller #577

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

t123yh
Copy link

@t123yh t123yh commented Jan 10, 2022

This patch adds support for multiple CS pin, useful if anyone wants 128MB or 256MB with SDRAM. It's tested on a board with 4 IS42S16320F SDRAM chips, forming 32bit x 2CS structure.

Note that it currently only works with the sdr controller, not xdr controller. I'm not sure how to modify xdr controller, so any help is appreciated.

@Dolu1990
Copy link
Member

I guess
https://github.com/SpinalHDL/SpinalHDL/runs/4786705600?check_suite_focus=true#step:10:167
regs.CSn(0) := io.ctrl.phases(0).CSn should fix the compilation error

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