Hi,

On 12/30/2011 11:24 AM, Jean-Francois Moine wrote:
On Fri, 30 Dec 2011 10:29:56 +0100
Hans de Goede<hdego...@redhat.com>  wrote:
        [snip]
The following changes since commit 1a5cd29631a6b75e49e6ad8a770ab9d69cda0fa2:

    [media] tda10021: Add support for DVB-C Annex C (2011-12-20 14:01:08 -0200)

are available in the git repository at:
    git://linuxtv.org/hgoede/gspca.git media-for_v3.3
        [snip]
Theodore Kilgore (1):
        gspca: add jl2005bcd sub driver
        [snip]

I have noticed some problems with the patch 2346c78dff71b003f:

- there should be no change in gspca.h (addition of two empty lines)

- there is no documentation about the new pixel format 'JL20'

- in jl2005bcd.c, the macro 'err' is used instead of 'pr_err'
   (there are also spaces at end of line, but this is less important..)


I took it as is from Theodore, I guess we should do a separate cleanup
patch on top to preserve the history / authorship. Since I'm busy testing
the new isoc bandwidth stuff today, could you perhaps do a cleanup patch for 
this?

Regards,

Hans
--
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

Reply via email to