On 2017-02-28 08:20 PM, Emil Velikov wrote:
Hi Harry,
On 1 March 2017 at 00:26, Harry Wentland <harry.wentl...@amd.com> wrote:
Change-Id: I48d38e4d0224c9f0e52055b3c4ddef8e872b3dac
Signed-off-by: Harry Wentland <harry.wentl...@amd.com>
Acked-by: Harry Wentland <harry.wentl...@amd.com>
Reviewed-by: Jordan Lazare <jordan.laz...@amd.com>
Reviewed-by: Tony Cheng <tony.ch...@amd.com>
---
drivers/gpu/drm/amd/display/Makefile | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/amd/display/Makefile
b/drivers/gpu/drm/amd/display/Makefile
index 8ba37dd9cf7f..c22475b55586 100644
--- a/drivers/gpu/drm/amd/display/Makefile
+++ b/drivers/gpu/drm/amd/display/Makefile
@@ -5,6 +5,8 @@
AMDDALPATH = $(RELATIVE_AMD_DISPLAY_PATH)
+subdir-ccflags-y += -Werror
+
Many kernel modules have tried this in the past and had to quickly
revert it. There's far too many compiler/toolchain combinations for
this to always be true.
By adding this you'll be adding serious annoyance to users and extra
burden on Alex.
-Emil
Point taken. I probably shouldn't have sent this one. This is something
we're doing in our internal trees to make sure we catch warnings at
pre-submission stage. Please ignore warnings too frequently and I'm
getting tired of fixing all of them.
Would be nice if there was some consistency to gcc's warnings, but
that's not something I see happening ever.
Harry
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
