Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-12 Thread Jonathan Cameron
On Wed, 11 Jul 2018 09:00:07 -0400 Brian Masney wrote: > On Wed, Jul 11, 2018 at 12:29:33PM +, Jean-Baptiste Maneyrol wrote: > > Hello, > > > > I really don't like the idea to have regulator handled inside the > > driver. I know this was done like that before for Nexus 5, but I > > think

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-12 Thread Jonathan Cameron
On Wed, 11 Jul 2018 09:00:07 -0400 Brian Masney wrote: > On Wed, Jul 11, 2018 at 12:29:33PM +, Jean-Baptiste Maneyrol wrote: > > Hello, > > > > I really don't like the idea to have regulator handled inside the > > driver. I know this was done like that before for Nexus 5, but I > > think

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread Brian Masney
On Wed, Jul 11, 2018 at 12:29:33PM +, Jean-Baptiste Maneyrol wrote: > Hello, > > I really don't like the idea to have regulator handled inside the driver. I > know this was done like that before for Nexus 5, but I think now this is > something that can be done using dts only. > Does anyone

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread Brian Masney
On Wed, Jul 11, 2018 at 12:29:33PM +, Jean-Baptiste Maneyrol wrote: > Hello, > > I really don't like the idea to have regulator handled inside the driver. I > know this was done like that before for Nexus 5, but I think now this is > something that can be done using dts only. > Does anyone

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread Jean-Baptiste Maneyrol
Hello, I really don't like the idea to have regulator handled inside the driver. I know this was done like that before for Nexus 5, but I think now this is something that can be done using dts only. Does anyone know if there is a way with dts to handle regulator automatically and prevent the

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread Jean-Baptiste Maneyrol
Hello, I really don't like the idea to have regulator handled inside the driver. I know this was done like that before for Nexus 5, but I think now this is something that can be done using dts only. Does anyone know if there is a way with dts to handle regulator automatically and prevent the

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread Brian Masney
On Wed, Jul 11, 2018 at 03:34:43PM +0530, Himanshu Jha wrote: > Hi Brain, > > On Tue, Jul 10, 2018 at 09:09:31PM -0400, Brian Masney wrote: > > This patch adds support for the regulator framework to the mpu6050 > > driver. > > > > Signed-off-by: Brian Masney > > Signed-off-by: Jonathan Marek >

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread Brian Masney
On Wed, Jul 11, 2018 at 03:34:43PM +0530, Himanshu Jha wrote: > Hi Brain, > > On Tue, Jul 10, 2018 at 09:09:31PM -0400, Brian Masney wrote: > > This patch adds support for the regulator framework to the mpu6050 > > driver. > > > > Signed-off-by: Brian Masney > > Signed-off-by: Jonathan Marek >

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread Himanshu Jha
Hi Brain, On Tue, Jul 10, 2018 at 09:09:31PM -0400, Brian Masney wrote: > This patch adds support for the regulator framework to the mpu6050 > driver. > > Signed-off-by: Brian Masney > Signed-off-by: Jonathan Marek > --- > } > EXPORT_SYMBOL_GPL(inv_mpu_core_probe); > > +int

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread Himanshu Jha
Hi Brain, On Tue, Jul 10, 2018 at 09:09:31PM -0400, Brian Masney wrote: > This patch adds support for the regulator framework to the mpu6050 > driver. > > Signed-off-by: Brian Masney > Signed-off-by: Jonathan Marek > --- > } > EXPORT_SYMBOL_GPL(inv_mpu_core_probe); > > +int

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread Brian Masney
On Wed, Jul 11, 2018 at 04:50:22PM +0800, kbuild test robot wrote: > Hi Brian, > > Thank you for the patch! Yet something to improve: > > [auto build test ERROR on iio/togreg] > [also build test ERROR on next-20180710] > [cannot apply to v4.18-rc4] > [if your patch is applied to the wrong git

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread Brian Masney
On Wed, Jul 11, 2018 at 04:50:22PM +0800, kbuild test robot wrote: > Hi Brian, > > Thank you for the patch! Yet something to improve: > > [auto build test ERROR on iio/togreg] > [also build test ERROR on next-20180710] > [cannot apply to v4.18-rc4] > [if your patch is applied to the wrong git

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread kbuild test robot
Hi Brian, Thank you for the patch! Yet something to improve: [auto build test ERROR on iio/togreg] [also build test ERROR on next-20180710] [cannot apply to v4.18-rc4] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator framework

2018-07-11 Thread kbuild test robot
Hi Brian, Thank you for the patch! Yet something to improve: [auto build test ERROR on iio/togreg] [also build test ERROR on next-20180710] [cannot apply to v4.18-rc4] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: