1. Patchew
  2. QEMU
  3. View series

[PATCH 0/7] hw/i386: Cleanups around 'hw/i386/pc.h'

Philippe Mathieu-Daudé posted 7 patches 9 months, 2 weeks ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20240213120153.90930-1-philmd@linaro.org
Maintainers: "Michael S. Tsirkin" <mst@redhat.com>, Igor Mammedov <imammedo@redhat.com>, Ani Sinha <anisinha@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Richard Henderson <richard.henderson@linaro.org>, Eduardo Habkost <eduardo@habkost.net>, Marcel Apfelbaum <marcel.apfelbaum@gmail.com>
hw/i386/acpi-common.h   | 3 +++
include/hw/i386/pc.h    | 6 ------
include/hw/timer/hpet.h | 2 ++
hw/acpi/cpu_hotplug.c   | 4 ++--
hw/i386/acpi-common.c   | 1 -
hw/i386/port92.c        | 1 +
hw/isa/lpc_ich9.c       | 1 -
hw/timer/hpet.c         | 1 -
target/i386/monitor.c   | 1 -
9 files changed, 8 insertions(+), 12 deletions(-)
[PATCH 0/7] hw/i386: Cleanups around 'hw/i386/pc.h'
Posted by Philippe Mathieu-Daudé 9 months, 2 weeks ago 
Trivial patches removing uses of "hw/i386/pc.h".

Philippe Mathieu-Daudé (7):
  target/i386/monitor: Remove unused 'hw/i386/pc.h' header
  hw/timer: Move HPET_INTCAP definition to "hpet.h"
  hw/isa/lpc_ich9: Remove unused 'hw/i386/pc.h'
  hw/i386/acpi: Declare pc_madt_cpu_entry() in 'acpi-common.h'
  hw/i386/port92: Add missing 'hw/isa/isa.h' header
  hw/acpi/cpu_hotplug: Include 'pci_device.h' instead of 'pci.h'
  hw/acpi/cpu_hotplug: Include 'x86.h' instead of 'pc.h'

 hw/i386/acpi-common.h   | 3 +++
 include/hw/i386/pc.h    | 6 ------
 include/hw/timer/hpet.h | 2 ++
 hw/acpi/cpu_hotplug.c   | 4 ++--
 hw/i386/acpi-common.c   | 1 -
 hw/i386/port92.c        | 1 +
 hw/isa/lpc_ich9.c       | 1 -
 hw/timer/hpet.c         | 1 -
 target/i386/monitor.c   | 1 -
 9 files changed, 8 insertions(+), 12 deletions(-)

-- 
2.41.0
Re: [PATCH 0/7] hw/i386: Cleanups around 'hw/i386/pc.h'
Posted by Michael Tokarev 9 months, 2 weeks ago 
13.02.2024 15:01, Philippe Mathieu-Daudé wrote:
> Trivial patches removing uses of "hw/i386/pc.h".
> 
> Philippe Mathieu-Daudé (7):
>    target/i386/monitor: Remove unused 'hw/i386/pc.h' header
>    hw/timer: Move HPET_INTCAP definition to "hpet.h"
>    hw/isa/lpc_ich9: Remove unused 'hw/i386/pc.h'
>    hw/i386/acpi: Declare pc_madt_cpu_entry() in 'acpi-common.h'
>    hw/i386/port92: Add missing 'hw/isa/isa.h' header
>    hw/acpi/cpu_hotplug: Include 'pci_device.h' instead of 'pci.h'
>    hw/acpi/cpu_hotplug: Include 'x86.h' instead of 'pc.h'
> 
>   hw/i386/acpi-common.h   | 3 +++
>   include/hw/i386/pc.h    | 6 ------
>   include/hw/timer/hpet.h | 2 ++
>   hw/acpi/cpu_hotplug.c   | 4 ++--
>   hw/i386/acpi-common.c   | 1 -
>   hw/i386/port92.c        | 1 +
>   hw/isa/lpc_ich9.c       | 1 -
>   hw/timer/hpet.c         | 1 -
>   target/i386/monitor.c   | 1 -
>   9 files changed, 8 insertions(+), 12 deletions(-)

