Skip to content

Commit

Permalink
[PATCH] Fix "value computed not used" warnings
Browse files Browse the repository at this point in the history
Fixes for annoying gcc-4.1 compile warnings "value computed not used".
Simply cast to void.

(akpm: Linus will go ballistic...)

Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
  • Loading branch information
Takashi Iwai authored and Linus Torvalds committed Mar 24, 2006
1 parent ac51589 commit 10e705f
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 10 deletions.
16 changes: 8 additions & 8 deletions drivers/block/paride/comm.c
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ static int comm_read_regr( PIA *pi, int cont, int regr )

case 2:
case 3:
case 4: w3(r+0x20); r1();
case 4: w3(r+0x20); (void)r1();
w2(0x24); h = r4(); w2(4);
return h;

Expand All @@ -82,7 +82,7 @@ static void comm_write_regr( PIA *pi, int cont, int regr, int val )

case 2:
case 3:
case 4: w3(r); r1(); w4(val);
case 4: w3(r); (void)r1(); w4(val);
break;
}
}
Expand Down Expand Up @@ -126,17 +126,17 @@ static void comm_read_block( PIA *pi, char * buf, int count )
w2(4);
break;

case 2: w3(0x68); r1(); w2(0x24);
case 2: w3(0x68); (void)r1(); w2(0x24);
for (i=0;i<count;i++) buf[i] = r4();
w2(4);
break;

case 3: w3(0x68); r1(); w2(0x24);
case 3: w3(0x68); (void)r1(); w2(0x24);
for (i=0;i<count/2;i++) ((u16 *)buf)[i] = r4w();
w2(4);
break;

case 4: w3(0x68); r1(); w2(0x24);
case 4: w3(0x68); (void)r1(); w2(0x24);
for (i=0;i<count/4;i++) ((u32 *)buf)[i] = r4l();
w2(4);
break;
Expand All @@ -160,15 +160,15 @@ static void comm_write_block( PIA *pi, char * buf, int count )
w2(5); w2(4);
break;

case 2: w3(0x48); r1();
case 2: w3(0x48); (void)r1();
for (k=0;k<count;k++) w4(buf[k^1]);
break;

case 3: w3(0x48); r1();
case 3: w3(0x48); (void)r1();
for (k=0;k<count/2;k++) w4w(pi_swab16(buf,k));
break;

case 4: w3(0x48); r1();
case 4: w3(0x48); (void)r1();
for (k=0;k<count/4;k++) w4l(pi_swab32(buf,k));
break;

Expand Down
2 changes: 1 addition & 1 deletion drivers/block/paride/on26.c
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ static int on26_read_regr( PIA *pi, int cont, int regr )
case 3:
case 4: w3(1); w3(1); w2(5); w4(r); w2(4);
w3(0); w3(0); w2(0x24); a = r4(); w2(4);
w2(0x24); r4(); w2(4);
w2(0x24); (void)r4(); w2(4);
return a;

}
Expand Down
2 changes: 1 addition & 1 deletion drivers/net/8139too.c
Original file line number Diff line number Diff line change
Expand Up @@ -1131,7 +1131,7 @@ static void __devexit rtl8139_remove_one (struct pci_dev *pdev)
No extra delay is needed with 33Mhz PCI, but 66Mhz may change this.
*/

#define eeprom_delay() RTL_R32(Cfg9346)
#define eeprom_delay() (void)RTL_R32(Cfg9346)

/* The EEPROM commands include the alway-set leading bit. */
#define EE_WRITE_CMD (5)
Expand Down

0 comments on commit 10e705f

Please sign in to comment.