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 selftest for sync and aux pins #152

Open
electroniceel opened this issue Aug 19, 2019 · 3 comments
Open

Add selftest for sync and aux pins #152

electroniceel opened this issue Aug 19, 2019 · 3 comments
Labels
selftest Applet: selftest

Comments

@electroniceel
Copy link
Member

electroniceel commented Aug 19, 2019

Currently there is no selftest for the sync and aux pins of revC. These should be added.

For a loop test they should be connected like on the test jig pcb: sync to A0, Aux A10 to A1 and Aux C9 to A2.

As the Aux pin headers are marked DNP, the tests for them should be optional. They should probably get their own command like "pins-aux" and "pins-aux-loop".

These tests also require the internal pullups on the ice40 which are not yet supported in nmigen.

@whitequark whitequark added the selftest Applet: selftest label Aug 19, 2019
@whitequark
Copy link
Member

whitequark commented Aug 19, 2019

Sync to A0? Aux could probably connected together.

@electroniceel
Copy link
Member Author

Yeah. When executing "pins-sync-loop" the user is required to connect the sync pin to port A pin 0.

I don't like looping Aux together as they could probably have a short between them. Connecting them to port A, which was tested independently by the other tests, seems to be a better idea to me.

@whitequark
Copy link
Member

Makes sense

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
selftest Applet: selftest
Projects
None yet
Development

No branches or pull requests

2 participants