[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 6/6] scsi-bus: remove overlapping entry
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 6/6] scsi-bus: remove overlapping entry |
Date: |
Thu, 02 Aug 2012 10:30:56 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.1 (gnu/linux) |
[cc: SCSI maintainer]
Blue Swirl <address@hidden> writes:
> LOAD_UNLOAD and START_STOP have same value, so the table
> entry is initialized twice. Spotted by Clang compiler.
>
> Remove LOAD_UNLOAD entry since START_STOP entry already
> represents both.
>
> Signed-off-by: Blue Swirl <address@hidden>
> ---
> hw/scsi-bus.c | 1 -
> 1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/hw/scsi-bus.c b/hw/scsi-bus.c
> index e4ec19e..a8759bc 100644
> --- a/hw/scsi-bus.c
> +++ b/hw/scsi-bus.c
> @@ -1355,7 +1355,6 @@ static const char *scsi_command_name(uint8_t cmd)
> [ BLANK ] = "BLANK",
> [ MOVE_MEDIUM ] = "MOVE_MEDIUM",
> [ EXCHANGE_MEDIUM ] = "EXCHANGE MEDIUM",
> - [ LOAD_UNLOAD ] = "LOAD_UNLOAD",
> [ READ_12 ] = "READ_12",
> [ WRITE_12 ] = "WRITE_12",
> [ ERASE_12 ] = "ERASE_12/GET_PERFORMANCE",
Missed in commit 15e58a21.
Reviewed-by: Markus Armbruster <address@hidden>
- [Qemu-devel] [PATCH v2 0/6] Clang patches, Blue Swirl, 2012/08/01
- [Qemu-devel] [PATCH v2 1/6] sparc: fix floppy TC line setup, Blue Swirl, 2012/08/01
- [Qemu-devel] [PATCH v2 2/6] sparc: fix expression with uninitialized initial value, Blue Swirl, 2012/08/01
- [Qemu-devel] [PATCH v2 3/6] qapi: avoid reserved keywords, Blue Swirl, 2012/08/01
- [Qemu-devel] [PATCH v2 4/6] user: fix accidental AREG0 use, Blue Swirl, 2012/08/01
- [Qemu-devel] [PATCH v2 6/6] scsi-bus: remove overlapping entry, Blue Swirl, 2012/08/01
- Re: [Qemu-devel] [PATCH v2 6/6] scsi-bus: remove overlapping entry,
Markus Armbruster <=
- [Qemu-devel] [PATCH v2 5/6] xilinx_axienet: avoid useless self-assignment, Blue Swirl, 2012/08/01