On Wed, 31 Aug 2016, Keerthy wrote:

On Wednesday 31 August 2016 01:26 PM, Lee Jones wrote:

quoted

On Fri, 12 Aug 2016, Keerthy wrote:

quoted

The LP873X chip is a power management IC for Portable Navigation Systems
    and Tablet Computing devices. It contains the following components:

     - Regulators.
     - Configurable General Purpose Output Signals(GPO).

[...]

quoted

quoted

+static const struct i2c_device_id lp873x_id_table[] = {
+	{ "lp873x", LP873X },
+	{ },
+};
+MODULE_DEVICE_TABLE(i2c, lp873x_id_table);

Don't set the ID in this table.

I did not understand this. Could you please elaborate?
You mean no need of LP873X can just be a '0' as i am seeing in other mfd
drivers?

Just leave it blank.  No need for the second attribute.

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