Re: [PATCH 1/3] mfd: lp87565: Add lp87565 PMIC support

Lee Jones
2017-05-23T06:57:42Z

On Tue, 23 May 2017, Keerthy wrote:
> On Monday 22 May 2017 11:18 PM, Lee Jones wrote:
> > On Fri, 19 May 2017, Keerthy wrote:
> >
> >> The LP87565 chip is a power management IC for Portable Navigation Systems
> >> and Tablet Computing devices. It contains the following components:
> >>
> >> - Configurable Bucks(Single and multi-phase).
> >> - Configurable General Purpose Output Signals (GPO).
> >>
> >> The LP87565-Q1 variant device uses two 2-phase outputs configuration,
> >> Buck0 is master for Buck0/1 output and Buck2 is master for Buck2/3
> >> output.
> >>
> >> Signed-off-by: Keerthy <j-keerthy-l0cyMroinI0@public.gmane.org>
> >> ---
> >> Documentation/devicetree/bindings/mfd/lp87565.txt | 44 ++++
> >> drivers/mfd/Kconfig | 14 ++
> >> drivers/mfd/Makefile | 1 +
> >> drivers/mfd/lp87565.c | 103 ++++++++
> >> include/linux/mfd/lp87565.h | 275 ++++++++++++++++++++++
> >> 5 files changed, 437 insertions(+)
> >> create mode 100644 Documentation/devicetree/bindings/mfd/lp87565.txt
> >> create mode 100644 drivers/mfd/lp87565.c
> >> create mode 100644 include/linux/mfd/lp87565.h
> >>

[...]

> >> +/*
> >> + * Copyright (C) 2017 Texas Instruments Incorporated - http://www.ti.com/
> >> + *
> >> + * Author: Keerthy <j-keerthy-l0cyMroinI0@public.gmane.org>
> >
> > We usually ask for full "first last" names.
>
> Keerthy is my full name and i do not have a surname.

What is the 'J' for in your email address?
[...]

> >> +static const struct i2c_device_id lp87565_id_table[] = {
> >> + { "lp87565-q1", 0 },
> >> + { },
> >> +};
> >> +MODULE_DEVICE_TABLE(i2c, lp87565_id_table);
> >
> > Is this an OF only driver, or not?

You didn't answer my question?

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