Re: [PATCH 10/14] FRV: Make calibrate_delay() optional

From: David Howells
Date: Tue Nov 02 2004 - 11:37:51 EST



> this changelog certainly does not apply to the delay loop calibration.

I just duplicated the banners from init/main.c and tacked some extra bits on
the front.

> any reason it's in lib? Better move this to kernel and properly compile
> it conditionally.

So that it get built and placed in an archive library, thus allowing the
linker to decide whether to include it or not, without having to use
conditional stuff and without having to change every other arch to enable it.

I suppose I could do something like this in init/Makefile:

if ($(CONFIG_DISABLE_GENERIC_CALIBRATE_DELAY),y)
obj-y += calibrate.o
endif

That would allow me to avoid having to change all the archs.

David
-
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/