]> sigrok.org Git - libsigrok.git/commit
rdtech-um: rephrase checksum verification routines
authorGerhard Sittig <redacted>
Wed, 15 Mar 2023 16:11:30 +0000 (17:11 +0100)
committerGerhard Sittig <redacted>
Thu, 16 Mar 2023 13:29:30 +0000 (14:29 +0100)
commit85accbc2e7c14b49039b5a9a3fc8afc169a9acc0
tree725ccdba351067d82dee0ac33553f29f05f382e7
parentfda34e5af6e7a11744035c1c8df2f2b2f95cf1d8
rdtech-um: rephrase checksum verification routines

Rename the unfortunately named poll_csum() callback. It neither polled
for receive data nor calculated checksums for the caller. It verifies
that a received packet's checksum is plausible, and returns a boolean.

Address coding style nits in the process. Adjust data types. Use size
types where appropriate. Don't use characters when bytes are meant. Use
booleans for the checksum test result. Shorten identifier names to trim
text line length. No need to "take the address" of a routine. Remove
dead code, model dependent checksum approaches are handled by registered
callbacks in profiles. Eliminate else after return. Prefer read_u16be()
endianess routines over RB16() preprocessor macros. Prefer lower case
hex digits. Make the code reflect that the last 16bit or 8bit entity in
the received data packet gets checked. Use "calc" and "recv" names for
awareness which variable holds which value.
src/hardware/rdtech-um/protocol.c
src/hardware/rdtech-um/protocol.h