For the series:

Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>

and picked up for trivial-patches.

Thanks,

/mjt
Re: [PATCH 0/7] hw/i386: Cleanups around 'hw/i386/pc.h'
Posted by Michael S. Tsirkin 9 months, 2 weeks ago 
On Tue, Feb 13, 2024 at 01:01:45PM +0100, Philippe Mathieu-Daudé wrote:
> Trivial patches removing uses of "hw/i386/pc.h".

How did you test that it's indeed unused?

> Philippe Mathieu-Daudé (7):
>   target/i386/monitor: Remove unused 'hw/i386/pc.h' header
>   hw/timer: Move HPET_INTCAP definition to "hpet.h"
>   hw/isa/lpc_ich9: Remove unused 'hw/i386/pc.h'
>   hw/i386/acpi: Declare pc_madt_cpu_entry() in 'acpi-common.h'
>   hw/i386/port92: Add missing 'hw/isa/isa.h' header
>   hw/acpi/cpu_hotplug: Include 'pci_device.h' instead of 'pci.h'
>   hw/acpi/cpu_hotplug: Include 'x86.h' instead of 'pc.h'
> 
>  hw/i386/acpi-common.h   | 3 +++
>  include/hw/i386/pc.h    | 6 ------
>  include/hw/timer/hpet.h | 2 ++
>  hw/acpi/cpu_hotplug.c   | 4 ++--
>  hw/i386/acpi-common.c   | 1 -
>  hw/i386/port92.c        | 1 +
>  hw/isa/lpc_ich9.c       | 1 -
>  hw/timer/hpet.c         | 1 -
>  target/i386/monitor.c   | 1 -
>  9 files changed, 8 insertions(+), 12 deletions(-)
> 
> -- 
> 2.41.0
Re: [PATCH 0/7] hw/i386: Cleanups around 'hw/i386/pc.h'
Posted by Philippe Mathieu-Daudé 9 months, 2 weeks ago 
On 13/2/24 16:55, Michael S. Tsirkin wrote:
> On Tue, Feb 13, 2024 at 01:01:45PM +0100, Philippe Mathieu-Daudé wrote:
>> Trivial patches removing uses of "hw/i386/pc.h".
> 
> How did you test that it's indeed unused?

