]> sigrok.org Git - libsigrokdecode.git/commit
adf435x: Move decoder logic to SPI transfers
authorVesa-Pekka Palmu <redacted>
Mon, 26 Dec 2022 17:18:36 +0000 (19:18 +0200)
committerGerhard Sittig <redacted>
Mon, 9 Jan 2023 19:11:44 +0000 (20:11 +0100)
commitadb8233a0bf30b1d9ee9176e1caa5dc8ae1830dd
tree8b4bdc252e5187e63d8dc0a717e232e3e5c45579
parentf534ce442c271c13af5d216e16f56322dc586822
adf435x: Move decoder logic to SPI transfers

The previous implementation of the ADF435x decoder assumed knowledge of
internal details which are the SPI transport layer's responsibility. And
encoded an inappropriate chip select polarity in the process (falling
CS edge). The datasheet specifies that previously clocked in data bits
get latched on rising LE edges.

Not all setups were affected, that's why the issue went unnoticed before.

Use the lower layer's TRANSFER annotation to process the completion of
an ADF435x register access, after BITS annotations made the location of
individual bits available. The LE (CS) signal's polarity remains a detail
of the SPI decoding layer, and must be configured there. The SPI decoder's
default matches the ADF435x chip's expectation.

This fixes bug #1814.

Reported-by: Martin Homuth-Rosemann <redacted>
decoders/adf435x/pd.py