Re: [PATCH 2/2] drm/amd/display: Don't build DCN1 when kcov is enabled
On 2018-08-16 09:44 PM, sunpeng...@amd.com wrote: > From: "Leo (Sunpeng) Li" > > DCN1 contains code that utilizes fp math. When > CONFIG_KCOV_INSTRUMENT_ALL and CONFIG_KCOV_ENABLE_COMPARISONS are > enabled, build errors are found. See this earlier patch for details: > > https://lists.freedesktop.org/archives/dri-devel/2018-August/186131.html > > As a short term solution, disable CONFIG_DRM_AMD_DC_DCN1_0 when > KCOV_INSTRUMENT_ALL and KCOV_ENABLE_COMPARISONS are enabled. In > addition, make it a fully derived config, taking into account > CONFIG_X86. > > Signed-off-by: Leo (Sunpeng) Li > --- > drivers/gpu/drm/amd/display/Kconfig | 8 +++- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/Kconfig > b/drivers/gpu/drm/amd/display/Kconfig > index 4c35625..ed654a7 100644 > --- a/drivers/gpu/drm/amd/display/Kconfig > +++ b/drivers/gpu/drm/amd/display/Kconfig > @@ -4,18 +4,16 @@ menu "Display Engine Configuration" > config DRM_AMD_DC > bool "AMD DC - Enable new display engine" > default y > + select DRM_AMD_DC_DCN1_0 if X86 && !(KCOV_INSTRUMENT_ALL && > KCOV_ENABLE_COMPARISONS) > help > Choose this option if you want to use the new display engine > support for AMDGPU. This adds required support for Vega and > Raven ASICs. > > config DRM_AMD_DC_DCN1_0 > - bool "DCN 1.0 Raven family" > - depends on DRM_AMD_DC && X86 > - default y > + def_bool n > help > - Choose this option if you want to have > - RV family for display engine > + RV family support for display engine > > config DEBUG_KERNEL_DC > bool "Enable kgdb break in DC" > Thanks Leo, Alex, Arnd et al for taking care of this! Apologies for the trouble caused by my change, and for not helping much with the solution (I was on vacation). This series is Reviewed-by: Michel Dänzer -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH 2/2] drm/amd/display: Don't build DCN1 when kcov is enabled
On Thu, Aug 16, 2018 at 9:45 PM wrote: > > From: "Leo (Sunpeng) Li" > > DCN1 contains code that utilizes fp math. When > CONFIG_KCOV_INSTRUMENT_ALL and CONFIG_KCOV_ENABLE_COMPARISONS are > enabled, build errors are found. See this earlier patch for details: > > https://lists.freedesktop.org/archives/dri-devel/2018-August/186131.html > > As a short term solution, disable CONFIG_DRM_AMD_DC_DCN1_0 when > KCOV_INSTRUMENT_ALL and KCOV_ENABLE_COMPARISONS are enabled. In > addition, make it a fully derived config, taking into account > CONFIG_X86. > > Signed-off-by: Leo (Sunpeng) Li Looks good to me, Acked-by: Arnd Bergmann ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
[PATCH 2/2] drm/amd/display: Don't build DCN1 when kcov is enabled
From: "Leo (Sunpeng) Li" DCN1 contains code that utilizes fp math. When CONFIG_KCOV_INSTRUMENT_ALL and CONFIG_KCOV_ENABLE_COMPARISONS are enabled, build errors are found. See this earlier patch for details: https://lists.freedesktop.org/archives/dri-devel/2018-August/186131.html As a short term solution, disable CONFIG_DRM_AMD_DC_DCN1_0 when KCOV_INSTRUMENT_ALL and KCOV_ENABLE_COMPARISONS are enabled. In addition, make it a fully derived config, taking into account CONFIG_X86. Signed-off-by: Leo (Sunpeng) Li --- drivers/gpu/drm/amd/display/Kconfig | 8 +++- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/amd/display/Kconfig b/drivers/gpu/drm/amd/display/Kconfig index 4c35625..ed654a7 100644 --- a/drivers/gpu/drm/amd/display/Kconfig +++ b/drivers/gpu/drm/amd/display/Kconfig @@ -4,18 +4,16 @@ menu "Display Engine Configuration" config DRM_AMD_DC bool "AMD DC - Enable new display engine" default y + select DRM_AMD_DC_DCN1_0 if X86 && !(KCOV_INSTRUMENT_ALL && KCOV_ENABLE_COMPARISONS) help Choose this option if you want to use the new display engine support for AMDGPU. This adds required support for Vega and Raven ASICs. config DRM_AMD_DC_DCN1_0 - bool "DCN 1.0 Raven family" - depends on DRM_AMD_DC && X86 - default y + def_bool n help - Choose this option if you want to have - RV family for display engine + RV family support for display engine config DEBUG_KERNEL_DC bool "Enable kgdb break in DC" -- 2.7.4 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx