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 _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
