:::::: 
:::::: Manual check reason: "low confidence static check warning: 
arch/arm/mm/dma-mapping.c:77:57: warning: Parameter 'virt' can be declared with 
const [constParameter]"
:::::: 

CC: kbuild-...@lists.01.org
BCC: l...@intel.com
TO: Christoph Hellwig <h...@lst.de>

tree:   git://git.infradead.org/users/hch/dma-mapping for-next
head:   4136ce90f079e812fec2c5e10732b2265f61f1bd
commit: ae626eb97376148bb63c3f3ca9517fde0f39bec3 [11/14] ARM/dma-mapping: use 
dma-direct unconditionally
:::::: branch date: 12 hours ago
:::::: commit date: 12 hours ago
compiler: arm-linux-gnueabi-gcc (GCC) 11.3.0
reproduce (cppcheck warning):
        # apt-get install cppcheck
        git checkout ae626eb97376148bb63c3f3ca9517fde0f39bec3
        cppcheck --quiet --enable=style,performance,portability --template=gcc 
FILE

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <l...@intel.com>


cppcheck possible warnings: (new ones prefixed by >>, may not real problems)

>> arch/arm/mm/dma-mapping.c:77:57: warning: Parameter 'virt' can be declared 
>> with const [constParameter]
   static struct arm_dma_buffer *arm_dma_buffer_find(void *virt)
                                                           ^
>> arch/arm/mm/dma-mapping.c:84:12: warning: Uninitialized variable: buf->virt 
>> [uninitvar]
     if (buf->virt == virt) {
              ^

vim +/virt +77 arch/arm/mm/dma-mapping.c

19e6e5e5392bd64 Rabin Vincent 2016-03-03  76  
19e6e5e5392bd64 Rabin Vincent 2016-03-03 @77  static struct arm_dma_buffer 
*arm_dma_buffer_find(void *virt)
19e6e5e5392bd64 Rabin Vincent 2016-03-03  78  {
19e6e5e5392bd64 Rabin Vincent 2016-03-03  79    struct arm_dma_buffer *buf, 
*found = NULL;
19e6e5e5392bd64 Rabin Vincent 2016-03-03  80    unsigned long flags;
19e6e5e5392bd64 Rabin Vincent 2016-03-03  81  
19e6e5e5392bd64 Rabin Vincent 2016-03-03  82    
spin_lock_irqsave(&arm_dma_bufs_lock, flags);
19e6e5e5392bd64 Rabin Vincent 2016-03-03  83    list_for_each_entry(buf, 
&arm_dma_bufs, list) {
19e6e5e5392bd64 Rabin Vincent 2016-03-03 @84            if (buf->virt == virt) {
19e6e5e5392bd64 Rabin Vincent 2016-03-03  85                    
list_del(&buf->list);
19e6e5e5392bd64 Rabin Vincent 2016-03-03  86                    found = buf;
19e6e5e5392bd64 Rabin Vincent 2016-03-03  87                    break;
19e6e5e5392bd64 Rabin Vincent 2016-03-03  88            }
19e6e5e5392bd64 Rabin Vincent 2016-03-03  89    }
19e6e5e5392bd64 Rabin Vincent 2016-03-03  90    
spin_unlock_irqrestore(&arm_dma_bufs_lock, flags);
19e6e5e5392bd64 Rabin Vincent 2016-03-03  91    return found;
19e6e5e5392bd64 Rabin Vincent 2016-03-03  92  }
19e6e5e5392bd64 Rabin Vincent 2016-03-03  93  

:::::: The code at line 77 was first introduced by commit
:::::: 19e6e5e5392bd646c93d9e2c7b2b58c8558cb041 ARM: 8547/1: dma-mapping: store 
buffer information

:::::: TO: Rabin Vincent <ra...@rab.in>
:::::: CC: Russell King <rmk+ker...@arm.linux.org.uk>

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp
_______________________________________________
kbuild mailing list -- kbuild@lists.01.org
To unsubscribe send an email to kbuild-le...@lists.01.org

Reply via email to