Re: [linuxtv-media:master 499/499] drivers/media/usb/usbtv/usbtv-core.c:119:22: sparse: symbol 'usbtv_id_table' was not declared. Should it be static?

[Mauro Carvalho Chehab <m.chehab@samsung.com>]

Em Wed, 5 Mar 2014 14:27:46 -0300
Mauro Carvalho Chehab <m.chehab@samsung.com> escreveu:

> Hi Fengguang,
> 
> This patch got obsoleted by another patch in the same series.
> 
> Unfortunately, I had to break sending the patch series into a few
> pushes, as my mailbomb script has a logic there that prevents it to
> send more than 30~50 emails (I never remember the exact setting).
> 
> So, I pushed this 80-series into a few pushes. You likely compiled the
> tree without waiting for the hole series to be upstreamed.

Sorry, I sent the reply to the wrong patch. The above comments are for
this patch:
	Subject: [PATCH linuxtv-media] drx-j: drxj_default_aud_data_g can be static

That was on the email with this title:
	[linuxtv-media:master 428/499] drivers/media/dvb-frontends/drx39xyj/drxj.c:1039:16: sparse: symbol 'drxj_default_aud_data_g' was not declared. Should

Regards,
Mauro

> 
> Regards,
> Mauro
> 
> Em Tue, 04 Feb 2014 17:02:02 +0800
> kbuild test robot <fengguang.wu@intel.com> escreveu:
> 
> > tree:   git://linuxtv.org/media_tree.git master
> > head:   a3550ea665acd1922df8275379028c1634675629
> > commit: a3550ea665acd1922df8275379028c1634675629 [499/499] [media] usbtv: split core and video implementation
> > reproduce: make C=1 CF=-D__CHECK_ENDIAN__
> > 
> > 
> > sparse warnings: (new ones prefixed by >>)
> > 
> > >> drivers/media/usb/usbtv/usbtv-core.c:119:22: sparse: symbol 'usbtv_id_table' was not declared. Should it be static?
> > >> drivers/media/usb/usbtv/usbtv-core.c:129:19: sparse: symbol 'usbtv_usb_driver' was not declared. Should it be static?
> > --
> > >> drivers/media/usb/usbtv/usbtv-video.c:285:14: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:285:14: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:285:14: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:285:14: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:285:14: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:285:14: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:287:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:287:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:287:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:287:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:287:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:287:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:288:15: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:288:15: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:288:15: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:288:15: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:288:15: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:288:15: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:289:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:289:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:289:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:289:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:289:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:289:20: sparse: cast to restricted __be32
> > >> drivers/media/usb/usbtv/usbtv-video.c:565:23: sparse: symbol 'usbtv_ioctl_ops' was not declared. Should it be static?
> > >> drivers/media/usb/usbtv/usbtv-video.c:587:29: sparse: symbol 'usbtv_fops' was not declared. Should it be static?
> > >> drivers/media/usb/usbtv/usbtv-video.c:648:16: sparse: symbol 'usbtv_vb2_ops' was not declared. Should it be static?
> > 
> > Please consider folding the attached diff :-)
> > 
> > ---
> > 0-DAY kernel build testing backend              Open Source Technology Center
> > http://lists.01.org/mailman/listinfo/kbuild                 Intel Corporation
> 
> 


-- 

Cheers,
Mauro