Re: [PATCH v5 2/2] dt-bindings: iio: st-sensors: add LIS2DS12 accelerometer

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1300 bytes --]

On Sun, Jun 23, 2024 at 11:03:05AM +0100, Jonathan Cameron wrote:
> On Sat, 22 Jun 2024 18:03:46 +0530
> Kaustabh Chakraborty <kauschluss@disroot.org> wrote:
> 
> > LIS2DS12 is an accelerometer by STMicroelectronics. It is identifiable by
> > its WhoAmI value 0x43.
> > 
> > Its register interface is not compatible with existing parts. For example:
> > 
> > - The full-scale values are present in register 0x20, in bits 2 and 3
> >   (mask 0x0c). Most other supported sensors have the register address set
> >   to 0x21, 0x23, 0x24, or 0x25. There is one sensor setting though
> >   (bearing WhoAmI 0x3b) which has it's address set to 0x20, but the mask is
> >   set to 0x20, not 0x0c.
> > 
> > - The full-scale values 2G, 4G, 8G, and 16G correspond to the register
> >   values 0x00, 0x02, 0x03, 0x01 respectively. None of the sensor settings
> >   have the value 0x01 associated with 16G.
> > 
> > Add the compatible string without any fallback.
> > 
> > Signed-off-by: Kaustabh Chakraborty <kauschluss@disroot.org>
> Given the only requested change was this description and everyone seemed
> happy with it in previous thread, I'll pick this series up now.

ye, fine in my books. I should've prob acked explicitly when I assented
to the wording on the previous version.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]