[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 15/63] i8259: Rename TYPE_I8259 to TYPE_PIC
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 15/63] i8259: Rename TYPE_I8259 to TYPE_PIC |
Date: |
Thu, 3 Sep 2020 14:47:33 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 9/3/20 12:42 AM, Eduardo Habkost wrote:
> This will make the type name constant consistent with the name of
> the type checking macro.
>
> Signed-off-by: Eduardo Habkost <ehabkost@redhat.com>
> ---
> Cc: "Michael S. Tsirkin" <mst@redhat.com>
> Cc: Paolo Bonzini <pbonzini@redhat.com>
> Cc: qemu-devel@nongnu.org
> ---
> hw/intc/i8259.c | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/hw/intc/i8259.c b/hw/intc/i8259.c
> index 344fd04db1..e536cc90be 100644
> --- a/hw/intc/i8259.c
> +++ b/hw/intc/i8259.c
> @@ -37,10 +37,10 @@
>
> //#define DEBUG_IRQ_LATENCY
>
> -#define TYPE_I8259 "isa-i8259"
> +#define TYPE_PIC "isa-i8259"
I disagree with this patch, as we have various PIC and only one I8259.
> typedef struct PICClass PICClass;
> DECLARE_CLASS_CHECKERS(PICClass, PIC,
> - TYPE_I8259)
> + TYPE_PIC)
>
> /**
> * PICClass:
> @@ -418,7 +418,7 @@ qemu_irq *i8259_init(ISABus *bus, qemu_irq parent_irq)
>
> irq_set = g_new0(qemu_irq, ISA_NUM_IRQS);
>
> - isadev = i8259_init_chip(TYPE_I8259, bus, true);
> + isadev = i8259_init_chip(TYPE_PIC, bus, true);
> dev = DEVICE(isadev);
>
> qdev_connect_gpio_out(dev, 0, parent_irq);
> @@ -428,7 +428,7 @@ qemu_irq *i8259_init(ISABus *bus, qemu_irq parent_irq)
>
> isa_pic = dev;
>
> - isadev = i8259_init_chip(TYPE_I8259, bus, false);
> + isadev = i8259_init_chip(TYPE_PIC, bus, false);
> dev = DEVICE(isadev);
>
> qdev_connect_gpio_out(dev, 0, irq_set[2]);
> @@ -451,7 +451,7 @@ static void i8259_class_init(ObjectClass *klass, void
> *data)
> }
>
> static const TypeInfo i8259_info = {
> - .name = TYPE_I8259,
> + .name = TYPE_PIC,
> .instance_size = sizeof(PICCommonState),
> .parent = TYPE_PIC_COMMON,
> .class_init = i8259_class_init,
>
- [PATCH 12/63] etrax: Rename TYPE_ETRAX_FS_SERIAL to TYPE_ETRAX_SERIAL, (continued)
- [PATCH 12/63] etrax: Rename TYPE_ETRAX_FS_SERIAL to TYPE_ETRAX_SERIAL, Eduardo Habkost, 2020/09/02
- [PATCH 13/63] etrax: Rename TYPE_ETRAX_FS_TIMER to TYPE_ETRAX_TIMER, Eduardo Habkost, 2020/09/02
- [PATCH 14/63] i8254: Rename TYPE_I8254 to TYPE_PIT, Eduardo Habkost, 2020/09/02
- Re: [PATCH 14/63] i8254: Rename TYPE_I8254 to TYPE_PIT, Philippe Mathieu-Daudé, 2020/09/03
- Re: [PATCH 14/63] i8254: Rename TYPE_I8254 to TYPE_PIT, Eduardo Habkost, 2020/09/03
- Re: [PATCH 14/63] i8254: Rename TYPE_I8254 to TYPE_PIT, Daniel P . Berrangé, 2020/09/03
- Re: [PATCH 14/63] i8254: Rename TYPE_I8254 to TYPE_PIT, Eduardo Habkost, 2020/09/03
- Re: [PATCH 14/63] i8254: Rename TYPE_I8254 to TYPE_PIT, Philippe Mathieu-Daudé, 2020/09/03
- Re: [PATCH 14/63] i8254: Rename TYPE_I8254 to TYPE_PIT, Eduardo Habkost, 2020/09/03
[PATCH 15/63] i8259: Rename TYPE_I8259 to TYPE_PIC, Eduardo Habkost, 2020/09/02
- Re: [PATCH 15/63] i8259: Rename TYPE_I8259 to TYPE_PIC,
Philippe Mathieu-Daudé <=
[PATCH 16/63] i8259: Rename TYPE_KVM_I8259 to TYPE_KVM_PIC, Eduardo Habkost, 2020/09/02
[PATCH 17/63] intel-hda: Rename TYPE_INTEL_HDA_GENERIC to TYPE_INTEL_HDA, Eduardo Habkost, 2020/09/02
[PATCH 20/63] pc87312: Rename TYPE_PC87312_SUPERIO to TYPE_PC87312, Eduardo Habkost, 2020/09/02