-
Notifications
You must be signed in to change notification settings - Fork 212
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
Use ProcessorTop templates in non osflow examples #77
Comments
Good point! Two ideas come to mind:
Some more:
|
While I acknowledge that Minimal vs MinimalBoot might be confusing, the main idea is that Minimal has the bootloader disabled and no other output than the PWM. Conversely, in MinimalBoot the usage of the bootloader is optional, as it is connecting the UART or GPIO. Therefore, I think that "indirectboot" and "directboot" are not descriptive enough. The purpose of MinimalBoot is to be used for both indirect and direct boot. It is correct that Minimal is direct only. Other than that, I think that Tiny, Small, Medium, etc. might be as good as any other naming you pick.
We are already doing that: https://github.com/stnolting/neorv32/blob/master/rtl/templates/system/neorv32_SystemTop_axi4lite.vhd. So, yes. |
I was just thinking out loud 😅
I am still not sure about this... |
Fixed ✔️
I am still struggling with this 😅 but that might be stuff for a different issue. I think this can be closed now. |
Currently, the non osflow examples (radiant, quartus) do not use a ProcessorTop or SystemTop template. Quartus is trying to use
neorv32_test_setup.vhd
(which was renamed and does not exist) and radiant is instantiating the NEORV32 top directly.It would be nice to update these examples for using
neorv32_ProcessorTop_Test
andneorv32_ProcessorTop_UP5KDemo
(or MinimalBoot).Vivado examples use
neorv32_ProcessorTop_Test
already.The text was updated successfully, but these errors were encountered: