Em Thu, 14 Aug 2014 11:32:48 +0200
Hans Verkuil <hansv...@cisco.com> escreveu:

> The following changes since commit 0f3bf3dc1ca394a8385079a5653088672b65c5c4:
>                                                                               
>                                                          
>    [media] cx23885: fix UNSET/TUNER_ABSENT confusion (2014-08-01 15:30:59 
> -0300)
>                                                                               
>                                                          
> are available in the git repository at:
>                                                                               
>                                                          
>    git://linuxtv.org/hverkuil/media_tree.git tw68
>                                                                               
>                                                          
> for you to fetch changes up to 64889b98f7ed20ab630a47eff4a5847c3aa0555e:
>                                                                               
>                                                          
>    MAINTAINERS: add tw68 entry (2014-08-10 10:36:10 +0200)
>                                                                               
>                                                          
> ----------------------------------------------------------------
> Hans Verkuil (2):
>        tw68: add support for Techwell tw68xx PCI grabber boards

NACK. This patch breaks compilation with allyesconfig:

drivers/media/pci/tw68/tw68-video.c: In function 'tw68_s_input':
drivers/media/pci/tw68/tw68-video.c:751:2: warning: comparison of unsigned 
expression < 0 is always false [-Wtype-limits]
  if (i < 0 || i >= TW68_INPUT_MAX)
  ^
drivers/media/pci/tw68/built-in.o:(.bss+0x568): multiple definition of 
`video_debug'
drivers/media/pci/saa7134/built-in.o:(.bss+0x3b08): first defined here
make[2]: *** [drivers/media/pci/built-in.o] Error 1
make[1]: *** [drivers/media/pci] Error 2
make: *** [_module_drivers/media] Error 2

PS.: It likely makes sense to also rename video_debug at saa7134, or
to get rid of both, in favor of using dynamic debug printks.

Regards,
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

Reply via email to