On Sat, 17 Nov 2018 09:37:54 -0600 Rob Herring <robh@xxxxxxxxxx> wrote: > On Sun, Nov 11, 2018 at 03:15:36PM +0100, Lorenzo Bianconi wrote: > > Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@xxxxxxxxxx> > > --- > > Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt b/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt > > index 879322ad50fd..69d53d98d0f0 100644 > > --- a/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt > > +++ b/Documentation/devicetree/bindings/iio/imu/st_lsm6dsx.txt > > @@ -13,6 +13,7 @@ Required properties: > > Optional properties: > > - st,drdy-int-pin: the pin on the package that will be used to signal > > "data ready" (valid values: 1 or 2). > > +- st,pullups : enable/disable internal i2c controller pullup resistors. > > bias-pull-up is the standard property for this. The problem here is 'of what?'. Perhaps it wasn't clear in the current binding either, but we are talking the pull ups on the auxilliary i2c bus (which doesn't really have a direct representation in DT at all). If we just go with bias-pull-up my instinct would be that it was applying to the drdy pin for example. Lorenzo, could you send a follow up patch fixing this once we have it pinned down? We have plenty of time to get it in before the end of the cycle, so I would rather do that than revert this series over it. Conversely I don't want a binding we are still discussing getting into a release. I should have given Rob time to get to this. Lots of precedence doesn't mean there isn't a better way! Sorry All. Thanks, Jonathan > > > - drive-open-drain: the interrupt/data ready line will be configured > > as open drain, which is useful if several sensors share the same > > interrupt line. This is a boolean property. > > -- > > 2.19.1 > >
