On Fri, Aug 11, 2017 at 02:29:00PM -0600, Scott Bauer wrote: > On Sat, Aug 12, 2017 at 04:37:33AM +0800, kbuild test robot wrote: > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci.git pci/host-vmd > > head: 6cf59d3b557e60a8fe3012761f336615d332a539 > > commit: 6cf59d3b557e60a8fe3012761f336615d332a539 [3/3] PCI: vmd: Free up IRQs on suspend path > > config: x86_64-allmodconfig (attached as .config) > > compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 > > reproduce: > > git checkout 6cf59d3b557e60a8fe3012761f336615d332a539 > > # save the attached .config to linux build tree > > make ARCH=x86_64 > > > > All warnings (new ones prefixed by >>): > > > > drivers/pci/host/vmd.c: In function 'vmd_resume': > > >> drivers/pci/host/vmd.c:791:2: warning: ignoring return value of 'pcim_enable_device', declared with attribute warn_unused_result [-Wunused-result] > > pcim_enable_device(pdev); > > > ugh, It was my intention to RESEND the original patch (without enable/disable). > I had forgotten when Jon suggested to disable/enable that I squashed that in. > I didn't do my due diligence before the resend to verify it was the original. > > Bjorn, how do you want me to fix this should I send a patch on top of this one, > removing the enable/disables? Or do you want to rip them out? Let me know what's > easiest for you. I dropped this one, so just send a new patch.
