lsi53c895a: avoid a warning from clang analyzer

Avoid this warning from clang analyzer by deleting the variable:
/src/qemu/hw/lsi53c895a.c:895:5: warning: Value stored to 'id' is never read
    id = (current_tag >> 8) & 0xf;

Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
This commit is contained in:
Blue Swirl 2011-09-04 11:22:31 +00:00
parent 337fc75870
commit a6c6f44ae4
1 changed files with 1 additions and 4 deletions

View File

@ -883,7 +883,6 @@ static void lsi_do_msgout(LSIState *s)
int len; int len;
uint32_t current_tag; uint32_t current_tag;
lsi_request *current_req, *p, *p_next; lsi_request *current_req, *p, *p_next;
int id;
if (s->current) { if (s->current) {
current_tag = s->current->tag; current_tag = s->current->tag;
@ -892,7 +891,6 @@ static void lsi_do_msgout(LSIState *s)
current_tag = s->select_tag; current_tag = s->select_tag;
current_req = lsi_find_by_tag(s, current_tag); current_req = lsi_find_by_tag(s, current_tag);
} }
id = (current_tag >> 8) & 0xf;
DPRINTF("MSG out len=%d\n", s->dbc); DPRINTF("MSG out len=%d\n", s->dbc);
while (s->dbc) { while (s->dbc) {
@ -977,9 +975,8 @@ static void lsi_do_msgout(LSIState *s)
device, but this is currently not implemented (and seems not device, but this is currently not implemented (and seems not
to be really necessary). So let's simply clear all queued to be really necessary). So let's simply clear all queued
commands for the current device: */ commands for the current device: */
id = current_tag & 0x0000ff00;
QTAILQ_FOREACH_SAFE(p, &s->queue, next, p_next) { QTAILQ_FOREACH_SAFE(p, &s->queue, next, p_next) {
if ((p->tag & 0x0000ff00) == id) { if ((p->tag & 0x0000ff00) == (current_tag & 0x0000ff00)) {
scsi_req_cancel(p->req); scsi_req_cancel(p->req);
} }
} }