From: kbuild test robot <fengguang.wu@intel.com>
Cc: kbuild-all@01.org,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Joe Perches <joe@perches.com>,
Andrew Morton <akpm@linux-foundation.org>,
Linux Memory Management List <linux-mm@kvack.org>,
Sasha Levin <sasha.levin@oracle.com>
Subject: [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: Mon, 20 Jun 2016 19:10:44 +0800 [thread overview]
Message-ID: <201606201937.lOb6fboA%fengguang.wu@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 6455 bytes --]
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@metzlerbros.de>
:::::: CC: Mauro Carvalho Chehab <mchehab@redhat.com>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 23196 bytes --]
reply other threads:[~2016-06-20 11:12 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201606201937.lOb6fboA%fengguang.wu@intel.com \
--to=fengguang.wu@intel.com \
--cc=akpm@linux-foundation.org \
--cc=gregkh@linuxfoundation.org \
--cc=joe@perches.com \
--cc=kbuild-all@01.org \
--cc=linux-mm@kvack.org \
--cc=sasha.levin@oracle.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox