[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