Re: [PATCH 000/493] remove CONFIG_HOTPLUG as an option

From: Grant Likely
Date: Fri Dec 07 2012 - 18:04:42 EST


On Fri, Dec 7, 2012 at 5:16 PM, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
> On Fri, Dec 07, 2012 at 01:47:48PM +0000, Grant Likely wrote:
>> On Wed, 5 Dec 2012 16:39:23 -0800, Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx> wrote:
>> > On Thu, Dec 06, 2012 at 12:27:42AM +0000, Grant Likely wrote:
>> > > On Wed, 21 Nov 2012 20:07:23 -0500, wfp5p@xxxxxxxxxxxxxxxxxxxxxxxxx (Bill Pemberton) wrote:
>> > > > Grant Likely writes:
>> > > > >
>> > > > > You mean this series wasn't created with a script? You did this by
>> > > > > hand? If so then I must say kudos on your dedication!
>> > > > >
>> > > > > But it makes me more nervous about the series. Too easy to fat
>> > > > > finger many things when touching that many files.
>> > > > >
>> > > >
>> > > > No, I didn't do them by hand, it was a script. Originally, it was a
>> > > > couple, all basically the same, but removing each __dev*. Then I'd do
>> > > > a word diff to eyeball them to make sure the script didn't do
>> > > > something goofy.
>> > > >
>> > > > The whack-a-mole part came along because I was working against
>> > > > linux-next and whatever patch series was right for one day wouldn't be
>> > > > right for the next day because of some of the faster moving trees.
>> > > >
>> > > >
>> > > > > Please do write a script and post that for review.
>> > > > >
>> > > >
>> > > > The all-in-one version of the script:
>> > > >
>> > > > #! /usr/bin/perl
>> > > >
>> > > > use strict;
>> > > > use IO::InSitu;
>> > > >
>> > > > sub processfile
>> > > > {
>> > > > my $fn = shift;
>> > > >
>> > > > my ($in, $out) = open_rw($fn, $fn);
>> > > >
>> > > > while (<$in>) {
>> > > > s|__devexit_p\(([^)]+)\)|$1|;
>> > > > s|\s__devexit\b||;
>> > > > s|\s__devinitconst\b||;
>> > > > s|\s__devinitdata\b||;
>> > > > s|\s__devinit\b||;
>> > >
>> > > Pretty straight forward, and works against the files I tried. :-)
>> > >
>> > > Greg, I'd much rather see the change applied all at once in this manner.
>> > > If that isn't possible, then at the least I'll use the script against
>> > > the code that I maintain and push th result out to Linus.
>> >
>> > Given that there are a lot of patches already in linux-next from Bill
>> > due to this work, I'm not going to do this for all files right now,
>> > sorry.
>> >
>> > But, if you want to use this for the files you maintain and push that
>> > out for 3.8-rc1, that would be great. I'll be walking the tree after
>> > 3.8-rc1 is out to catch the stragglers with a script like this.
>>
>> Okay. Can you drop any commits you have against drivers/{spi,gpio,of}?
>
> Hm, I only applied the gpio ones to my tree, you got an email when that
> happened. I didn't apply the spi or of ones.
>
>> Or are they in a tree that you will not rebase?
>
> They are in my driver-core.git tree, the driver-core-next branch, which
> will not be rebased, and has been in linux-next for a while now.
>
> I can revert the 5 gpio patches if you want me to, just let me know.

no. Don't revert. I won't have the gpio changes in my tree, but I'll
do spi since they aren't in your tree yet.

g.


--
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/