drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:543:54: sparse: sparse: incorrect type in argument 2 (different address spaces)

From: kernel test robot
Date: Sat Jun 06 2020 - 14:51:15 EST

Next message: kernel test robot: "[rcu:dev.2020.06.02a 67/90] include/linux/kern_levels.h:5:18: warning: format '%zu' expects argument of type 'size_t', but argument 2 has type 'long unsigned int'"
Previous message: Manivannan Sadhasivam: "Re: [PATCH v3 1/2] usb: serial: Add MaxLinear/Exar USB to Serial driver"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 9fa88c5d3f5eae3e68ef20d226c3f13e21490668
commit: a425b6e1c69ba907b72b737a4d44f8cfbc43ce3c hinic: add mailbox function support
date: 6 weeks ago
config: arm64-randconfig-s032-20200607 (attached as .config)
compiler: aarch64-linux-gcc (GCC) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.1-247-gcadbd124-dirty
git checkout a425b6e1c69ba907b72b737a4d44f8cfbc43ce3c
# save the attached .config to linux build tree
make W=1 C=1 ARCH=arm64 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'

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

sparse warnings: (new ones prefixed by >>)

>> drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:543:54: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] <asn:2> *addr @@ got unsigned char [usertype] * @@
>> drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:543:54: sparse: expected void volatile [noderef] <asn:2> *addr
>> drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:543:54: sparse: got unsigned char [usertype] *
drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:566:58: sparse: sparse: incorrect type in argument 2 (different address spaces) @@ expected void volatile [noderef] <asn:2> *addr @@ got unsigned char [usertype] * @@
drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:566:58: sparse: expected void volatile [noderef] <asn:2> *addr
drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:566:58: sparse: got unsigned char [usertype] *
drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:601:6: sparse: sparse: symbol 'dump_mox_reg' was not declared. Should it be static?
>> drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:618:22: sparse: sparse: cast to restricted __be64
>> drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:1057:25: sparse: sparse: incorrect type in assignment (different address spaces) @@ expected unsigned char [usertype] *data @@ got void [noderef] <asn:2> * @@
>> drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:1057:25: sparse: expected unsigned char [usertype] *data
>> drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c:1057:25: sparse: got void [noderef] <asn:2> *
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected unsigned int [usertype] val @@ got restricted __be32 [usertype] @@
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: expected unsigned int [usertype] val
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: got restricted __be32 [usertype]
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected unsigned int [usertype] val @@ got restricted __be32 [usertype] @@
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: expected unsigned int [usertype] val
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: got restricted __be32 [usertype]
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:250:16: sparse: sparse: cast to restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:250:16: sparse: sparse: cast to restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected unsigned int [usertype] val @@ got restricted __be32 [usertype] @@
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: expected unsigned int [usertype] val
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: got restricted __be32 [usertype]
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected unsigned int [usertype] val @@ got restricted __be32 [usertype] @@
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: expected unsigned int [usertype] val
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: got restricted __be32 [usertype]
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected unsigned int [usertype] val @@ got restricted __be32 [usertype] @@
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: expected unsigned int [usertype] val
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: got restricted __be32 [usertype]
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: incorrect type in argument 1 (different base types) @@ expected unsigned int [usertype] val @@ got restricted __be32 [usertype] @@
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: expected unsigned int [usertype] val
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: got restricted __be32 [usertype]
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32
drivers/net/ethernet/huawei/hinic/hinic_hw_if.h:256:9: sparse: sparse: cast from restricted __be32

vim +543 drivers/net/ethernet/huawei/hinic/hinic_hw_mbox.c

