X-Git-Url: https://sigrok.org/gitweb/?p=libsigrokdecode.git;a=blobdiff_plain;f=decoders%2Fspi%2Fpd.py;h=88487bb77f6ef0772403330fe1f27a0c12138bad;hp=b18d021b85c592bcc67105fc3dc590924f4f84ee;hb=12549f1161748215072541ed9a9c8625da041131;hpb=8a3c879201df27615f30b7c6b3a4072526c0e378 diff --git a/decoders/spi/pd.py b/decoders/spi/pd.py index b18d021..88487bb 100644 --- a/decoders/spi/pd.py +++ b/decoders/spi/pd.py @@ -2,7 +2,7 @@ ## This file is part of the libsigrokdecode project. ## ## Copyright (C) 2011 Gareth McMullin -## Copyright (C) 2012-2013 Uwe Hermann +## Copyright (C) 2012-2014 Uwe Hermann ## ## This program is free software; you can redistribute it and/or modify ## it under the terms of the GNU General Public License as published by @@ -19,8 +19,6 @@ ## Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA ## -# SPI protocol decoder - import sigrokdecode as srd ''' @@ -32,7 +30,8 @@ SPI packet: Commands: - 'DATA': contains the MISO data, contains the MOSI data. The data is _usually_ 8 bits (but can also be fewer or more bits). - Both data items are Python numbers, not strings. + Both data items are Python numbers (not strings), or None if the respective + probe was not supplied. - 'CS CHANGE': is the old CS# pin value, is the new value. Both data items are Python numbers (0/1), not strings. @@ -40,6 +39,8 @@ Examples: ['CS-CHANGE', 1, 0] ['DATA', 0xff, 0x3a] ['DATA', 0x65, 0x00] + ['DATA', 0xa8, None] + ['DATA', None, 0x55] ['CS-CHANGE', 0, 1] ''' @@ -63,13 +64,13 @@ class Decoder(srd.Decoder): inputs = ['logic'] outputs = ['spi'] probes = [ - {'id': 'miso', 'name': 'MISO', - 'desc': 'SPI MISO line (Master in, slave out)'}, - {'id': 'mosi', 'name': 'MOSI', - 'desc': 'SPI MOSI line (Master out, slave in)'}, {'id': 'sck', 'name': 'CLK', 'desc': 'SPI clock line'}, ] optional_probes = [ + {'id': 'miso', 'name': 'MISO', + 'desc': 'SPI MISO line (master in, slave out)'}, + {'id': 'mosi', 'name': 'MOSI', + 'desc': 'SPI MOSI line (master out, slave in)'}, {'id': 'cs', 'name': 'CS#', 'desc': 'SPI chip-select line'}, ] options = { @@ -81,10 +82,9 @@ class Decoder(srd.Decoder): 'format': ['Data format', 'hex'], } annotations = [ - ['MISO/MOSI data', 'MISO/MOSI SPI data'], - ['MISO data', 'MISO SPI data'], - ['MOSI data', 'MOSI SPI data'], - ['Warnings', 'Human-readable warnings'], + ['miso-data', 'MISO SPI data'], + ['mosi-data', 'MOSI SPI data'], + ['warnings', 'Human-readable warnings'], ] def __init__(self): @@ -93,12 +93,14 @@ class Decoder(srd.Decoder): self.bitcount = 0 self.mosidata = 0 self.misodata = 0 - self.bytesreceived = 0 self.startsample = -1 self.samplenum = -1 self.cs_was_deasserted_during_data_word = 0 self.oldcs = -1 self.oldpins = None + self.have_cs = None + self.have_miso = None + self.have_mosi = None self.state = 'IDLE' def metadata(self, key, value): @@ -106,14 +108,11 @@ class Decoder(srd.Decoder): self.samplerate = value def start(self): - self.out_proto = self.add(srd.OUTPUT_PYTHON, 'spi') - self.out_ann = self.add(srd.OUTPUT_ANN, 'spi') + self.out_proto = self.register(srd.OUTPUT_PYTHON) + self.out_ann = self.register(srd.OUTPUT_ANN) self.out_bitrate = self.register(srd.OUTPUT_META, meta=(int, 'Bitrate', 'Bitrate during transfers')) - def report(self): - return 'SPI: %d bytes received' % self.bytesreceived - def putpw(self, data): self.put(self.startsample, self.samplenum, self.out_proto, data) @@ -133,16 +132,18 @@ class Decoder(srd.Decoder): ws = self.options['wordsize'] # Receive MOSI bit into our shift register. - if self.options['bitorder'] == 'msb-first': - self.mosidata |= mosi << (ws - 1 - self.bitcount) - else: - self.mosidata |= mosi << self.bitcount + if self.have_mosi: + if self.options['bitorder'] == 'msb-first': + self.mosidata |= mosi << (ws - 1 - self.bitcount) + else: + self.mosidata |= mosi << self.bitcount # Receive MISO bit into our shift register. - if self.options['bitorder'] == 'msb-first': - self.misodata |= miso << (ws - 1 - self.bitcount) - else: - self.misodata |= miso << self.bitcount + if self.have_miso: + if self.options['bitorder'] == 'msb-first': + self.misodata |= miso << (ws - 1 - self.bitcount) + else: + self.misodata |= miso << self.bitcount self.bitcount += 1 @@ -150,27 +151,30 @@ class Decoder(srd.Decoder): if self.bitcount != ws: return - # Pass MOSI and MISO to the next PD up the stack - self.putpw(['DATA', self.mosidata, self.misodata]) + si = self.mosidata if self.have_mosi else None + so = self.misodata if self.have_miso else None + + # Pass MOSI and MISO to the next PD up the stack. + self.putpw(['DATA', si, so]) - # Annotations - self.putw([0, ['%02X/%02X' % (self.mosidata, self.misodata)]]) - self.putw([1, ['%02X' % self.misodata]]) - self.putw([2, ['%02X' % self.mosidata]]) + # Annotations. + if self.have_miso: + self.putw([0, ['%02X' % self.misodata]]) + if self.have_mosi: + self.putw([1, ['%02X' % self.mosidata]]) - # Meta bitrate + # Meta bitrate. elapsed = 1 / float(self.samplerate) * (self.samplenum - self.startsample + 1) bitrate = int(1 / elapsed * self.options['wordsize']) self.put(self.startsample, self.samplenum, self.out_bitrate, bitrate) - if self.cs_was_deasserted_during_data_word: - self.putw([3, ['CS# was deasserted during this data word!']]) + if self.have_cs and self.cs_was_deasserted_during_data_word: + self.putw([2, ['CS# was deasserted during this data word!']]) # Reset decoder state. - self.mosidata = self.misodata = self.bitcount = 0 - - # Keep stats for summary. - self.bytesreceived += 1 + self.misodata = 0 if self.have_miso else None + self.mosidata = 0 if self.have_mosi else None + self.bitcount = 0 def find_clk_edge(self, miso, mosi, sck, cs): if self.have_cs and self.oldcs != cs: @@ -179,7 +183,9 @@ class Decoder(srd.Decoder): ['CS-CHANGE', self.oldcs, cs]) self.oldcs = cs # Reset decoder state when CS# changes (and the CS# pin is used). - self.mosidata = self.misodata = self.bitcount= 0 + self.misodata = 0 if self.have_miso else None + self.mosidata = 0 if self.have_mosi else None + self.bitcount = 0 # Ignore sample if the clock pin hasn't changed. if sck == self.oldsck: @@ -204,13 +210,15 @@ class Decoder(srd.Decoder): def decode(self, ss, es, data): if self.samplerate is None: raise Exception("Cannot decode without samplerate.") - # TODO: Either MISO or MOSI could be optional. CS# is optional. + # Either MISO or MOSI can be omitted (but not both). CS# is optional. for (self.samplenum, pins) in data: # Ignore identical samples early on (for performance reasons). if self.oldpins == pins: continue - self.oldpins, (miso, mosi, sck, cs) = pins, pins + self.oldpins, (sck, miso, mosi, cs) = pins, pins + self.have_miso = (miso in (0, 1)) + self.have_mosi = (mosi in (0, 1)) self.have_cs = (cs in (0, 1)) # State machine.