From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: [jimc:dd-drm-rfc 13/18] drivers/gpu/drm/arm/display/komeda/komeda_kms.c:121:22: warning: Value stored to 'a' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
Date: Sun, 23 Jan 2022 04:15:30 +0800 [thread overview]
Message-ID: <202201230137.mBVZ14jX-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 33981 bytes --]
CC: kbuild-all(a)lists.01.org
TO: Jim Cromie <jim.cromie@gmail.com>
tree: https://github.com/jimc/linux.git dd-drm-rfc
head: c86deb7d09199fd29665d9c68f1e70850132e49a
commit: ed4425a17de1eb5638703d40ed1d002b28963687 [13/18] drm_print: add choice to use dynamic debug in drm-debug
:::::: branch date: 4 months ago
:::::: commit date: 4 months ago
config: x86_64-randconfig-c007-20211009 (https://download.01.org/0day-ci/archive/20220123/202201230137.mBVZ14jX-lkp(a)intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 30caca39f401ae17927439c0a0bd6d1b1916dd6a)
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/jimc/linux/commit/ed4425a17de1eb5638703d40ed1d002b28963687
git remote add jimc https://github.com/jimc/linux.git
git fetch --no-tags jimc dd-drm-rfc
git checkout ed4425a17de1eb5638703d40ed1d002b28963687
# save the config file to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=x86_64 clang-analyzer
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
clang-analyzer warnings: (new ones prefixed by >>)
^
include/linux/mfd/twl.h:157:9: note: expanded from macro 'TWL_CLASS_IS'
return ((id) == (GET_TWL_REV)) ? 1 : 0; \
^
include/linux/mfd/twl.h:160:1: note: Returning the value 1, which participates in a condition later
TWL_CLASS_IS(4030, TWL4030_CLASS_ID)
^
include/linux/mfd/twl.h:157:2: note: expanded from macro 'TWL_CLASS_IS'
return ((id) == (GET_TWL_REV)) ? 1 : 0; \
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mfd/twl-core.c:1201:6: note: Returning from 'twl_class_is_4030'
if (twl_class_is_4030()) {
^~~~~~~~~~~~~~~~~~~
drivers/mfd/twl-core.c:1201:2: note: Taking true branch
if (twl_class_is_4030()) {
^
drivers/mfd/twl-core.c:1202:3: note: 'temp' declared without an initial value
u8 temp;
^~~~~~~
drivers/mfd/twl-core.c:1204:3: note: Calling 'twl_i2c_read_u8'
twl_i2c_read_u8(TWL4030_MODULE_INTBR, &temp, REG_GPPUPDCTR1);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/mfd/twl.h:180:9: note: Calling 'twl_i2c_read'
return twl_i2c_read(mod_no, val, reg, 1);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mfd/twl-core.c:473:7: note: 'regmap' is null
if (!regmap)
^~~~~~
drivers/mfd/twl-core.c:473:2: note: Taking true branch
if (!regmap)
^
drivers/mfd/twl-core.c:474:3: note: Returning without writing to '*value'
return -EPERM;
^
include/linux/mfd/twl.h:180:9: note: Returning from 'twl_i2c_read'
return twl_i2c_read(mod_no, val, reg, 1);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/mfd/twl.h:180:2: note: Returning without writing to '*val'
return twl_i2c_read(mod_no, val, reg, 1);
^
drivers/mfd/twl-core.c:1204:3: note: Returning from 'twl_i2c_read_u8'
twl_i2c_read_u8(TWL4030_MODULE_INTBR, &temp, REG_GPPUPDCTR1);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/mfd/twl-core.c:1205:8: note: The left expression of the compound assignment is an uninitialized value. The computed value will also be garbage
temp &= ~(SR_I2C_SDA_CTRL_PU | SR_I2C_SCL_CTRL_PU | \
~~~~ ^
Suppressed 6 warnings (6 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
3 warnings generated.
Suppressed 3 warnings (3 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
3 warnings generated.
Suppressed 3 warnings (3 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
8 warnings generated.
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:514:8: warning: Branch condition evaluates to a garbage value [clang-analyzer-core.uninitialized.Branch]
if (!(dflow->en_scaling || dflow->en_img_enhancement))
^
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:1164:8: note: Calling 'komeda_splitter_validate'
err = komeda_splitter_validate(pipe->splitter, conn_st,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:572:6: note: Assuming 'splitter' is non-null
if (!splitter) {
^~~~~~~~~
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:572:2: note: Taking false branch
if (!splitter) {
^
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:577:2: note: Taking false branch
if (!in_range(&splitter->hsize, dflow->in_w)) {
^
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:583:2: note: Taking false branch
if (!in_range(&splitter->vsize, dflow->in_h)) {
^
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:592:2: note: Taking true branch
if (IS_ERR(c_st))
^
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:593:3: note: Returning without writing to 'l_output->en_scaling'
return PTR_ERR(c_st);
^
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:1164:8: note: Returning from 'komeda_splitter_validate'
err = komeda_splitter_validate(pipe->splitter, conn_st,
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:1166:6: note: Assuming 'err' is 0
if (err)
^~~
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:1166:2: note: Taking false branch
if (err)
^
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:1168:8: note: Calling 'komeda_scaler_validate'
err = komeda_scaler_validate(conn, kcrtc_st, &l_dflow);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/arm/display/komeda/komeda_pipeline_state.c:514:8: note: Branch condition evaluates to a garbage value
if (!(dflow->en_scaling || dflow->en_img_enhancement))
^~~~~~~~~~~~~~~~~
Suppressed 7 warnings (7 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
7 warnings generated.
Suppressed 7 warnings (7 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
10 warnings generated.
>> drivers/gpu/drm/arm/display/komeda/komeda_kms.c:121:22: warning: Value stored to 'a' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
struct drm_plane *a = node->base.plane;
^ ~~~~~~~~~~~~~~~~
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:121:22: note: Value stored to 'a' during its initialization is never read
struct drm_plane *a = node->base.plane;
^ ~~~~~~~~~~~~~~~~
>> drivers/gpu/drm/arm/display/komeda/komeda_kms.c:122:22: warning: Value stored to 'b' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
struct drm_plane *b = new->base.plane;
^ ~~~~~~~~~~~~~~~
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:122:22: note: Value stored to 'b' during its initialization is never read
struct drm_plane *b = new->base.plane;
^ ~~~~~~~~~~~~~~~
include/linux/list.h:73:2: warning: Dereference of null pointer [clang-analyzer-core.NullDereference]
WRITE_ONCE(prev->next, new);
^
include/asm-generic/rwonce.h:61:2: note: expanded from macro 'WRITE_ONCE'
__WRITE_ONCE(x, val); \
^
include/asm-generic/rwonce.h:55:30: note: expanded from macro '__WRITE_ONCE'
*(volatile typeof(x) *)&(x) = (val); \
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:205:6: note: Assuming 'err' is 0
if (err)
^~~
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:205:2: note: Taking false branch
if (err)
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:212:2: note: Assuming 'i' is < field 'num_crtc'
for_each_new_crtc_in_state(state, crtc, new_crtc_st, i) {
^
include/drm/drm_atomic.h:846:7: note: expanded from macro 'for_each_new_crtc_in_state'
(__i) < (__state)->dev->mode_config.num_crtc; \
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:212:2: note: Loop condition is true. Entering loop body
for_each_new_crtc_in_state(state, crtc, new_crtc_st, i) {
^
include/drm/drm_atomic.h:845:2: note: expanded from macro 'for_each_new_crtc_in_state'
for ((__i) = 0; \
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:212:2: note: Assuming field 'ptr' is non-null
for_each_new_crtc_in_state(state, crtc, new_crtc_st, i) {
^
include/drm/drm_atomic.h:848:16: note: expanded from macro 'for_each_new_crtc_in_state'
for_each_if ((__state)->crtcs[__i].ptr && \
~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/drm/drm_util.h:63:38: note: expanded from macro 'for_each_if'
#define for_each_if(condition) if (!(condition)) {} else
^~~~~~~~~
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:212:2: note: Left side of '&&' is true
for_each_new_crtc_in_state(state, crtc, new_crtc_st, i) {
^
include/drm/drm_atomic.h:848:16: note: expanded from macro 'for_each_new_crtc_in_state'
for_each_if ((__state)->crtcs[__i].ptr && \
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:212:2: note: Taking false branch
for_each_new_crtc_in_state(state, crtc, new_crtc_st, i) {
^
include/drm/drm_atomic.h:848:3: note: expanded from macro 'for_each_new_crtc_in_state'
for_each_if ((__state)->crtcs[__i].ptr && \
^
include/drm/drm_util.h:63:32: note: expanded from macro 'for_each_if'
#define for_each_if(condition) if (!(condition)) {} else
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:214:7: note: Assuming 'err' is 0
if (err)
^~~
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:214:3: note: Taking false branch
if (err)
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:217:9: note: Calling 'komeda_crtc_normalize_zpos'
err = komeda_crtc_normalize_zpos(crtc, new_crtc_st);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:140:30: note: Left side of '&&' is false
struct komeda_crtc *kcrtc = to_kcrtc(crtc);
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.h:131:21: note: expanded from macro 'to_kcrtc'
#define to_kcrtc(p) container_of(p, struct komeda_crtc, base)
^
include/linux/kernel.h:495:61: note: expanded from macro 'container_of'
BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:140:30: note: Taking false branch
struct komeda_crtc *kcrtc = to_kcrtc(crtc);
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.h:131:21: note: expanded from macro 'to_kcrtc'
#define to_kcrtc(p) container_of(p, struct komeda_crtc, base)
^
include/linux/kernel.h:495:2: note: expanded from macro 'container_of'
BUILD_BUG_ON_MSG(!__same_type(*(ptr), ((type *)0)->member) && \
^
include/linux/build_bug.h:39:37: note: expanded from macro 'BUILD_BUG_ON_MSG'
#define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
^
include/linux/compiler_types.h:322:2: note: expanded from macro 'compiletime_assert'
_compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
^
include/linux/compiler_types.h:310:2: note: expanded from macro '_compiletime_assert'
__compiletime_assert(condition, msg, prefix, suffix)
^
include/linux/compiler_types.h:302:3: note: expanded from macro '__compiletime_assert'
if (!(condition)) \
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.c:140:30: note: Loop condition is false. Exiting loop
struct komeda_crtc *kcrtc = to_kcrtc(crtc);
^
drivers/gpu/drm/arm/display/komeda/komeda_kms.h:131:21: note: expanded from macro 'to_kcrtc'
#define to_kcrtc(p) container_of(p, struct komeda_crtc, base)
--
10 warnings generated.
Suppressed 10 warnings (10 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
14 warnings generated.
drivers/target/target_core_pr.c:227:2: warning: Value stored to 'tpg' is never read [clang-analyzer-deadcode.DeadStores]
tpg = sess->se_tpg;
^ ~~~~~~~~~~~~
drivers/target/target_core_pr.c:227:2: note: Value stored to 'tpg' is never read
tpg = sess->se_tpg;
^ ~~~~~~~~~~~~
drivers/target/target_core_pr.c:1022:26: warning: Value stored to 'se_tpg' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
struct se_portal_group *se_tpg = nacl->se_tpg;
^~~~~~ ~~~~~~~~~~~~
drivers/target/target_core_pr.c:1022:26: note: Value stored to 'se_tpg' during its initialization is never read
struct se_portal_group *se_tpg = nacl->se_tpg;
^~~~~~ ~~~~~~~~~~~~
drivers/target/target_core_pr.c:1279:39: warning: Value stored to 'tfo' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
const struct target_core_fabric_ops *tfo =
^~~
drivers/target/target_core_pr.c:1279:39: note: Value stored to 'tfo' during its initialization is never read
const struct target_core_fabric_ops *tfo =
^~~
drivers/target/target_core_pr.c:1796:3: warning: Value stored to 'dest_se_deve' is never read [clang-analyzer-deadcode.DeadStores]
dest_se_deve = tidh->dest_se_deve;
^ ~~~~~~~~~~~~~~~~~~
drivers/target/target_core_pr.c:1796:3: note: Value stored to 'dest_se_deve' is never read
dest_se_deve = tidh->dest_se_deve;
^ ~~~~~~~~~~~~~~~~~~
drivers/target/target_core_pr.c:1954:3: warning: Value stored to 'len' is never read [clang-analyzer-deadcode.DeadStores]
len += sprintf(buf+len, "No Registrations or Reservations");
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/target/target_core_pr.c:1954:3: note: Value stored to 'len' is never read
len += sprintf(buf+len, "No Registrations or Reservations");
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/target/target_core_pr.c:2450:39: warning: Value stored to 'tfo' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
const struct target_core_fabric_ops *tfo = se_nacl->se_tpg->se_tpg_tfo;
^~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/target/target_core_pr.c:2450:39: note: Value stored to 'tfo' during its initialization is never read
const struct target_core_fabric_ops *tfo = se_nacl->se_tpg->se_tpg_tfo;
^~~ ~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/target/target_core_pr.c:2760:39: warning: Value stored to 'tfo' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
const struct target_core_fabric_ops *tfo = nacl->se_tpg->se_tpg_tfo;
^~~ ~~~~~~~~~~~~~~~~~~~~~~~~
drivers/target/target_core_pr.c:2760:39: note: Value stored to 'tfo' during its initialization is never read
const struct target_core_fabric_ops *tfo = nacl->se_tpg->se_tpg_tfo;
^~~ ~~~~~~~~~~~~~~~~~~~~~~~~
drivers/target/target_core_pr.c:3163:2: warning: Value stored to 'tf_ops' is never read [clang-analyzer-deadcode.DeadStores]
tf_ops = se_tpg->se_tpg_tfo;
^ ~~~~~~~~~~~~~~~~~~
drivers/target/target_core_pr.c:3163:2: note: Value stored to 'tf_ops' is never read
tf_ops = se_tpg->se_tpg_tfo;
^ ~~~~~~~~~~~~~~~~~~
drivers/target/target_core_pr.c:3924:3: warning: Value stored to 'add_desc_len' is never read [clang-analyzer-deadcode.DeadStores]
add_desc_len = 0;
^ ~
drivers/target/target_core_pr.c:3924:3: note: Value stored to 'add_desc_len' is never read
add_desc_len = 0;
^ ~
Suppressed 5 warnings (5 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
7 warnings generated.
drivers/media/dvb-frontends/horus3a.c:141:23: warning: Value stored to 'priv' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
struct horus3a_priv *priv = fe->tuner_priv;
^~~~ ~~~~~~~~~~~~~~
drivers/media/dvb-frontends/horus3a.c:141:23: note: Value stored to 'priv' during its initialization is never read
struct horus3a_priv *priv = fe->tuner_priv;
^~~~ ~~~~~~~~~~~~~~
drivers/media/dvb-frontends/horus3a.c:149:23: warning: Value stored to 'priv' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
struct horus3a_priv *priv = fe->tuner_priv;
^~~~ ~~~~~~~~~~~~~~
drivers/media/dvb-frontends/horus3a.c:149:23: note: Value stored to 'priv' during its initialization is never read
struct horus3a_priv *priv = fe->tuner_priv;
^~~~ ~~~~~~~~~~~~~~
Suppressed 5 warnings (5 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
7 warnings generated.
drivers/media/dvb-frontends/helene.c:432:22: warning: Value stored to 'priv' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
struct helene_priv *priv = fe->tuner_priv;
^~~~ ~~~~~~~~~~~~~~
drivers/media/dvb-frontends/helene.c:432:22: note: Value stored to 'priv' during its initialization is never read
struct helene_priv *priv = fe->tuner_priv;
^~~~ ~~~~~~~~~~~~~~
drivers/media/dvb-frontends/helene.c:452:22: warning: Value stored to 'priv' during its initialization is never read [clang-analyzer-deadcode.DeadStores]
struct helene_priv *priv = fe->tuner_priv;
^~~~ ~~~~~~~~~~~~~~
drivers/media/dvb-frontends/helene.c:452:22: note: Value stored to 'priv' during its initialization is never read
struct helene_priv *priv = fe->tuner_priv;
^~~~ ~~~~~~~~~~~~~~
Suppressed 5 warnings (5 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
4 warnings generated.
Suppressed 4 warnings (4 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
7 warnings generated.
Suppressed 7 warnings (7 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
7 warnings generated.
Suppressed 7 warnings (7 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
10 warnings generated.
>> drivers/gpu/drm/i2c/tda998x_drv.c:798:3: warning: Value stored to 'flag0' is never read [clang-analyzer-deadcode.DeadStores]
flag0 = reg_read(priv, REG_INT_FLAGS_0);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i2c/tda998x_drv.c:798:3: note: Value stored to 'flag0' is never read
flag0 = reg_read(priv, REG_INT_FLAGS_0);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/gpu/drm/i2c/tda998x_drv.c:799:3: warning: Value stored to 'flag1' is never read [clang-analyzer-deadcode.DeadStores]
flag1 = reg_read(priv, REG_INT_FLAGS_1);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/i2c/tda998x_drv.c:799:3: note: Value stored to 'flag1' is never read
flag1 = reg_read(priv, REG_INT_FLAGS_1);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Suppressed 8 warnings (8 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
5 warnings generated.
Suppressed 5 warnings (5 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
6 warnings generated.
Suppressed 6 warnings (6 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
7 warnings generated.
Suppressed 7 warnings (7 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
8 warnings generated.
drivers/gpu/drm/panel/panel-kingdisplay-kd097d04.c:366:3: warning: Value stored to 'err' is never read [clang-analyzer-deadcode.DeadStores]
err = PTR_ERR(kingdisplay->enable_gpio);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/panel/panel-kingdisplay-kd097d04.c:366:3: note: Value stored to 'err' is never read
err = PTR_ERR(kingdisplay->enable_gpio);
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Suppressed 7 warnings (7 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
6 warnings generated.
Suppressed 6 warnings (6 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
7 warnings generated.
Suppressed 7 warnings (7 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
7 warnings generated.
Suppressed 7 warnings (7 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
7 warnings generated.
Suppressed 7 warnings (7 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
5 warnings generated.
Suppressed 5 warnings (5 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
5 warnings generated.
Suppressed 5 warnings (5 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
5 warnings generated.
Suppressed 5 warnings (5 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
6 warnings generated.
drivers/usb/dwc3/core.c:56:2: warning: Value stored to 'mode' is never read [clang-analyzer-deadcode.DeadStores]
mode = dwc->dr_mode;
^ ~~~~~~~~~~~~
drivers/usb/dwc3/core.c:56:2: note: Value stored to 'mode' is never read
mode = dwc->dr_mode;
^ ~~~~~~~~~~~~
Suppressed 5 warnings (5 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
13 warnings generated.
Suppressed 13 warnings (13 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.
10 warnings generated.
drivers/usb/dwc3/gadget.c:687:2: warning: Value stored to 'dep' is never read [clang-analyzer-deadcode.DeadStores]
dep = dwc->eps[1];
^ ~~~~~~~~~~~
drivers/usb/dwc3/gadget.c:687:2: note: Value stored to 'dep' is never read
dep = dwc->eps[1];
^ ~~~~~~~~~~~
drivers/usb/dwc3/gadget.c:698:3: warning: Value stored to 'dep' is never read [clang-analyzer-deadcode.DeadStores]
dep = dwc->eps[num];
^ ~~~~~~~~~~~~~
drivers/usb/dwc3/gadget.c:698:3: note: Value stored to 'dep' is never read
dep = dwc->eps[num];
^ ~~~~~~~~~~~~~
drivers/usb/dwc3/gadget.c:1681:39: warning: The result of the left shift is undefined due to shifting '0' by '14', which is unrepresentable in the unsigned version of the return type 'int' [clang-analyzer-core.UndefinedBinaryOperatorResult]
test_frame_number |= dep->combo_num << 14;
^
drivers/usb/dwc3/gadget.c:3450:6: note: Assuming the condition is false
if (!(dep->flags & DWC3_EP_ENABLED)) {
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/usb/dwc3/gadget.c:3450:2: note: Taking false branch
if (!(dep->flags & DWC3_EP_ENABLED)) {
^
drivers/usb/dwc3/gadget.c:3459:6: note: Assuming 'epnum' is not equal to 0
if (epnum == 0 || epnum == 1) {
^~~~~~~~~~
drivers/usb/dwc3/gadget.c:3459:6: note: Left side of '||' is false
drivers/usb/dwc3/gadget.c:3459:20: note: Assuming 'epnum' is not equal to 1
if (epnum == 0 || epnum == 1) {
^~~~~~~~~~
drivers/usb/dwc3/gadget.c:3459:2: note: Taking false branch
if (epnum == 0 || epnum == 1) {
^
drivers/usb/dwc3/gadget.c:3464:2: note: Control jumps to 'case 3:' at line 3468
switch (event->endpoint_event) {
^
drivers/usb/dwc3/gadget.c:3469:3: note: Calling 'dwc3_gadget_endpoint_transfer_not_ready'
dwc3_gadget_endpoint_transfer_not_ready(dep, event);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/usb/dwc3/gadget.c:3325:2: note: Calling 'dwc3_gadget_endpoint_frame_from_event'
dwc3_gadget_endpoint_frame_from_event(dep, event);
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/usb/dwc3/gadget.c:3241:1: note: Returning without writing to 'dep->combo_num'
vim +/a +121 drivers/gpu/drm/arm/display/komeda/komeda_kms.c
61f1c4a8ab7575 james qian wang (Arm Technology China 2019-01-03 96)
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 97) static int komeda_plane_state_list_add(struct drm_plane_state *plane_st,
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 98) struct list_head *zorder_list)
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 99) {
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 100) struct komeda_plane_state *new = to_kplane_st(plane_st);
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 101) struct komeda_plane_state *node, *last;
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 102)
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 103) last = list_empty(zorder_list) ?
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 104) NULL : list_last_entry(zorder_list, typeof(*last), zlist_node);
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 105)
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 106) /* Considering the list sequence is zpos increasing, so if list is empty
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 107) * or the zpos of new node bigger than the last node in list, no need
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 108) * loop and just insert the new one to the tail of the list.
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 109) */
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 110) if (!last || (new->base.zpos > last->base.zpos)) {
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 111) list_add_tail(&new->zlist_node, zorder_list);
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 112) return 0;
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 113) }
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 114)
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 115) /* Build the list by zpos increasing */
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 116) list_for_each_entry(node, zorder_list, zlist_node) {
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 117) if (new->base.zpos < node->base.zpos) {
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 118) list_add_tail(&new->zlist_node, &node->zlist_node);
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 119) break;
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 120) } else if (node->base.zpos == new->base.zpos) {
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 @121) struct drm_plane *a = node->base.plane;
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 @122) struct drm_plane *b = new->base.plane;
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 123)
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 124) /* Komeda doesn't support setting a same zpos for
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 125) * different planes.
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 126) */
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 127) DRM_DEBUG_ATOMIC("PLANE: %s and PLANE: %s are configured same zpos: %d.\n",
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 128) a->name, b->name, node->base.zpos);
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 129) return -EINVAL;
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 130) }
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 131) }
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 132)
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 133) return 0;
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 134) }
109bd7d5f4c2db Lowry Li (Arm Technology China 2019-05-20 135)
:::::: The code at line 121 was first introduced by commit
:::::: 109bd7d5f4c2db66541272e5a41aeabd6cfeef95 drm/komeda: Adds zorder support
:::::: TO: Lowry Li (Arm Technology China) <Lowry.Li@arm.com>
:::::: CC: Liviu Dudau <Liviu.Dudau@arm.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
next reply other threads:[~2022-01-22 20:15 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-22 20:15 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2022-01-25 14:43 [jimc:dd-drm-rfc 13/18] drivers/gpu/drm/arm/display/komeda/komeda_kms.c:121:22: warning: Value stored to 'a' during its initialization is never read [clang-analyzer-deadcode.DeadStores] kernel test robot
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=202201230137.mBVZ14jX-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild@lists.01.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.