1. 08 Jul, 2016 5 commits
  2. 28 Jun, 2016 24 commits
  3. 24 Jun, 2016 11 commits
    • Mauro Carvalho Chehab's avatar
      bdisp: move the V/H filter spec to bdisp-hw.c · 068adc45
      Mauro Carvalho Chehab authored
      Those structs are used only at bdisp-hw, so they shouldn't be
      there in a header file that it is used elsewhere.
      
      This fixes the following Gcc 6.1 warnings:
      
      In file included from drivers/media/platform/sti/bdisp/bdisp-debug.c:11:0:
      drivers/media/platform/sti/bdisp/bdisp-filter.h:207:65: warning: ‘bdisp_v_spec’ defined but not used [-Wunused-const-variable=]
       static const struct __maybe_unused bdisp_filter_v_spec bdisp_v_spec[] = {
                                                                       ^~~~~~~~~
      In file included from drivers/media/platform/sti/bdisp/bdisp-debug.c:11:0:
      drivers/media/platform/sti/bdisp/bdisp-filter.h:23:65: warning: ‘bdisp_h_spec’ defined but not used [-Wunused-const-variable=]
       static const struct __maybe_unused bdisp_filter_h_spec bdisp_h_spec[] = {
                                                                       ^~~~~~~~~
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      068adc45
    • Mauro Carvalho Chehab's avatar
      adv7842: comment out a table useful for debug · 60eb9579
      Mauro Carvalho Chehab authored
      Gcc 6.1 warns about an unused table:
      
      drivers/media/i2c/adv7842.c:2400:27: warning: 'prim_mode_txt' defined but not used [-Wunused-const-variable=]
       static const char * const prim_mode_txt[] = {
                                 ^~~~~~~~~~~~~
      
      That seems to be useful for debug, and likely were used before.
      While we could simply remove, let's comment it out, for now.
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      60eb9579
    • Mauro Carvalho Chehab's avatar
      em28xx-dvb: remove some left over · 5776fbec
      Mauro Carvalho Chehab authored
      Gcc 6.1 warns about an unused table:
      
      drivers/media/usb/em28xx/em28xx-dvb.c:907:38: warning: 'pctv_461e_m88ds3103_config' defined but not used [-Wunused-const-variable=]
       static const struct m88ds3103_config pctv_461e_m88ds3103_config = {
                                            ^~~~~~~~~~~~~~~~~~~~~~~~~~
      
      That's a left over of patch 76b91be3 ('em28xx: PCTV 461e use I2C
      client for demod and SEC').
      
      Remove the dead code.
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      5776fbec
    • Mauro Carvalho Chehab's avatar
      vivid: remove some unused vars · ba4cd399
      Mauro Carvalho Chehab authored
      Gcc 6.1 warns about some unused vars. Remove them:
      drivers/media/platform/vivid/vivid-vid-cap.c:40:2: warning: 'tpf_default' defined but not used [-Wunused-const-variable=]
        tpf_default = {.numerator = 1,  .denominator = 30};
        ^~~~~~~~~~~
      drivers/media/platform/vivid/vivid-sdr-cap.c:54:27: warning: 'NUM_FORMATS' defined but not used [-Wunused-const-variable=]
       static const unsigned int NUM_FORMATS = ARRAY_SIZE(formats);
                                 ^~~~~~~~~~~
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      ba4cd399
    • Mauro Carvalho Chehab's avatar
      zr36016: remove some unused tables · 40bcfdac
      Mauro Carvalho Chehab authored
      Gcc 6.1 warns about some unused tables:
      
      drivers/media/pci/zoran/zr36016.c:251:18: warning: 'zr016_yoff' defined but not used [-Wunused-const-variable=]
       static const int zr016_yoff[] = { 8, 9, 7 };
                        ^~~~~~~~~~
      drivers/media/pci/zoran/zr36016.c:250:18: warning: 'zr016_xoff' defined but not used [-Wunused-const-variable=]
       static const int zr016_xoff[] = { 20, 20, 20 };
                        ^~~~~~~~~~
      
      Those tables aren't used anywere. So, remove them.
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      40bcfdac
    • Mauro Carvalho Chehab's avatar
      r820t: comment out two ancillary tables · 29025f69
      Mauro Carvalho Chehab authored
      As Gcc6.1 warned, those tables are currently unused:
      	drivers/media/tuners/r820t.c:349:18: warning: 'r820t_mixer_gain_steps' defined but not used [-Wunused-const-variable=]
      	 static const int r820t_mixer_gain_steps[]  = {
      	                  ^~~~~~~~~~~~~~~~~~~~~~
      	drivers/media/tuners/r820t.c:345:18: warning: 'r820t_lna_gain_steps' defined but not used [-Wunused-const-variable=]
      	 static const int r820t_lna_gain_steps[]  = {
      	                  ^~~~~~~~~~~~~~~~~~~~
      
      They're actually used only by a routine that it is currently
      commented out. So, move those tables to be together with such
      code and comment them out.
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      29025f69
    • Mauro Carvalho Chehab's avatar
      dib0090: comment out the unused tables · cf47faca
      Mauro Carvalho Chehab authored
      Those tables are currently unused, so comment them out:
      
      drivers/media/dvb-frontends/dib0090.c:852:18: warning: 'rf_ramp_pwm_sband' defined but not used [-Wunused-const-variable=]
       static const u16 rf_ramp_pwm_sband[] = {
                        ^~~~~~~~~~~~~~~~~
      drivers/media/dvb-frontends/dib0090.c:800:18: warning: 'bb_ramp_pwm_boost' defined but not used [-Wunused-const-variable=]
       static const u16 bb_ramp_pwm_boost[] = {
                        ^~~~~~~~~~~~~~~~~
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      cf47faca
    • Mauro Carvalho Chehab's avatar
      drxj: comment out the unused nicam_presc_table_val table · 01d7d436
      Mauro Carvalho Chehab authored
      Avoid this warning:
      
      drivers/media/dvb-frontends/drx39xyj/drxj.c:1243:18: warning: 'nicam_presc_table_val' defined but not used [-Wunused-const-variable=]
       static const u16 nicam_presc_table_val[43] = {
                        ^~~~~~~~~~~~~~~~~~~~~
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      01d7d436
    • Mauro Carvalho Chehab's avatar
      cx25821-alsa: shutup a Gcc 6.1 warning · 114e877a
      Mauro Carvalho Chehab authored
      The PCI device ID table is only used if compiled with modules
      support. When compiled with modules disabled, this is now
      producing this bogus warning:
      
      drivers/media/pci/cx25821/cx25821-alsa.c:696:35: warning: 'cx25821_audio_pci_tbl' defined but not used [-Wunused-const-variable=]
       static const struct pci_device_id cx25821_audio_pci_tbl[] = {
                                         ^~~~~~~~~~~~~~~~~~~~~
      
      Fix it by annotating that the function may not be used.
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      114e877a
    • Mauro Carvalho Chehab's avatar
      m5602_ov7660: move skeletons to the .c file · 123818ee
      Mauro Carvalho Chehab authored
      The header file has some private static structures that
      are used only by the C file. Move those structures to the C file,
      in order to shut up gcc 6.1 warnings.
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      123818ee
    • Mauro Carvalho Chehab's avatar
      m5602_po1030: move skeletons to the .c file · 804c7812
      Mauro Carvalho Chehab authored
      The header file has some private static structures that
      are used only by the C file. Move those structures to the C file,
      in order to shut up gcc 6.1 warnings.
      Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
      804c7812