Generally "delete code" patches are easy to review.  But sometimes you
have to change formatting and remove variables and curly braces.

$ grep LOG drivers/staging/unisys/ -R | wc -l
415

There isn't a firm rule on way a patch is just too big and annoying to
review.  Probably break it up into:

[patch 1/x] delete LOGINF()
[patch 2/x] delete LOGERR()
...

regards,
dan carpenter

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to