From: Uwe Hermann <redacted>
Date: Sun, 4 Mar 2012 09:55:30 +0000 (+0100)
Subject: srd: I2C: Update PD protocol docs.
X-Git-Tag: libsigrokdecode-0.1.0~35
X-Git-Url: https://sigrok.org/gitaction?a=commitdiff_plain;h=f9d87f641e55cd643037d81d7221ddc169cdae6b;p=libsigrokdecode.git

srd: I2C: Update PD protocol docs.
---

diff --git a/decoders/i2c/__init__.py b/decoders/i2c/__init__.py
index 3c29aa1..5216e01 100644
--- a/decoders/i2c/__init__.py
+++ b/decoders/i2c/__init__.py
@@ -59,21 +59,24 @@ http://en.wikipedia.org/wiki/I2C
 Protocol output format:
 
 I2C packet:
-[<i2c_command>, <data>, <ack_bit>]
+[<cmd>, <data>]
 
-<i2c_command> is one of:
+<cmd> is one of:
   - 'START' (START condition)
-  - 'START REPEAT' (Repeated START)
-  - 'ADDRESS READ' (Address, read)
-  - 'ADDRESS WRITE' (Address, write)
+  - 'START REPEAT' (Repeated START condition)
+  - 'ADDRESS READ' (Slave address, read)
+  - 'ADDRESS WRITE' (Slave address, write)
   - 'DATA READ' (Data, read)
   - 'DATA WRITE' (Data, write)
   - 'STOP' (STOP condition)
+  - 'ACK' (ACK bit)
+  - 'NACK' (NACK bit)
 
 <data> is the data or address byte associated with the 'ADDRESS*' and 'DATA*'
-command. For 'START', 'START REPEAT' and 'STOP', this is None.
+command. Slave addresses do not include bit 0 (the READ/WRITE indication bit).
+For example, a slave address field could be 0x51 (instead of 0xa2).
+For 'START', 'START REPEAT', 'STOP', 'ACK', and 'NACK' <data> is None.
 
-<ack_bit> is either 'ACK' or 'NACK', but may also be None.
 '''
 
 from .i2c import *