Re: [PATCH] ARM: OMAP: Remove unused CONFIG MACH_OMAP_HTCWIZARD

From: Robert P. J. Day
Date: Sat Sep 26 2009 - 21:51:59 EST


On Sun, 27 Sep 2009, Alistair Buxton wrote:

> 2009/9/26 Jaswinder Singh Rajput <jaswinder@xxxxxxxxxx>:
> >
> > Remove unused CONFIG MACH_OMAP_HTCWIZARD from Kconfig
>
> Don't do it please! We're working on it.... board file is coming
> very soon.

i've noticed that happening fairly often -- feature infrastructure
is added before the feature itself. at the risk of sounding all
preachy and everything, that sort of thing should be avoided in order
to keep the history cleaner. and because that's *exactly* the sort of
thing that, occasionally, produces the kind of historical cruft that
leaves unused CONFIG variables lying around when a feature is
abandoned in midstream.

ok, i'll stop preaching now.

rday
--


========================================================================
Robert P. J. Day Waterloo, Ontario, CANADA

Linux Consulting, Training and Annoying Kernel Pedantry.

Web page: http://crashcourse.ca
Twitter: http://twitter.com/rpjday
========================================================================
--
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/