[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] isa: configure serial+parallel by index.
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] isa: configure serial+parallel by index. |
Date: |
Fri, 20 Nov 2009 16:01:09 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Gerd Hoffmann <address@hidden> writes:
> This patch adds a 'index' property to the isa-parallel and isa-serial
> devices. This can be used to create devices with the default isa irqs
> and ioports by simply specifying the index, i.e.
>
> -device isa-serial,index=1
>
> instead of
>
> -device isa-serial,iobase=0x2f8,irq=3
>
> for ttyS1 aka com2. Likewise for parallel ports.
>
> Signed-off-by: Gerd Hoffmann <address@hidden>
> ---
> hw/parallel.c | 2 +-
> hw/serial.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/parallel.c b/hw/parallel.c
> index 5ae8348..12693d4 100644
> --- a/hw/parallel.c
> +++ b/hw/parallel.c
> @@ -587,7 +587,7 @@ static ISADeviceInfo parallel_isa_info = {
> .qdev.size = sizeof(ISAParallelState),
> .init = parallel_isa_initfn,
> .qdev.props = (Property[]) {
> - DEFINE_PROP_HEX32("index", ISAParallelState, index, -1),
> + DEFINE_PROP_UINT32("index", ISAParallelState, index, -1),
> DEFINE_PROP_HEX32("iobase", ISAParallelState, iobase, -1),
> DEFINE_PROP_UINT32("irq", ISAParallelState, isairq, 7),
> DEFINE_PROP_CHR("chardev", ISAParallelState, state.chr),
> diff --git a/hw/serial.c b/hw/serial.c
> index 0063260..e7538ac 100644
> --- a/hw/serial.c
> +++ b/hw/serial.c
> @@ -900,7 +900,7 @@ static ISADeviceInfo serial_isa_info = {
> .qdev.size = sizeof(ISASerialState),
> .init = serial_isa_initfn,
> .qdev.props = (Property[]) {
> - DEFINE_PROP_HEX32("index", ISASerialState, index, -1),
> + DEFINE_PROP_UINT32("index", ISASerialState, index, -1),
> DEFINE_PROP_HEX32("iobase", ISASerialState, iobase, -1),
> DEFINE_PROP_UINT32("irq", ISASerialState, isairq, -1),
> DEFINE_PROP_CHR("chardev", ISASerialState, state.chr),
The commit message is bogus. The patch is a minor fix of commit
e8ee28fb, and the message is a copy of that commit's message.