[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-kernel
Subject:    Re: [PATCH] Isapnp warning
From:       Sean Neakums <sneakums () zork ! net>
Date:       2003-06-21 15:04:35
[Download RAW message or body]

Geert Uytterhoeven <geert@linux-m68k.org> writes:

> On 21 Jun 2003, Alan Cox wrote:
>> On Sul, 2003-06-15 at 19:36, Geert Uytterhoeven wrote:
>> > Isapnp: Kill warning if CONFIG_PCI is not set
>> > 
>> > --- linux-2.5.x/drivers/pnp/resource.c	Tue May 27 19:03:04 2003
>> > +++ linux-m68k-2.5.x/drivers/pnp/resource.c	Sun Jun  8 13:31:20 2003
>> > @@ -97,7 +97,9 @@
>> >  
>> >  int pnp_add_irq_resource(struct pnp_dev *dev, int depnum, struct pnp_irq *data)
>> >  {
>> > +#ifdef CONFIG_PCI
>> >  	int i;
>> > +#endif
>> 
>> This is far uglier than te warning
>
> It depends on your goals. These warnings distract us from the real harmful
> warnings. Will we ever have a kernel that compiles with -Werror?

Unless GCC never emits an invalid warning, no.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/
[prev in list] [next in list] [prev in thread] [next in thread]