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