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

revC1: add io port pinout to back silkscreen #120

Merged
merged 1 commit into from Apr 5, 2019

Conversation

electroniceel
Copy link
Member

my suggestion for silkscreen as described in #119

@whitequark
Copy link
Member

Screenshot?

@marcan
Copy link
Member

marcan commented Apr 4, 2019

image

I think it's too busy (and non-symmetrical with the top); I would favor a sparser layout with a single character per pin. V for Vio, S for sense, 0-7 for the port pins, and the NC pins can just be left unmarked. The goal of putting the pinout on there isn't to tell people what the pins do (they should know that already), but just to remind existing users of which pin is which among the pins they already know are on there.

Also, please make sure all the labels are perfectly aligned with the port pins (yes, this is likely to be annoying given the mixed imperial/metric grids used for those connectors, but we're anal about such things).

To differentiate the A and B ports, a single A/B to the left of each (in the same text size as the A/B on top) is sufficient, we don't need to mark every IO pin separately.

@electroniceel
Copy link
Member Author

How about this:
silkscreen

@marcan
Copy link
Member

marcan commented Apr 4, 2019

Much better. Horizontal alignment seems a bit off, though. @whitequark, any opinions on Vio vs V for that pin? + could be another option.

@whitequark
Copy link
Member

I'd say V fits nicely.

Shortened version with just numbers, V instead of Vio, aligned numbers
@electroniceel
Copy link
Member Author

Updated version using "V" instead of "Vio". I also aligned all the numbers and letters to the respective holes of the connectors.

silkscreen

@whitequark whitequark merged commit f2202ca into GlasgowEmbedded:master Apr 5, 2019
@whitequark
Copy link
Member

Thanks!

@electroniceel electroniceel deleted the ioport-silkscreen branch April 5, 2019 20:41
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.

None yet

3 participants