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

On Tuesday 23 May 2017 02:15 PM, Lee Jones wrote:

On Tue, 23 May 2017, Keerthy wrote:

quoted

On Tuesday 23 May 2017 12:27 PM, Lee Jones wrote:

quoted

On Tue, 23 May 2017, Keerthy wrote:

quoted

On Monday 22 May 2017 11:18 PM, Lee Jones wrote:

quoted

On Fri, 19 May 2017, Keerthy wrote:

quoted

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@ti.com>
---
 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

[...]

quoted

quoted

quoted

+/*
+ * Copyright (C) 2017 Texas Instruments Incorporated - http://www.ti.com/
+ *
+ * Author: Keerthy [off-list ref]

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.

Okay.

quoted

quoted

[...]

quoted

quoted

quoted

+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.

Okay, then you don't need this table and you should depend on OF in
the Kconfig file.

Okay. Cool thanks for this info.

I am seeing this commit: 4895e493f91bff7f96f328b883c7f6448aa6abc6

mfd: Export OF module alias information in missing drivers

So no need of i2c_device_id for any of the OF-only mfd drivers?