Re: [rtc-linux] [PATCH 2/6] mfd: max77620: add core driver for MAX77620/MAX20024

From: Lee Jones
Date: Mon Jan 11 2016 - 00:47:07 EST


On Fri, 08 Jan 2016, Krzysztof Kozlowski wrote:

Thanks for taking the time to review.

> ()2016-01-07 23:38 GMT+09:00 Laxman Dewangan <ldewangan@xxxxxxxxxx>:
> > MAX77620/MAX20024 are Power Management IC from the MAXIM.
> > It supports RTC, multiple GPIOs, multiple DCDC and LDOs,
> > watchdog, clock etc.
> >
> > Add MFD drier to provides common support for accessing the
> > device; additional drivers is developed on respected subsystem
> > in order to use the functionality of the device.
> >
> > Signed-off-by: Laxman Dewangan <ldewangan@xxxxxxxxxx>
> > Signed-off-by: Chaitanya Bandi <bandik@xxxxxxxxxx>
> > Signed-off-by: Mallikarjun Kasoju <mkasoju@xxxxxxxxxx>
> > Tested-by: Venkat Reddy Talla <vreddytalla@xxxxxxxxxx>
>
> The Testing and Reviewed are statements (see SubmittingPatches) so
> they should be made explicitly by people. As this is v1 how they could
> make a public statement so far?

SubmittingPatches bears no mention that Reviewed-by/Tested-by
statements have to be provided on one of the public mailing lists.
These can be provided privately prior to upstream submission v1.

> > ---
> > drivers/mfd/Kconfig | 15 +
> > drivers/mfd/Makefile | 1 +
> > drivers/mfd/max77620.c | 926 +++++++++++++++++++++++++++++++++++++++++++
> > include/linux/mfd/max77620.h | 503 +++++++++++++++++++++++
> > 4 files changed, 1445 insertions(+)
> > create mode 100644 drivers/mfd/max77620.c
> > create mode 100644 include/linux/mfd/max77620.h

[...]

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog