[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 14/15] piix4: rename PIIX4 object to piix4-is
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH v2 14/15] piix4: rename PIIX4 object to piix4-isa |
Date: |
Sat, 16 Dec 2017 22:14:07 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 |
Hi Hervé,
On 12/16/2017 06:30 PM, Hervé Poussineau wrote:
> Other piix4 parts are already named piix4-ide and piix4-usb-uhci.
>
> Signed-off-by: Hervé Poussineau <address@hidden>
> ---
> hw/isa/piix4.c | 2 +-
> hw/mips/mips_malta.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/isa/piix4.c b/hw/isa/piix4.c
> index dc7146ee14..8bf260594f 100644
> --- a/hw/isa/piix4.c
> +++ b/hw/isa/piix4.c
> @@ -52,7 +52,7 @@ typedef struct PIIX4State {
> uint8_t rcr;
> } PIIX4State;
>
> -#define TYPE_PIIX4_PCI_DEVICE "PIIX4"
> +#define TYPE_PIIX4_PCI_DEVICE "piix4-isa"
> #define PIIX4_PCI_DEVICE(obj) \
> OBJECT_CHECK(PIIX4State, (obj), TYPE_PIIX4_PCI_DEVICE)
>
> diff --git a/hw/mips/mips_malta.c b/hw/mips/mips_malta.c
> index 3d304a6e0a..28d91c9668 100644
> --- a/hw/mips/mips_malta.c
> +++ b/hw/mips/mips_malta.c
> @@ -1178,7 +1178,7 @@ void mips_malta_init(MachineState *machine)
> ide_drive_get(hd, ARRAY_SIZE(hd));
>
> pci = pci_create_multifunction(pci_bus, PCI_DEVFN(10, 0),
> - true, "PIIX4");
> + true, "piix4-isa");
Can you use TYPE_PIIX4_PCI_DEVICE instead?
Then:
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> dev = DEVICE(pci);
>
> /* Floppy */
>
- [Qemu-devel] [PATCH v2 04/15] mc146818rtc: always register rtc to rtc list, (continued)
- [Qemu-devel] [PATCH v2 04/15] mc146818rtc: always register rtc to rtc list, Hervé Poussineau, 2017/12/16
- [Qemu-devel] [PATCH v2 06/15] piix4: add Reset Control Register, Hervé Poussineau, 2017/12/16
- [Qemu-devel] [PATCH v2 07/15] piix4: add a i8259 interrupt controller as specified in datasheet, Hervé Poussineau, 2017/12/16
- [Qemu-devel] [PATCH v2 08/15] piix4: add a i8257 dma controller as specified in datasheet, Hervé Poussineau, 2017/12/16
- [Qemu-devel] [PATCH v2 09/15] piix4: add a i8254 pit controller as specified in datasheet, Hervé Poussineau, 2017/12/16
- [Qemu-devel] [PATCH v2 10/15] piix4: add a i8042 keyboard/mouse controller as specified in datasheet, Hervé Poussineau, 2017/12/16
- [Qemu-devel] [PATCH v2 11/15] piix4: add a floppy controller, 1 parallel port and 2 serial ports, Hervé Poussineau, 2017/12/16
- [Qemu-devel] [PATCH v2 12/15] piix4: add a mc146818rtc controller as specified in datasheet, Hervé Poussineau, 2017/12/16
- [Qemu-devel] [PATCH v2 13/15] piix4: add a speaker as specified in datasheet, Hervé Poussineau, 2017/12/16
- [Qemu-devel] [PATCH v2 14/15] piix4: rename PIIX4 object to piix4-isa, Hervé Poussineau, 2017/12/16
- Re: [Qemu-devel] [PATCH v2 14/15] piix4: rename PIIX4 object to piix4-isa,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH v2 15/15] piix4: we can now instanciate a PIIX4 with -device, Hervé Poussineau, 2017/12/16