CC: [email protected]
CC: [email protected]
TO: Krzysztof Kozlowski <[email protected]>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
master
head:   228a65d4544af5086bd167dcc5a0cb4fae2c42b4
commit: ea0c0ad6b6eb36726088991d97a55b99cae456d0 memory: Enable compile testing 
for most of the drivers
date:   5 months ago
:::::: branch date: 3 hours ago
:::::: commit date: 5 months ago
compiler: ia64-linux-gcc (GCC) 9.3.0

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


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

>> drivers/memory/emif.c:628:30: warning: Shifting signed 32-bit value by 31 
>> bits is undefined behaviour [shiftTooManyBitsSigned]
    alert |= (cs1_used ? 1 : 0) << TA_CS1EN_SHIFT;
                                ^
   drivers/memory/emif.c:849:20: warning: Shifting signed 32-bit value by 31 
bits is undefined behaviour [shiftTooManyBitsSigned]
     writel(DDR_MR4 | CS_MASK, base + EMIF_LPDDR2_MODE_REG_CONFIG);
                      ^

vim +628 drivers/memory/emif.c

98231c4fc3b95a0e Aneesh V 2012-04-27  597  
98231c4fc3b95a0e Aneesh V 2012-04-27  598  static u32 
get_temp_alert_config(const struct lpddr2_addressing *addressing,
98231c4fc3b95a0e Aneesh V 2012-04-27  599               const struct 
emif_custom_configs *custom_configs, bool cs1_used,
98231c4fc3b95a0e Aneesh V 2012-04-27  600               u32 sdram_io_width, u32 
emif_bus_width)
98231c4fc3b95a0e Aneesh V 2012-04-27  601  {
98231c4fc3b95a0e Aneesh V 2012-04-27  602       u32 alert = 0, interval, devcnt;
98231c4fc3b95a0e Aneesh V 2012-04-27  603  
98231c4fc3b95a0e Aneesh V 2012-04-27  604       if (custom_configs && 
(custom_configs->mask &
98231c4fc3b95a0e Aneesh V 2012-04-27  605                               
EMIF_CUSTOM_CONFIG_TEMP_ALERT_POLL_INTERVAL))
98231c4fc3b95a0e Aneesh V 2012-04-27  606               interval = 
custom_configs->temp_alert_poll_interval_ms;
98231c4fc3b95a0e Aneesh V 2012-04-27  607       else
98231c4fc3b95a0e Aneesh V 2012-04-27  608               interval = 
TEMP_ALERT_POLL_INTERVAL_DEFAULT_MS;
98231c4fc3b95a0e Aneesh V 2012-04-27  609  
98231c4fc3b95a0e Aneesh V 2012-04-27  610       interval *= 1000000;            
        /* Convert to ns */
98231c4fc3b95a0e Aneesh V 2012-04-27  611       interval /= 
addressing->tREFI_ns;       /* Convert to refresh cycles */
98231c4fc3b95a0e Aneesh V 2012-04-27  612       alert |= (interval << 
TA_REFINTERVAL_SHIFT);
98231c4fc3b95a0e Aneesh V 2012-04-27  613  
98231c4fc3b95a0e Aneesh V 2012-04-27  614       /*
98231c4fc3b95a0e Aneesh V 2012-04-27  615        * sdram_io_width is in 
'log2(x) - 1' form. Convert emif_bus_width
98231c4fc3b95a0e Aneesh V 2012-04-27  616        * also to this form and 
subtract to get TA_DEVCNT, which is
98231c4fc3b95a0e Aneesh V 2012-04-27  617        * in log2(x) form.
98231c4fc3b95a0e Aneesh V 2012-04-27  618        */
98231c4fc3b95a0e Aneesh V 2012-04-27  619       emif_bus_width = 
__fls(emif_bus_width) - 1;
98231c4fc3b95a0e Aneesh V 2012-04-27  620       devcnt = emif_bus_width - 
sdram_io_width;
98231c4fc3b95a0e Aneesh V 2012-04-27  621       alert |= devcnt << 
TA_DEVCNT_SHIFT;
98231c4fc3b95a0e Aneesh V 2012-04-27  622  
98231c4fc3b95a0e Aneesh V 2012-04-27  623       /* DEVWDT is in 'log2(x) - 3' 
form */
98231c4fc3b95a0e Aneesh V 2012-04-27  624       alert |= (sdram_io_width - 2) 
<< TA_DEVWDT_SHIFT;
98231c4fc3b95a0e Aneesh V 2012-04-27  625  
98231c4fc3b95a0e Aneesh V 2012-04-27  626       alert |= 1 << TA_SFEXITEN_SHIFT;
98231c4fc3b95a0e Aneesh V 2012-04-27  627       alert |= 1 << TA_CS0EN_SHIFT;
98231c4fc3b95a0e Aneesh V 2012-04-27 @628       alert |= (cs1_used ? 1 : 0) << 
TA_CS1EN_SHIFT;
98231c4fc3b95a0e Aneesh V 2012-04-27  629  
98231c4fc3b95a0e Aneesh V 2012-04-27  630       return alert;
98231c4fc3b95a0e Aneesh V 2012-04-27  631  }
98231c4fc3b95a0e Aneesh V 2012-04-27  632  

:::::: The code at line 628 was first introduced by commit
:::::: 98231c4fc3b95a0e9d37ba3711ad993bb27605c8 memory: emif: add one-time 
settings

:::::: TO: Aneesh V <[email protected]>
:::::: CC: Greg Kroah-Hartman <[email protected]>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/[email protected]
_______________________________________________
kbuild mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to