Re: [PATCH 1/1] regulator: max77620: check for valid regulator info

2016-06-28 Thread Mark Brown
On Tue, Jun 28, 2016 at 04:35:56AM +, Venkat Reddy Talla wrote: > > This sounds like it's papering over a bug in the driver - the driver should > > be > > able to instantiate without anything beyond the registration of the device. > > What's the driver relying on from the DT? > Sorry for

Re: [PATCH 1/1] regulator: max77620: check for valid regulator info

2016-06-28 Thread Mark Brown
On Tue, Jun 28, 2016 at 04:35:56AM +, Venkat Reddy Talla wrote: > > This sounds like it's papering over a bug in the driver - the driver should > > be > > able to instantiate without anything beyond the registration of the device. > > What's the driver relying on from the DT? > Sorry for

RE: [PATCH 1/1] regulator: max77620: check for valid regulator info

2016-06-27 Thread Venkat Reddy Talla
> * PGP Signed by an unknown key > > On Mon, Jun 27, 2016 at 05:13:44PM +0530, Venkat Reddy Talla wrote: > > Check for valid regulator information data before configuring FPS > > source and FPS power up/down period to avoid NULL pointer exception if > > entries for PMIC regulators not provided

RE: [PATCH 1/1] regulator: max77620: check for valid regulator info

2016-06-27 Thread Venkat Reddy Talla
> * PGP Signed by an unknown key > > On Mon, Jun 27, 2016 at 05:13:44PM +0530, Venkat Reddy Talla wrote: > > Check for valid regulator information data before configuring FPS > > source and FPS power up/down period to avoid NULL pointer exception if > > entries for PMIC regulators not provided

Re: [PATCH 1/1] regulator: max77620: check for valid regulator info

2016-06-27 Thread Mark Brown
On Mon, Jun 27, 2016 at 05:13:44PM +0530, Venkat Reddy Talla wrote: > Check for valid regulator information data before > configuring FPS source and FPS power up/down > period to avoid NULL pointer exception if entries for > PMIC regulators not provided through device tree. This sounds like it's

Re: [PATCH 1/1] regulator: max77620: check for valid regulator info

2016-06-27 Thread Mark Brown
On Mon, Jun 27, 2016 at 05:13:44PM +0530, Venkat Reddy Talla wrote: > Check for valid regulator information data before > configuring FPS source and FPS power up/down > period to avoid NULL pointer exception if entries for > PMIC regulators not provided through device tree. This sounds like it's

[PATCH 1/1] regulator: max77620: check for valid regulator info

2016-06-27 Thread Venkat Reddy Talla
Check for valid regulator information data before configuring FPS source and FPS power up/down period to avoid NULL pointer exception if entries for PMIC regulators not provided through device tree. SD4 regulator is not supported by MAX77620 PMIC, removing SD4 entry from regulator information

[PATCH 1/1] regulator: max77620: check for valid regulator info

2016-06-27 Thread Venkat Reddy Talla
Check for valid regulator information data before configuring FPS source and FPS power up/down period to avoid NULL pointer exception if entries for PMIC regulators not provided through device tree. SD4 regulator is not supported by MAX77620 PMIC, removing SD4 entry from regulator information