Re: CVS Update@oss.sgi.com: linux

[Ralf Baechle <ralf@oss.sgi.com>]

On Mon, Jan 15, 2001 at 06:03:10PM +0100, Maciej W. Rozycki wrote:

> > Modified files:
> >     arch/mips/dec  : Makefile rtc-dec.c 
> 
>  Hmm, shouldn't it use <linux/spinlock.h> instead?  Anyway it's
> include/linux/mc146818rtc.h that should be fixed instead -- there's
> nothing in rtc-dec.c that would need spinlocks. 

Sure, but that's !#@$#@^ typical Linux.  The include file structure sucks.
Frequently the only reason why things get written as macros is to avoid
having to include one header into another which makes the code more
confusing.  In the end the actual kernel code includes a giant load
of include files and nobody knows why they're necessary.

  Ralf