Olof Johansson wrote at Friday, January 06, 2012 10:45 AM: > On Fri, Jan 6, 2012 at 8:47 AM, Stephen Warren <swarren@xxxxxxxxxx> 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@xxxxxx> > >> > > 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@xxxxxxxxxx> > >> > > Signed-off-by: Shubhrajyoti D <shubhrajyoti@xxxxxx> > >> > > >> > Acked-by: Stephen Warren <swarren@xxxxxxxxxx> > >> > > >> > 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@xxxxxxxxx> > > > > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html