[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/10] fdc: add CCR (Configuration Control Regis
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH 08/10] fdc: add CCR (Configuration Control Register) write register |
Date: |
Wed, 11 Jan 2012 21:30:12 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:9.0) Gecko/20111220 Thunderbird/9.0 |
Am 08.01.2012 21:27, schrieb Hervé Poussineau:
>
> Signed-off-by: Hervé Poussineau <address@hidden>
> ---
> hw/fdc.c | 21 +++++++++++++++++++++
> 1 files changed, 21 insertions(+), 0 deletions(-)
>
> diff --git a/hw/fdc.c b/hw/fdc.c
> index ddfa91f..67cd14f 100644
> --- a/hw/fdc.c
> +++ b/hw/fdc.c
> @@ -227,6 +227,7 @@ static void fdctrl_write_rate(FDCtrl *fdctrl, uint32_t
> value);
> static uint32_t fdctrl_read_data(FDCtrl *fdctrl);
> static void fdctrl_write_data(FDCtrl *fdctrl, uint32_t value);
> static uint32_t fdctrl_read_dir(FDCtrl *fdctrl);
> +static void fdctrl_write_ccr(FDCtrl *fdctrl, uint32_t value);
>
> enum {
> FD_DIR_WRITE = 0,
> @@ -251,6 +252,7 @@ enum {
> FD_REG_DSR = 0x04,
> FD_REG_FIFO = 0x05,
> FD_REG_DIR = 0x07,
> + FD_REG_CCR = 0x07,
I'm not familiar with FDCs, so this reads weird: Do DIR and CCR share
the same address, with DIR being read-only and CCR write-only? If so, an
explanatory sentence in the commit message would be nice. (dito for most
other patches in the series)
> };
>
> enum {
> @@ -495,6 +497,8 @@ static void fdctrl_write (void *opaque, uint32_t reg,
> uint32_t value)
> case FD_REG_FIFO:
> fdctrl_write_data(fdctrl, value);
> break;
> + case FD_REG_CCR:
> + fdctrl_write_ccr(fdctrl, value);
break; please, to avoid future errors or complaints by static analysis
tools.
Andreas
> default:
> break;
> }
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- [Qemu-devel] [PATCH 02/10] fdc: set busy bit when starting a command, (continued)
- [Qemu-devel] [PATCH 02/10] fdc: set busy bit when starting a command, Hervé Poussineau, 2012/01/08
- [Qemu-devel] [PATCH 03/10] fdc: most control commands do not generate interrupts, Hervé Poussineau, 2012/01/08
- [Qemu-devel] [PATCH 04/10] fdc: emulate stepping 0, Hervé Poussineau, 2012/01/08
- [Qemu-devel] [PATCH 01/10] fdc: take head count into account, Hervé Poussineau, 2012/01/08
- [Qemu-devel] [PATCH 05/10] fdc: handle read-only floppies (abort early on write commands), Hervé Poussineau, 2012/01/08
- [Qemu-devel] [PATCH 06/10] block: add a transfer rate for floppy types, Hervé Poussineau, 2012/01/08
- [Qemu-devel] [PATCH 07/10] fdc: check if media rate is correct before doing any transfer, Hervé Poussineau, 2012/01/08
- [Qemu-devel] [PATCH 08/10] fdc: add CCR (Configuration Control Register) write register, Hervé Poussineau, 2012/01/08
- [Qemu-devel] [PATCH 10/10] fdc: DIR (Digital Input Register) should return status of current drive..., Hervé Poussineau, 2012/01/08
- [Qemu-devel] [PATCH 09/10] fdc: fix seek command, which shouldn't check tracks, Hervé Poussineau, 2012/01/08