Re: [PATCH] lm85: extend to support emc6d103 chips

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

 



Hallo Jan,

On Fri, 18 Feb 2011 08:18:26 +0000, Jan Beulich wrote:
> The interface is identical emc6d102, so all that needs to be added are
> some definitions and their uses.
> 
> Registers apparently missing in emc6d103:A2 compared to emc6d103:A0,
> emc6d103:A1, and emc6d102 (according to the data sheets), but used
> unconditionally in the driver: 62[5:7], 6D[0:7], and 6E[0:7]. For that
> reason, A2 stepping chips don't get enabled for the time being.

The EMC6D103 datasheet on smsc.com doesn't mention revision A2, so I'm
curious where you got this information?

BTW, could you please provide a dump of your chip? I'd like to add it
to my collection. Use i2cdump with the i2c-dev kernel driver.

> 
> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxxxx>
> 
> ---
>  drivers/hwmon/Kconfig |    2 +-
>  drivers/hwmon/lm85.c  |   20 ++++++++++++++++++--
>  2 files changed, 19 insertions(+), 3 deletions(-)

Sorry for being a little late in the game, but can you please also
update Documentation/hwmon/lm85?

The patch looks good otherwise.

-- 
Jean Delvare

_______________________________________________
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