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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/17/23 13:44, Conor Dooley wrote:
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.


Hmm, yes, that would have been a duplicate. If iio support for lm75
compatible sensors is needed for some reason, a hwmon -> iio bridge
would make more sense.

Thanks,
Guenter


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.





[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux