[coreboot] [PATCH] MCP55 azalia driver

[Stefan Reinauer]

On 6/1/10 8:02 AM, Peter Stuge wrote:
> So is there absolutely no way for us to avoid that this code is
> duplicated for *every* board with such a select?
>
> It seems wrong to automatically add a driver during build, and then
> need to call it manually in boards where it was automatically built!
>   
That's actually not what happens. You only need to call it for those
boards that need to load a HDA "VERB table". Many boards should be able
to work without loading a VERB. Unfortuntately I could not test this on
all MCP55 boards, so hence the "safe way".

Providing the HDA VERB table is a mainboard specific thing. So it
belongs into the mainboard.

We could add some magic to load the VERB upon CONFIG_LOAD_VERB. I
considered this back when I implemented the algorithm, but it would even
more mean jumping between components with the code and making it harder
to understand.

> I think this is pretty important. It seems also related to the VSA
> situation for the Geode that I worked around in r5581.
>   
In which way?

Stefan

-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: +49 761 7668825 • Fax: +49 761 7664613
Email: info at coresystems.de  • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866