[dpdk-dev] [PATCH] net/qede: fix gcc compiler option checks

2016-11-07 Thread Thomas Monjalon
2016-10-27 23:37, Rasesh Mody: > From: Rasesh Mody > > Using GCC_VERSION to check gcc version and decide whether to include > that compiler option. > > Fixes: ec94dbc57362 ("qede: add base driver") No need for the above line. It is fixing the commit below (which fixes above one). > Fixes: ecc7

[dpdk-dev] [PATCH] net/qede: fix gcc compiler option checks

2016-11-07 Thread Thomas Monjalon
2016-10-28 22:49, Mody, Rasesh: > > From: Stephen Hemminger > > > ifeq ($(CONFIG_RTE_TOOLCHAIN_GCC),y) > > > -ifeq ($(shell gcc -Wno-unused-but-set-variable -Werror -E - < /dev/null > > > > /dev/null 2>&1; echo $$?),0) > > > +ifeq ($(shell test $(GCC_VERSION) -ge 44 && echo 1), 1) > > > CFLAGS_B

[dpdk-dev] [PATCH] net/qede: fix gcc compiler option checks

2016-10-28 Thread Mody, Rasesh
Hi Stephen, > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Stephen > Hemminger > Sent: Friday, October 28, 2016 3:12 PM > > On Thu, 27 Oct 2016 23:37:57 -0700 > Rasesh Mody wrote: > > > From: Rasesh Mody > > > > Using GCC_VERSION to check gcc version and decide whether to include >

[dpdk-dev] [PATCH] net/qede: fix gcc compiler option checks

2016-10-28 Thread Stephen Hemminger
On Thu, 27 Oct 2016 23:37:57 -0700 Rasesh Mody wrote: > From: Rasesh Mody > > Using GCC_VERSION to check gcc version and decide whether to include > that compiler option. > > Fixes: ec94dbc57362 ("qede: add base driver") > Fixes: ecc7a5a27ffe ("net/qede/base: fix 32-bit build") > > Signed-off

[dpdk-dev] [PATCH] net/qede: fix gcc compiler option checks

2016-10-27 Thread Rasesh Mody
From: Rasesh Mody Using GCC_VERSION to check gcc version and decide whether to include that compiler option. Fixes: ec94dbc57362 ("qede: add base driver") Fixes: ecc7a5a27ffe ("net/qede/base: fix 32-bit build") Signed-off-by: Rasesh Mody --- drivers/net/qede/Makefile | 4 ++-- 1 file changed,