drivers/scsi/advansys won't compile

Eyal Lebedinsky (eyal@fir.canberra.edu.au)
Wed, 8 May 1996 09:24:38 +1000 (EST)


And this is due to the BYTE and WORD types that got imported from
<linux/amigaffs.h>.

It is generaly bad practice to pollute the headers with all these standard
types - Linux should enforce one set of types (already available) and avoid
private ones in common headers.

As the number of kernel componnents grows, it would be nice if people limit
the number of types and #defines in headers when these are not necessary
outside the driver - I often see people breaking all the definitions for a
driver into the header which is used only by that driver. This header
sometimes ends up as a globally visible one.

In this specific case (see subject) the amigaffs header got in through the
general fs.h header.