search

GlasgowEmbedded / glasgow

Scots Army Knife for electronics
BSD Zero Clause License
1.92k stars 187 forks source link

Add selftest for sync and aux pins #152

Open electroniceel opened 5 years ago

electroniceel commented 5 years ago

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 commented 5 years ago

Sync to A0? Aux could probably connected together.

electroniceel commented 5 years ago

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 commented 5 years ago

Makes sense