IA32 - 4 New warnings

IA32 - 4 New warnings

[John Cherry]

On the nightly IA32 compiles, I am now flagging NEW warnings that were
introduced into the build in the last 24 hours.  I'll send these out
automatically for awhile with the builds as it has some value at this
stage.

drivers/net/wan/cosa.c:516: warning: implicit declaration of function `sti'
drivers/net/wan/cosa.c:661: warning: `MOD_INC_USE_COUNT' is deprecated
(declared at include/linux/module.h:482)
drivers/net/wan/cosa.c:669: warning: `MOD_DEC_USE_COUNT' is deprecated
(declared at include/linux/module.h:494)
drivers/net/wan/cosa.c:729: warning: `MOD_DEC_USE_COUNT' is deprecated
(declared at include/linux/module.h:494)

John

Re: IA32 - 4 New warnings

[David S. Miller]

On Wed, 3 Sep 2003 23:48:54 -0700 (PDT)
John Cherry <cherry@osdl.org> wrote:

> drivers/net/wan/cosa.c:516: warning: implicit declaration of function `sti'
> drivers/net/wan/cosa.c:661: warning: `MOD_INC_USE_COUNT' is deprecated
> (declared at include/linux/module.h:482)

There is no way these warnings were added in the past
24 hours, they've been there nearly the entire 2.5.x
series.

Re: IA32 - 4 New warnings

[John Cherry]

David,

You are absolutely correct that these are not new warnings.  However,
they are new with respect to CLEAN_COMPILE being enabled by default.  On
Sept 2, Linus add the following patch...

-config BROKEN
-	bool "Prompt for old and known-broken drivers"
-	depends on EXPERIMENTAL
-	default n
+config CLEAN_COMPILE
+	bool "Don't select drivers known to be broken" if EXPERIMENTAL
+	default y 	help
-	  This option allows you to choose whether you want to try to
-	  compile (and fix) old drivers that haven't been updated to
-	  new infrastructure.
+	  Select this option if you don't even want to see the option
+	  to configure known-broken drivers.
+
+	  If unsure, say Y 
-	  If unsure, say N.
+config BROKEN
+	bool
+	depends on !CLEAN_COMPILE
+	default y

By default, the builds did not include known broken drivers.  The build 
yesterday picked up bunk's patch for COSA...

-	depends on WAN && ISA && m && BROKEN
+	depends on WAN && ISA && m

This allowed the driver to be built again, and the warnings reappeared.
So these ARE new warnings in the sense that bunk specifically declared
this driver NOT to be broken.  In fact, the changeset comment was "COSA
is no longer BROKEN".

Actually, to remain consistent with warning and error statistics, I will
need to over-ride the default CLEAN_COMPILE config option in the
automated builds and continue to build broken drivers.  Perhaps I will
build both.  Hey, it is just CPU time.  :)

John


On Thu, 2003-09-04 at 00:24, David S. Miller wrote:
> On Wed, 3 Sep 2003 23:48:54 -0700 (PDT)
> John Cherry <cherry@osdl.org> wrote:
> 
> > drivers/net/wan/cosa.c:516: warning: implicit declaration of function `sti'
> > drivers/net/wan/cosa.c:661: warning: `MOD_INC_USE_COUNT' is deprecated
> > (declared at include/linux/module.h:482)
> 
> There is no way these warnings were added in the past
> 24 hours, they've been there nearly the entire 2.5.x
> series.
> -
> 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/

IA32 - 4 New warnings

[John Cherry]

sound/oss/pss.c:1004: warning: `check_region' is deprecated (declared at include/linux/ioport.h:117)
sound/oss/pss.c:191: warning: `check_region' is deprecated (declared at include/linux/ioport.h:117)
sound/oss/pss.c:640: warning: `check_region' is deprecated (declared at include/linux/ioport.h:117)
sound/oss/pss.c:710: warning: `check_region' is deprecated (declared at include/linux/ioport.h:117)