| Message ID | 20220503004156.3559940-2-swboyd@chromium.org (mailing list archive) |
|---|---|
| State | Accepted |
| Headers | show |
| Series | iio: sx9324: Support CS idle mode | expand |
On Mon, May 02, 2022 at 05:41:55PM -0700, Stephen Boyd wrote: > We need to configure the sleep and idle mode of the CS pins for this > device. Add a DT property to do this so pins are in a proper state > during sleep (either hi-z, grounded, or pulled up to vdd). > > Cc: Gwendal Grignou <gwendal@chromium.org> > Signed-off-by: Stephen Boyd <swboyd@chromium.org> > --- > .../devicetree/bindings/iio/proximity/semtech,sx9324.yaml | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml b/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml > index d265eb5258c8..6cec3481c8b2 100644 > --- a/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml > +++ b/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml > @@ -126,6 +126,14 @@ properties: > UINT_MAX (4294967295) represents infinite. Other values > represent 1-1/N. > > + semtech,cs-idle-sleep: > + description: > + State of CS pins during sleep mode and idle time. > + enum: > + - hz I was trying to figure out how Hertz was a pin state. 'hiz' or 'hi-z' instead? > + - gnd > + - vdd > + > semtech,int-comp-resistor: > description: > Internal resistor setting for compensation. > -- > https://chromeos.dev > >
Quoting Rob Herring (2022-05-03 07:18:44) > On Mon, May 02, 2022 at 05:41:55PM -0700, Stephen Boyd wrote: > > We need to configure the sleep and idle mode of the CS pins for this > > device. Add a DT property to do this so pins are in a proper state > > during sleep (either hi-z, grounded, or pulled up to vdd). > > > > Cc: Gwendal Grignou <gwendal@chromium.org> > > Signed-off-by: Stephen Boyd <swboyd@chromium.org> > > --- > > .../devicetree/bindings/iio/proximity/semtech,sx9324.yaml | 8 ++++++++ > > 1 file changed, 8 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml b/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml > > index d265eb5258c8..6cec3481c8b2 100644 > > --- a/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml > > +++ b/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml > > @@ -126,6 +126,14 @@ properties: > > UINT_MAX (4294967295) represents infinite. Other values > > represent 1-1/N. > > > > + semtech,cs-idle-sleep: > > + description: > > + State of CS pins during sleep mode and idle time. > > + enum: > > + - hz > > I was trying to figure out how Hertz was a pin state. 'hiz' or 'hi-z' > instead? Sure. I was following the datasheet that has "HZ" but hi-z is better.
diff --git a/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml b/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml index d265eb5258c8..6cec3481c8b2 100644 --- a/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml +++ b/Documentation/devicetree/bindings/iio/proximity/semtech,sx9324.yaml @@ -126,6 +126,14 @@ properties: UINT_MAX (4294967295) represents infinite. Other values represent 1-1/N. + semtech,cs-idle-sleep: + description: + State of CS pins during sleep mode and idle time. + enum: + - hz + - gnd + - vdd + semtech,int-comp-resistor: description: Internal resistor setting for compensation.
We need to configure the sleep and idle mode of the CS pins for this device. Add a DT property to do this so pins are in a proper state during sleep (either hi-z, grounded, or pulled up to vdd). Cc: Gwendal Grignou <gwendal@chromium.org> Signed-off-by: Stephen Boyd <swboyd@chromium.org> --- .../devicetree/bindings/iio/proximity/semtech,sx9324.yaml | 8 ++++++++ 1 file changed, 8 insertions(+)