From cdfe17df88b335269ddabc7ade7a6148a1a20f0d Mon Sep 17 00:00:00 2001 From: Blue Swirl Date: Sun, 19 Dec 2010 10:43:09 +0000 Subject: [PATCH] ahci: delete write-only variables (v2) Avoid these warnings with GCC 4.6.0: /src/qemu/hw/ide/ahci.c: In function 'ahci_reset_port': /src/qemu/hw/ide/ahci.c:810:14: error: variable 'tfd' set but not used [-Werror=unused-but-set-variable] /src/qemu/hw/ide/ahci.c: In function 'handle_cmd': /src/qemu/hw/ide/ahci.c:1103:19: error: variable 'pr' set but not used [-Werror=unused-but-set-variable] In the tfd variable case, fix the logic also. CC: Alexander Graf CC: Kevin Wolf Acked-by: Alexander Graf Signed-off-by: Blue Swirl --- hw/ide/ahci.c | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/hw/ide/ahci.c b/hw/ide/ahci.c index 8ae236af30..968fdcecaf 100644 --- a/hw/ide/ahci.c +++ b/hw/ide/ahci.c @@ -807,7 +807,6 @@ static void ahci_reset_port(AHCIState *s, int port) AHCIPortRegs *pr = &d->port_regs; IDEState *ide_state = &d->port.ifs[0]; uint8_t init_fis[0x20]; - uint32_t tfd; int i; DPRINTF(port, "reset port\n"); @@ -848,7 +847,7 @@ static void ahci_reset_port(AHCIState *s, int port) s->dev[port].port_state = STATE_RUN; if (!ide_state->bs) { s->dev[port].port_regs.sig = 0; - tfd = (1 << 8) | SEEK_STAT | WRERR_STAT; + ide_state->status = SEEK_STAT | WRERR_STAT; } else if (ide_state->drive_kind == IDE_CD) { s->dev[port].port_regs.sig = SATA_SIGNATURE_CDROM; ide_state->lcyl = 0x14; @@ -1100,7 +1099,6 @@ static void process_ncq_command(AHCIState *s, int port, uint8_t *cmd_fis, static int handle_cmd(AHCIState *s, int port, int slot) { IDEState *ide_state; - AHCIPortRegs *pr; uint32_t opts; uint64_t tbl_addr; AHCICmdHdr *cmd; @@ -1113,7 +1111,6 @@ static int handle_cmd(AHCIState *s, int port, int slot) return -1; } - pr = &s->dev[port].port_regs; cmd = &((AHCICmdHdr *)s->dev[port].lst)[slot]; if (!s->dev[port].lst) {