diff options
| author | Blue Swirl | 2012-07-08 08:56:53 +0200 |
|---|---|---|
| committer | Blue Swirl | 2012-07-28 11:23:11 +0200 |
| commit | 0ed8b6f67f26acc2e88dfc6be4954e63f943bd28 (patch) | |
| tree | 8490bfb39362e56f8238ba9208114764bc938d7e /hw/ide/piix.c | |
| parent | hw/escc: Drop duplicate definition of 'disabled' property (diff) | |
| download | qemu-0ed8b6f67f26acc2e88dfc6be4954e63f943bd28.tar.gz qemu-0ed8b6f67f26acc2e88dfc6be4954e63f943bd28.tar.xz qemu-0ed8b6f67f26acc2e88dfc6be4954e63f943bd28.zip | |
Avoid returning void
It's silly and non-conforming to standards to return void,
don't do it.
Signed-off-by: Blue Swirl <blauwirbel@gmail.com>
Diffstat (limited to 'hw/ide/piix.c')
| -rw-r--r-- | hw/ide/piix.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/hw/ide/piix.c b/hw/ide/piix.c index f5a74c293a..66527610a8 100644 --- a/hw/ide/piix.c +++ b/hw/ide/piix.c @@ -73,7 +73,8 @@ static void bmdma_write(void *opaque, target_phys_addr_t addr, #endif switch(addr & 3) { case 0: - return bmdma_cmd_writeb(bm, val); + bmdma_cmd_writeb(bm, val); + break; case 2: bm->status = (val & 0x60) | (bm->status & 1) | (bm->status & ~val & 0x06); break; |