[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20180711103129.ql4txni63i2m7m4r@d08157822050>
Date: Wed, 11 Jul 2018 06:31:29 -0400
From: Brian Masney <masneyb@...tation.org>
To: Himanshu Jha <himanshujha199640@...il.com>
Cc: jic23@...nel.org, robh+dt@...nel.org, mark.rutland@....com,
andy.gross@...aro.org, david.brown@...aro.org,
linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-msm@...r.kernel.org,
linux-soc@...r.kernel.org, jonathan@...ek.ca,
jmaneyrol@...ensense.com, knaack.h@....de, lars@...afoo.de,
pmeerw@...erw.net, mkelly@...o.com, fischerdouglasc@...il.com,
bshah@....org, ctatlor97@...il.com
Subject: Re: [PATCH 2/3] iio: imu: mpu6050: add support for regulator
framework
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 <masneyb@...tation.org>
> > Signed-off-by: Jonathan Marek <jonathan@...ek.ca>
> > ---
> > }
> > EXPORT_SYMBOL_GPL(inv_mpu_core_probe);
> >
> > +int inv_mpu_core_remove(struct inv_mpu6050_state *st)
> > +{
> > + return regulator_disable(st->vddio_supply);
> > +}
> EXPORT_SYMBOL_GPL(inv_mpu_core_remove); ?
>
> This is causing 0-day test build failure perhaps.
You are right. It compiles if I have CONFIG_INV_MPU6050_IIO built into
the kernel, which is what I'm using to boot the board, but fails when
compiled as a module. Good catch 0-day!
The patch won't apply to Linus's current master branch and I assumed
that iio/togreg didn't have the required changes as well. Sorry about
the noise.
I'll send a v2 before the weekend.
Brian
Powered by blists - more mailing lists