Re: [PATCH 1/2] dt-bindings: hwmon: Add AMS AS6200 temperature sensor

From: Conor Dooley
Date: Sun Dec 17 2023 - 16:44:52 EST


On Sun, Dec 17, 2023 at 01:39:37PM -0800, Guenter Roeck wrote:
> On 12/17/23 12:58, Conor Dooley wrote:
> > On Sat, Dec 16, 2023 at 11:39:29AM -0500, Abdel Alkuor wrote:
> > > as6200 is a temperature sensor with a range between -40°C to
> > > 125°C degrees and an accuracy of ±0.4°C degree between 0
> > > and 65°C and ±1°C for the other ranges.
> > >
> > > Signed-off-by: Abdel Alkuor <alkuor@xxxxxxxxx>
> >
> > Is this not v3?
>
> FWIW, I don't recall seeing it.

Ah, I think this was originally submitted to iio, went through 2
iterations and then ended up here on Jonathan's advice.

>
> > Either way,
> > Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> >
>
> As I pointed out in my reply to the other patch, this chip is mostly
> compatible to lm75 and almost register-compatible to tmp112, the only
> difference being some configuration register bits.
>
> Bindings for this chip should be added to
> Documentation/devicetree/bindings/hwmon/lm75.yaml.

Ah. In that case, I would expect my tag not to be picked up by Abdel.

Attachment: signature.asc
Description: PGP signature