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

[Keerthy]

On Tuesday 23 May 2017 12:27 PM, Lee Jones wrote:
> 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@xxxxxx>
>>>> ---
>>>>  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@xxxxxx>
>>>
>>> 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?

Father's initials.

> [...]
> 
>>>> +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?
> 

Sorry i replied on top again. It will be an OF only driver.

Regards,
Keerthy