[kbuild-all] [iio:testing 34/42] drivers/iio/adc/at91_adc.c:450:16: warning: Shifting signed 32-bit value by 31 bits is undefined behaviour

tree: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git testing head: c82164898b6f6663d6a723f7eff42abf5011ba5f commit: cfc11c189488dff60d032ea9e82d12790549aca8 [34/42] iio: Kconfig: at91_adc: add COMPILE_TEST dependency to driver compiler: gcc-9 (Debian 9.3.0-13) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; cppcheck warnings: (new ones prefixed by >>) if (status & AT91_ADC_ISR_PENS) { ^ vim +450 drivers/iio/adc/at91_adc.c 84882b060301c3 Alexandre Belloni 2014-04-15 417 84882b060301c3 Alexandre Belloni 2014-04-15 418 static irqreturn_t at91_adc_9x5_interrupt(int irq, void *private) c8b11de0404d31 Josh Wu 2013-10-08 419 { c8b11de0404d31 Josh Wu 2013-10-08 420 struct iio_dev *idev = private; c8b11de0404d31 Josh Wu 2013-10-08 421 struct at91_adc_state *st = iio_priv(idev); c8b11de0404d31 Josh Wu 2013-10-08 422 u32 status = at91_adc_readl(st, st->registers->status_register); c8b11de0404d31 Josh Wu 2013-10-08 423 const uint32_t ts_data_irq_mask = c8b11de0404d31 Josh Wu 2013-10-08 424 AT91_ADC_IER_XRDY | c8b11de0404d31 Josh Wu 2013-10-08 425 AT91_ADC_IER_YRDY | c8b11de0404d31 Josh Wu 2013-10-08 426 AT91_ADC_IER_PRDY; c8b11de0404d31 Josh Wu 2013-10-08 427 d4f51956ac8ad3 Ludovic Desroches 2014-10-09 428 if (status & GENMASK(st->num_channels - 1, 0)) c8b11de0404d31 Josh Wu 2013-10-08 429 handle_adc_eoc_trigger(irq, idev); c8b11de0404d31 Josh Wu 2013-10-08 430 c8b11de0404d31 Josh Wu 2013-10-08 431 if (status & AT91_ADC_IER_PEN) { c8b11de0404d31 Josh Wu 2013-10-08 432 at91_adc_writel(st, AT91_ADC_IDR, AT91_ADC_IER_PEN); c8b11de0404d31 Josh Wu 2013-10-08 433 at91_adc_writel(st, AT91_ADC_IER, AT91_ADC_IER_NOPEN | c8b11de0404d31 Josh Wu 2013-10-08 434 ts_data_irq_mask); c8b11de0404d31 Josh Wu 2013-10-08 435 /* Set up period trigger for sampling */ c8b11de0404d31 Josh Wu 2013-10-08 436 at91_adc_writel(st, st->registers->trigger_register, c8b11de0404d31 Josh Wu 2013-10-08 437 AT91_ADC_TRGR_MOD_PERIOD_TRIG | c8b11de0404d31 Josh Wu 2013-10-08 438 AT91_ADC_TRGR_TRGPER_(st->ts_sample_period_val)); c8b11de0404d31 Josh Wu 2013-10-08 439 } else if (status & AT91_ADC_IER_NOPEN) { c8b11de0404d31 Josh Wu 2013-10-08 440 at91_adc_writel(st, st->registers->trigger_register, 0); c8b11de0404d31 Josh Wu 2013-10-08 441 at91_adc_writel(st, AT91_ADC_IDR, AT91_ADC_IER_NOPEN | c8b11de0404d31 Josh Wu 2013-10-08 442 ts_data_irq_mask); c8b11de0404d31 Josh Wu 2013-10-08 443 at91_adc_writel(st, AT91_ADC_IER, AT91_ADC_IER_PEN); c8b11de0404d31 Josh Wu 2013-10-08 444 c8b11de0404d31 Josh Wu 2013-10-08 445 input_report_key(st->ts_input, BTN_TOUCH, 0); c8b11de0404d31 Josh Wu 2013-10-08 446 input_sync(st->ts_input); c8b11de0404d31 Josh Wu 2013-10-08 447 } else if ((status & ts_data_irq_mask) == ts_data_irq_mask) { c8b11de0404d31 Josh Wu 2013-10-08 448 /* Now all touchscreen data is ready */ c8b11de0404d31 Josh Wu 2013-10-08 449 c8b11de0404d31 Josh Wu 2013-10-08 @450 if (status & AT91_ADC_ISR_PENS) { c8b11de0404d31 Josh Wu 2013-10-08 451 /* validate data by pen contact */ c8b11de0404d31 Josh Wu 2013-10-08 452 at91_ts_sample(st); c8b11de0404d31 Josh Wu 2013-10-08 453 } else { c8b11de0404d31 Josh Wu 2013-10-08 454 /* triggered by event that is no pen contact, just read c8b11de0404d31 Josh Wu 2013-10-08 455 * them to clean the interrupt and discard all. c8b11de0404d31 Josh Wu 2013-10-08 456 */ c8b11de0404d31 Josh Wu 2013-10-08 457 at91_adc_readl(st, AT91_ADC_TSXPOSR); c8b11de0404d31 Josh Wu 2013-10-08 458 at91_adc_readl(st, AT91_ADC_TSYPOSR); c8b11de0404d31 Josh Wu 2013-10-08 459 at91_adc_readl(st, AT91_ADC_TSPRESSR); c8b11de0404d31 Josh Wu 2013-10-08 460 } c8b11de0404d31 Josh Wu 2013-10-08 461 } 0e589d5fb3172b Maxime Ripard 2012-05-11 462 0e589d5fb3172b Maxime Ripard 2012-05-11 463 return IRQ_HANDLED; 0e589d5fb3172b Maxime Ripard 2012-05-11 464 } 0e589d5fb3172b Maxime Ripard 2012-05-11 465 :::::: The code at line 450 was first introduced by commit :::::: c8b11de0404d318c4a67bf6b9066663b9d93786c iio: at91: introduce touch screen support in iio adc driver :::::: TO: Josh Wu <josh.wu(a)atmel.com&gt; :::::: CC: Jonathan Cameron <jic23(a)kernel.org&gt; --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org