Dear Larry,
thanks for your review.
Larry Finger wrote:
On 08/28/2015 03:45 PM, Luca Ceresoli wrote:
Some of the report_*() functions are internal and will be declared
static,
and their declaration removed from rtw_mlme_ext.h, in a later commit.
This would break compilation, since they are are referenced before their
definition.
Reorder these functions so that symbols are defined before they are
referenced, without the need for forward declarations.
Also move near the beginning of the file the collect_bss_info() and
process_80211d(), since they are called from the report_*() functions.
This commit only reorders code, there is no content change.
Signed-off-by: Luca Ceresoli <luca@xxxxxxxxxxxxxxxx>
Cc: Larry Finger <Larry.Finger@xxxxxxxxxxxx>
Cc: Greg Kroah-Hartman <gregkh@xxxxxxxxx>
Even though you are just moving code, I think you should fix the long
line warnings from scripts/checkpatch, otherwise note that they are not
fixed here, but they will be fixed in a later patch. As for the other
warnings, they should be noted and fixed later.
I understand your point. However, the coding style of rtw_mlme_ext.c is
so far away from the Linux requirements that cleaning it up is a
daunting task.
There are 151 lines over 80 characters, and several (maybe one third)
cannot or should not be fixed by simply adding newlines. Restructuring
the code is needed in some cases, and the complexity of the change
(and of the code review, correspondingly) can well be larger than the
whole impact of my first patchset. In some cases it will require more
understanding of the code that it took to produce this patchset.
Nevertheless I started the cleanup job, but it will take time. When
completed I expect the number of patches in this patchset to at least
double...
--
Luca
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
