Re: [PATCH 056/493] edac: remove use of __devexit_p

2012-11-24 Thread Greg KH
On Sat, Nov 24, 2012 at 10:14:53AM +0100, Borislav Petkov wrote: > On Fri, Nov 23, 2012 at 08:40:24AM -0800, Greg KH wrote: > > No, as there will not be any discarded sections anymore because > > CONFIG_HOTPLUG is always enabled. We will just delete this whole > > #if/#endif section once all users

Re: [PATCH 056/493] edac: remove use of __devexit_p

2012-11-24 Thread Borislav Petkov
On Fri, Nov 23, 2012 at 08:40:24AM -0800, Greg KH wrote: > No, as there will not be any discarded sections anymore because > CONFIG_HOTPLUG is always enabled. We will just delete this whole > #if/#endif section once all users of __devexit_p() are gone from the > tree. > > Does that help out? Yes,

Re: [PATCH 056/493] edac: remove use of __devexit_p

2012-11-23 Thread Greg KH
On Fri, Nov 23, 2012 at 11:06:35AM +0100, Borislav Petkov wrote: > On Thu, Nov 22, 2012 at 10:22:22AM -0800, Greg KH wrote: > > On Thu, Nov 22, 2012 at 02:44:51PM +0100, Borislav Petkov wrote: > > > On Mon, Nov 19, 2012 at 01:20:05PM -0500, Bill Pemberton wrote: > > > > CONFIG_HOTPLUG is going away

Re: [PATCH 056/493] edac: remove use of __devexit_p

2012-11-23 Thread Borislav Petkov
On Thu, Nov 22, 2012 at 10:22:22AM -0800, Greg KH wrote: > On Thu, Nov 22, 2012 at 02:44:51PM +0100, Borislav Petkov wrote: > > On Mon, Nov 19, 2012 at 01:20:05PM -0500, Bill Pemberton wrote: > > > CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer > > > needed. > > > > Erm, I d

Re: [PATCH 056/493] edac: remove use of __devexit_p

2012-11-22 Thread Greg KH
On Thu, Nov 22, 2012 at 02:44:51PM +0100, Borislav Petkov wrote: > On Mon, Nov 19, 2012 at 01:20:05PM -0500, Bill Pemberton wrote: > > CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer > > needed. > > Erm, I don't understand. __devexit_p is defined also for modules not > only f

Re: [PATCH 056/493] edac: remove use of __devexit_p

2012-11-22 Thread Borislav Petkov
On Mon, Nov 19, 2012 at 01:20:05PM -0500, Bill Pemberton wrote: > CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer > needed. Erm, I don't understand. __devexit_p is defined also for modules not only for CONFIG_HOTPLUG: #if defined(MODULE) || defined(CONFIG_HOTPLUG) #define __

[PATCH 056/493] edac: remove use of __devexit_p

2012-11-19 Thread Bill Pemberton
CONFIG_HOTPLUG is going away as an option so __devexit_p is no longer needed. Signed-off-by: Bill Pemberton Cc: Doug Thompson Cc: Borislav Petkov Cc: Mark Gross Cc: Jason Uhlenkott Cc: Mauro Carvalho Chehab Cc: Tim Small Cc: Ranganathan Desikan Cc: "Arvind R." Cc: Egor Martovetsky