CC: [email protected]
CC: [email protected]
CC: Linux Memory Management List <[email protected]>
TO: AngeloGioacchino Del Regno <[email protected]>
CC: Sam Ravnborg <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git 
master
head:   5191249f880367a4cd675825cd721a8d78f26a45
commit: a19125a281128c836faae249c1001eb8d7a18b48 [312/2290] drm/panel: Add BOE 
BF060Y8M-AJ0 5.99" AMOLED panel driver
:::::: branch date: 2 days ago
:::::: commit date: 5 weeks ago
config: arm-randconfig-c002-20211119 (attached as .config)
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
        # install arm cross compiling tool for clang build
        # apt-get install binutils-arm-linux-gnueabi
        # 
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=a19125a281128c836faae249c1001eb8d7a18b48
        git remote add linux-next 
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git
        git fetch --no-tags linux-next master
        git checkout a19125a281128c836faae249c1001eb8d7a18b48
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm 
clang-analyzer 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <[email protected]>


clang-analyzer warnings: (new ones prefixed by >>)
           if ((lis3->whoami == WAI_3DC) || (lis3->whoami == WAI_3DLH)) {
                ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/misc/lis3lv02d/lis3lv02d.c:289:32: note: Left side of '||' is true
           if ((lis3->whoami == WAI_3DC) || (lis3->whoami == WAI_3DLH)) {
                                         ^
   drivers/misc/lis3lv02d/lis3lv02d.c:303:6: note: 'ret' is -6
           if (ret)
               ^~~
   drivers/misc/lis3lv02d/lis3lv02d.c:303:2: note: Taking true branch
           if (ret)
           ^
   drivers/misc/lis3lv02d/lis3lv02d.c:304:3: note: Control jumps to line 356
                   goto fail;
                   ^
   drivers/misc/lis3lv02d/lis3lv02d.c:357:2: note: Returning without writing to 
'*results'
           return ret;
           ^
   drivers/misc/lis3lv02d/lis3lv02d.c:794:10: note: Returning from 
'lis3lv02d_selftest'
           switch (lis3lv02d_selftest(lis3, values)) {
                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/misc/lis3lv02d/lis3lv02d.c:794:2: note: Control jumps to the 
'default' case at line 802
           switch (lis3lv02d_selftest(lis3, values)) {
           ^
   drivers/misc/lis3lv02d/lis3lv02d.c:804:3: note:  Execution continues on line 
806
                   break;
                   ^
   drivers/misc/lis3lv02d/lis3lv02d.c:806:9: note: 4th function call argument 
is an uninitialized value
           return sprintf(buf, "%s %d %d %d\n", res,
                  ^
   Suppressed 1 warnings (1 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.
   1 warning generated.
   drivers/clk/bcm/clk-iproc-pll.c:182:3: warning: Value stored to 'val' is 
never read [clang-analyzer-deadcode.DeadStores]
                   val = readl(base + offset);
                   ^
   drivers/clk/bcm/clk-iproc-pll.c:182:3: note: Value stored to 'val' is never 
read
   2 warnings generated.
   drivers/gpu/drm/tegra/fb.c:126:14: warning: Assigned value is garbage or 
undefined [clang-analyzer-core.uninitialized.Assign]
                   fb->obj[i] = &planes[i]->gem;
                              ^
   drivers/gpu/drm/tegra/fb.c:150:14: note: Assuming 'i' is >= field 
'num_planes'
           for (i = 0; i < info->num_planes; i++) {
                       ^~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/tegra/fb.c:150:2: note: Loop condition is false. Execution 
continues on line 174
           for (i = 0; i < info->num_planes; i++) {
           ^
   drivers/gpu/drm/tegra/fb.c:174:7: note: Calling 'tegra_fb_alloc'
           fb = tegra_fb_alloc(drm, cmd, planes, i);
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/tegra/fb.c:119:7: note: Calling 'kzalloc'
           fb = kzalloc(sizeof(*fb), GFP_KERNEL);
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:721:9: note: Calling 'kmalloc'
           return kmalloc(size, flags | __GFP_ZERO);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:579:2: note: Taking false branch
           if (__builtin_constant_p(size)) {
           ^
   include/linux/slab.h:596:2: note: Returning pointer, which participates in a 
condition later
           return __kmalloc(size, flags);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:721:9: note: Returning from 'kmalloc'
           return kmalloc(size, flags | __GFP_ZERO);
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/slab.h:721:2: note: Returning pointer, which participates in a 
condition later
           return kmalloc(size, flags | __GFP_ZERO);
           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/tegra/fb.c:119:7: note: Returning from 'kzalloc'
           fb = kzalloc(sizeof(*fb), GFP_KERNEL);
                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/tegra/fb.c:120:6: note: Assuming 'fb' is non-null
           if (!fb)
               ^~~
   drivers/gpu/drm/tegra/fb.c:120:2: note: Taking false branch
           if (!fb)
           ^
   drivers/gpu/drm/tegra/fb.c:125:7: note: The value 0 is assigned to 'i'
           for (i = 0; i < fb->format->num_planes; i++)
                ^~~~~
   drivers/gpu/drm/tegra/fb.c:125:14: note: Assuming 'i' is < field 'num_planes'
           for (i = 0; i < fb->format->num_planes; i++)
                       ^~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/tegra/fb.c:125:2: note: Loop condition is true.  Entering 
loop body
           for (i = 0; i < fb->format->num_planes; i++)
           ^
   drivers/gpu/drm/tegra/fb.c:126:14: note: Assigned value is garbage or 
undefined
                   fb->obj[i] = &planes[i]->gem;
                              ^ ~~~~~~~~~~~~~~~
   Suppressed 1 warnings (1 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.
   1 warning generated.
   Suppressed 1 warnings (1 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.
   1 warning generated.
   Suppressed 1 warnings (1 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.
   1 warning generated.
   Suppressed 1 warnings (1 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.
   2 warnings generated.
>> drivers/gpu/drm/panel/panel-boe-bf060y8m-aj0.c:205:2: warning: Value stored 
>> to 'ret' is never read [clang-analyzer-deadcode.DeadStores]
           ret = regulator_bulk_disable(ARRAY_SIZE(boe->vregs), boe->vregs);
           ^     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/panel/panel-boe-bf060y8m-aj0.c:205:2: note: Value stored to 
'ret' is never read
           ret = regulator_bulk_disable(ARRAY_SIZE(boe->vregs), boe->vregs);
           ^     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   Suppressed 1 warnings (1 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.
   1 warning generated.
   Suppressed 1 warnings (1 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.
   1 warning generated.
   Suppressed 1 warnings (1 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.
   2 warnings generated.
   drivers/gpu/drm/panel/panel-elida-kd35t133.c:65:17: warning: Value stored to 
'dev' during its initialization is never read 
[clang-analyzer-deadcode.DeadStores]
           struct device *dev = ctx->dev;
                          ^~~   ~~~~~~~~
   drivers/gpu/drm/panel/panel-elida-kd35t133.c:65:17: note: Value stored to 
'dev' during its initialization is never read
           struct device *dev = ctx->dev;
                          ^~~   ~~~~~~~~
   Suppressed 1 warnings (1 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.
   1 warning generated.
   Suppressed 1 warnings (1 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.
   2 warnings generated.
   drivers/usb/serial/ir-usb.c:96:17: warning: Value stored to 'dev' during its 
initialization is never read [clang-analyzer-deadcode.DeadStores]
           struct device *dev = &serial->dev->dev;
                          ^~~   ~~~~~~~~~~~~~~~~~
   drivers/usb/serial/ir-usb.c:96:17: note: Value stored to 'dev' during its 
initialization is never read
           struct device *dev = &serial->dev->dev;
                          ^~~   ~~~~~~~~~~~~~~~~~
   drivers/usb/serial/ir-usb.c:209:2: warning: Value stored to 'rates' is never 
read [clang-analyzer-deadcode.DeadStores]
           rates = le16_to_cpu(irda_desc->wBaudRate);
           ^
   drivers/usb/serial/ir-usb.c:209:2: note: Value stored to 'rates' is never 
read
   10 warnings generated.
   drivers/usb/serial/iuu_phoenix.c:241:6: warning: Value stored to 'status' 
during its initialization is never read [clang-analyzer-deadcode.DeadStores]
           int status = urb->status;
               ^~~~~~   ~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:241:6: note: Value stored to 'status' 
during its initialization is never read
           int status = urb->status;
               ^~~~~~   ~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:530:17: warning: Value stored to 'dev' 
during its initialization is never read [clang-analyzer-deadcode.DeadStores]
           struct device *dev = &port->dev;
                          ^~~   ~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:530:17: note: Value stored to 'dev' during 
its initialization is never read
           struct device *dev = &port->dev;
                          ^~~   ~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:669:3: warning: Value stored to 'status' is 
never read [clang-analyzer-deadcode.DeadStores]
                   status = iuu_read_buf(port, len);
                   ^        ~~~~~~~~~~~~~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:669:3: note: Value stored to 'status' is 
never read
                   status = iuu_read_buf(port, len);
                   ^        ~~~~~~~~~~~~~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:674:3: warning: Value stored to 'status' is 
never read [clang-analyzer-deadcode.DeadStores]
                   status = iuu_status(port);
                   ^        ~~~~~~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:674:3: note: Value stored to 'status' is 
never read
                   status = iuu_status(port);
                   ^        ~~~~~~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:682:3: warning: Value stored to 'status' is 
never read [clang-analyzer-deadcode.DeadStores]
                   status = iuu_reset(port, 0xC);
                   ^        ~~~~~~~~~~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:682:3: note: Value stored to 'status' is 
never read
                   status = iuu_reset(port, 0xC);
                   ^        ~~~~~~~~~~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:689:3: warning: Value stored to 'status' is 
never read [clang-analyzer-deadcode.DeadStores]
                   status = iuu_bulk_write(port);
                   ^        ~~~~~~~~~~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:689:3: note: Value stored to 'status' is 
never read
                   status = iuu_bulk_write(port);
                   ^        ~~~~~~~~~~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:735:2: warning: Value stored to 'result' is 
never read [clang-analyzer-deadcode.DeadStores]
           result = usb_submit_urb(port->read_urb, GFP_ATOMIC);
           ^        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:735:2: note: Value stored to 'result' is 
never read
           result = usb_submit_urb(port->read_urb, GFP_ATOMIC);
           ^        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/usb/serial/iuu_phoenix.c:996:3: warning: Value stored to 'result' is 
never read [clang-analyzer-deadcode.DeadStores]
                   result =
                   ^
   drivers/usb/serial/iuu_phoenix.c:996:3: note: Value stored to 'result' is 
never read
                   result =
                   ^
   drivers/usb/serial/iuu_phoenix.c:1004:3: warning: Value stored to 'result' 
is never read [clang-analyzer-deadcode.DeadStores]
                   result =
                   ^
   drivers/usb/serial/iuu_phoenix.c:1004:3: note: Value stored to 'result' is 
never read
                   result =
                   ^
   drivers/usb/serial/iuu_phoenix.c:1011:3: warning: Value stored to 'result' 
is never read [clang-analyzer-deadcode.DeadStores]
                   result =
                   ^
   drivers/usb/serial/iuu_phoenix.c:1011:3: note: Value stored to 'result' is 
never read
                   result =
                   ^
   1 warning generated.
   Suppressed 1 warnings (1 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.

vim +/ret +205 drivers/gpu/drm/panel/panel-boe-bf060y8m-aj0.c

a19125a281128c AngeloGioacchino Del Regno 2021-09-01  190  
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  191  static int 
boe_bf060y8m_aj0_unprepare(struct drm_panel *panel)
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  192  {
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  193       struct 
boe_bf060y8m_aj0 *boe = to_boe_bf060y8m_aj0(panel);
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  194       struct device 
*dev = &boe->dsi->dev;
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  195       int ret;
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  196  
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  197       if 
(!boe->prepared)
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  198               return 
0;
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  199  
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  200       ret = 
boe_bf060y8m_aj0_off(boe);
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  201       if (ret < 0)
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  202               
dev_err(dev, "Failed to un-initialize panel: %d\n", ret);
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  203  
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  204       
gpiod_set_value_cansleep(boe->reset_gpio, 1);
a19125a281128c AngeloGioacchino Del Regno 2021-09-01 @205       ret = 
regulator_bulk_disable(ARRAY_SIZE(boe->vregs), boe->vregs);
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  206  
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  207       boe->prepared = 
false;
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  208       return 0;
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  209  }
a19125a281128c AngeloGioacchino Del Regno 2021-09-01  210  

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]

Attachment: .config.gz
Description: application/gzip

_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to