Well, it is what it is, will send the patch the right way. Best, Chaitanya On Tue, May 27, 2014 at 1:39 PM, Dan Carpenter <dan.carpenter@xxxxxxxxxx> wrote: > On Tue, May 27, 2014 at 01:09:45PM -0700, Chaitanya Hazarey wrote: >> Fixed the following: >> >> ERROR: do not use C99 // comments >> ERROR: space required after that ',' (ctx:VxV) >> ERROR: space required after that ';' (ctx:VxV) >> ERROR: space required before the open brace '{' >> ERROR: spaces required around that '=' (ctx:VxV) >> ERROR: spaces required around that '<' (ctx:VxV) >> ERROR: space required after that close brace '}' >> ERROR: spaces required around that '==' (ctx:VxV) >> ERROR: space required before the open parenthesis '(' >> > > All of these patches have the same subject. And also they aren't > broken up in the correct way. Basically for this kind of thing we want > to see one kind of cleanup per patch. > > [patch 1/6] Staging: rtl8192u: add missing spaces > [patch 2/6] Staging: rtl8192u: delete commented out code > [patch 3/6] Staging: rtl8192u: clean up c99 comments > [patch 4/6] Staging: rtl8192u: reformat long lines > ... > > Etc. > > Sorry for asking people to redo these again... > > regards, > dan carpenter > _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel