On 09/02/2015 01:45 AM, Jean Delvare wrote:
On Mon, 31 Aug 2015 16:57:33 -0700, Guenter Roeck wrote:
NCT6793D is register compatible with NCT6792D.
Tested-by: Grazvydas Ignotas <notasas@xxxxxxxxx>
Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>
Documentation/hwmon/nct6775 | 4 ++++
drivers/hwmon/Kconfig | 4 ++--
drivers/hwmon/nct6775.c | 29 ++++++++++++++++++++++-------
3 files changed, 28 insertions(+), 9 deletions(-)
Looks good to me (didn't check the datasheet, trusting you on the fact
I only cound one difference: The "3-wire enable" register (index 0x31 in
various banks) no longer exists. It isn't used by the driver, so no change
was needed. Of course, I may have missed some swapped registers ...
that the chips are compatible) except that you forgot to add a name
string for the new chip to nct6775_sio_names.
With that fixed:
Reviewed-by: Jean Delvare <jdelvare@xxxxxxx>
Note: given that enum kinds is used to index nct6775_sio_names, it
would feel safer to have nct6775_sio_names defined closer to enum
kinds, as nct6775_device_names is already.
I did that.
Thanks a lot for the review.
lm-sensors mailing list