[coreboot] dead Kconfig variables in v3
Marc Jones
Marc.Jones at AMD.com
Thu Feb 14 17:38:17 CET 2008
I'll comment on a few.
Carl-Daniel Hailfinger wrote:
> The following config variables are tested in the code, but don't exist
> in Kconfig:
> CONFIG_ADDR
>
This is actually defined but never used in v2 or v3. Remove it.
> CONFIG_AGP_PLUGIN_SUPPORT
> CONFIG_BIG_ENDIAN
> CONFIG_CARDBUS_PLUGIN_SUPPORT
> CONFIG_CARTEST
>
Looks like old test code the was #if0 in v2. Remove it.
This is also related to a question I have below about stage0_i586.S CAR.
> CONFIG_CMD
> CONFIG_HYPERTRANSPORT_PLUGIN_SUPPORT
> CONFIG_MMCONF_SUPPORT
>
This will be needed when FAM10 support is added but I don't think it
should be removed. The mmconf PCI code is there and ready to be used.
> CONFIG_PCIE_PLUGIN_SUPPORT
> CONFIG_PCIX_PLUGIN_SUPPORT
> CONFIG_ROMSIZE
>
> CONFIG_STAGE0
> CONFIG_VIDEO_MB
>
CONFIG_VIDEO_MB is needed for Geode graphics. I will try to fix this
one today.
> CONFIG_XIP_ROM_BASE
> CONFIG_XIP_ROM_SIZE
>
>
XIP_ROM_ BASE and SIZE are AMD K8 and FAM10 specific. It is used to
cache the ROM area while doing CAR. In K8/FAM10 the cache is always on
to careful use of mtrrs keeps that cache from being flushed. Anyway, I
don't understand why stage0_i586.S would be so similar AMD CAR. Is it
just for emulation? I think that the emulator will ignore most cache
settings and can be cleaned up. There seems to be some history here
since v2 has cpu/x86/car/cache_as_ram.inc. Anyone know why?
> Should we just kill the code they are protecting or should we introcude
> Kconfig variables for them?
>
>
For the most part, I think that they should be removed until code is
ported that requires them but each section needs to be examined more
closely. As with the XIP_ROM it may bring up questions about what the
code around them is doing.
Marc
--
Marc Jones
Senior Firmware Engineer
(970) 226-9684 Office
mailto:Marc.Jones at amd.com
http://www.amd.com/embeddedprocessors
More information about the coreboot
mailing list