535
536 static void mbox_copy_header(struct hinic_hwdev *hwdev,
537 struct hinic_send_mbox *mbox, u64 *header)
538 {
539 u32 i, idx_max = MBOX_HEADER_SZ / sizeof(u32);
540 u32 *data = (u32 *)header;
541
542 for (i = 0; i < idx_max; i++)
> 543 __raw_writel(*(data + i), mbox->data + i * sizeof(u32));
544 }
545
546 static void mbox_copy_send_data(struct hinic_hwdev *hwdev,
547 struct hinic_send_mbox *mbox, void *seg,
548 u16 seg_len)
549 {
550 u8 mbox_max_buf[MBOX_SEG_LEN] = {0};
551 u32 data_len, chk_sz = sizeof(u32);
552 u32 *data = seg;
553 u32 i, idx_max;
554
555 /* The mbox message should be aligned in 4 bytes. */
556 if (seg_len % chk_sz) {
557 memcpy(mbox_max_buf, seg, seg_len);
558 data = (u32 *)mbox_max_buf;
559 }
560
561 data_len = seg_len;
562 idx_max = ALIGN(data_len, chk_sz) / chk_sz;
563
564 for (i = 0; i < idx_max; i++)
565 __raw_writel(*(data + i),
566 mbox->data + MBOX_HEADER_SZ + i * sizeof(u32));
567 }
568
569 static void write_mbox_msg_attr(struct hinic_mbox_func_to_func *func_to_func,
570 u16 dst_func, u16 dst_aeqn, u16 seg_len,
571 int poll)
572 {
573 u16 rsp_aeq = (dst_aeqn == 0) ? 0 : HINIC_MBOX_RSP_AEQN;
574 u32 mbox_int, mbox_ctrl;
575
576 mbox_int = HINIC_MBOX_INT_SET(dst_func, DST_FUNC) |
577 HINIC_MBOX_INT_SET(dst_aeqn, DST_AEQN) |
578 HINIC_MBOX_INT_SET(rsp_aeq, SRC_RESP_AEQN) |
579 HINIC_MBOX_INT_SET(NO_DMA_ATTRIBUTE_VAL, STAT_DMA) |
580 HINIC_MBOX_INT_SET(ALIGN(MBOX_SEG_LEN + MBOX_HEADER_SZ +
581 MBOX_INFO_SZ, MBOX_SEG_LEN_ALIGN) >> 2,
582 TX_SIZE) |
583 HINIC_MBOX_INT_SET(STRONG_ORDER, STAT_DMA_SO_RO) |
584 HINIC_MBOX_INT_SET(WRITE_BACK, WB_EN);
585
586 hinic_hwif_write_reg(func_to_func->hwif,
587 HINIC_FUNC_CSR_MAILBOX_INT_OFFSET_OFF, mbox_int);
588
589 wmb(); /* writing the mbox int attributes */
590 mbox_ctrl = HINIC_MBOX_CTRL_SET(TX_NOT_DONE, TX_STATUS);
591
592 if (poll)
593 mbox_ctrl |= HINIC_MBOX_CTRL_SET(NOT_TRIGGER, TRIGGER_AEQE);
594 else
595 mbox_ctrl |= HINIC_MBOX_CTRL_SET(TRIGGER, TRIGGER_AEQE);
596
597 hinic_hwif_write_reg(func_to_func->hwif,
598 HINIC_FUNC_CSR_MAILBOX_CONTROL_OFF, mbox_ctrl);
599 }
600
601 void dump_mox_reg(struct hinic_hwdev *hwdev)
602 {
603 u32 val;
604
605 val = hinic_hwif_read_reg(hwdev->hwif,
606 HINIC_FUNC_CSR_MAILBOX_CONTROL_OFF);
607 dev_err(&hwdev->hwif->pdev->dev, "Mailbox control reg: 0x%x\n", val);
608
609 val = hinic_hwif_read_reg(hwdev->hwif,
610 HINIC_FUNC_CSR_MAILBOX_INT_OFFSET_OFF);
611 dev_err(&hwdev->hwif->pdev->dev, "Mailbox interrupt offset: 0x%x\n",
612 val);
613 }
614
615 static u16 get_mbox_status(struct hinic_send_mbox *mbox)
616 {
617 /* write back is 16B, but only use first 4B */
> 618 u64 wb_val = be64_to_cpu(*mbox->wb_status);
619
620 rmb(); /* verify reading before check */
621
622 return (u16)(wb_val & MBOX_WB_STATUS_ERRCODE_MASK);
623 }
624

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

Attachment: .config.gz
Description: application/gzip

Next message: kernel test robot: "[rcu:dev.2020.06.02a 67/90] include/linux/kern_levels.h:5:18: warning: format '%zu' expects argument of type 'size_t', but argument 2 has type 'long unsigned int'"
Previous message: Manivannan Sadhasivam: "Re: [PATCH v3 1/2] usb: serial: Add MaxLinear/Exar USB to Serial driver"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]