Re: [PATCH v3 2/3] drivers: hwmon: Add W83773G driver

2017-11-12 Thread Lei YU
On Fri, Nov 10, 2017 at 10:49 PM, Guenter Roeck wrote: > On 11/08/2017 11:09 PM, Lei YU wrote: >> >> Nuvoton W83773G is a hardware monitor IC providing one local >> temperature and two remote temperature sensors. >> >> Signed-off-by: Lei YU > > > Nicely done. Couple of nitpicks left. > > Thanks!

Re: [PATCH v3 2/3] drivers: hwmon: Add W83773G driver

2017-11-10 Thread Guenter Roeck
On 11/08/2017 11:09 PM, Lei YU wrote: Nuvoton W83773G is a hardware monitor IC providing one local temperature and two remote temperature sensors. Signed-off-by: Lei YU Nicely done. Couple of nitpicks left. Thanks! Guenter --- v2: - Rewrite the driver using regmap - Add offset and upda

[PATCH v3 2/3] drivers: hwmon: Add W83773G driver

2017-11-08 Thread Lei YU
Nuvoton W83773G is a hardware monitor IC providing one local temperature and two remote temperature sensors. Signed-off-by: Lei YU --- v2: - Rewrite the driver using regmap - Add offset and update_interval v3: - Use devm_hwmon_device_register_with_info() with is_visible/read/write functions