[Camel to lower case? (was Re: [PATCH 507/641] Staging: rtxxxx: unify AsicSwitchChannel())

Greg KH gregkh at suse.de
Thu Sep 17 13:23:02 UTC 2009


On Thu, Sep 17, 2009 at 07:58:39AM -0500, Nishanth Menon wrote:
> Greg KH said the following on 09/16/2009 09:11 PM:
> > On Wed, Sep 16, 2009 at 09:08:07PM -0500, Nishanth Menon wrote:
> >   
> >> Greg Kroah-Hartman said the following on 09/15/2009 02:12 PM:
> >>     
> >>> From: Bartlomiej Zolnierkiewicz <bzolnier at gmail.com>
> >>>
> >>> Unify RT30xx and !RT30xx code in AsicSwitchChannel().
> >>>
> >>> There should be no functional changes caused by this patch.
> >>>
> >>> Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier at gmail.com>
> >>> Signed-off-by: Greg Kroah-Hartman <gregkh at suse.de>
> >>> ---
> >>>  drivers/staging/rt2860/common/mlme.c |   46 ++++++---------------------------
> >>>  1 files changed, 9 insertions(+), 37 deletions(-)
> >>>
> >>> diff --git a/drivers/staging/rt2860/common/mlme.c b/drivers/staging/rt2860/common/mlme.c
> >>> index 12444e3..3364d67 100644
> >>> --- a/drivers/staging/rt2860/common/mlme.c
> >>> +++ b/drivers/staging/rt2860/common/mlme.c
> >>> @@ -5787,8 +5787,7 @@ VOID AsicSwitchChannel(
> >>>
> >>>   
> >>>       
> >> generically - does it make sense for rename of camel casing to lower casing?
> >>     
> >
> > Yes, it does, but be smart about it.
> >   
> from [1], my tiny little script (I think it can be done better):

Heh, nice, thanks for the script.

greg k-h



More information about the devel mailing list