[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v9 02/10] acpi: move aml builder code for floppy device
From: |
Thomas Huth |
Subject: |
Re: [PATCH v9 02/10] acpi: move aml builder code for floppy device |
Date: |
Wed, 17 Jun 2020 13:34:06 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 |
On 17/06/2020 13.31, Gerd Hoffmann wrote:
> On Wed, Jun 17, 2020 at 10:40:24AM +0200, Thomas Huth wrote:
>> On 17/06/2020 09.11, Gerd Hoffmann wrote:
>>> DSDT change: isa device order changes in case MI1 (ipmi) is present.
>>>
>>> Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
>>> Reviewed-by: Igor Mammedov <imammedo@redhat.com>
>>> ---
>>> hw/block/fdc.c | 83 ++++++++++++++++++++++++++++++++
>>> hw/i386/acpi-build.c | 83 --------------------------------
>>> stubs/cmos.c | 7 +++
>>> stubs/Makefile.objs | 1 +
>>> tests/data/acpi/pc/DSDT.ipmikcs | Bin 5086 -> 5086 bytes
>>> 5 files changed, 91 insertions(+), 83 deletions(-)
>>> create mode 100644 stubs/cmos.c
>>
>> Hmm, not all targets that use CONFIG_FDC also have ACPI ... would it be
>> possible to move the ACPI-related code into a new file, say fdc-acpi.c,
>> instead and only compile that if both, CONFIG_FDC and CONFIG_ACPI are
>> enabled?
>
> Possible sure, but is that really worth it? It isn't that much
> (possibly dead) code, and we have stubs for the aml_*() functions so it
> builds just fine with CONFIG_ACPI=n ...
Ok, fine for me, too. It's really not that much code, indeed.
Thomas
- [PATCH v9 00/10] acpi: i386 tweaks, Gerd Hoffmann, 2020/06/17
- [PATCH v9 03/10] floppy: make isa_fdc_get_drive_max_chs static, Gerd Hoffmann, 2020/06/17
- [PATCH v9 10/10] acpi: q35: drop _SB.PCI0.ISA.LPCD opregion., Gerd Hoffmann, 2020/06/17
- [PATCH v9 06/10] acpi: factor out fw_cfg_add_acpi_dsdt(), Gerd Hoffmann, 2020/06/17
- [PATCH v9 07/10] acpi: simplify build_isa_devices_aml(), Gerd Hoffmann, 2020/06/17
- [PATCH v9 05/10] acpi: move aml builder code for i8042 (kbd+mouse) device, Gerd Hoffmann, 2020/06/17
- [PATCH v9 09/10] acpi: drop build_piix4_pm(), Gerd Hoffmann, 2020/06/17
- [PATCH v9 08/10] acpi: drop serial/parallel enable bits from dsdt, Gerd Hoffmann, 2020/06/17