[PATCH 08/21] staging: brcm80211: changed module wlc_mac80211 to wlc_main

Dan Carpenter error27 at gmail.com
Fri Feb 25 11:54:19 UTC 2011


On Fri, Feb 25, 2011 at 11:31:27AM +0100, Arend van Spriel wrote:
> The source and include file for the wlc_mac80211 module has been
> renamed to wlc_main and subsequently the include statement in
> other source files. This module provides the main interface towards
> wl_mac80211 module.
> 

I know that we say you should run checkpatch on all patches, but moving
code around is the exception.  If it had just moved the code around
without the changes, then we have automated ways of reviewing that.

This patch is 20k lines long.  Ideally it would have been several
patches.  One to move the code without changes.  It would be a big
patch to fast to review automatically.  One to fix checkpatch warnings.
One to add comments.

I'm not going to say that you should redo it, but please don't do it
this way next time.

Reviewed-by: Dan Carpenter <error27 at gmail.com>

regards,
dan carpenter



More information about the devel mailing list