[Getting back to the older stuff...]>
On Fri, 09 Apr 2010 22:34:16 +0200
Florian Tobias Schandinat <FlorianSchandinat@xxxxxx> wrote:
Just a minor nit:
Could we change the default so that if someone adds support for a new IGP (and misses this function) we default to either the newest or preferably to none? I've just seen too much poorly maintained code in this driver and defaulting to the oldest is hence a bad idea.
Otherwise it's fine.
In the absence of that, the only course of action which makes sense is
to simply fail the initialization if an unknown chip type shows up
there. That's easy, and I can do it. But, given that this was a
"minor nit," can we leave it as-is for now?
There's a *lot* of things to clean up in this driver, I'd like to make it> better a step at a time rather than trying to do the whole thing at once.