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

Litex add litespi flash and watchdog #21

Merged
merged 9 commits into from
Sep 17, 2024

Conversation

maass-hamburg
Copy link
Contributor

@maass-hamburg maass-hamburg commented Jun 19, 2024

  • Be able to use spi flash from LiteSPI.
  • Adds watchdog

update all submodules.

Signed-off-by: Fin Maaß <[email protected]>
use the new existing function from litex to add the spi master.

Signed-off-by: Fin Maaß <[email protected]>
add watchdog

Signed-off-by: Fin Maaß <[email protected]>
@maass-hamburg maass-hamburg force-pushed the litex_add_litespi_flash branch from db2db86 to c255c19 Compare June 25, 2024 14:18
@maass-hamburg maass-hamburg changed the title Litex add litespi flash Litex add litespi flash and watchdog Jun 25, 2024
- adds etherbone option
- uses functions from litex for ethernet directly

Signed-off-by: Fin Maaß <[email protected]>
@maass-hamburg maass-hamburg force-pushed the litex_add_litespi_flash branch 12 times, most recently from f34cddf to eeef8b6 Compare August 20, 2024 08:29
Signed-off-by: Fin Maaß <[email protected]>
update version of actions.

Signed-off-by: Fin Maaß <[email protected]>
@maass-hamburg maass-hamburg force-pushed the litex_add_litespi_flash branch from eeef8b6 to a28e226 Compare August 20, 2024 08:39
@maass-hamburg
Copy link
Contributor Author

Fixed the ci and updated the it to use https://github.com/YosysHQ/oss-cad-suite-build, which contains yosys, prjoxide and nextpnr, so we don't have to compile and install it our self.

@maass-hamburg
Copy link
Contributor Author

@enjoy-digital @tgorochowik you might want to take a look

Copy link
Member

@tgorochowik tgorochowik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable, thanks! I assume you verified it on hardware?

@maass-hamburg
Copy link
Contributor Author

@tgorochowik I tested it successful on my arty

@tgorochowik tgorochowik merged commit e539467 into litex-hub:master Sep 17, 2024
2 checks passed
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.

2 participants