linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Marc Kleine-Budde <mkl@pengutronix.de>
To: kernel test robot <lkp@intel.com>,
	Henning Colliander <henning.colliander@evidente.se>,
	Greg Ungerer <gerg@linux-m68k.org>
Cc: kbuild-all@lists.01.org, linux-kernel@vger.kernel.org,
	Jimmy Assarsson <extja@kvaser.com>,
	Christer Beskow <chbe@kvaser.com>
Subject: Re: drivers/net/can/kvaser_pciefd.c:801:17: sparse: sparse: cast removes address space '<asn:2>' of expression
Date: Fri, 12 Jun 2020 09:48:46 +0200	[thread overview]
Message-ID: <e93fe895-a6b8-34ff-f9c8-00ec7138b1c1@pengutronix.de> (raw)
In-Reply-To: <202006121356.lKucoVPo%lkp@intel.com>

Hello Greg,

the k-build robot found this sparse problem, triggered by building a CAN driver
for m68k. Is this a problem in our CAN driver or in the m68k headers?

Marc

On 6/12/20 7:28 AM, kernel test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head:   b791d1bdf9212d944d749a5c7ff6febdba241771
> commit: 26ad340e582d3d5958ed8456a1911d79cfb567b4 can: kvaser_pciefd: Add driver for Kvaser PCIEcan devices
> date:   11 months ago
> config: m68k-randconfig-s032-20200612 (attached as .config)
> compiler: m68k-linux-gcc (GCC) 9.3.0
> reproduce:
>         # apt-get install sparse
>         # sparse version: v0.6.1-250-g42323db3-dirty
>         git checkout 26ad340e582d3d5958ed8456a1911d79cfb567b4
>         # save the attached .config to linux build tree
>         make W=1 C=1 ARCH=m68k CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
> 
> If you fix the issue, kindly add following tag as appropriate
> Reported-by: kernel test robot <lkp@intel.com>
> 
> 
> sparse warnings: (new ones prefixed by >>)
> 
>>> drivers/net/can/kvaser_pciefd.c:801:17: sparse: sparse: cast removes address space '<asn:2>' of expression
>    drivers/net/can/kvaser_pciefd.c:805:17: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:77:24: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:77:24: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:77:24: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:77:24: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast removes address space '<asn:2>' of expression
>    arch/m68k/include/asm/io_no.h:78:16: sparse: sparse: cast to restricted __le32
> 
> vim +801 drivers/net/can/kvaser_pciefd.c
> 
>    764	
>    765	static netdev_tx_t kvaser_pciefd_start_xmit(struct sk_buff *skb,
>    766						    struct net_device *netdev)
>    767	{
>    768		struct kvaser_pciefd_can *can = netdev_priv(netdev);
>    769		unsigned long irq_flags;
>    770		struct kvaser_pciefd_tx_packet packet;
>    771		int nwords;
>    772		u8 count;
>    773	
>    774		if (can_dropped_invalid_skb(netdev, skb))
>    775			return NETDEV_TX_OK;
>    776	
>    777		nwords = kvaser_pciefd_prepare_tx_packet(&packet, can, skb);
>    778	
>    779		spin_lock_irqsave(&can->echo_lock, irq_flags);
>    780	
>    781		/* Prepare and save echo skb in internal slot */
>    782		can_put_echo_skb(skb, netdev, can->echo_idx);
>    783	
>    784		/* Move echo index to the next slot */
>    785		can->echo_idx = (can->echo_idx + 1) % can->can.echo_skb_max;
>    786	
>    787		/* Write header to fifo */
>    788		iowrite32(packet.header[0],
>    789			  can->reg_base + KVASER_PCIEFD_KCAN_FIFO_REG);
>    790		iowrite32(packet.header[1],
>    791			  can->reg_base + KVASER_PCIEFD_KCAN_FIFO_REG);
>    792	
>    793		if (nwords) {
>    794			u32 data_last = ((u32 *)packet.data)[nwords - 1];
>    795	
>    796			/* Write data to fifo, except last word */
>    797			iowrite32_rep(can->reg_base +
>    798				      KVASER_PCIEFD_KCAN_FIFO_REG, packet.data,
>    799				      nwords - 1);
>    800			/* Write last word to end of fifo */
>  > 801			__raw_writel(data_last, can->reg_base +
>    802				     KVASER_PCIEFD_KCAN_FIFO_LAST_REG);
>    803		} else {
>    804			/* Complete write to fifo */
>    805			__raw_writel(0, can->reg_base +
>    806				     KVASER_PCIEFD_KCAN_FIFO_LAST_REG);
>    807		}
>    808	
>    809		count = ioread32(can->reg_base + KVASER_PCIEFD_KCAN_TX_NPACKETS_REG);
>    810		/* No room for a new message, stop the queue until at least one
>    811		 * successful transmit
>    812		 */
>    813		if (count >= KVASER_PCIEFD_CAN_TX_MAX_COUNT ||
>    814		    can->can.echo_skb[can->echo_idx])
>    815			netif_stop_queue(netdev);
>    816	
>    817		spin_unlock_irqrestore(&can->echo_lock, irq_flags);
>    818	
>    819		return NETDEV_TX_OK;
>    820	}
>    821	
> 
> ---
> 0-DAY CI Kernel Test Service, Intel Corporation
> https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
> 


-- 
Pengutronix e.K.                 | Marc Kleine-Budde           |
Embedded Linux                   | https://www.pengutronix.de  |
Vertretung West/Dortmund         | Phone: +49-231-2826-924     |
Amtsgericht Hildesheim, HRA 2686 | Fax:   +49-5121-206917-5555 |

  reply	other threads:[~2020-06-12  7:49 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-12  5:28 drivers/net/can/kvaser_pciefd.c:801:17: sparse: sparse: cast removes address space '<asn:2>' of expression kernel test robot
2020-06-12  7:48 ` Marc Kleine-Budde [this message]
2020-06-12 15:33   ` Greg Ungerer
2020-06-12 16:35     ` Luc Van Oostenryck
2020-06-13  2:19       ` Greg Ungerer
2020-06-15  7:07       ` Greg Ungerer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e93fe895-a6b8-34ff-f9c8-00ec7138b1c1@pengutronix.de \
    --to=mkl@pengutronix.de \
    --cc=chbe@kvaser.com \
    --cc=extja@kvaser.com \
    --cc=gerg@linux-m68k.org \
    --cc=henning.colliander@evidente.se \
    --cc=kbuild-all@lists.01.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lkp@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).