All of lore.kernel.org
 help / color / mirror / Atom feed
* [hch-block:block-bounce-removal 5/5] drivers/net/ethernet/apple/mace.c:831:8: warning: assignment to 'volatile struct dbdma_cmd *' from 'int' makes pointer from integer without a cast
@ 2021-04-05 18:50 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2021-04-05 18:50 UTC (permalink / raw)
  To: kbuild-all

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

tree:   git://git.infradead.org/users/hch/block.git block-bounce-removal
head:   a134b61f80c05569ba06a6fab7feae4b05918c32
commit: a134b61f80c05569ba06a6fab7feae4b05918c32 [5/5] powerpc: stop implementing virt_to_bus
config: powerpc-pmac32_defconfig (attached as .config)
compiler: powerpc-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git remote add hch-block git://git.infradead.org/users/hch/block.git
        git fetch --no-tags hch-block block-bounce-removal
        git checkout a134b61f80c05569ba06a6fab7feae4b05918c32
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=powerpc 

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

All warnings (new ones prefixed by >>):

   In file included from include/linux/swab.h:5,
                    from include/uapi/linux/byteorder/big_endian.h:13,
                    from include/linux/byteorder/big_endian.h:5,
                    from arch/powerpc/include/uapi/asm/byteorder.h:14,
                    from include/asm-generic/bitops/le.h:6,
                    from arch/powerpc/include/asm/bitops.h:265,
                    from include/linux/bitops.h:32,
                    from include/linux/kernel.h:11,
                    from include/linux/list.h:9,
                    from include/linux/module.h:12,
                    from drivers/net/ethernet/apple/mace.c:9:
   drivers/net/ethernet/apple/mace.c: In function 'mace_open':
   drivers/net/ethernet/apple/mace.c:457:29: error: implicit declaration of function 'virt_to_bus'; did you mean 'virt_to_phys'? [-Werror=implicit-function-declaration]
     457 |  cp->phy_addr = cpu_to_le32(virt_to_bus(data));
         |                             ^~~~~~~~~~~
   include/uapi/linux/swab.h:115:54: note: in definition of macro '__swab32'
     115 | #define __swab32(x) (__u32)__builtin_bswap32((__u32)(x))
         |                                                      ^
   include/linux/byteorder/generic.h:88:21: note: in expansion of macro '__cpu_to_le32'
      88 | #define cpu_to_le32 __cpu_to_le32
         |                     ^~~~~~~~~~~~~
   drivers/net/ethernet/apple/mace.c:457:17: note: in expansion of macro 'cpu_to_le32'
     457 |  cp->phy_addr = cpu_to_le32(virt_to_bus(data));
         |                 ^~~~~~~~~~~
   drivers/net/ethernet/apple/mace.c: In function 'mace_interrupt':
   drivers/net/ethernet/apple/mace.c:664:22: warning: variable 'stat' set but not used [-Wunused-but-set-variable]
     664 |     int intr, fs, i, stat, x;
         |                      ^~~~
   drivers/net/ethernet/apple/mace.c: In function 'mace_tx_timeout':
   drivers/net/ethernet/apple/mace.c:831:10: error: implicit declaration of function 'bus_to_virt'; did you mean 'phys_to_virt'? [-Werror=implicit-function-declaration]
     831 |     cp = bus_to_virt(le32_to_cpu(rd->cmdptr));
         |          ^~~~~~~~~~~
         |          phys_to_virt
>> drivers/net/ethernet/apple/mace.c:831:8: warning: assignment to 'volatile struct dbdma_cmd *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
     831 |     cp = bus_to_virt(le32_to_cpu(rd->cmdptr));
         |        ^
   cc1: some warnings being treated as errors
--
   drivers/net/ethernet/apple/bmac.c: In function 'bmac_construct_xmt':
   drivers/net/ethernet/apple/bmac.c:569:10: error: implicit declaration of function 'virt_to_bus'; did you mean 'virt_to_phys'? [-Werror=implicit-function-declaration]
     569 |  baddr = virt_to_bus(vaddr);
         |          ^~~~~~~~~~~
         |          virt_to_phys
   drivers/net/ethernet/apple/bmac.c: In function 'bmac_txdma_intr':
   drivers/net/ethernet/apple/bmac.c:775:14: error: implicit declaration of function 'bus_to_virt'; did you mean 'phys_to_virt'? [-Werror=implicit-function-declaration]
     775 |    if (cp == bus_to_virt(in_le32(&bp->tx_dma->cmdptr)))
         |              ^~~~~~~~~~~
         |              phys_to_virt
>> drivers/net/ethernet/apple/bmac.c:775:11: warning: comparison between pointer and integer
     775 |    if (cp == bus_to_virt(in_le32(&bp->tx_dma->cmdptr)))
         |           ^~
   drivers/net/ethernet/apple/bmac.c: In function 'bmac_verify_checksum':
   drivers/net/ethernet/apple/bmac.c:1169:23: warning: variable 'storedCS' set but not used [-Wunused-but-set-variable]
    1169 |  unsigned short data, storedCS;
         |                       ^~~~~~~~
   drivers/net/ethernet/apple/bmac.c: In function 'bmac_tx_timeout':
