glasgow icon indicating copy to clipboard operation
glasgow copied to clipboard

Add selftest for sync and aux pins

Open electroniceel opened this issue 5 years ago • 3 comments

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.

electroniceel avatar Aug 19 '19 13:08 electroniceel

Sync to A0? Aux could probably connected together.

whitequark avatar Aug 19 '19 13:08 whitequark

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.

electroniceel avatar Aug 19 '19 13:08 electroniceel

Makes sense

whitequark avatar Aug 19 '19 13:08 whitequark