On Sat, Apr 19, 2014 at 11:09 AM, Jinqiang Zeng <jinqiangz...@gmail.com> wrote: > --- > drivers/staging/media/sn9c102/sn9c102.h | 30 +- > drivers/staging/media/sn9c102/sn9c102_core.c | 342 > ++++++++++---------- > drivers/staging/media/sn9c102/sn9c102_devtable.h | 22 +- > drivers/staging/media/sn9c102/sn9c102_hv7131d.c | 22 +- > drivers/staging/media/sn9c102/sn9c102_hv7131r.c | 22 +- > drivers/staging/media/sn9c102/sn9c102_mi0343.c | 30 +- > drivers/staging/media/sn9c102/sn9c102_mi0360.c | 30 +- > drivers/staging/media/sn9c102/sn9c102_ov7630.c | 22 +- > drivers/staging/media/sn9c102/sn9c102_ov7660.c | 22 +- > drivers/staging/media/sn9c1
Hi, Jinqiang, you just missed something. 1. a "Signed-off-by" line, if you use git, git-format-patch is a good tool.;-) 2. better add a module name in cover later, like this: "sn9c102: fix the coding style errors" 3. maybe some changelog is good, but for this patch, leaving it out is OK. You would better read the "Documentation/SubmittingPatches" . Thanks, Jianyu Zhan -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html