From: "Russell, Kent" <Kent.Russell@amd.com> To: "Ho, Kenny" <Kenny.Ho@amd.com>, Alex Deucher <alexdeucher@gmail.com>, "Mahfooz, Hamza" <Hamza.Mahfooz@amd.com> Cc: "Li, Sun peng \(Leo\)" <Sunpeng.Li@amd.com>, "Kuehling, Felix" <Felix.Kuehling@amd.com>, "Pan, Xinhui" <Xinhui.Pan@amd.com>, "Siqueira, Rodrigo" <Rodrigo.Siqueira@amd.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, "amd-gfx@lists.freedesktop.org" <amd-gfx@lists.freedesktop.org>, "Deucher, Alexander" <Alexander.Deucher@amd.com>, "Koenig, Christian" <Christian.Koenig@amd.com> Subject: RE: [PATCH v2] drm/amd/display: enable more strict compile checks Date: Wed, 24 May 2023 19:41:02 +0000 [thread overview] Message-ID: <BL1PR12MB589849F37FBE98A3A06A316185419@BL1PR12MB5898.namprd12.prod.outlook.com> (raw) In-Reply-To: <DM6PR12MB3996C4248CBC968A7B08341583419@DM6PR12MB3996.namprd12.prod.outlook.com> [-- Attachment #1: Type: text/plain, Size: 4193 bytes --] [AMD Official Use Only - General] (Adding Felix in CC) I’m a fan of adding it to KFD as well. Felix, can you foresee any issues here? Kent From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> On Behalf Of Ho, Kenny Sent: Wednesday, May 24, 2023 3:23 PM To: Alex Deucher <alexdeucher@gmail.com>; Mahfooz, Hamza <Hamza.Mahfooz@amd.com> Cc: Li, Sun peng (Leo) <Sunpeng.Li@amd.com>; Wentland, Harry <Harry.Wentland@amd.com>; Pan, Xinhui <Xinhui.Pan@amd.com>; Siqueira, Rodrigo <Rodrigo.Siqueira@amd.com>; linux-kernel@vger.kernel.org; dri-devel@lists.freedesktop.org; amd-gfx@lists.freedesktop.org; Daniel Vetter <daniel@ffwll.ch>; Deucher, Alexander <Alexander.Deucher@amd.com>; David Airlie <airlied@gmail.com>; Koenig, Christian <Christian.Koenig@amd.com> Subject: Re: [PATCH v2] drm/amd/display: enable more strict compile checks [AMD Official Use Only - General] [AMD Official Use Only - General] (+ Felix) Should we do the same for other modules under amd (amdkfd)? I was going to enable full kernel werror in the kconfig used by my CI but this is probably better. Kenny ________________________________ From: Alex Deucher <alexdeucher@gmail.com<mailto:alexdeucher@gmail.com>> Sent: Wednesday, May 24, 2023 3:22 PM To: Mahfooz, Hamza <Hamza.Mahfooz@amd.com<mailto:Hamza.Mahfooz@amd.com>> Cc: amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> <amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org>>; Li, Sun peng (Leo) <Sunpeng.Li@amd.com<mailto:Sunpeng.Li@amd.com>>; Ho, Kenny <Kenny.Ho@amd.com<mailto:Kenny.Ho@amd.com>>; Pan, Xinhui <Xinhui.Pan@amd.com<mailto:Xinhui.Pan@amd.com>>; Siqueira, Rodrigo <Rodrigo.Siqueira@amd.com<mailto:Rodrigo.Siqueira@amd.com>>; linux-kernel@vger.kernel.org<mailto:linux-kernel@vger.kernel.org> <linux-kernel@vger.kernel.org<mailto:linux-kernel@vger.kernel.org>>; dri-devel@lists.freedesktop.org<mailto:dri-devel@lists.freedesktop.org> <dri-devel@lists.freedesktop.org<mailto:dri-devel@lists.freedesktop.org>>; Daniel Vetter <daniel@ffwll.ch<mailto:daniel@ffwll.ch>>; Deucher, Alexander <Alexander.Deucher@amd.com<mailto:Alexander.Deucher@amd.com>>; David Airlie <airlied@gmail.com<mailto:airlied@gmail.com>>; Wentland, Harry <Harry.Wentland@amd.com<mailto:Harry.Wentland@amd.com>>; Koenig, Christian <Christian.Koenig@amd.com<mailto:Christian.Koenig@amd.com>> Subject: Re: [PATCH v2] drm/amd/display: enable more strict compile checks On Wed, May 24, 2023 at 3:20 PM Hamza Mahfooz <hamza.mahfooz@amd.com<mailto:hamza.mahfooz@amd.com>> wrote: > > Currently, there are quite a number of issues that are quite easy for > the CI to catch, that slip through the cracks. Among them, there are > unused variable and indentation issues. Also, we should consider all > warnings to be compile errors, since the community will eventually end > up complaining about them. So, enable -Werror, -Wunused and > -Wmisleading-indentation for all kernel builds. > > Cc: Alex Deucher <alexander.deucher@amd.com<mailto:alexander.deucher@amd.com>> > Cc: Harry Wentland <harry.wentland@amd.com<mailto:harry.wentland@amd.com>> > Cc: Kenny Ho <kenny.ho@amd.com<mailto:kenny.ho@amd.com>> > Signed-off-by: Hamza Mahfooz <hamza.mahfooz@amd.com<mailto:hamza.mahfooz@amd.com>> > --- > v2: fix grammatical error > --- > 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 0d610cb376bb..3c44162ebe21 100644 > --- a/drivers/gpu/drm/amd/display/Makefile > +++ b/drivers/gpu/drm/amd/display/Makefile > @@ -26,6 +26,8 @@ > > AMDDALPATH = $(RELATIVE_AMD_DISPLAY_PATH) > > +subdir-ccflags-y += -Werror -Wunused -Wmisleading-indentation > + Care to enable this for the rest of amdgpu as well? Or send out an additional patch to do that? Either way: Reviewed-by: Alex Deucher <alexander.deucher@amd.com<mailto:alexander.deucher@amd.com>> Alex > subdir-ccflags-y += -I$(FULL_AMD_DISPLAY_PATH)/dc/inc/ > subdir-ccflags-y += -I$(FULL_AMD_DISPLAY_PATH)/dc/inc/hw > subdir-ccflags-y += -I$(FULL_AMD_DISPLAY_PATH)/dc/clk_mgr > -- > 2.40.1 > [-- Attachment #2: Type: text/html, Size: 8738 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: "Russell, Kent" <Kent.Russell@amd.com> To: "Ho, Kenny" <Kenny.Ho@amd.com>, Alex Deucher <alexdeucher@gmail.com>, "Mahfooz, Hamza" <Hamza.Mahfooz@amd.com> Cc: "Li, Sun peng \(Leo\)" <Sunpeng.Li@amd.com>, David Airlie <airlied@gmail.com>, "Kuehling, Felix" <Felix.Kuehling@amd.com>, "Pan, Xinhui" <Xinhui.Pan@amd.com>, "Siqueira, Rodrigo" <Rodrigo.Siqueira@amd.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, "amd-gfx@lists.freedesktop.org" <amd-gfx@lists.freedesktop.org>, Daniel Vetter <daniel@ffwll.ch>, "Deucher, Alexander" <Alexander.Deucher@amd.com>, "Wentland, Harry" <Harry.Wentland@amd.com>, "Koenig, Christian" <Christian.Koenig@amd.com> Subject: RE: [PATCH v2] drm/amd/display: enable more strict compile checks Date: Wed, 24 May 2023 19:41:02 +0000 [thread overview] Message-ID: <BL1PR12MB589849F37FBE98A3A06A316185419@BL1PR12MB5898.namprd12.prod.outlook.com> (raw) In-Reply-To: <DM6PR12MB3996C4248CBC968A7B08341583419@DM6PR12MB3996.namprd12.prod.outlook.com> [-- Attachment #1: Type: text/plain, Size: 4193 bytes --] [AMD Official Use Only - General] (Adding Felix in CC) I’m a fan of adding it to KFD as well. Felix, can you foresee any issues here? Kent From: amd-gfx <amd-gfx-bounces@lists.freedesktop.org> On Behalf Of Ho, Kenny Sent: Wednesday, May 24, 2023 3:23 PM To: Alex Deucher <alexdeucher@gmail.com>; Mahfooz, Hamza <Hamza.Mahfooz@amd.com> Cc: Li, Sun peng (Leo) <Sunpeng.Li@amd.com>; Wentland, Harry <Harry.Wentland@amd.com>; Pan, Xinhui <Xinhui.Pan@amd.com>; Siqueira, Rodrigo <Rodrigo.Siqueira@amd.com>; linux-kernel@vger.kernel.org; dri-devel@lists.freedesktop.org; amd-gfx@lists.freedesktop.org; Daniel Vetter <daniel@ffwll.ch>; Deucher, Alexander <Alexander.Deucher@amd.com>; David Airlie <airlied@gmail.com>; Koenig, Christian <Christian.Koenig@amd.com> Subject: Re: [PATCH v2] drm/amd/display: enable more strict compile checks [AMD Official Use Only - General] [AMD Official Use Only - General] (+ Felix) Should we do the same for other modules under amd (amdkfd)? I was going to enable full kernel werror in the kconfig used by my CI but this is probably better. Kenny ________________________________ From: Alex Deucher <alexdeucher@gmail.com<mailto:alexdeucher@gmail.com>> Sent: Wednesday, May 24, 2023 3:22 PM To: Mahfooz, Hamza <Hamza.Mahfooz@amd.com<mailto:Hamza.Mahfooz@amd.com>> Cc: amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org> <amd-gfx@lists.freedesktop.org<mailto:amd-gfx@lists.freedesktop.org>>; Li, Sun peng (Leo) <Sunpeng.Li@amd.com<mailto:Sunpeng.Li@amd.com>>; Ho, Kenny <Kenny.Ho@amd.com<mailto:Kenny.Ho@amd.com>>; Pan, Xinhui <Xinhui.Pan@amd.com<mailto:Xinhui.Pan@amd.com>>; Siqueira, Rodrigo <Rodrigo.Siqueira@amd.com<mailto:Rodrigo.Siqueira@amd.com>>; linux-kernel@vger.kernel.org<mailto:linux-kernel@vger.kernel.org> <linux-kernel@vger.kernel.org<mailto:linux-kernel@vger.kernel.org>>; dri-devel@lists.freedesktop.org<mailto:dri-devel@lists.freedesktop.org> <dri-devel@lists.freedesktop.org<mailto:dri-devel@lists.freedesktop.org>>; Daniel Vetter <daniel@ffwll.ch<mailto:daniel@ffwll.ch>>; Deucher, Alexander <Alexander.Deucher@amd.com<mailto:Alexander.Deucher@amd.com>>; David Airlie <airlied@gmail.com<mailto:airlied@gmail.com>>; Wentland, Harry <Harry.Wentland@amd.com<mailto:Harry.Wentland@amd.com>>; Koenig, Christian <Christian.Koenig@amd.com<mailto:Christian.Koenig@amd.com>> Subject: Re: [PATCH v2] drm/amd/display: enable more strict compile checks On Wed, May 24, 2023 at 3:20 PM Hamza Mahfooz <hamza.mahfooz@amd.com<mailto:hamza.mahfooz@amd.com>> wrote: > > Currently, there are quite a number of issues that are quite easy for > the CI to catch, that slip through the cracks. Among them, there are > unused variable and indentation issues. Also, we should consider all > warnings to be compile errors, since the community will eventually end > up complaining about them. So, enable -Werror, -Wunused and > -Wmisleading-indentation for all kernel builds. > > Cc: Alex Deucher <alexander.deucher@amd.com<mailto:alexander.deucher@amd.com>> > Cc: Harry Wentland <harry.wentland@amd.com<mailto:harry.wentland@amd.com>> > Cc: Kenny Ho <kenny.ho@amd.com<mailto:kenny.ho@amd.com>> > Signed-off-by: Hamza Mahfooz <hamza.mahfooz@amd.com<mailto:hamza.mahfooz@amd.com>> > --- > v2: fix grammatical error > --- > 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 0d610cb376bb..3c44162ebe21 100644 > --- a/drivers/gpu/drm/amd/display/Makefile > +++ b/drivers/gpu/drm/amd/display/Makefile > @@ -26,6 +26,8 @@ > > AMDDALPATH = $(RELATIVE_AMD_DISPLAY_PATH) > > +subdir-ccflags-y += -Werror -Wunused -Wmisleading-indentation > + Care to enable this for the rest of amdgpu as well? Or send out an additional patch to do that? Either way: Reviewed-by: Alex Deucher <alexander.deucher@amd.com<mailto:alexander.deucher@amd.com>> Alex > subdir-ccflags-y += -I$(FULL_AMD_DISPLAY_PATH)/dc/inc/ > subdir-ccflags-y += -I$(FULL_AMD_DISPLAY_PATH)/dc/inc/hw > subdir-ccflags-y += -I$(FULL_AMD_DISPLAY_PATH)/dc/clk_mgr > -- > 2.40.1 > [-- Attachment #2: Type: text/html, Size: 8738 bytes --]
next prev parent reply other threads:[~2023-05-24 19:41 UTC|newest] Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-05-24 19:19 [PATCH v2] drm/amd/display: enable more strict compile checks Hamza Mahfooz 2023-05-24 19:19 ` Hamza Mahfooz 2023-05-24 19:19 ` Hamza Mahfooz 2023-05-24 19:22 ` Alex Deucher 2023-05-24 19:22 ` Alex Deucher 2023-05-24 19:22 ` Alex Deucher 2023-05-24 19:23 ` Ho, Kenny 2023-05-24 19:23 ` Ho, Kenny 2023-05-24 19:41 ` Russell, Kent [this message] 2023-05-24 19:41 ` Russell, Kent 2023-05-24 19:45 ` Felix Kuehling 2023-05-24 19:45 ` Felix Kuehling 2023-05-24 19:45 ` Felix Kuehling 2023-05-24 19:50 ` Alex Deucher 2023-05-24 19:50 ` Alex Deucher 2023-05-24 19:50 ` Alex Deucher 2023-05-24 19:56 ` Ho, Kenny 2023-05-24 19:56 ` Ho, Kenny 2023-05-24 19:56 ` Ho, Kenny 2023-05-24 20:27 ` Hamza Mahfooz 2023-05-24 20:27 ` Hamza Mahfooz 2023-05-24 20:27 ` Hamza Mahfooz 2023-05-25 7:48 ` Jani Nikula 2023-05-25 7:48 ` Jani Nikula 2023-05-25 15:37 ` Kees Cook 2023-05-25 15:37 ` Kees Cook 2023-05-25 15:37 ` Kees Cook 2023-05-25 16:26 ` Nathan Chancellor 2023-05-25 16:26 ` Nathan Chancellor 2023-05-25 16:26 ` Nathan Chancellor 2023-05-24 19:27 ` Hamza Mahfooz 2023-05-24 19:27 ` Hamza Mahfooz 2023-05-24 19:27 ` Hamza Mahfooz 2023-05-24 19:54 ` Harry Wentland 2023-05-24 19:54 ` Harry Wentland 2023-05-24 19:54 ` Harry Wentland 2023-05-24 19:57 ` Hamza Mahfooz 2023-05-24 19:57 ` Hamza Mahfooz 2023-05-24 19:57 ` Hamza Mahfooz 2023-05-25 9:43 ` Christoph Hellwig 2023-05-25 9:43 ` Christoph Hellwig 2023-05-26 5:00 ` kernel test robot 2023-05-26 5:00 ` kernel test robot
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=BL1PR12MB589849F37FBE98A3A06A316185419@BL1PR12MB5898.namprd12.prod.outlook.com \ --to=kent.russell@amd.com \ --cc=Alexander.Deucher@amd.com \ --cc=Christian.Koenig@amd.com \ --cc=Felix.Kuehling@amd.com \ --cc=Hamza.Mahfooz@amd.com \ --cc=Kenny.Ho@amd.com \ --cc=Rodrigo.Siqueira@amd.com \ --cc=Sunpeng.Li@amd.com \ --cc=Xinhui.Pan@amd.com \ --cc=alexdeucher@gmail.com \ --cc=amd-gfx@lists.freedesktop.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=linux-kernel@vger.kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.