[coreboot] [PATCH] AMD MMCONF Support

Arne Georg Gleditsch arne.gleditsch at numascale.com
Wed Sep 8 17:00:18 CEST 2010


Myles Watson <mylesgw at gmail.com> writes:
> I'm confused why you wouldn't add the resource from the northbridge
> code.  Is there a reason to have it be a mainboard resource?

Not really, but there's no existing infrastructure for having
add_northbridge_resources be called except by way of
add_mainboard_resources.  As far as I can tell.  I think this may
warrant changing, but preferrably as a separate step.

> +config MMCONF_SUPPORT
> +	bool
> +	default y
> +	depends on NORTHBRIDGE_AMD_AMDFAM10
>
> You could use select for MMCONF_SUPPORT.  I think it should be
> selected in the northbridge or in the mainboard, but not both.

MMCONF_SUPPORT is selected in the northbridge, MMCONF_SUPPORT_DEFAULT is
selected in the mainboard config to actually activate the functionality.
This was the way I read the existing code, perhaps this two-level
approach is not be required?  Either way, I think you want to select
this on a mainboard-by-mainboard basis, at least initially.  There is a
potential for breakage, like the one we experienced with the nvidia
southbridge.

> Instead of adding the reserved area directly to the coreboot tables,
> you should add it before resource allocation and let the rest happen
> automatically.

I'm sorry, I'm not at all familiar with the resource allocation
framework.  I tried to model this on the corresponding code for relevant
Intel mainboards.  How would you change it, precisely?

-- 
							Arne.




More information about the coreboot mailing list