Re: + clocksource-driver-initialize-list-value.patch added to -mm tree

From: Ingo Molnar
Date: Thu Apr 05 2007 - 03:52:18 EST



* Daniel Walker <dwalker@xxxxxxxxxx> wrote:

> [...] Adding a list initializer doesn't change anything..

Daniel, you are still in denial. OF COURSE it changes something: it adds
a line of code to a driver, where that line was not needed before.
That's against the fundamental task of a driver model: TO KEEP THINGS
SIMPLE. Yes, this concept includes single-line changes as well.

yes, a single line might not sound much, but it's exactly one line more
than necessary.

Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/