On Thu, May 03, 2012 at 05:49:03PM +0300, Tomas Melin wrote: > Removed one additional level of if() by combining two if() statements into one. > The patch looks good obviously. All three redone patches have the same subject. It would be better if it were: [patch 1/4] Staging: Comedi adv_pci1710: move a check forward [patch 2/4] Staging: Comedi adv_pci1710: fix some line breaks [patch 3/4] Staging: Comedi adv_pci1710: clarify some comments [patch 4/4] Staging: Comedi adv_pci1710: combine two conditions I kind of feel bad for being nit-picky like this... It's just that it's your first few patches so you may as well learn what we look for. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html