Re: [PATCH 1/4] hw/timer/i8254_common: Share "iobase" property via base class

2023-05-15 Thread Philippe Mathieu-Daudé
On 13/5/23 12:09, Bernhard Beschow wrote: Both TYPE_KVM_I8254 and TYPE_I8254 have their own but same implementation of the "iobase" property. The storage for the property already resides in PITCommonState, so also move the property definition there. Signed-off-by: Bernhard Beschow --- hw/i386

Re: [PATCH 1/4] hw/timer/i8254_common: Share "iobase" property via base class

2023-05-14 Thread Mark Cave-Ayland
On 13/05/2023 11:09, Bernhard Beschow wrote: Both TYPE_KVM_I8254 and TYPE_I8254 have their own but same implementation of the "iobase" property. The storage for the property already resides in PITCommonState, so also move the property definition there. Signed-off-by: Bernhard Beschow --- hw/

[PATCH 1/4] hw/timer/i8254_common: Share "iobase" property via base class

2023-05-13 Thread Bernhard Beschow
Both TYPE_KVM_I8254 and TYPE_I8254 have their own but same implementation of the "iobase" property. The storage for the property already resides in PITCommonState, so also move the property definition there. Signed-off-by: Bernhard Beschow --- hw/i386/kvm/i8254.c | 1 - hw/timer/i8254.c