Re: [PATCH] Fixup warning for acenic

Matthew Wilcox (willy@debian.org)
Thu, 20 Mar 2003 15:59:43 +0000


On Thu, Mar 20, 2003 at 07:50:55AM -0800, Martin J. Bligh wrote:
> > Please don't delete this table. At some point when Jes gets his head
> > out of the "must support Linux 1.2" space, this table will be used and
> > then this driver will support hotplugging.
>
> Fair enough ... but can we wrap it in CONFIG_SOMETHING? or #if 0 ?

If you must, you could wrap it in MODULE. I don't see the value in
removing every single warning from the kernel build. If you're intent on
chasing all these pointless things, try installing gcc 3.3 and compiling
a kernel with that. It'll pump out more warnings than you can shake
a pointy stick at. Or turn on -W with gcc 2.96 -- it has much the
same effect. I made an effort to remove some of the -W warnings from
the header files a while ago so I could compile individual files with
-W as I find it tends to point out some mistakes I often make.

-- 
"It's not Hollywood.  War is real, war is primarily not about defeat or
victory, it is about death.  I've seen thousands and thousands of dead bodies.
Do you think I want to have an academic debate on this subject?" -- Robert Fisk
-
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/