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

Altera apu agilex7 #2635

Closed

Conversation

AngelaGonzalezMarino
Copy link
Contributor

@AngelaGonzalezMarino AngelaGonzalezMarino commented Nov 27, 2024

This PR is adding the sources required to generate a bitstream for Altera FPGA.

  • Altera proprietary IPs
  • Avalon to AXI conversion modules
  • Submodules to use PULP gpio in APU instead of Xilinx GPIO IP
  • AXI data width adapter (non xilinx IP)
  • adaptation of debug module to use with vJTAG IP (from Thales AVS)
  • Top level of cva6 APU + cva6 peripherals adapted to Altera
  • Pin mapping
  • FPGA constraints

It does not include integration in the Makefile, that will be the last PR of the Altera support.

Copy link
Contributor

❌ failed run, report available here.

Copy link
Contributor

✔️ successful run, report available here.

Copy link
Contributor

✔️ successful run, report available here.

Copy link
Contributor

✔️ successful run, report available here.

Copy link
Contributor

✔️ successful run, report available here.

2 similar comments
Copy link
Contributor

✔️ successful run, report available here.

Copy link
Contributor

✔️ successful run, report available here.

@JeanRochCoulon
Copy link
Contributor

JeanRochCoulon commented Nov 28, 2024

Do you have time next CVA6 meeting to explain why all Altera techno files are added, this is huge and not sure it would be a good idea to overload the repository
300klines are added

@AngelaGonzalezMarino
Copy link
Contributor Author

Do you have time next CVA6 meeting to explain why all Altera techno files are added, this is huge and not sure it would be a good idea to overload the repository 300klines are added

Sure, we can discuss next Tuesday. The files added are for Altera IPs needed in the APU. I think it could be possible to add only the IPs packaged, but I thought it was better to give the sources in case someone wants to modify (e.g. change frequency)

@JeanRochCoulon
Copy link
Contributor

I agree, it is better to provide the sources. In this case, why not creating a project with Altera library, the cva6 script would clone this project when needed
@MikeOpenHWGroup

@AngelaGonzalezMarino
Copy link
Contributor Author

AngelaGonzalezMarino commented Nov 28, 2024

I agree, it is better to provide the sources. In this case, why not creating a project with Altera library, the cva6 script would clone this project when needed @MikeOpenHWGroup

I found the way to regenerate the sources from the .ip file, so it is much lighter now. This will be integrated in the next PR with the automation flow.

The remaining ones are files automatically generated by Quartus to connect the DDR IP with the AXI. Should these be moved to a library or is it OK to keep them here? I mean the ones under folders like "corev_apu/altera/src/altera_xXx

Thanks

Copy link
Contributor

✔️ successful run, report available here.

Copy link
Contributor

✔️ successful run, report available here.

Copy link
Contributor

❌ failed run, report available here.

Copy link
Contributor

github-actions bot commented Dec 3, 2024

❌ failed run, report available here.

@valentinThomazic
Copy link
Contributor

@JeanRochCoulon @AngelaGonzalezMarino fpga boot is ok

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.

3 participants