Re: [PATCH v2 02/17] Staging: quickstart: Prefix remaining functions names with quickstart_

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 10 Feb 2012 19:58:23 +0100
Szymon Janc <szymon@xxxxxxxxxxx> wrote:

> Hi,
> 
> > > I'll take this, but you do know there is no reason for this, as these
> > > are static functions.  I'd recommend keeping them as short as possible,
> > > but hey, it's not my code...
> > > 
> > > thanks,
> > > 
> > > greg k-h
> > 
> > I agree with Greg on this one. At the time I didn't prefix these
> > functions with the module name because I wanted to make them clear they
> > were different. The difference lies in the fact that those functions
> > are only used to implement the sysfs interface.
> 
> Yeah, maybe you are right. I thought it was just an oversight. I can send a 
> patch that will get orig names back if you prefer.

Well, if the patch is already applied, then no need to revert, as Greg said
the relevant functions are static, so the patch ends up being just cosmetic. :)

Thanks for pushing this out of staging.
- Angelo
> 
> 
> -- 
> Szymon K. Janc
> szymon@xxxxxxxxxxx
> 
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Development]     [Kernel Announce]     [Kernel Newbies]     [Linux Networking Development]     [Share Photos]     [IDE]     [Security]     [Git]     [Netfilter]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Device Mapper]

  Powered by Linux