On Oct 15 Piotr Chmura wrote: > Staging submission: PCTV 74e driver (as102) > > From: Devin Heitmueller<dheitmueller@xxxxxxxxxxxxxx> > > pull as102 driver from > > This is driver for PCTV 74e DVB-T USB tuner, taken from [1], > written by Devin Heitmueller using the GPL reference driver provided by Abilis. > > The only change needed to compile it in current git tree [2] > was changing calls usb_buffer_alloc() and usb_buffer_free() to > usb_alloc_coherent() and usb_free_coherent(). It's included in this patch. > > Patch was tested by me on amd64. > > [1]http://kernellabs.com/hg/~dheitmueller/v4l-dvb-as102-2/ > [2] git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git kernel-3.1.0-git9+ > > Signed-off-by: Piotr Chmura<chmooreck@xxxxxxxxxxxxxx> > Cc: Devin Heitmueller<dheitmueller@xxxxxxxxxxxxxx> > Cc: Greg HK<gregkh@xxxxxxx> > > diff -Nur linux.clean/drivers/staging/as102/as102_drv.c linux.as102_initial/drivers/staging/as102/as102_drv.c > --- linux.clean/drivers/staging/as102/as102_drv.c 1970-01-01 01:00:00.000000000 +0100 > +++ linux.as102_initial/drivers/staging/as102/as102_drv.c 2011-10-14 17:55:02.000000000 +0200 > @@ -0,0 +1,360 @@ > +/* > + * Abilis Systems Single DVB-T Receiver > + * Copyright (C) 2008 Pierrick Hascoet<pierrick.hascoet@xxxxxxxxxx> > + * Copyright (C) 2010 Devin Heitmueller<dheitmueller@xxxxxxxxxxxxxx> Hi Piotr, thanks for getting this going again. - I have not yet looked through the source but have some small remarks on the patch format. - In your changelogs and in the diffs, somehow the space between real name and e-mail address got lost. - The repetition of the Subject: line as first line in the changelog is unnecessary (and would cause an undesired duplication e.g. when git-am is used, last time I checked). - AFAICT, author of patch 1/7 is not Devin but you. Hence the From: line right above the changelog is wrong. - The reference to the source hg tree is very helpful. However, since the as102 related history in there is very well laid out, it may be beneficial to quote some of this prior history. I suggest, include the changelog of "as102: import initial as102 driver", http://kernellabs.com/hg/~dheitmueller/v4l-dvb-as102-2/rev/a78bda1e1a0b (but mark it clearly as a quote from the out-of-tree repo), and include a shortlog from this commit inclusive until the head commit inclusive. (Note, the hg author field appears to be wrong; some of the changes apparently need to be attributed to Pierrick Hascoet as author.) This would IMO improve the picture of who contributed what when this goes into mainline git history, even though the hg history needed to be discarded. - A diffstat is always very nice to have in a patch posting. Most tools for patch generation make it easy to add, and it helps the recipients of the patch posting. (Also, a diffstat of all patches combined would be good to have in the introductory PATCH 0/n posting, but not many developers take the time to do so.) Again, thanks for the effort and also thanks to Devin for making it possible. -- Stefan Richter -=====-==-== =-=- =---- http://arcgraph.de/sr/ _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
