[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 3/7] hw/acpi/Kconfig: Do not needlessly build TYPE_PIIX4_P
From: |
Bernhard Beschow |
Subject: |
Re: [PATCH v2 3/7] hw/acpi/Kconfig: Do not needlessly build TYPE_PIIX4_PM in non-PC/Malta machines |
Date: |
Fri, 16 Dec 2022 15:10:00 +0000 |
Am 16. Dezember 2022 14:44:33 UTC schrieb BALATON Zoltan <balaton@eik.bme.hu>:
>On Fri, 16 Dec 2022, Bernhard Beschow wrote:
>> TYPE_PIIX4_PM is only used in machines where PIIX chipsets are used
>> which is currently PC and Malta. There is no point building it for the
>> other ACPI_X86 machines.
>>
>> Note that this also removes unneeded ACPI_PIIX4 from PEGASOS2.
>>
>> Signed-off-by: Bernhard Beschow <shentey@gmail.com>
>
>Reviewed-by: BALATON Zoltan <balaton@eik.bme.hu>
>
>Isn't 440fx using PIIX3 whereas Malta uses PIIX4?
Correct.
> If so there may be still some inconsistency in the naming of this config
> option (should it be ACPI_PIIX maybe?)
I440fx does indeed use the PIIX4 PM controller, often referred to as a
"Frankenstein" device here on the list. That's why I'm currently consolidating
the PIIX south bridges -- to make PIIX4 bridge usable in i440fx machine.
Best regards,
Bernhard
> but it should not be needed for pegasos2 in any case.
>
>Regards,
>BALATON Zoltan
>
>> ---
>> hw/acpi/Kconfig | 1 -
>> hw/i386/Kconfig | 1 +
>> 2 files changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/hw/acpi/Kconfig b/hw/acpi/Kconfig
>> index 704cbc6236..b7006e18c8 100644
>> --- a/hw/acpi/Kconfig
>> +++ b/hw/acpi/Kconfig
>> @@ -9,7 +9,6 @@ config ACPI_X86
>> select ACPI_CPU_HOTPLUG
>> select ACPI_MEMORY_HOTPLUG
>> select ACPI_HMAT
>> - select ACPI_PIIX4
>> select ACPI_PCIHP
>> select ACPI_ERST
>>
>> diff --git a/hw/i386/Kconfig b/hw/i386/Kconfig
>> index d22ac4a4b9..3a92566701 100644
>> --- a/hw/i386/Kconfig
>> +++ b/hw/i386/Kconfig
>> @@ -69,6 +69,7 @@ config I440FX
>> imply E1000_PCI
>> imply VMPORT
>> imply VMMOUSE
>> + select ACPI_PIIX4
>> select PC_PCI
>> select PC_ACPI
>> select ACPI_SMBUS
>>
- Re: [PATCH v2 5/7] hw/isa/Kconfig: Add missing dependency to VT82C686, (continued)
- [PATCH v2 6/7] i386, mips: Resolve redundant ACPI and APM dependencies, Bernhard Beschow, 2022/12/16
- [PATCH v2 4/7] hw/acpi/Kconfig: Add missing dependencies to ACPI_PIIX4, Bernhard Beschow, 2022/12/16
- [PATCH v2 1/7] hw/acpi/Kconfig: Rename ACPI_X86_ICH to ACPI_ICH9, Bernhard Beschow, 2022/12/16
- [PATCH v2 3/7] hw/acpi/Kconfig: Do not needlessly build TYPE_PIIX4_PM in non-PC/Malta machines, Bernhard Beschow, 2022/12/16
- Re: [PATCH v2 0/7] Clean up dependencies of ACPI controllers, Bernhard Beschow, 2022/12/17
- Re: [PATCH v2 0/7] Clean up dependencies of ACPI controllers, Bernhard Beschow, 2022/12/21