>> drivers/net/ethernet/apple/bmac.c:1502:5: warning: assignment to 'volatile struct dbdma_cmd *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
    1502 |  cp = bus_to_virt(le32_to_cpu(rd->cmdptr));
         |     ^
   cc1: some warnings being treated as errors


vim +831 drivers/net/ethernet/apple/mace.c

^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  802  
de892f8f2cc817 drivers/net/ethernet/apple/mace.c Kees Cook      2017-10-16  803  static void mace_tx_timeout(struct timer_list *t)
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  804  {
de892f8f2cc817 drivers/net/ethernet/apple/mace.c Kees Cook      2017-10-16  805      struct mace_data *mp = from_timer(mp, t, tx_timeout);
de892f8f2cc817 drivers/net/ethernet/apple/mace.c Kees Cook      2017-10-16  806      struct net_device *dev = macio_get_drvdata(mp->mdev);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  807      volatile struct mace __iomem *mb = mp->mace;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  808      volatile struct dbdma_regs __iomem *td = mp->tx_dma;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  809      volatile struct dbdma_regs __iomem *rd = mp->rx_dma;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  810      volatile struct dbdma_cmd *cp;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  811      unsigned long flags;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  812      int i;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  813  
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  814      spin_lock_irqsave(&mp->lock, flags);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  815      mp->timeout_active = 0;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  816      if (mp->tx_active == 0 && !mp->tx_bad_runt)
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  817  	goto out;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  818  
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  819      /* update various counters */
09f75cd7bf1372 drivers/net/mace.c                Jeff Garzik    2007-10-03  820      mace_handle_misc_intrs(mp, in_8(&mb->ir), dev);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  821  
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  822      cp = mp->tx_cmds + NCMDS_TX * mp->tx_empty;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  823  
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  824      /* turn off both tx and rx and reset the chip */
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  825      out_8(&mb->maccc, 0);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  826      printk(KERN_ERR "mace: transmit timeout - resetting\n");
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  827      dbdma_reset(td);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  828      mace_reset(dev);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  829  
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  830      /* restart rx dma */
f5718726714cd6 drivers/net/ethernet/apple/mace.c David Gibson   2015-02-03 @831      cp = bus_to_virt(le32_to_cpu(rd->cmdptr));
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  832      dbdma_reset(rd);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  833      out_le16(&cp->xfer_status, 0);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  834      out_le32(&rd->cmdptr, virt_to_bus(cp));
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  835      out_le32(&rd->control, (RUN << 16) | RUN);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  836  
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  837      /* fix up the transmit side */
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  838      i = mp->tx_empty;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  839      mp->tx_active = 0;
09f75cd7bf1372 drivers/net/mace.c                Jeff Garzik    2007-10-03  840      ++dev->stats.tx_errors;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  841      if (mp->tx_bad_runt) {
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  842  	mp->tx_bad_runt = 0;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  843      } else if (i != mp->tx_fill) {
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  844  	dev_kfree_skb(mp->tx_bufs[i]);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  845  	if (++i >= N_TX_RING)
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  846  	    i = 0;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  847  	mp->tx_empty = i;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  848      }
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  849      mp->tx_fullup = 0;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  850      netif_wake_queue(dev);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  851      if (i != mp->tx_fill) {
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  852  	cp = mp->tx_cmds + NCMDS_TX * i;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  853  	out_le16(&cp->xfer_status, 0);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  854  	out_le16(&cp->command, OUTPUT_LAST);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  855  	out_le32(&td->cmdptr, virt_to_bus(cp));
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  856  	out_le32(&td->control, (RUN << 16) | RUN);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  857  	++mp->tx_active;
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  858  	mace_set_timeout(dev);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  859      }
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  860  
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  861      /* turn it back on */
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  862      out_8(&mb->imr, RCVINT);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  863      out_8(&mb->maccc, mp->maccc);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  864  
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  865  out:
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  866      spin_unlock_irqrestore(&mp->lock, flags);
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  867  }
^1da177e4c3f41 drivers/net/mace.c                Linus Torvalds 2005-04-16  868  

:::::: The code at line 831 was first introduced by commit
:::::: f5718726714cd6114876c4e3ca9b6992ab81176c powerpc: Move Power Macintosh drivers to generic byteswappers

:::::: TO: David Gibson <david@gibson.dropbear.id.au>
:::::: CC: Benjamin Herrenschmidt <benh@kernel.crashing.org>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 25274 bytes --]

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2021-04-05 18:50 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-05 18:50 [hch-block:block-bounce-removal 5/5] drivers/net/ethernet/apple/mace.c:831:8: warning: assignment to 'volatile struct dbdma_cmd *' from 'int' makes pointer from integer without a cast kernel test robot

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.