[PATCH 14/15] drm/amd/display: Enable warnings as errors
harry.wentland at amd.com
Wed Mar 1 15:04:01 UTC 2017
On 2017-02-28 08:20 PM, Emil Velikov wrote:
> 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.
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.
More information about the amd-gfx