Re: [PATCH i2c-next v9 1/5] dt-bindings: i2c: Add 'bus-timeout-ms' and '#retries' properties as common optional

[Jae Hyun Yoo]

On 11/28/2018 3:54 PM, Wolfram Sang wrote:
> On Tue, Oct 30, 2018 at 02:09:12PM -0700, Jae Hyun Yoo wrote:
> > This commit adds 'bus-timeout-ms' and '#retries' properties as
> > common optional properties that can be used for setting 'timeout'
> > and 'retries' values of 'struct i2c_adapter'. With this patch, the
> > bus timeout value and the master transfer retries count can be set
> > through these properties at the registration time of an adapter.
> > 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 this document.
> >
> > Signed-off-by: Jae Hyun Yoo <jae.hyun.yoo@xxxxxxxxxxxxxxx>
> > Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
>
> Definately NACK on the #retries part. This is purely a configuration
> thing. Besides that it is kind of ancient cruft, too. I don't recommend
> it, at all.
>
> On the timeout thing, I am still not fully convinced that this is a HW
> description. I said that before. But maybe I need to understand your
> problem case better.

Okay, I got it. I'll leave those as configuration things. Will remove
this part in the next version.

Thanks for your comment.

Jae