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

Add buttons_arty target #998

Closed
duck2 opened this issue Sep 16, 2019 · 3 comments
Closed

Add buttons_arty target #998

duck2 opened this issue Sep 16, 2019 · 3 comments

Comments

@duck2
Copy link
Collaborator

duck2 commented Sep 16, 2019

I tried to add a buttons_arty test by adding an arty target to xc7/tests/buttons and creating the required arty.pcf and buttons_arty.v files, but building it failed with this error:

PCF constraint "set_io in[0] A8" from line 2 constraints pad A8 which is not in available pad map:
A9
C2
D10
E3
T10

This seems to because of the way the arty device is defined:

add_xc7_device_define_type(
...
  ROI_DIR ${PRJXRAY_DB_DIR}/artix7/harness/arty-a7/uart
...
)

In the prjxray repository, there is also swbut and pmod harnesses, but it doesn't seem to be possible to specify the harness in the test. How to go about this? Make a harness for arty including IO for both uart and buttons or try and generalize test targets?

@duck2 duck2 changed the title Add buttons_arty test Add buttons_arty target Sep 16, 2019
@acomodi
Copy link
Contributor

acomodi commented Sep 16, 2019

@duck2 I believe the easiest way to add a buttons target for arty and use the swbut harness is to create another directory in https://github.com/SymbiFlow/symbiflow-arch-defs/tree/master/xc7/archs/artix7/devices and call it something like: "xc7a50t-arty-swbut-roi-virt".

Than in there use the same CMakeLists.txt as for the other arty device and change the ROI to be used.

@acomodi
Copy link
Contributor

acomodi commented Sep 16, 2019

Then, in the arty_button tests you will have to specify the correct I/Os in the arty.pcf files as they appear in the swbut ROI and add the arty_buttons target to the CMakeLists.txt using the newly added device

@duck2
Copy link
Collaborator Author

duck2 commented Sep 25, 2019

#1006

@duck2 duck2 closed this as completed Sep 25, 2019
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

No branches or pull requests

2 participants