Patch looks ok. Just a few comments: Em 06-08-2012 10:38, Ezequiel Garcia escreveu: > This driver adds support for stk1160 usb bridge as used in some > video/audio usb capture devices. > It is a complete rewrite of staging/media/easycap driver and > it's expected as a replacement. > ---
Please don't add a "---" here. Everything after a --- are discarded by my scripts (and by most other kernel developer scripts). > Cc: Mauro Carvalho Chehab <mche...@redhat.com> > Cc: Takashi Iwai <ti...@suse.de> > Cc: Hans Verkuil <hverk...@xs4all.nl> > Cc: Sylwester Nawrocki <sylvester.nawro...@gmail.com> Hmm... weren't it reviewed already be them? > Signed-off-by: Ezequiel Garcia <elezegar...@gmail.com> > diff --git a/drivers/media/video/stk1160/Makefile > b/drivers/media/video/stk1160/Makefile > new file mode 100644 > index 0000000..8f66a78 > --- /dev/null > +++ b/drivers/media/video/stk1160/Makefile > @@ -0,0 +1,12 @@ > +obj-stk1160-ac97-$(CONFIG_VIDEO_STK1160_AC97) := stk1160-ac97.o > + > +stk1160-y := stk1160-core.o \ > + stk1160-v4l.o \ > + stk1160-video.o \ > + stk1160-i2c.o \ > + $(obj-stk1160-ac97-y) > + > +obj-$(CONFIG_VIDEO_STK1160) += stk1160.o > + > +ccflags-y += -Wall You shouldn't be adding the above here. > +ccflags-y += -Idrivers/media/video Ah, please split this patch into two patches: one with the new driver addition, and another one with the removal of the driver at staging. That will help to make the patch smaller, and avoids mixing two different things at the same place. Thanks, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
