[linux-stable-rc:linux-3.14.y 4656/4754] drivers/media/dvb-frontends/drxd_hard.c:2631:3: warning: this 'if' clause does not guard...

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



tree:   https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-3.14.y
head:   96f343e0621a0d5afa9c2a34e2aee69f33b6eb5c
commit: 3711edaf01a01818f2aed9f21efe29b9818134b9 [4656/4754] compiler-gcc: integrate the various compiler-gcc[345].h files
config: i386-randconfig-s1-201625 (attached as .config)
compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
reproduce:
        git checkout 3711edaf01a01818f2aed9f21efe29b9818134b9
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   drivers/media/dvb-frontends/drxd_hard.c: In function 'DRXD_init':
>> drivers/media/dvb-frontends/drxd_hard.c:2631:3: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
      if (!state->type_A && state->m_HiI2cPatch != NULL)
      ^~
   drivers/media/dvb-frontends/drxd_hard.c:2633:4: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
       if (status < 0)
       ^~
   drivers/media/dvb-frontends/drxd_hard.c:2691:3: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
      if (state->type_A)
      ^~
   drivers/media/dvb-frontends/drxd_hard.c:2693:4: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
       if (status < 0)
       ^~
--
   drivers/media/dvb-frontends/drxk_hard.c: In function 'set_agc_rf':
>> drivers/media/dvb-frontends/drxk_hard.c:2223:3: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
      if (p_if_agc_settings->ctrl_mode == DRXK_AGC_CTRL_AUTO)
      ^~
   drivers/media/dvb-frontends/drxk_hard.c:2227:4: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
       if (status < 0)
       ^~
--
   In file included from include/linux/thermal.h:28:0,
                    from drivers/thermal/of-thermal.c:25:
   drivers/thermal/of-thermal.c: In function 'thermal_of_build_thermal_zone':
>> include/linux/of.h:663:2: warning: this 'for' clause does not guard... [-Wmisleading-indentation]
     for (child = of_get_next_child(parent, NULL); child != NULL; \
     ^
>> drivers/thermal/of-thermal.c:715:2: note: in expansion of macro 'for_each_child_of_node'
     for_each_child_of_node(child, gchild)
     ^~~~~~~~~~~~~~~~~~~~~~
   drivers/thermal/of-thermal.c:718:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'for'
      if (ret)
      ^~
--
>> drivers/usb/gadget/mv_u3d_core.c:38:19: warning: 'driver_desc' defined but not used [-Wunused-const-variable=]
    static const char driver_desc[] = DRIVER_DESC;
                      ^~~~~~~~~~~
--
   kernel/cgroup.c: In function 'parse_cgroupfs_options':
>> kernel/cgroup.c:1235:5: warning: suggest explicit braces to avoid ambiguous 'else' [-Wparentheses]
     if (all_ss || (!one_ss && !opts->none && !opts->name))
        ^
   At top level:
>> kernel/cgroup.c:3839:36: warning: 'cgroup_pidlist_seq_operations' defined but not used [-Wunused-const-variable=]
    static const struct seq_operations cgroup_pidlist_seq_operations = {
                                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~

vim +/if +2631 drivers/media/dvb-frontends/drxd_hard.c

126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2615  	int status = 0;
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2616  	u32 driverVersion;
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2617  
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2618  	if (state->init_done)
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2619  		return 0;
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2620  
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2621  	CDRXD(state, state->config.IF ? state->config.IF : 36000000);
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2622  
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2623  	do {
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2624  		state->operation_mode = OM_Default;
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2625  
58d5eaec drivers/media/dvb/frontends/drxd_hard.c Mauro Carvalho Chehab 2011-03-25  2626  		status = SetDeviceTypeId(state);
58d5eaec drivers/media/dvb/frontends/drxd_hard.c Mauro Carvalho Chehab 2011-03-25  2627  		if (status < 0)
58d5eaec drivers/media/dvb/frontends/drxd_hard.c Mauro Carvalho Chehab 2011-03-25  2628  			break;
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2629  
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2630  		/* Apply I2c address patch to B1 */
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12 @2631  		if (!state->type_A && state->m_HiI2cPatch != NULL)
58d5eaec drivers/media/dvb/frontends/drxd_hard.c Mauro Carvalho Chehab 2011-03-25  2632  			status = WriteTable(state, state->m_HiI2cPatch);
58d5eaec drivers/media/dvb/frontends/drxd_hard.c Mauro Carvalho Chehab 2011-03-25  2633  			if (status < 0)
58d5eaec drivers/media/dvb/frontends/drxd_hard.c Mauro Carvalho Chehab 2011-03-25  2634  				break;
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2635  
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2636  		if (state->type_A) {
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2637  			/* HI firmware patch for UIO readout,
126f1e61 drivers/media/dvb/frontends/drxd_hard.c Ralph Metzler         2011-03-12  2638  			   avoid clearing of result register */
58d5eaec drivers/media/dvb/frontends/drxd_hard.c Mauro Carvalho Chehab 2011-03-25  2639  			status = Write16(state, 0x43012D, 0x047f, 0);

:::::: The code at line 2631 was first introduced by commit
:::::: 126f1e61887085aa2c2cfa7644aee8295a94e1f7 drx: add initial drx-d driver

:::::: TO: Ralph Metzler <rjkm@xxxxxxxxxxxxxx>
:::::: CC: Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: Binary data


[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]