[linux-stable-rc:linux-3.14.y 4656/4754] arch/x86/pci/sta2x11-fixup.c:186:10: warning: initialization from incompatible pointer type

[linux-stable-rc:linux-3.14.y 4656/4754] arch/x86/pci/sta2x11-fixup.c:186:10: warning: initialization from incompatible pointer type

[kbuild test robot]

[-- Attachment #1: Type: text/plain, Size: 4764 bytes --]

Hi,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-3.14.y
head:   96f343e0621a0d5afa9c2a34e2aee69f33b6eb5c
commit: 3711edaf01a01818f2aed9f21efe29b9818134b9 [4656/4754] compiler-gcc: integrate the various compiler-gcc[345].h files
config: i386-randconfig-s0-201625 (attached as .config)
compiler: gcc-6 (Debian 6.1.1-1) 6.1.1 20160430
reproduce:
        git checkout 3711edaf01a01818f2aed9f21efe29b9818134b9
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

>> arch/x86/pci/sta2x11-fixup.c:186:10: warning: initialization from incompatible pointer type [-Wincompatible-pointer-types]
     .free = swiotlb_free_coherent,
             ^~~~~~~~~~~~~~~~~~~~~
   arch/x86/pci/sta2x11-fixup.c:186:10: note: (near initialization for 'sta2x11_dma_ops.free')
--
>> drivers/acpi/video.c:213:19: warning: 'device_decode' defined but not used [-Wunused-const-variable=]
    static const char device_decode[][30] = {
                      ^~~~~~~~~~~~~
--
   drivers/staging/iio/resolver/ad2s1200.c: In function 'ad2s1200_probe':
>> drivers/staging/iio/resolver/ad2s1200.c:109:2: warning: this 'for' clause does not guard... [-Wmisleading-indentation]
     for (pn = 0; pn < AD2S1200_PN; pn++)
     ^~~
   drivers/staging/iio/resolver/ad2s1200.c:112:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'for'
      if (ret) {
      ^~
--
   drivers/video/s3fb.c: In function 's3_pci_probe':
>> drivers/video/s3fb.c:1185:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
     par->state.vgabase = (void __iomem *) vga_res.start;
                          ^
--
   drivers/video/arkfb.c: In function 'ark_pci_probe':
>> drivers/video/arkfb.c:1019:23: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
     par->state.vgabase = (void __iomem *) vga_res.start;
                          ^
--
   fs/ncpfs/dir.c: In function 'ncp_lookup':
>> fs/ncpfs/dir.c:855:3: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
      if (!res)
      ^~
   fs/ncpfs/dir.c:857:4: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
       if (!res)
       ^~

vim +186 arch/x86/pci/sta2x11-fixup.c

83125a3a Alessandro Rubini 2012-04-04  170  					    dma_addr_t *dma_handle,
83125a3a Alessandro Rubini 2012-04-04  171  					    gfp_t flags,
83125a3a Alessandro Rubini 2012-04-04  172  					    struct dma_attrs *attrs)
83125a3a Alessandro Rubini 2012-04-04  173  {
83125a3a Alessandro Rubini 2012-04-04  174  	void *vaddr;
83125a3a Alessandro Rubini 2012-04-04  175  
83125a3a Alessandro Rubini 2012-04-04  176  	vaddr = dma_generic_alloc_coherent(dev, size, dma_handle, flags, attrs);
83125a3a Alessandro Rubini 2012-04-04  177  	if (!vaddr)
83125a3a Alessandro Rubini 2012-04-04  178  		vaddr = swiotlb_alloc_coherent(dev, size, dma_handle, flags);
83125a3a Alessandro Rubini 2012-04-04  179  	*dma_handle = p2a(*dma_handle, to_pci_dev(dev));
83125a3a Alessandro Rubini 2012-04-04  180  	return vaddr;
83125a3a Alessandro Rubini 2012-04-04  181  }
83125a3a Alessandro Rubini 2012-04-04  182  
83125a3a Alessandro Rubini 2012-04-04  183  /* We have our own dma_ops: the same as swiotlb but from alloc (above) */
83125a3a Alessandro Rubini 2012-04-04  184  static struct dma_map_ops sta2x11_dma_ops = {
83125a3a Alessandro Rubini 2012-04-04  185  	.alloc = sta2x11_swiotlb_alloc_coherent,
83125a3a Alessandro Rubini 2012-04-04 @186  	.free = swiotlb_free_coherent,
83125a3a Alessandro Rubini 2012-04-04  187  	.map_page = swiotlb_map_page,
83125a3a Alessandro Rubini 2012-04-04  188  	.unmap_page = swiotlb_unmap_page,
83125a3a Alessandro Rubini 2012-04-04  189  	.map_sg = swiotlb_map_sg_attrs,
83125a3a Alessandro Rubini 2012-04-04  190  	.unmap_sg = swiotlb_unmap_sg_attrs,
83125a3a Alessandro Rubini 2012-04-04  191  	.sync_single_for_cpu = swiotlb_sync_single_for_cpu,
83125a3a Alessandro Rubini 2012-04-04  192  	.sync_single_for_device = swiotlb_sync_single_for_device,
83125a3a Alessandro Rubini 2012-04-04  193  	.sync_sg_for_cpu = swiotlb_sync_sg_for_cpu,
83125a3a Alessandro Rubini 2012-04-04  194  	.sync_sg_for_device = swiotlb_sync_sg_for_device,

:::::: The code at line 186 was first introduced by commit
:::::: 83125a3a189ec34fb22a04e8efad69ae6d52674a x86, platform: Initial support for sta2x11 I/O hub

:::::: TO: Alessandro Rubini <rubini@gnudd.com>
:::::: CC: H. Peter Anvin <hpa@zytor.com>

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

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 21465 bytes --]