Thread (18 messages) 18 messages, 9 authors, 2011-08-01

[PATCH] Remove remaining references of CONFIG_GENERIC_TIME

From: geert@linux-m68k.org (Geert Uytterhoeven)
Date: 2011-08-01 10:09:10
Also in: linux-um, lkml

On Mon, Aug 1, 2011 at 10:36, Russell King - ARM Linux
[off-list ref] wrote:
On Sat, Jul 30, 2011 at 06:14:38PM +0200, Zoltan Devai wrote:
quoted
Commit 592913ecb87a9e06f98ddb55b298f1a66bf94c6b has killed off any
use of this config option long ago.
I don't see the point of this - we were free of GENERIC_TIME on ARM
shortly after it was originally killed off. ?The problem is you can't
stop people introducing new uses of this - because it existed once and
there's nothing which errors out on its presence, people are going to
continue submitting patches with it in. ?And it's going to continue
being missed at the review stage.

I've a similar problem with folk on ARM including mach/gpio.h as their
sole gpio header file rather than linux/gpio.h - I've been trying for
the last 1-2 years to educate people to use linux/ in preference. ?You
can't do it, and I'm still just about the only one who picks up on that.
(SoC maintainers don't care.) ?They will end up caring when I push a
change during the next merge window though, so I'll eventually stop
mach/gpio.h being included. ?(Instead, it'll be asm/gpio.h).

GENERIC_TIME though... I don't think you'll ever stop new uses of it
creeping in unless you can arrange for something to error out.
Doesn't kconf error out when trying to select a non-existent symbol?

Gr{oetje,eeting}s,

? ? ? ? ? ? ? ? ? ? ? ? Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert at linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
? ? ? ? ? ? ? ? ? ? ? ? ? ?? ?? -- Linus Torvalds
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help