[PATCH 14/15] drm/amd/display: Enable warnings as errors

[Emil Velikov]

Hi Harry,

On 1 March 2017 at 00:26, Harry Wentland <harry.wentland at amd.com> wrote:
> Change-Id: I48d38e4d0224c9f0e52055b3c4ddef8e872b3dac
> Signed-off-by: Harry Wentland <harry.wentland at amd.com>
> Acked-by: Harry Wentland <Harry.Wentland at amd.com>
> Reviewed-by: Jordan Lazare <Jordan.Lazare at amd.com>
> Reviewed-by: Tony Cheng <Tony.Cheng at 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