Thread (21 messages) 21 messages, 5 authors, 2016-04-01

Re: [PATCH v2 2/7] drivers/net/e1000: Fix missing brackets

From: Aaron Conole <aconole@redhat.com>
Date: 2016-03-30 13:14:43

Thomas Monjalon [off-list ref] writes:
2016-03-24 08:54, Panu Matilainen:
quoted
--- a/drivers/net/e1000/Makefile
+++ b/drivers/net/e1000/Makefile
@@ -54,6 +54,9 @@ else
  #
  CFLAGS_BASE_DRIVER = -Wno-uninitialized -Wno-unused-parameter
  CFLAGS_BASE_DRIVER += -Wno-unused-variable
+ifeq ($(shell test $(GCC_VERSION) -ge 60 && echo 1), 1)
+CFLAGS_BASE_DRIVER += -Wno-misleading-indentation
+endif
Aaron, have you tested this solution?
Are you going to provide a v3?
I haven't yet tested this solution, but if folks are that opposed to
changing the code, then I will test it and post a v3 of this particular
patch in the series. 

Thanks so much for the reviews and time on this (Panu AND Thomas :-)),
-Aaron
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help