Skip to content

Add support for the PYNQ-Z2 board #49

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nuntipat
Copy link

@nuntipat nuntipat commented Jun 6, 2023

This pull request adds support for the TUL PYNQ-Z2 board. BTN3 is used as a reset button, leaving 4 LEDs (LD0-3), 2 RGB LEDs (LD4-5), 2 slide switches (SW0-1), 4 GPIOs (PMODA) and 3 push buttons (BTN 0-2) for user applications. UART connection to the host PC is possible with the on-board FTDI chip with minimal modification to the board as described in the README.

One question. Because the board has a 125 MHz clock source rather than the Arty board's 100 MHz, a new clock generator module is given in rtl/fpga/clkgen_pynqz2.sv based on vendor/lowrisc_ibex/shared/rtl/fpga/xilinx/clkgen_xil7series.sv. Is it acceptable to place clkgen_pynqz2.sv in the rtl/fpga directory?

@virajwadhwa
Copy link

virajwadhwa commented Apr 27, 2024

Thanks for working on this, really cool! Your work helped me get a hello-world demo working on a Pynq Z2 board!

@marnovandermaas
Copy link
Contributor

marnovandermaas commented Mar 5, 2025

Sorry for not reviewing this sooner. Would you mind updating the PR with the latest main branch? Some of your changes to load_demo_system.sh should not be necessary any more because we can specify the TCL script now. I would like to get this work into the repo though.

@nuntipat
Copy link
Author

nuntipat commented Mar 6, 2025

@marnovandermaas Thank you for the feedback. I'll take a look at the change this weekend.

@marnovandermaas marnovandermaas marked this pull request as draft April 8, 2025 16:18
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