[PATCH i2c-next v7 2/5] i2c: core: Add support reading of 'bus-timeout-ms' and '#retries' properties

2018-10-05 Thread Jae Hyun Yoo
This commit adds support for 'bus-timeout-ms' and '#retries'
properties to set 'timeout' and 'retries' values in
'struct i2c_adapter' in case an adapter node has the properties.
Still the values can be set by I2C_TIMEOUT and I2C_RETRIES ioctls
on cdev at runtime too.

These properties may not be supported by all drivers. However, if
a driver wants to support one of them, it should adapt the
bindings in the dt-bindings document.

Signed-off-by: Jae Hyun Yoo 
---
 drivers/i2c/i2c-core-base.c | 12 ++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/drivers/i2c/i2c-core-base.c b/drivers/i2c/i2c-core-base.c
index 799776c6d421..aa2a365d374a 100644
--- a/drivers/i2c/i2c-core-base.c
+++ b/drivers/i2c/i2c-core-base.c
@@ -1214,6 +1214,7 @@ EXPORT_SYMBOL_GPL(i2c_handle_smbus_host_notify);
 static int i2c_register_adapter(struct i2c_adapter *adap)
 {
int res = -EINVAL;
+   u32 bus_timeout_ms = 0;
 
/* Can't register until after driver model init */
if (WARN_ON(!is_registered)) {
@@ -1239,8 +1240,15 @@ static int i2c_register_adapter(struct i2c_adapter *adap)
INIT_LIST_HEAD(>userspace_clients);
 
/* Set default timeout to 1 second if not already set */
-   if (adap->timeout == 0)
-   adap->timeout = HZ;
+   if (adap->timeout == 0) {
+   device_property_read_u32(>dev, "bus-timeout-ms",
+_timeout_ms);
+   adap->timeout = bus_timeout_ms ?
+   msecs_to_jiffies(bus_timeout_ms) : HZ;
+   }
+
+   /* Set retries count if it has the property setting */
+   device_property_read_u32(>dev, "#retries", >retries);
 
/* register soft irqs for Host Notify */
res = i2c_setup_host_notify_irq_domain(adap);
-- 
2.19.0



[PATCH i2c-next v7 2/5] i2c: core: Add support reading of 'bus-timeout-ms' and '#retries' properties

2018-10-05 Thread Jae Hyun Yoo
This commit adds support for 'bus-timeout-ms' and '#retries'
properties to set 'timeout' and 'retries' values in
'struct i2c_adapter' in case an adapter node has the properties.
Still the values can be set by I2C_TIMEOUT and I2C_RETRIES ioctls
on cdev at runtime too.

These properties may not be supported by all drivers. However, if
a driver wants to support one of them, it should adapt the
bindings in the dt-bindings document.

Signed-off-by: Jae Hyun Yoo 
---
 drivers/i2c/i2c-core-base.c | 12 ++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/drivers/i2c/i2c-core-base.c b/drivers/i2c/i2c-core-base.c
index 799776c6d421..aa2a365d374a 100644
--- a/drivers/i2c/i2c-core-base.c
+++ b/drivers/i2c/i2c-core-base.c
@@ -1214,6 +1214,7 @@ EXPORT_SYMBOL_GPL(i2c_handle_smbus_host_notify);
 static int i2c_register_adapter(struct i2c_adapter *adap)
 {
int res = -EINVAL;
+   u32 bus_timeout_ms = 0;
 
/* Can't register until after driver model init */
if (WARN_ON(!is_registered)) {
@@ -1239,8 +1240,15 @@ static int i2c_register_adapter(struct i2c_adapter *adap)
INIT_LIST_HEAD(>userspace_clients);
 
/* Set default timeout to 1 second if not already set */
-   if (adap->timeout == 0)
-   adap->timeout = HZ;
+   if (adap->timeout == 0) {
+   device_property_read_u32(>dev, "bus-timeout-ms",
+_timeout_ms);
+   adap->timeout = bus_timeout_ms ?
+   msecs_to_jiffies(bus_timeout_ms) : HZ;
+   }
+
+   /* Set retries count if it has the property setting */
+   device_property_read_u32(>dev, "#retries", >retries);
 
/* register soft irqs for Host Notify */
res = i2c_setup_host_notify_irq_domain(adap);
-- 
2.19.0