From e02e918eba3f0803d354faf3d28072ee2307f7e5 Mon Sep 17 00:00:00 2001 From: Jeremy Compostella Date: Tue, 24 Jan 2023 13:59:46 -0700 Subject: [PATCH] drivers/intel/gma: Dump output setting only if DEBUG_ADA_CODE is set This patch restricts the dump of the vebose graphics output settings to configuration with the `DEBUG_ADA_CODE' flag set. BUG=b:264526798 BRANCH=firmware-brya-14505.B TEST=Configuration dump is seen only if DEBUG_ADA_CODE is set Change-Id: Iadd6c9552b184f7d6ec8df9d0d392634864ba50c Signed-off-by: Jeremy Compostella Reviewed-on: https://review.coreboot.org/c/coreboot/+/72418 Reviewed-by: Tarun Tuli Tested-by: build bot (Jenkins) Reviewed-by: Nick Vaccaro --- src/drivers/intel/gma/hires_fb/gma-gfx_init.adb | 2 +- src/drivers/intel/gma/text_fb/gma-gfx_init.adb | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/drivers/intel/gma/hires_fb/gma-gfx_init.adb b/src/drivers/intel/gma/hires_fb/gma-gfx_init.adb index 048b7bd9da..420dc6e7d1 100644 --- a/src/drivers/intel/gma/hires_fb/gma-gfx_init.adb +++ b/src/drivers/intel/gma/hires_fb/gma-gfx_init.adb @@ -70,7 +70,7 @@ is configs (i).Framebuffer := fb; end loop; - HW.GFX.GMA.Dump_Configs (configs); + pragma Debug (HW.GFX.GMA.Dump_Configs (configs)); HW.GFX.GMA.Setup_Default_FB (FB => fb, diff --git a/src/drivers/intel/gma/text_fb/gma-gfx_init.adb b/src/drivers/intel/gma/text_fb/gma-gfx_init.adb index 026cb16b5c..cec21b5310 100644 --- a/src/drivers/intel/gma/text_fb/gma-gfx_init.adb +++ b/src/drivers/intel/gma/text_fb/gma-gfx_init.adb @@ -48,7 +48,7 @@ is configs (Primary).Framebuffer.Offset := VGA_PLANE_FRAMEBUFFER_OFFSET; - HW.GFX.GMA.Dump_Configs (configs); + pragma Debug (HW.GFX.GMA.Dump_Configs (configs)); HW.GFX.GMA.Update_Outputs (configs); lightup_ok := 1;