[coreboot-gerrit] Change in coreboot[master]: soc/intel/graphics: Create Kconfig for shifting graphic memory base

[Wonkyu Kim (Code Review)]

Attention is currently required from: Subrata Banik, Nico Huber, Ethan Tsao, Ravishankar Sarawadi, Paul Menzel, Raj Astekar, Patrick Rudolph.
Wonkyu Kim has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/61389 )

Change subject: soc/intel/graphics: Create Kconfig for shifting graphic memory base
......................................................................

Patch Set 11: Code-Review+1

(2 comments)

File src/soc/intel/common/block/graphics/graphics.c:

https://review.coreboot.org/c/coreboot/+/61389/comment/98e7b53e_b144d85b 
PS11, Line 122: 	printk(BIOS_INFO, "gfx memory bar(0x18) = 0x%lx GFX_MEMBASE_OFFSET = 0x%x\n",
              : 		memory_base, CONFIG_SOC_INTEL_GFX_MEMBASE_OFFSET);
> The purpose of this message is to print gfx basic information without adding debug message and re-bu […]

Can we also print out final memory_base?
We can print out 3 values or final memory_base with memory bar should be ok as we can figureout offset from 2 values.
BTW, how many times does this printout happen during boot up?
As this can be common API, we may need to optimize log if there are too many.

https://review.coreboot.org/c/coreboot/+/61389/comment/f8240f48_af74ea67 
PS11, Line 129: memory_base = memory_base + CONFIG_SOC_INTEL_GFX_MEMBASE_OFFSET;
> I think the purpose of if clause is checking if memory bar is zero. […]

I also think above memory_base checking for checking PCI config space 0x18 is programmed or not.