]> sigrok.org Git - libsigrok.git/commit
kingst-la2016: shuffle code order to follow application perspective
authorGerhard Sittig <redacted>
Sat, 22 Jan 2022 10:28:13 +0000 (11:28 +0100)
committerGerhard Sittig <redacted>
Sun, 6 Feb 2022 17:53:53 +0000 (18:53 +0100)
commitc34f4a89d16c3001d4687a1281b98b63d9e61715
tree6d52d8c11209b2c8a018624398b16c797443992f
parentda2cb50d7311fe66a50169e33ee1be9f7b901709
kingst-la2016: shuffle code order to follow application perspective

Sort "set threshold voltage" code blocks into numerical order of voltage
levels for improved readability. It's surprising to find ">= 2.9V" then
"<= -0.4V" and then "between those" at the end.

Move the la2016_has_triggered() routine next to run_state() since both
of them inspect the device's current run state. Closer inspection even
suggests that one is named inappropriately (not renamed in this commit,
to help reviewers).

The purpose of the set_run_mode() routine's argument most probably is
"mode", and not the surprising "fast blinking". Indicators may just be
a byproduct of setting mode to run or halt.

Make the set_defaults() invocation in la2016_init_device() stand out
more perceivably. Don't hide it in a return statement.
src/hardware/kingst-la2016/protocol.c