| Message ID | 20181005214507.26315-4-jae.hyun.yoo@linux.intel.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | i2c: aspeed: Add bus idle waiting logic for multi-master use cases | expand |
On Fri, Oct 5, 2018 at 2:45 PM Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com> wrote: > > This commit adds 'bus-timeout-ms' property as an optional property > which can be used for setting the bus timeout value of an adapter. > With this patch, the bus timeout value can be set through this > property at the probing time of this module. Still the bus timeout > value can be set by an I2C_TIMEOUT ioctl on cdev at runtime too. > > Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com> Reviewed-by: Brendan Higgins <brendanhiggins@google.com> Thanks for putting all the work in to do it this way!
On 10/9/2018 5:00 PM, Brendan Higgins wrote: > > Reviewed-by: Brendan Higgins <brendanhiggins@google.com> > > Thanks for putting all the work in to do it this way! > Thanks a lot for your review, Brendan!
On Fri, 5 Oct 2018 14:45:05 -0700, Jae Hyun Yoo wrote: > This commit adds 'bus-timeout-ms' property as an optional property > which can be used for setting the bus timeout value of an adapter. > With this patch, the bus timeout value can be set through this > property at the probing time of this module. Still the bus timeout > value can be set by an I2C_TIMEOUT ioctl on cdev at runtime too. > > Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com> > --- > Documentation/devicetree/bindings/i2c/i2c-aspeed.txt | 3 +++ > 1 file changed, 3 insertions(+) > Reviewed-by: Rob Herring <robh@kernel.org>
On 10/11/2018 3:12 PM, Rob Herring wrote: > > Reviewed-by: Rob Herring <robh@kernel.org> > Thanks much for your review, Rob! -Jae
diff --git a/Documentation/devicetree/bindings/i2c/i2c-aspeed.txt b/Documentation/devicetree/bindings/i2c/i2c-aspeed.txt index 8fbd8633a387..ce1f07620368 100644 --- a/Documentation/devicetree/bindings/i2c/i2c-aspeed.txt +++ b/Documentation/devicetree/bindings/i2c/i2c-aspeed.txt @@ -17,6 +17,9 @@ Optional Properties: specified - multi-master : states that there is another master active on this bus. +- bus-timeout-ms: bus timeout in milliseconds defaults to 1 second when not + specified. + Example: i2c {
This commit adds 'bus-timeout-ms' property as an optional property which can be used for setting the bus timeout value of an adapter. With this patch, the bus timeout value can be set through this property at the probing time of this module. Still the bus timeout value can be set by an I2C_TIMEOUT ioctl on cdev at runtime too. Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@linux.intel.com> --- Documentation/devicetree/bindings/i2c/i2c-aspeed.txt | 3 +++ 1 file changed, 3 insertions(+)