On Oct 15 Piotr Chmura wrote:
> Staging submission: PCTV 74e driver (as102)
>
> From: Devin Heitmueller<[email protected]>
>
> 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<[email protected]>
> Cc: Devin Heitmueller<[email protected]>
> Cc: Greg HK<[email protected]>
>
> 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<[email protected]>
> + * Copyright (C) 2010 Devin Heitmueller<[email protected]>
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
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel
