From: Jens Steinhauser <redacted>
Date: Sat, 18 Oct 2014 09:31:16 +0000 (+0200)
Subject: spi: Tell stacked decoders about missing CS# signal.
X-Git-Tag: libsigrokdecode-0.4.0~151
X-Git-Url: https://sigrok.org/gitaction?a=commitdiff_plain;h=1c49e875623918f8893308307f487a1f551daec4;p=libsigrokdecode.git

spi: Tell stacked decoders about missing CS# signal.

This could previously be detected by the absence of a CS-CHANGE packet
before the first data packet, but it makes the stacked decoder simpler
if it is told directly.
---

diff --git a/decoders/spi/pd.py b/decoders/spi/pd.py
index 8579258..25a9f85 100644
--- a/decoders/spi/pd.py
+++ b/decoders/spi/pd.py
@@ -35,7 +35,9 @@ Packet:
  - 'BITS': <data1>/<data2> contain a list of bit values in this MOSI/MISO data
    item, and for each of those also their respective start-/endsample numbers.
  - 'CS CHANGE': <data1> is the old CS# pin value, <data2> is the new value.
-   Both data items are Python numbers (0/1), not strings.
+   Both data items are Python numbers (0/1), not strings. At the beginning of
+   the decoding a packet is generated with <data1> = -1 and <data2> being the
+   initial state of the CS# pin or -1 if the chip select pin is not supplied.
 
 Examples:
  ['CS-CHANGE', 1, 0]
@@ -122,6 +124,7 @@ class Decoder(srd.Decoder):
         self.oldcs = -1
         self.oldpins = None
         self.have_cs = self.have_miso = self.have_mosi = None
+        self.no_cs_notification = False
 
     def metadata(self, key, value):
         if key == srd.SRD_CONF_SAMPLERATE:
@@ -281,4 +284,9 @@ class Decoder(srd.Decoder):
             if not (self.have_miso or self.have_mosi):
                 raise ChannelError('Either MISO or MOSI (or both) pins required.')
 
+            # Tell stacked decoders that we don't have a CS# signal.
+            if not self.no_cs_notification and not self.have_cs:
+                self.put(0, 0, self.out_python, ['CS-CHANGE', -1, -1])
+                self.no_cs_notification = True
+
             self.find_clk_edge(miso, mosi, clk, cs)