RE: [PATCH] ips driver for 2.4.23-pre4

[James Bottomley <James.Bottomley@steeleye.com>]

On Thu, 2004-01-08 at 15:18, Hammer, Jack wrote:
> The reason this compiler warning exists is that these are the only hardware
> platforms that support this adapter.  Use of this adapter is any other box
> can cause serious problems and is not supported by anyone.  Removal of this
> warning has no effect on the executable code and would not improve the
> driver in my opinion.  If I was including a module in my system that is not
> going to work, I'd like to know about it.
[...]
> > -#if !defined(__i386__) && !defined(__ia64__) -#error "This driver has 
> > only been tested on the x86/ia64 platforms"
> > +#if !defined(__i386__) && !defined(__ia64__) && !defined(__x86_64__) 
> > +#error "This driver has only been tested on the x86/ia64/x86_64
> platforms"
> >  #endif

This is actually an #error though, not a #warning...could we at least
downgrade it to #warning (like has already been done for ips) so I don't
get spurious compile failure reports from people doing make allyes
configurations on non-x86 boxes?

Thanks,

James