-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
litex: remove minimal build option #2768
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One nit pick. Looks good otherwise. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! Thanks!
Looks like you need to merge in master. ulab changed but github's merge doesn't look smart enough to update the submodule. Not sure why it only broke fomu though. |
Add a mp_hal_delay_us, which is required by some of the modules, that simply calls mp_hal_delay_ms / 1000. Signed-off-by: Sean Cross <sean@xobs.io>
Signed-off-by: Sean Cross <sean@xobs.io>
Manually specify available modules and disable any modules that are currently unsupported on the litex target. Signed-off-by: Sean Cross <sean@xobs.io>
I did a rebase onto master. Let's see if that fixes it. |
Signed-off-by: Sean Cross <sean@xobs.io>
The issue was the micropython config was missing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
This removes
CIRCUITPY_MINIMAL_BUILD
and adds config options to get circuitpython building on Fomu.It also adds required functions such as
mp_hal_delay_us
and theos
module, which are necessary for doing a non-minimal build.