Skip to content

Open/short detection#8

Draft
JohnAZoidberg wants to merge 3 commits intomainfrom
open-short
Draft

Open/short detection#8
JohnAZoidberg wants to merge 3 commits intomainfrom
open-short

Conversation

@JohnAZoidberg
Copy link
Member

@JohnAZoidberg JohnAZoidberg commented Sep 18, 2023

Detect which LEDs are connected and which aren't.
Useful to detect which kind of module we're running on.
E.g. early Framework prototype, later prototype, Adafruit module, ...

Doesn't quite work, hm :/ (Data seems wrong)

@JohnAZoidberg JohnAZoidberg changed the title Make sure sw_enablement doesn't overwrite Open/short detection Sep 19, 2023
@JohnAZoidberg JohnAZoidberg marked this pull request as draft September 19, 2023 02:50
JohnAZoidberg and others added 2 commits September 19, 2023 11:12
Useful if not all SW pins are connected. It's good to disable
disconnected ones. Otherwise it causes higher voltages spikes which can
cause audible noise.

Signed-off-by: Daniel Schaefer <git@danielschaefer.me>
Signed-off-by: Daniel Schaefer <dhs@frame.work>
@JohnAZoidberg JohnAZoidberg changed the base branch from sw-enablement to enable-reading September 19, 2023 03:13
@JohnAZoidberg JohnAZoidberg force-pushed the open-short branch 2 times, most recently from 767b9c4 to 84bab3e Compare September 19, 2023 03:15
@JohnAZoidberg
Copy link
Member Author

References:

```
let open1 = matrix.device.check_open(&mut delay, 0x03, true).unwrap();
let open2 = matrix.device.check_open(&mut delay, 0x07, true).unwrap();
let open3 = matrix.device.check_open(&mut delay, 0x08, true).unwrap();
let open4 = matrix.device.check_open(&mut delay, 0x0C, true).unwrap();

let short1 = matrix.device.check_open(&mut delay, 0x03, false).unwrap();
let short2 = matrix.device.check_open(&mut delay, 0x07, false).unwrap();
let short2 = matrix.device.check_open(&mut delay, 0x07, false).unwrap();
let short3 = matrix.device.check_open(&mut delay, 0x08, false).unwrap();
let short4 = matrix.device.check_open(&mut delay, 0x0C, false).unwrap();
```

Signed-off-by: Daniel Schaefer <git@danielschaefer.me>
Base automatically changed from enable-reading to main October 8, 2023 10:08
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.

1 participant