I put and #error in hw/i386/pc.h and built :)
(I'm trying to remove the PC dependency on generic devices.)

The few generic devices left are not as trivial.

>> Philippe Mathieu-Daudé (7):
>>    target/i386/monitor: Remove unused 'hw/i386/pc.h' header
>>    hw/timer: Move HPET_INTCAP definition to "hpet.h"
>>    hw/isa/lpc_ich9: Remove unused 'hw/i386/pc.h'
>>    hw/i386/acpi: Declare pc_madt_cpu_entry() in 'acpi-common.h'
>>    hw/i386/port92: Add missing 'hw/isa/isa.h' header
>>    hw/acpi/cpu_hotplug: Include 'pci_device.h' instead of 'pci.h'
>>    hw/acpi/cpu_hotplug: Include 'x86.h' instead of 'pc.h'
>>
>>   hw/i386/acpi-common.h   | 3 +++
>>   include/hw/i386/pc.h    | 6 ------
>>   include/hw/timer/hpet.h | 2 ++
>>   hw/acpi/cpu_hotplug.c   | 4 ++--
>>   hw/i386/acpi-common.c   | 1 -
>>   hw/i386/port92.c        | 1 +
>>   hw/isa/lpc_ich9.c       | 1 -
>>   hw/timer/hpet.c         | 1 -
>>   target/i386/monitor.c   | 1 -
>>   9 files changed, 8 insertions(+), 12 deletions(-)
>>
>> -- 
>> 2.41.0
>
Re: [PATCH 0/7] hw/i386: Cleanups around 'hw/i386/pc.h'
Posted by Bernhard Beschow 9 months, 2 weeks ago 

Am 13. Februar 2024 16:00:26 UTC schrieb "Philippe Mathieu-Daudé" <philmd@linaro.org>:
>On 13/2/24 16:55, Michael S. Tsirkin wrote:
>> On Tue, Feb 13, 2024 at 01:01:45PM +0100, Philippe Mathieu-Daudé wrote:
>>> Trivial patches removing uses of "hw/i386/pc.h".
>> 
>> How did you test that it's indeed unused?
>
>I put and #error in hw/i386/pc.h and built :)
>(I'm trying to remove the PC dependency on generic devices.)
>
>The few generic devices left are not as trivial.
>
>>> Philippe Mathieu-Daudé (7):
>>>    target/i386/monitor: Remove unused 'hw/i386/pc.h' header
>>>    hw/timer: Move HPET_INTCAP definition to "hpet.h"
>>>    hw/isa/lpc_ich9: Remove unused 'hw/i386/pc.h'
>>>    hw/i386/acpi: Declare pc_madt_cpu_entry() in 'acpi-common.h'
>>>    hw/i386/port92: Add missing 'hw/isa/isa.h' header
>>>    hw/acpi/cpu_hotplug: Include 'pci_device.h' instead of 'pci.h'
>>>    hw/acpi/cpu_hotplug: Include 'x86.h' instead of 'pc.h'
>>> 
>>>   hw/i386/acpi-common.h   | 3 +++
>>>   include/hw/i386/pc.h    | 6 ------
>>>   include/hw/timer/hpet.h | 2 ++
>>>   hw/acpi/cpu_hotplug.c   | 4 ++--
>>>   hw/i386/acpi-common.c   | 1 -
>>>   hw/i386/port92.c        | 1 +

I'd take care of port92 and make it part of all our three south bridge families (which models the real hardware more closely).

Best regards,
Bernhard

>>>   hw/isa/lpc_ich9.c       | 1 -
>>>   hw/timer/hpet.c         | 1 -
>>>   target/i386/monitor.c   | 1 -
>>>   9 files changed, 8 insertions(+), 12 deletions(-)
>>> 
>>> -- 
>>> 2.41.0
>> 
>
Re: [PATCH 0/7] hw/i386: Cleanups around 'hw/i386/pc.h'
Posted by Luc Michel 9 months, 2 weeks ago 
On 13:01 Tue 13 Feb     , Philippe Mathieu-Daudé wrote:
> Trivial patches removing uses of "hw/i386/pc.h".
> 
> Philippe Mathieu-Daudé (7):
>   target/i386/monitor: Remove unused 'hw/i386/pc.h' header
>   hw/timer: Move HPET_INTCAP definition to "hpet.h"
>   hw/isa/lpc_ich9: Remove unused 'hw/i386/pc.h'
>   hw/i386/acpi: Declare pc_madt_cpu_entry() in 'acpi-common.h'
>   hw/i386/port92: Add missing 'hw/isa/isa.h' header
>   hw/acpi/cpu_hotplug: Include 'pci_device.h' instead of 'pci.h'
>   hw/acpi/cpu_hotplug: Include 'x86.h' instead of 'pc.h'
> 
>  hw/i386/acpi-common.h   | 3 +++
>  include/hw/i386/pc.h    | 6 ------
>  include/hw/timer/hpet.h | 2 ++
>  hw/acpi/cpu_hotplug.c   | 4 ++--
>  hw/i386/acpi-common.c   | 1 -
>  hw/i386/port92.c        | 1 +
>  hw/isa/lpc_ich9.c       | 1 -
>  hw/timer/hpet.c         | 1 -
>  target/i386/monitor.c   | 1 -
>  9 files changed, 8 insertions(+), 12 deletions(-)
> 
> --
> 2.41.0
> 
> 

For the series:
Reviewed-by: Luc Michel <luc.michel@amd.com>

Patchew 2.1-devel-b67e4c2

© 2016 - 2024 Red Hat, Inc.