diff mbox series

drivers: fix ICC compilation issue with i686 for octeontx2

Message ID 20190701180346.42906-1-jerinj@marvell.com (mailing list archive)
State Accepted, archived
Delegated to: Thomas Monjalon
Headers
Series drivers: fix ICC compilation issue with i686 for octeontx2 |

Checks

Context Check Description
ci/checkpatch success coding style OK
ci/intel-Performance-Testing success Performance Testing PASS
ci/mellanox-Performance-Testing success Performance Testing PASS
ci/Intel-compilation success Compilation OK

Commit Message

Jerin Jacob Kollanukkaran July 1, 2019, 6:03 p.m. UTC 
  From: Amit Gupta <agupta3@marvell.com>

Fix the following ICC specific compilation issue with i686 build.

dpdk/drivers/common/octeontx2/otx2_mbox.c(47): error #2259:
non-pointer conversion from "unsigned long long" to "struct mbox_hdr *"
may lose significant bits
(struct mbox_hdr *)((uintptr_t)mdev->mbase + mbox->tx_start);

Fixes: 732377a6792e ("mempool/octeontx2: add build infra and device probe")
Fixes: 371d3212cbed ("common/octeontx2: add build infrastructure and HW definition")

Signed-off-by: Amit Gupta <agupta3@marvell.com>
---
 drivers/common/octeontx2/Makefile  | 3 +++
 drivers/mempool/octeontx2/Makefile | 3 +++
 2 files changed, 6 insertions(+)

Comments

Thomas Monjalon July 4, 2019, 2:46 p.m. UTC | #1 
01/07/2019 20:03, jerinj@marvell.com:
> From: Amit Gupta <agupta3@marvell.com>
> 
> Fix the following ICC specific compilation issue with i686 build.
> 
> dpdk/drivers/common/octeontx2/otx2_mbox.c(47): error #2259:
> non-pointer conversion from "unsigned long long" to "struct mbox_hdr *"
> may lose significant bits
> (struct mbox_hdr *)((uintptr_t)mdev->mbase + mbox->tx_start);
> 
> Fixes: 732377a6792e ("mempool/octeontx2: add build infra and device probe")
> Fixes: 371d3212cbed ("common/octeontx2: add build infrastructure and HW definition")
> 
> Signed-off-by: Amit Gupta <agupta3@marvell.com>

Applied, thanks
diff mbox series

Patch

diff --git a/drivers/common/octeontx2/Makefile b/drivers/common/octeontx2/Makefile
index fabc32537..afe570817 100644
--- a/drivers/common/octeontx2/Makefile
+++ b/drivers/common/octeontx2/Makefile
@@ -17,6 +17,9 @@  CFLAGS += -I$(RTE_SDK)/drivers/bus/pci
 ifneq ($(CONFIG_RTE_ARCH_64),y)
 CFLAGS += -Wno-int-to-pointer-cast
 CFLAGS += -Wno-pointer-to-int-cast
+ifeq ($(CONFIG_RTE_TOOLCHAIN_ICC),y)
+CFLAGS += -diag-disable 2259
+endif
 endif
 
 EXPORT_MAP := rte_common_octeontx2_version.map
diff --git a/drivers/mempool/octeontx2/Makefile b/drivers/mempool/octeontx2/Makefile
index b3568443e..87cce22c6 100644
--- a/drivers/mempool/octeontx2/Makefile
+++ b/drivers/mempool/octeontx2/Makefile
@@ -18,6 +18,9 @@  CFLAGS += -O3
 ifneq ($(CONFIG_RTE_ARCH_64),y)
 CFLAGS += -Wno-int-to-pointer-cast
 CFLAGS += -Wno-pointer-to-int-cast
+ifeq ($(CONFIG_RTE_TOOLCHAIN_ICC),y)
+CFLAGS += -diag-disable 2259
+endif
 endif
 
 EXPORT_MAP := rte_mempool_octeontx2_version.map