Re: [PATCH v3 1/3] mfd: da9063: Add HWMON dependencies

From: Lee Jones
Date: Thu Aug 05 2021 - 07:40:22 EST


On Wed, 07 Jul 2021, Vincent Pelletier wrote:

> From: "Opensource [Steve Twiss]" <stwiss.opensource@xxxxxxxxxxx>
>
> Dependencies required for DA9063 HWMON support.
>
> Signed-off-by: Opensource [Steve Twiss] <stwiss.opensource@xxxxxxxxxxx>
>
> Moved temperature offset reading to hwmon driver.
>
> Signed-off-by: Vincent Pelletier <plr.vincent@xxxxxxxxx>


Nit: Please remove the empty lines between tags.

And format like this please:

[vincent: Moved temperature offset reading to hwmon driver]

> ---
> Changes in v3:
> - moved temperature offset reading to hwmon driver
>
> Changes in v2:
> - registers.h changes moved from patch 2
>
> Originally submitted by Steve Twiss in 2014:
> https://marc.info/?l=linux-kernel&m=139560864709852&w=2
>
> include/linux/mfd/da9063/registers.h | 34 ++++++++++++++++++++++++++++
> 1 file changed, 34 insertions(+)

Once fixed:

Acked-by: Lee Jones <lee.jones@xxxxxxxxxx>

--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog