Re: [PATCH net-next 5/8] net: mscc: Add initial Ocelot switch support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Alexandre,

I love your patch! Perhaps something to improve:

[auto build test WARNING on next-20180323]
[also build test WARNING on v4.16-rc7]
[cannot apply to net-next/master net/master robh/for-next v4.16-rc6 v4.16-rc5 v4.16-rc4]
[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/Alexandre-Belloni/Microsemi-Ocelot-switch-support/20180325-234932
reproduce:
        # apt-get install sparse
        make ARCH=x86_64 allmodconfig
        make C=1 CF=-D__CHECK_ENDIAN__


sparse warnings: (new ones prefixed by >>)

>> drivers/net/ethernet/mscc/ocelot.c:395:17: sparse: incorrect type in argument 2 (different base types) @@    expected unsigned int [unsigned] [usertype] val @@    got ed int [unsigned] [usertype] val @@
   drivers/net/ethernet/mscc/ocelot.c:395:17:    expected unsigned int [unsigned] [usertype] val
   drivers/net/ethernet/mscc/ocelot.c:395:17:    got restricted __le32 [usertype] <noident>
--
>> drivers/net/ethernet/mscc/ocelot_io.c:110:24: sparse: incorrect type in return expression (different address spaces) @@    expected struct regmap * @@    got void [noderef] <asnstruct regmap * @@
   drivers/net/ethernet/mscc/ocelot_io.c:110:24:    expected struct regmap *
   drivers/net/ethernet/mscc/ocelot_io.c:110:24:    got void [noderef] <asn:2>*[assigned] regs
--
>> drivers/net/ethernet/mscc/ocelot_board.c:41:26: sparse: cast to restricted __be32
>> drivers/net/ethernet/mscc/ocelot_board.c:41:26: sparse: cast to restricted __be32
>> drivers/net/ethernet/mscc/ocelot_board.c:41:26: sparse: cast to restricted __be32
>> drivers/net/ethernet/mscc/ocelot_board.c:41:26: sparse: cast to restricted __be32
>> drivers/net/ethernet/mscc/ocelot_board.c:41:26: sparse: cast to restricted __be32
>> drivers/net/ethernet/mscc/ocelot_board.c:41:26: sparse: cast to restricted __be32
>> drivers/net/ethernet/mscc/ocelot_board.c:142:34: sparse: cast to restricted __le32

vim +395 drivers/net/ethernet/mscc/ocelot.c

   367	
   368	static int ocelot_port_xmit(struct sk_buff *skb, struct net_device *dev)
   369	{
   370		struct ocelot_port *port = netdev_priv(dev);
   371		struct ocelot *ocelot = port->ocelot;
   372		u32 val, ifh[IFH_LEN];
   373		struct frame_info info = {};
   374		u8 grp = 0; /* Send everything on CPU group 0 */
   375		int i, count, last;
   376	
   377		val = ocelot_read(ocelot, QS_INJ_STATUS);
   378		if (!(val & QS_INJ_STATUS_FIFO_RDY(BIT(grp))) ||
   379		    (val & QS_INJ_STATUS_WMARK_REACHED(BIT(grp))))
   380			return NETDEV_TX_BUSY;
   381	
   382		ocelot_write_rix(ocelot, QS_INJ_CTRL_GAP_SIZE(1) |
   383				 QS_INJ_CTRL_SOF, QS_INJ_CTRL, grp);
   384	
   385		info.port = BIT(port->chip_port);
   386		info.cpuq = 0xff;
   387		ocelot_gen_ifh(ifh, &info);
   388	
   389		for (i = 0; i < IFH_LEN; i++)
   390			ocelot_write_rix(ocelot, ifh[i], QS_INJ_WR, grp);
   391	
   392		count = (skb->len + 3) / 4;
   393		last = skb->len % 4;
   394		for (i = 0; i < count; i++) {
 > 395			ocelot_write_rix(ocelot, cpu_to_le32(((u32 *)skb->data)[i]),
   396					 QS_INJ_WR, grp);
   397		}
   398	
   399		/* Add padding */
   400		while (i < (OCELOT_BUFFER_CELL_SZ / 4)) {
   401			ocelot_write_rix(ocelot, 0, QS_INJ_WR, grp);
   402			i++;
   403		}
   404	
   405		/* Indicate EOF and valid bytes in last word */
   406		ocelot_write_rix(ocelot, QS_INJ_CTRL_GAP_SIZE(1) |
   407				 QS_INJ_CTRL_VLD_BYTES(skb->len < OCELOT_BUFFER_CELL_SZ ? 0 : last) |
   408				 QS_INJ_CTRL_EOF,
   409				 QS_INJ_CTRL, grp);
   410	
   411		/* Add dummy CRC */
   412		ocelot_write_rix(ocelot, 0, QS_INJ_WR, grp);
   413		skb_tx_timestamp(skb);
   414	
   415		dev->stats.tx_packets++;
   416		dev->stats.tx_bytes += skb->len;
   417		dev_kfree_skb_any(skb);
   418	
   419		return NETDEV_TX_OK;
   420	}
   421	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux