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

liteeth: remove dw parameterization #70

Closed
sbourdeauducq opened this issue Dec 16, 2017 · 2 comments
Closed

liteeth: remove dw parameterization #70

sbourdeauducq opened this issue Dec 16, 2017 · 2 comments
Assignees

Comments

@sbourdeauducq
Copy link
Member

The data path width parameter is dragged around the core but the difficult issues (e.g. unaligned SFD, unaligned CRC) are not addressed. Having a configurable data path width is not straightforward and has little practical value, all current PHYs are 1-byte wide. The parameter should be removed to reduce complexity/clutter a bit and stop giving the false impression that the core supports dw != 8.

@whitequark
Copy link
Contributor

whitequark commented Jan 17, 2018

SaymaAMC, Kasli and KC705 MiniSoCs currently use dw=32.

@sbourdeauducq
Copy link
Member Author

That's the dw for the PHY. That one is 8 everywhere AFAIK. Where do you see 32?

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

No branches or pull requests

3 participants