Gary emailed me privately that fixing my comments would be a problem.
1) Greg complained about your earlier patch where you moved the
comments from this:
/* comment */
code();
To this:
code(); /* comment */
Greg's complaint was not about that you were doing more than one thing
per patch it was that the comment style was better in the original.
2) A lot of newbies have problems with understanding where the line
between doing one thing per patch and doing two things in a patch is.
You are allowed to make *tiny* changes if they are closely related to
the central thrust of the patch.
So in this case the patch would be:
Staging: bcm: DDRInit: fix up some comments
I have removed the C99 comments. Some comments were bogus so instead of
fixing them, I just deleted them. I made a few minor grammar, spelling
and white space fixes as well.
It's still one thing which is "fixing comments".
regards,
dan carpenter
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
