* [PATCH] qede: fix icc build error
@ 2016-05-10 11:23 Ferruh Yigit
2016-05-10 20:02 ` Harish Patil
0 siblings, 1 reply; 4+ messages in thread
From: Ferruh Yigit @ 2016-05-10 11:23 UTC (permalink / raw)
To: dev; +Cc: Harish Patil, Rasesh Mody, Sony Chacko, Ferruh Yigit
fix errors:
icc: command line warning #10006: ignoring unknown option '-Wno-unused-value'
icc: command line warning #10006: ignoring unknown option '-Wno-format-nonliteral'
icc: command line warning #10006: ignoring unknown option '-Wno-shift-negative-value'
qede/base/ecore_dev.c(1643): error #188: enumerated type mixed with another type
return 0;
^
Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
---
drivers/net/qede/Makefile | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/net/qede/Makefile b/drivers/net/qede/Makefile
index 47e01be..d0c1361 100644
--- a/drivers/net/qede/Makefile
+++ b/drivers/net/qede/Makefile
@@ -27,7 +27,6 @@ OS_TYPE := $(shell uname -s)
# CFLAGS
#
CFLAGS_BASE_DRIVER = -Wno-unused-parameter
-CFLAGS_BASE_DRIVER += -Wno-unused-value
CFLAGS_BASE_DRIVER += -Wno-sign-compare
CFLAGS_BASE_DRIVER += -Wno-missing-prototypes
CFLAGS_BASE_DRIVER += -Wno-cast-qual
@@ -35,12 +34,16 @@ CFLAGS_BASE_DRIVER += -Wno-unused-function
CFLAGS_BASE_DRIVER += -Wno-unused-variable
CFLAGS_BASE_DRIVER += -Wno-strict-aliasing
CFLAGS_BASE_DRIVER += -Wno-missing-prototypes
+
+ifneq ($(CONFIG_RTE_TOOLCHAIN_ICC),y)
+CFLAGS_BASE_DRIVER += -Wno-unused-value
CFLAGS_BASE_DRIVER += -Wno-format-nonliteral
ifeq ($(OS_TYPE),Linux)
ifeq ($(shell clang -Wno-shift-negative-value -Werror -E - < /dev/null > /dev/null 2>&1; echo $$?),0)
CFLAGS_BASE_DRIVER += -Wno-shift-negative-value
endif
endif
+endif
ifneq (,$(filter gcc gcc48,$(CC)))
CFLAGS_BASE_DRIVER += -Wno-unused-but-set-variable
@@ -57,7 +60,7 @@ ifeq ($(shell clang -Wno-pointer-bool-conversion -Werror -E - < /dev/null > /dev
CFLAGS_BASE_DRIVER += -Wno-pointer-bool-conversion
endif
else
-#icc flags
+CFLAGS_BASE_DRIVER += -wd188 #188: enumerated type mixed with another type
endif
#
--
2.5.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] qede: fix icc build error
2016-05-10 11:23 [PATCH] qede: fix icc build error Ferruh Yigit
@ 2016-05-10 20:02 ` Harish Patil
2016-05-11 17:17 ` Ferruh Yigit
2016-05-13 15:27 ` Thomas Monjalon
0 siblings, 2 replies; 4+ messages in thread
From: Harish Patil @ 2016-05-10 20:02 UTC (permalink / raw)
To: Ferruh Yigit, dev; +Cc: Rasesh Mody, Sony Chacko
Hi Ferruh,
>fix errors:
>icc: command line warning #10006: ignoring unknown option
>'-Wno-unused-value'
>icc: command line warning #10006: ignoring unknown option
>'-Wno-format-nonliteral'
>icc: command line warning #10006: ignoring unknown option
>'-Wno-shift-negative-value'
>qede/base/ecore_dev.c(1643): error #188: enumerated type mixed with
>another type
> return 0;
> ^
>
>Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>---
> drivers/net/qede/Makefile | 7 +++++--
> 1 file changed, 5 insertions(+), 2 deletions(-)
>
>diff --git a/drivers/net/qede/Makefile b/drivers/net/qede/Makefile
>index 47e01be..d0c1361 100644
>--- a/drivers/net/qede/Makefile
>+++ b/drivers/net/qede/Makefile
>@@ -27,7 +27,6 @@ OS_TYPE := $(shell uname -s)
> # CFLAGS
> #
> CFLAGS_BASE_DRIVER = -Wno-unused-parameter
>-CFLAGS_BASE_DRIVER += -Wno-unused-value
> CFLAGS_BASE_DRIVER += -Wno-sign-compare
> CFLAGS_BASE_DRIVER += -Wno-missing-prototypes
> CFLAGS_BASE_DRIVER += -Wno-cast-qual
>@@ -35,12 +34,16 @@ CFLAGS_BASE_DRIVER += -Wno-unused-function
> CFLAGS_BASE_DRIVER += -Wno-unused-variable
> CFLAGS_BASE_DRIVER += -Wno-strict-aliasing
> CFLAGS_BASE_DRIVER += -Wno-missing-prototypes
>+
>+ifneq ($(CONFIG_RTE_TOOLCHAIN_ICC),y)
>+CFLAGS_BASE_DRIVER += -Wno-unused-value
This change is not for the actual build error but to mask the warning:
icc: command line warning #10006: ignoring unknown option
'-Wno-unused-value’, right?
If so, what about for Wno-format-nonliteral and Wno-shift-negative-value?
> CFLAGS_BASE_DRIVER += -Wno-format-nonliteral
> ifeq ($(OS_TYPE),Linux)
> ifeq ($(shell clang -Wno-shift-negative-value -Werror -E - < /dev/null >
>/dev/null 2>&1; echo $$?),0)
> CFLAGS_BASE_DRIVER += -Wno-shift-negative-value
> endif
> endif
>+endif
>
> ifneq (,$(filter gcc gcc48,$(CC)))
> CFLAGS_BASE_DRIVER += -Wno-unused-but-set-variable
>@@ -57,7 +60,7 @@ ifeq ($(shell clang -Wno-pointer-bool-conversion
>-Werror -E - < /dev/null > /dev
> CFLAGS_BASE_DRIVER += -Wno-pointer-bool-conversion
> endif
> else
>-#icc flags
>+CFLAGS_BASE_DRIVER += -wd188 #188: enumerated type mixed with another
>type
> endif
>
> #
>--
>2.5.5
>
>
Looks good.
I hope its retested against gcc/clang also to make sure the change doesn’t
cause build errors.
Acked-by: Harish Patil <harish.patil@qlogic.com>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] qede: fix icc build error
2016-05-10 20:02 ` Harish Patil
@ 2016-05-11 17:17 ` Ferruh Yigit
2016-05-13 15:27 ` Thomas Monjalon
1 sibling, 0 replies; 4+ messages in thread
From: Ferruh Yigit @ 2016-05-11 17:17 UTC (permalink / raw)
To: Harish Patil, dev; +Cc: Rasesh Mody, Sony Chacko
On 5/10/2016 9:02 PM, Harish Patil wrote:
> Hi Ferruh,
>
>> fix errors:
>> icc: command line warning #10006: ignoring unknown option
>> '-Wno-unused-value'
>> icc: command line warning #10006: ignoring unknown option
>> '-Wno-format-nonliteral'
>> icc: command line warning #10006: ignoring unknown option
>> '-Wno-shift-negative-value'
>> qede/base/ecore_dev.c(1643): error #188: enumerated type mixed with
>> another type
>> return 0;
>> ^
>>
>> Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>> ---
>> drivers/net/qede/Makefile | 7 +++++--
>> 1 file changed, 5 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/net/qede/Makefile b/drivers/net/qede/Makefile
>> index 47e01be..d0c1361 100644
>> --- a/drivers/net/qede/Makefile
>> +++ b/drivers/net/qede/Makefile
>> @@ -27,7 +27,6 @@ OS_TYPE := $(shell uname -s)
>> # CFLAGS
>> #
>> CFLAGS_BASE_DRIVER = -Wno-unused-parameter
>> -CFLAGS_BASE_DRIVER += -Wno-unused-value
>> CFLAGS_BASE_DRIVER += -Wno-sign-compare
>> CFLAGS_BASE_DRIVER += -Wno-missing-prototypes
>> CFLAGS_BASE_DRIVER += -Wno-cast-qual
>> @@ -35,12 +34,16 @@ CFLAGS_BASE_DRIVER += -Wno-unused-function
>> CFLAGS_BASE_DRIVER += -Wno-unused-variable
>> CFLAGS_BASE_DRIVER += -Wno-strict-aliasing
>> CFLAGS_BASE_DRIVER += -Wno-missing-prototypes
>> +
>> +ifneq ($(CONFIG_RTE_TOOLCHAIN_ICC),y)
>> +CFLAGS_BASE_DRIVER += -Wno-unused-value
>
> This change is not for the actual build error but to mask the warning:
>
> icc: command line warning #10006: ignoring unknown option
> '-Wno-unused-value’, right?
>
>
> If so, what about for Wno-format-nonliteral and Wno-shift-negative-value?
These three flags are not supported by icc, please check commit log, it
has three of them in warnings, so removing them for icc. This doesn't
cause a compilation error, just warnings from compiler but it is too
noisy, I believe good to remove.
>
>> CFLAGS_BASE_DRIVER += -Wno-format-nonliteral
>> ifeq ($(OS_TYPE),Linux)
>> ifeq ($(shell clang -Wno-shift-negative-value -Werror -E - < /dev/null >
>> /dev/null 2>&1; echo $$?),0)
>> CFLAGS_BASE_DRIVER += -Wno-shift-negative-value
>> endif
>> endif
>> +endif
>>
>> ifneq (,$(filter gcc gcc48,$(CC)))
>> CFLAGS_BASE_DRIVER += -Wno-unused-but-set-variable
>> @@ -57,7 +60,7 @@ ifeq ($(shell clang -Wno-pointer-bool-conversion
>> -Werror -E - < /dev/null > /dev
>> CFLAGS_BASE_DRIVER += -Wno-pointer-bool-conversion
>> endif
>> else
>> -#icc flags
>> +CFLAGS_BASE_DRIVER += -wd188 #188: enumerated type mixed with another
>> type
>> endif
>>
>> #
>> --
>> 2.5.5
>>
>>
> Looks good.
> I hope its retested against gcc/clang also to make sure the change doesn’t
> cause build errors.
Yes, did a gcc and clang compile test.
> Acked-by: Harish Patil <harish.patil@qlogic.com>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] qede: fix icc build error
2016-05-10 20:02 ` Harish Patil
2016-05-11 17:17 ` Ferruh Yigit
@ 2016-05-13 15:27 ` Thomas Monjalon
1 sibling, 0 replies; 4+ messages in thread
From: Thomas Monjalon @ 2016-05-13 15:27 UTC (permalink / raw)
To: Ferruh Yigit; +Cc: dev, Harish Patil, Rasesh Mody, Sony Chacko
> >fix errors:
> >icc: command line warning #10006: ignoring unknown option
> >'-Wno-unused-value'
> >icc: command line warning #10006: ignoring unknown option
> >'-Wno-format-nonliteral'
> >icc: command line warning #10006: ignoring unknown option
> >'-Wno-shift-negative-value'
> >qede/base/ecore_dev.c(1643): error #188: enumerated type mixed with
> >another type
> > return 0;
> > ^
> >
> >Signed-off-by: Ferruh Yigit <ferruh.yigit@intel.com>
>
> Looks good.
> I hope its retested against gcc/clang also to make sure the change doesn’t
> cause build errors.
>
> Acked-by: Harish Patil <harish.patil@qlogic.com>
Fixes: ec94dbc57362 ("qede: add base driver")
Applied, thanks
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2016-05-13 15:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-10 11:23 [PATCH] qede: fix icc build error Ferruh Yigit
2016-05-10 20:02 ` Harish Patil
2016-05-11 17:17 ` Ferruh Yigit
2016-05-13 15:27 ` Thomas Monjalon
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.