]> sigrok.org Git - libsigrok.git/commit
kingst-la2016: use symbolic names for device run state
authorGerhard Sittig <redacted>
Sun, 23 Jan 2022 17:08:45 +0000 (18:08 +0100)
committerGerhard Sittig <redacted>
Sun, 6 Feb 2022 17:53:53 +0000 (18:53 +0100)
commitb711fd8eeef00501f9a59d2d226bd4272828205e
tree1a93bacbf74979349ba64b7dc759b4ca0c0cdbfd
parent695aeaf7392e5d4a6186861520df15a3ec7e4b0d
kingst-la2016: use symbolic names for device run state

Rephrase the run_state() routine in terms of symbolic identifiers,
eliminate magic numbers in the code. Extend comments on run state even
more to help reviewers. Reduce indentation in the optional logging when
the value changes.

It's assumed that the "current state and mask equals a typical pattern?"
phrase better reflects the purpose of a check, and is easier to verify
than hex numbers in a context where bit field meanings matter and not
all adjacent bits are involved).

This implementation no longer assumes that "sample to DRAM" is set. In
theory capture to internal RAM versus streaming via USB should not matter
when the acquisition state is queried.
src/hardware/kingst-la2016/protocol.c