Re: [PATCH] hwmon: (jc42) Add support for additional IDT temperature sensors

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

 



Hi Guenter,

Sorry for the late review.

On Thu, 12 Feb 2015 11:26:12 -0800, Guenter Roeck wrote:
> TS3000GB0 has a new device ID (0x2913). Since IDT's datasheets suggest
> that the upper 8 bit of the device ID reflect the chip ID and the lower
> 8 bit reflect the version number, modify the code to accept all chips
> with ID 0x29xx.

You did that for all other IDT chips as well. While this is in
accordance with the datasheets, this might lead to some false
positives, as the driver probes 8 popular I2C addressed and the
detection code doesn't check for that many bits anymore.

That being said I don't have a strong feeling about it. We can leave it
this way for now and revisit it later if false positives are actually
reported.

Reviewed-by: Jean Delvare <jdelvare@xxxxxxx>

What I would like though is that the detection code in sensors-detect
is updated to be in line with what the driver does now. It gets
confusing if sensors-detect finds a chip which the driver rejects or
misses a chip which the driver would support.

> Also add support for TS3001 and TSE2004.

That needs to be done in sensors-detect as well.

All these chips could be added to the Devices page in the wiki too.

> Some of the datasheets for older chips are no longer available from
> the IDT web site, so replace explicit links in the documentation with
> a generic note.
> 
> Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>
> ---
>  Documentation/hwmon/jc42 |  8 +++-----
>  drivers/hwmon/Kconfig    |  2 +-
>  drivers/hwmon/jc42.c     | 16 ++++++++++------
>  3 files changed, 14 insertions(+), 12 deletions(-)
> (...)

Thanks,
-- 
Jean Delvare
SUSE L3 Support

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors




[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux