linux-mips
[Top] [All Lists]

Re: Compiler warnings / linux-2.6

To: Ralf Rösch <linux@cantastic.de>
Subject: Re: Compiler warnings / linux-2.6
From: Ralf Baechle <ralf@linux-mips.org>
Date: Tue, 7 Jun 2005 12:37:31 +0100
Cc: linux-mips@linux-mips.org
In-reply-to: <42A47252.2010906@cantastic.de>
Original-recipient: rfc822;linux-mips@linux-mips.org
References: <42A47252.2010906@cantastic.de>
Sender: linux-mips-bounce@linux-mips.org
User-agent: Mutt/1.4.1i
On Mon, Jun 06, 2005 at 05:57:06PM +0200, Ralf Rösch wrote:

> Since a few days I get a lot of compiler warnings:
> 
> include2/asm/dsp.h: In function `__init_dsp':
> include2/asm/dsp.h:34: warning: statement with no effect
> 
> What could I do to prevent these?

Reporting them.  A fairly obvious and harmless problem.  I'm surprised my
compiler (gcc 3.4) doesn't throw these warnings.

  Ralf

Index: include/asm-mips/mipsregs.h
===================================================================
RCS file: /home/cvs/linux/include/asm-mips/mipsregs.h,v
retrieving revision 1.73
diff -u -r1.73 mipsregs.h
--- include/asm-mips/mipsregs.h 31 May 2005 11:49:19 -0000      1.73
+++ include/asm-mips/mipsregs.h 7 Jun 2005 11:39:10 -0000
@@ -1010,8 +1010,6 @@
 
 #define wrdsp(val, mask)                                               \
 do {                                                                   \
-       unsigned int __res;                                             \
-                                                                       \
        __asm__ __volatile__(                                           \
        "       .set    push                                    \n"     \
        "       .set    noat                                    \n"     \
@@ -1021,7 +1019,6 @@
        "       .set    pop                                     \n"     \
         :                                                              \
        : "r" (val), "i" (mask));                                       \
-        __res;                                                         \
 } while (0)
 
 #if 0  /* Need DSP ASE capable assembler ... */

<Prev in Thread] Current Thread [Next in Thread>