coherent_ht_mainboard()
Li-Ta Lo
ollie at lanl.gov
Wed Mar 24 11:33:01 CET 2004
On Wed, 2004-03-24 at 09:41, Stefan Reinauer wrote:
> Yinghai,
>
> tyan/*/hypertransport.c contain some fixed hypertransport register
> values poked into the AMD K8 northbridge. This code was earlier called
> by coherent_ht.c:setup_coherent_ht_domain() but I removed the call some
> time ago, since I could not see why it is still needed. Does this code
> provide any crucial functionality that can't be realized in the dynamic
> code of the current CVS tree?
>
> If not, I suggest we drop this code.
>
> The same thing could btw work for the different versions of
> resourcemap.c. The biggest reason for the split off here was that the
> default resource map did always assume the AMD8111 southbridge on link0
> which is not always true. This is fixed by now.
> The bus numbers should meanwhile be entered correctly by the generic
> code.
> So, we could reduce the amount of involved code a lot, flattening out
> all the nasty exceptions we invented to bend the rules..
>
Can you tell me in which file you did obth of these ?
Ollie
> Stefan
More information about the coreboot
mailing list