]> sigrok.org Git - libsigrokdecode.git/commit
i2c: more idiomatic use of Python list, reduces redundancy, comments
authorGerhard Sittig <redacted>
Fri, 14 Jul 2023 15:43:08 +0000 (17:43 +0200)
committerGerhard Sittig <redacted>
Tue, 18 Jul 2023 19:09:40 +0000 (21:09 +0200)
commit647aba6af7745c14961674c5feebf83fd18303a3
tree3ff40234546f9663255eb4f0290b2f49c5f54361
parent42fb0f33ff23c1633324bde8b3cf400493237d97
i2c: more idiomatic use of Python list, reduces redundancy, comments

Collect all bits of a byte time in a Python list (as was done before).
Eliminate the bit counter and the value accumulator, use the list's
length during accumulation and common conversion support after the
accumulation instead.

Also comment on the non-trivial start/end sample number update logic.
The decoder implementation likes to claim data validity outside of the
high SCL phase, which does not agree with the strict I2C protocol idea.
Increases usability though (data visibility at zoom levels). This and
backwards compatibility makes us keep the logic, as long as we remain
aware of its implications.

Comment on the rather unexpected LSB first emission of annotations and
stacked layer data passing, while the I2C protocol itself is MSB first.
decoders/i2c/pd.py