]> sigrok.org Git - libsigrokdecode.git/commitdiff
sle44xx: minor developer comment and style nits
authorGerhard Sittig <redacted>
Tue, 18 Aug 2020 19:05:17 +0000 (21:05 +0200)
committerGerhard Sittig <redacted>
Sun, 30 Aug 2020 05:23:58 +0000 (07:23 +0200)
These are just nits, behaviour remains. The comment may be incorrect
how command and data modes are detected. Add a developer comment for
later revisit. Drop unneeded parentheses in a ternary operation, and
move an ATR related comment around.

decoders/sle44xx/pd.py

index f84bea65efd4d52608417a2b82f778b718c1d797..775ee3c8a8fb78c9d29aa826a683e480e5da1885 100644 (file)
@@ -105,14 +105,16 @@ class Decoder(srd.Decoder):
         cls, texts = lookup_proto_ann_txt(self.cmd, {})
         self.putx([cls, texts])
         self.bits = []
-        self.cmd = 'ATR' # Next data bytes will be ATR
+        # Next data bytes will be Answer To Reset.
+        self.cmd = 'ATR'
 
     def handle_command(self, pins):
         rst, clk, io = pins
         self.ss, self.es = self.samplenum, self.samplenum
+        # XXX Is the comment inverted?
         # If I/O is rising -> command START
         # if I/O is falling -> command STOP and response data incoming
-        self.cmd = 'CMD' if (io == 0) else 'DATA'
+        self.cmd = 'CMD' if io == 0 else 'DATA'
         self.bits = []
 
     # Gather 8 bits of data