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

PCF files do not match the actual pinout for GPIOs #20

Closed
jeremyherbert opened this issue Jul 5, 2021 · 4 comments
Closed

PCF files do not match the actual pinout for GPIOs #20

jeremyherbert opened this issue Jul 5, 2021 · 4 comments

Comments

@jeremyherbert
Copy link

It appears that the GPIO pins in the pcf file here: https://github.com/gregdavill/OrangeCrab-examples/blob/main/verilog/orangecrab_r0.2.pcf are not correct.

They do not match the pinout here: https://gregdavill.github.io/OrangeCrab/r0.2/docs/pinout/ and at least from my quick look at the schematic, the picture is the correct reference.

@jeremyherbert
Copy link
Author

I had a closer look at this, it seems that the pinout numbers on the silkscreen do not match the GPIO numbering; is this intended?

@umarcor
Copy link

umarcor commented Jul 7, 2021

@jeremyherbert, my perception is that the constraints file is correct, in the sense that it responds to some criteria; however, the criteria is not coherent with the silkscreen. I would propose the following modifications to the names in the constraints file:

constraint silkscreen pin proposal
GPIO0[0] 0 N17 GPIO_0
GPIO0[1] 1 M18 GPIO_1
GPIO0[2] 5 B10 GPIO_5
GPIO0[3] 6 B9 GPIO_6
GPIO0[4] 9 C8 GPIO_9
GPIO0[5] 10 B8 GPIO_10
GPIO0[6] 11 A8 GPIO_11
GPIO0[7] 12 H2 GPIO_12
GPIO0[8] 13 J2 GPIO_13
GPIO0[9] A0 L4 GPIO_A0
GPIO0[10] A1 N3 GPIO_A1
GPIO0[11] A2 N4 GPIO_A2
GPIO0[12] A3 H4 GPIO_A3

@gregdavill
Copy link
Collaborator

The pinouts on the silkscreen are following the standard 'feather' numbering, which I think has some historical reasons for the numbering.

As @umarcor has pointed out, the GPIO numbers line up, but they have gaps.
I think we should update the pcf to cover the numbers actually present on the silkscreen.

@jeremyherbert
Copy link
Author

Thanks!

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

3 participants