CC: kbuild-...@lists.01.org
TO: Arnd Bergmann <a...@arndb.de>
CC: Krzysztof Kozlowski <k...@kernel.org>

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git 
master
head:   7ce53e3a447bced7b85ed181c4d027e93c062e07
commit: 71b9114d2c13a648fbe6523dd859e611c316ad90 [1952/6654] ARM: s3c: move 
into a common directory
:::::: branch date: 26 hours ago
:::::: commit date: 3 weeks ago
compiler: arm-linux-gnueabi-gcc (GCC) 9.3.0

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


cppcheck warnings: (new ones prefixed by >>)

>> arch/arm/mach-s3c/irq-s3c24xx.c:399:14: warning: Shifting signed 32-bit 
>> value by 31 bits is undefined behaviour. See condition at line 396. 
>> [shiftTooManyBitsSigned]
     intmod = 1 << offs;
                ^
   arch/arm/mach-s3c/irq-s3c24xx.c:396:12: note: Assuming that condition 
'offs>31' is not redundant
     if (offs > 31)
              ^
   arch/arm/mach-s3c/irq-s3c24xx.c:399:14: note: Shift
     intmod = 1 << offs;
                ^

# 
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?id=71b9114d2c13a648fbe6523dd859e611c316ad90
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 71b9114d2c13a648fbe6523dd859e611c316ad90
vim +399 arch/arm/mach-s3c/irq-s3c24xx.c

17453dd2e7df20 arch/arm/mach-s3c24xx/irq.c         Heiko Stuebner 2013-03-07  
374  
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
375  #ifdef CONFIG_FIQ
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
376  /**
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
377   * s3c24xx_set_fiq - set the FIQ routing
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
378   * @irq: IRQ number to route to FIQ on processor.
cd4bd8f9435ddf arch/arm/mach-s3c24xx/irq-s3c24xx.c Arnd Bergmann  2020-08-06  
379   * @ack_ptr: pointer to a location for storing the bit mask
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
380   * @on: Whether to route @irq to the FIQ, or to remove the FIQ routing.
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
381   *
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
382   * Change the state of the IRQ to FIQ routing depending on @irq and @on. If
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
383   * @on is true, the @irq is checked to see if it can be routed and the
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
384   * interrupt controller updated to route the IRQ. If @on is false, the FIQ
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
385   * routing is cleared, regardless of which @irq is specified.
cd4bd8f9435ddf arch/arm/mach-s3c24xx/irq-s3c24xx.c Arnd Bergmann  2020-08-06  
386   *
cd4bd8f9435ddf arch/arm/mach-s3c24xx/irq-s3c24xx.c Arnd Bergmann  2020-08-06  
387   * returns the mask value for the register.
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
388   */
cd4bd8f9435ddf arch/arm/mach-s3c24xx/irq-s3c24xx.c Arnd Bergmann  2020-08-06  
389  int s3c24xx_set_fiq(unsigned int irq, u32 *ack_ptr, bool on)
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
390  {
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
391       u32 intmod;
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
392       unsigned offs;
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
393  
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
394       if (on) {
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
395               offs = irq - FIQ_START;
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03 
@396               if (offs > 31)
cd4bd8f9435ddf arch/arm/mach-s3c24xx/irq-s3c24xx.c Arnd Bergmann  2020-08-06  
397                       return 0;
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
398  
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03 
@399               intmod = 1 << offs;
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
400       } else {
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
401               intmod = 0;
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
402       }
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
403  
cd4bd8f9435ddf arch/arm/mach-s3c24xx/irq-s3c24xx.c Arnd Bergmann  2020-08-06  
404       if (ack_ptr)
cd4bd8f9435ddf arch/arm/mach-s3c24xx/irq-s3c24xx.c Arnd Bergmann  2020-08-06  
405               *ack_ptr = intmod;
353332855eef20 drivers/irqchip/irq-s3c24xx.c       Ben Dooks      2016-06-21  
406       writel_relaxed(intmod, S3C2410_INTMOD);
cd4bd8f9435ddf arch/arm/mach-s3c24xx/irq-s3c24xx.c Arnd Bergmann  2020-08-06  
407  
cd4bd8f9435ddf arch/arm/mach-s3c24xx/irq-s3c24xx.c Arnd Bergmann  2020-08-06  
408       return intmod;
229fd8ffba57df arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-08-03  
409  }
0f13c8248040d6 arch/arm/plat-s3c24xx/irq.c         Ben Dooks      2009-12-07  
410  

:::::: The code at line 399 was first introduced by commit
:::::: 229fd8ffba57dfaea59078b9144371369ffdb0a3 ARM: S3C24XX: Add FIQ IRQ 
routing support

:::::: TO: Ben Dooks <b...@simtec.co.uk>
:::::: CC: Ben Dooks <ben-li...@fluff.org>

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

Reply via email to