| Message ID | 20191210203101.2663341-1-arnd@arndb.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | drm/amd/display: fix undefined struct member reference | expand |
> -----Original Message----- > From: Arnd Bergmann <arnd@arndb.de> > Sent: 2019/December/10, Tuesday 3:31 PM > To: Wentland, Harry <Harry.Wentland@amd.com>; Li, Sun peng (Leo) > <Sunpeng.Li@amd.com>; Deucher, Alexander > <Alexander.Deucher@amd.com>; Koenig, Christian > <Christian.Koenig@amd.com>; Zhou, David(ChunMing) > <David1.Zhou@amd.com>; David Airlie <airlied@linux.ie>; Daniel Vetter > <daniel@ffwll.ch>; Liu, Zhan <Zhan.Liu@amd.com> > Cc: Arnd Bergmann <arnd@arndb.de>; Laktyushkin, Dmytro > <Dmytro.Laktyushkin@amd.com>; Lakha, Bhawanpreet > <Bhawanpreet.Lakha@amd.com>; Lei, Jun <Jun.Lei@amd.com>; Liu, > Charlene <Charlene.Liu@amd.com>; Yang, Eric <Eric.Yang2@amd.com>; > Cornij, Nikola <Nikola.Cornij@amd.com>; amd-gfx@lists.freedesktop.org; > dri-devel@lists.freedesktop.org; linux-kernel@vger.kernel.org > Subject: [PATCH] drm/amd/display: fix undefined struct member reference > > An initialization was added for two optional struct members. One of these is > always present in the dcn20_resource file, but the other one depends on > CONFIG_DRM_AMD_DC_DSC_SUPPORT and causes a build failure if that is > missing: > > drivers/gpu/drm/amd/amdgpu/../display/dc/dcn20/dcn20_resource.c:926:1 > 4: error: excess elements in struct initializer [-Werror] > .num_dsc = 5, > > Add another #ifdef around the assignment. > > Fixes: c3d03c5a196f ("drm/amd/display: Include num_vmid and num_dsc > within NV14's resource caps") > Signed-off-by: Arnd Bergmann <arnd@arndb.de> Thank you for catching that
On 2019-12-10 3:54 p.m., Liu, Zhan wrote: > > >> -----Original Message----- >> From: Arnd Bergmann <arnd@arndb.de> >> Sent: 2019/December/10, Tuesday 3:31 PM >> To: Wentland, Harry <Harry.Wentland@amd.com>; Li, Sun peng (Leo) >> <Sunpeng.Li@amd.com>; Deucher, Alexander >> <Alexander.Deucher@amd.com>; Koenig, Christian >> <Christian.Koenig@amd.com>; Zhou, David(ChunMing) >> <David1.Zhou@amd.com>; David Airlie <airlied@linux.ie>; Daniel Vetter >> <daniel@ffwll.ch>; Liu, Zhan <Zhan.Liu@amd.com> >> Cc: Arnd Bergmann <arnd@arndb.de>; Laktyushkin, Dmytro >> <Dmytro.Laktyushkin@amd.com>; Lakha, Bhawanpreet >> <Bhawanpreet.Lakha@amd.com>; Lei, Jun <Jun.Lei@amd.com>; Liu, >> Charlene <Charlene.Liu@amd.com>; Yang, Eric <Eric.Yang2@amd.com>; >> Cornij, Nikola <Nikola.Cornij@amd.com>; amd-gfx@lists.freedesktop.org; >> dri-devel@lists.freedesktop.org; linux-kernel@vger.kernel.org >> Subject: [PATCH] drm/amd/display: fix undefined struct member reference >> >> An initialization was added for two optional struct members. One of these is >> always present in the dcn20_resource file, but the other one depends on >> CONFIG_DRM_AMD_DC_DSC_SUPPORT and causes a build failure if that is >> missing: >> >> drivers/gpu/drm/amd/amdgpu/../display/dc/dcn20/dcn20_resource.c:926:1 >> 4: error: excess elements in struct initializer [-Werror] >> .num_dsc = 5, >> >> Add another #ifdef around the assignment. >> >> Fixes: c3d03c5a196f ("drm/amd/display: Include num_vmid and num_dsc >> within NV14's resource caps") >> Signed-off-by: Arnd Bergmann <arnd@arndb.de> > > Thank you for catching that
On Tue, Dec 10, 2019 at 9:56 PM Kazlauskas, Nicholas <nicholas.kazlauskas@amd.com> wrote: > On 2019-12-10 3:54 p.m., Liu, Zhan wrote: > >> > >> Fixes: c3d03c5a196f ("drm/amd/display: Include num_vmid and num_dsc > >> within NV14's resource caps") > >> Signed-off-by: Arnd Bergmann <arnd@arndb.de> > > > > Thank you for catching that On my side I kept that flag enabled all the time, so I didn't realize there was a warning hidden here. > > > > Reviewed-by: Zhan Liu <zhan.liu@amd.com> > > What tree is this reported on? This is plain linux-5.5-rc1. > We dropped this flag whenever building DCN. Sounds like we're missing a > patch if you're getting this. > > So this is a NAK from me for going into amd-staging-drm-next at least. Fair enough, please revert c3d03c5a196f ("drm/amd/display: Include num_vmid and num_dsc within NV14's resource caps") for 5.5-rc2 then. Arnd
> -----Original Message----- > From: Kazlauskas, Nicholas <Nicholas.Kazlauskas@amd.com> > Sent: Tuesday, December 10, 2019 3:56 PM > To: Liu, Zhan <Zhan.Liu@amd.com>; Arnd Bergmann <arnd@arndb.de>; > Wentland, Harry <Harry.Wentland@amd.com>; Li, Sun peng (Leo) > <Sunpeng.Li@amd.com>; Deucher, Alexander > <Alexander.Deucher@amd.com>; Koenig, Christian > <Christian.Koenig@amd.com>; Zhou, David(ChunMing) > <David1.Zhou@amd.com>; David Airlie <airlied@linux.ie>; Daniel Vetter > <daniel@ffwll.ch> > Cc: Liu, Charlene <Charlene.Liu@amd.com>; Yang, Eric > <Eric.Yang2@amd.com>; linux-kernel@vger.kernel.org; amd- > gfx@lists.freedesktop.org; Cornij, Nikola <Nikola.Cornij@amd.com>; > Laktyushkin, Dmytro <Dmytro.Laktyushkin@amd.com>; dri- > devel@lists.freedesktop.org; Lei, Jun <Jun.Lei@amd.com>; Lakha, > Bhawanpreet <Bhawanpreet.Lakha@amd.com> > Subject: Re: [PATCH] drm/amd/display: fix undefined struct member > reference > > On 2019-12-10 3:54 p.m., Liu, Zhan wrote: > > > > > >> -----Original Message----- > >> From: Arnd Bergmann <arnd@arndb.de> > >> Sent: 2019/December/10, Tuesday 3:31 PM > >> To: Wentland, Harry <Harry.Wentland@amd.com>; Li, Sun peng (Leo) > >> <Sunpeng.Li@amd.com>; Deucher, Alexander > <Alexander.Deucher@amd.com>; > >> Koenig, Christian <Christian.Koenig@amd.com>; Zhou, David(ChunMing) > >> <David1.Zhou@amd.com>; David Airlie <airlied@linux.ie>; Daniel Vetter > >> <daniel@ffwll.ch>; Liu, Zhan <Zhan.Liu@amd.com> > >> Cc: Arnd Bergmann <arnd@arndb.de>; Laktyushkin, Dmytro > >> <Dmytro.Laktyushkin@amd.com>; Lakha, Bhawanpreet > >> <Bhawanpreet.Lakha@amd.com>; Lei, Jun <Jun.Lei@amd.com>; Liu, > >> Charlene <Charlene.Liu@amd.com>; Yang, Eric <Eric.Yang2@amd.com>; > >> Cornij, Nikola <Nikola.Cornij@amd.com>; > >> amd-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org; > >> linux-kernel@vger.kernel.org > >> Subject: [PATCH] drm/amd/display: fix undefined struct member > >> reference > >> > >> An initialization was added for two optional struct members. One of > >> these is always present in the dcn20_resource file, but the other one > >> depends on CONFIG_DRM_AMD_DC_DSC_SUPPORT and causes a build > failure > >> if that is > >> missing: > >> > >> > drivers/gpu/drm/amd/amdgpu/../display/dc/dcn20/dcn20_resource.c:926:1 > >> 4: error: excess elements in struct initializer [-Werror] > >> .num_dsc = 5, > >> > >> Add another #ifdef around the assignment. > >> > >> Fixes: c3d03c5a196f ("drm/amd/display: Include num_vmid and num_dsc > >> within NV14's resource caps") > >> Signed-off-by: Arnd Bergmann <arnd@arndb.de> > > > > Thank you for catching that
diff --git a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c index faab89d1e694..fdf93e6edf43 100644 --- a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c +++ b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c @@ -923,7 +923,9 @@ static const struct resource_caps res_cap_nv14 = { .num_dwb = 1, .num_ddc = 5, .num_vmid = 16, +#ifdef CONFIG_DRM_AMD_DC_DSC_SUPPORT .num_dsc = 5, +#endif }; static const struct dc_debug_options debug_defaults_drv = {
An initialization was added for two optional struct members. One of these is always present in the dcn20_resource file, but the other one depends on CONFIG_DRM_AMD_DC_DSC_SUPPORT and causes a build failure if that is missing: drivers/gpu/drm/amd/amdgpu/../display/dc/dcn20/dcn20_resource.c:926:14: error: excess elements in struct initializer [-Werror] .num_dsc = 5, Add another #ifdef around the assignment. Fixes: c3d03c5a196f ("drm/amd/display: Include num_vmid and num_dsc within NV14's resource caps") Signed-off-by: Arnd Bergmann <arnd@arndb.de> --- drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c | 2 ++ 1 file changed, 2 insertions(+)