Inter-revision diff: cover letter

Comparing v4 (message) to v6 (message)

--- v4
+++ v6
@@ -2,17 +2,30 @@
 exactly when peer master sends data to this driver's slave so a
 case can be happened that this master tries to send data through
 the master_xfer function but slave data from peer master is still
-being processed by this driver.
+being processed by this driver. To prevent state corruption in the
+case, this patch adds checking code if any slave operation is
+ongoing and it waits up to the bus timeout duration before starting
+a master_xfer operation.
 
-To prevent state corruption in the case, this patch adds checking
-code if any slave operation is ongoing and it waits up to the
-timeout duration before starting a master_xfer operation.
+To support this change, it introduces changes on i2c-core-base to
+make that able to read the bus timeout and master transfer retries
+count values from device tree properties.
 
 Please review this patch set.
 
 Thanks,
 
 -Jae
+
+Changes since v5:
+- Changed using of property reading API to device_property_read_u32.
+
+Changes since v4:
+- Moved the property reading code into i2c-base-core and changed the
+  property name to 'timeout-ms'. Also, added '#retries' property reading
+  code.
+- Changed bus busy checking logic to make that check slave_state instead
+  of 'Transfer Mode State Machine' reg value.
 
 Changes since v3:
 - Changed the property name to 'timeout' and made it use the
@@ -25,15 +38,20 @@
 Changes since v1:
 - Changed define names of timeout related.
 
-Jae Hyun Yoo (3):
-  dt-bindings: i2c: aspeed: Add 'timeout' property as an optional
+Jae Hyun Yoo (5):
+  dt-bindings: i2c: Add 'timeout-ms' and '#retries' properties as common
+    optional
+  i2c: Add support reading of 'timeout-ms' and '#retries' properties
+  dt-bindings: i2c: aspeed: Add 'timeout-ms' property as an optional
     property
-  i2c: aspeed: Add 'timeout' DT property reading code
+  i2c: aspeed: Remove hard-coded bus timeout value setting
   i2c: aspeed: Add bus idle waiting logic for multi-master use cases
 
- .../devicetree/bindings/i2c/i2c-aspeed.txt    |  3 +
- drivers/i2c/busses/i2c-aspeed.c               | 70 ++++++++++++++-----
- 2 files changed, 57 insertions(+), 16 deletions(-)
+ .../devicetree/bindings/i2c/i2c-aspeed.txt    |  3 ++
+ Documentation/devicetree/bindings/i2c/i2c.txt |  6 +++
+ drivers/i2c/busses/i2c-aspeed.c               | 54 +++++++++++++------
+ drivers/i2c/i2c-core-base.c                   | 10 +++-
+ 4 files changed, 55 insertions(+), 18 deletions(-)
 
 -- 
 2.19.0
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help