diff options
| author | Stefan Weil <weil@mail.berlios.de> | 2010-03-02 22:37:56 +0100 |
|---|---|---|
| committer | Michael S. Tsirkin <mst@redhat.com> | 2010-03-03 13:34:19 +0200 |
| commit | ec1d02d8de08ceda59b8a6e7a74ec1ae42a40ee6 (patch) | |
| tree | 50775cf77348260456681e3eda9db0a3413c2966 | |
| parent | cc02c66cc8c856ce6359efbbe5fc4535a34e605f (diff) | |
eepro100: Use tx.status
There is no need for a local variable "status".
Using tx.status makes it clearer which status
is addressed.
Signed-off-by: Stefan Weil <weil@mail.berlios.de>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
| -rw-r--r-- | hw/eepro100.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/hw/eepro100.c b/hw/eepro100.c index cf4e0ac24..f5aa3068a 100644 --- a/hw/eepro100.c +++ b/hw/eepro100.c @@ -903,10 +903,10 @@ static void action_command(EEPRO100State *s) for (;;) { s->cb_address = s->cu_base + s->cu_offset; cpu_physical_memory_read(s->cb_address, (uint8_t *)&s->tx, sizeof(s->tx)); - uint16_t status = le16_to_cpu(s->tx.status); uint16_t command = le16_to_cpu(s->tx.command); + s->tx.status = le16_to_cpu(s->tx.status); logout("val=(cu start), status=0x%04x, command=0x%04x, link=0x%08x\n", - status, command, s->tx.link); + s->tx.status, command, s->tx.link); bool bit_el = ((command & COMMAND_EL) != 0); bool bit_s = ((command & COMMAND_S) != 0); bool bit_i = ((command & COMMAND_I) != 0); @@ -950,7 +950,7 @@ static void action_command(EEPRO100State *s) break; } /* Write new status. */ - stw_phys(s->cb_address, status | STATUS_C | (success ? STATUS_OK : 0)); + stw_phys(s->cb_address, s->tx.status | STATUS_C | (success ? STATUS_OK : 0)); if (bit_i) { /* CU completed action. */ eepro100_cx_interrupt(s); |