Re: [PATCH] dmaengine: ipu: Revert ipu_irq_handler changes.

From: kbuild test robot
Date: Fri Dec 23 2016 - 09:40:32 EST


Hi Ivan,

[auto build test WARNING on linus/master]
[also build test WARNING on v4.9 next-20161223]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url: https://github.com/0day-ci/linux/commits/ivan-stoyanov-amk-drives-bg/dmaengine-ipu-Revert-ipu_irq_handler-changes/20161223-211424
config: arm-multi_v5_defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=arm

Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings

All warnings (new ones prefixed by >>):

drivers/dma/ipu/ipu_irq.c: In function 'ipu_irq_fn':
>> drivers/dma/ipu/ipu_irq.c:342:4: warning: 'irq' may be used uninitialized in this function [-Wmaybe-uninitialized]
generic_handle_irq(irq);
^~~~~~~~~~~~~~~~~~~~~~~

vim +/irq +342 drivers/dma/ipu/ipu_irq.c

326 unsigned int irq;
327
328 line--;
329 status &= ~(1UL << line);
330
331 raw_spin_lock(&bank_lock);
332 map = src2map(32 * i + line);
333 if (map)
334 irq = map->irq;
335 raw_spin_unlock(&bank_lock);
336
337 if (!map) {
338 pr_err("IPU: Interrupt on unmapped source %u bank %d\n",
339 line, i);
340 continue;
341 }
> 342 generic_handle_irq(irq);
343 }
344 }
345 }
346
347 static struct irq_chip ipu_irq_chip = {
348 .name = "ipu_irq",
349 .irq_ack = ipu_irq_ack,
350 .irq_mask = ipu_irq_mask,

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

Attachment: .config.gz
Description: application/gzip