[PATCH V2 1/4] i2c: tegra: Add __devinit/exit to probe/remove

Stephen Warren swarren at nvidia.com
Sat Feb 4 11:05:49 EST 2012


Olof Johansson wrote at Friday, January 06, 2012 10:45 AM:
> On Fri, Jan 6, 2012 at 8:47 AM, Stephen Warren <swarren at nvidia.com> wrote:
> > Ben Dooks wrote at Friday, January 06, 2012 9:26 AM:
> >> On Tue, Dec 20, 2011 at 01:47:29PM -0800, Stephen Warren wrote:
> >> > Shubhrajyoti Datta wrote at Monday, December 19, 2011 11:15 PM:
> >> > ...
> >> > > Hi Stephen ,
> >> > > could you check this one also
> >> > > thanks.
> >> > >
> >> > > From: Shubhrajyoti D <shubhrajyoti at ti.com>
> >> > > Date: Tue, 20 Dec 2011 11:40:51 +0530
> >> > > Subject: [PATCH] I2C : tegra : devexit in remove
> >> > >
> >> > > Adding I2C __devexxit in remove
> >> > >
> >> > > Cc: Stephen Warren <swarren at nvidia.com>
> >> > > Signed-off-by: Shubhrajyoti D <shubhrajyoti at ti.com>
> >> >
> >> > Acked-by: Stephen Warren <swarren at nvidia.com>
> >> >
> >> > Ben or Olof,
> >> >
> >> > > -       .remove  = tegra_i2c_remove,
> >> > > +       .remove  =__devexit_p(tegra_i2c_remove),
> >> >
> >> > When you apply this, can you fix it up to add a space after the = in
> >> > the modified line? If you want, I can post an updated version to do that.
> >>
> >> Let me know if you want anything applying or a pull-request processed.
> >>
> >> I think once fixed the patches would be fine and I ack them:
> >>
> >> Acked-by: Ben Dooks <ben-linux at fluff.org>
> >
> > Olof, since you took the original through the Tegra tree, I think it
> > makes sense for us to take this fix through there too, and for 3.3?
> > The patch above is http://patchwork.ozlabs.org/patch/132349/
> 
> Yep, will do.

I haven't seen this show up anywhere yet.

-- 
nvpublic



More information about the devicetree-discuss mailing list