[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH 1/3] hw/i386/pc.c: remove unused
From: |
Wei Yang |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH 1/3] hw/i386/pc.c: remove unused function pc_acpi_init() |
Date: |
Tue, 29 Jan 2019 08:01:57 +0800 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Mon, Jan 28, 2019 at 02:32:30PM +0100, Igor Mammedov wrote:
>On Fri, 25 Jan 2019 14:01:55 +0800
>Wei Yang <address@hidden> wrote:
>
>> Function pc_acpi_init() is now used in no place.
>s/is now used in no place./in not used anymore,/
>
>fixup the same phrase in other patches.
>
>otherwise series looks good, so fix commit messages and resubmit series.
>
>Also when sending multi-patch series, pls use cover letter
>(see help for git format-patch --cover-letter)
Thanks :-)
Will change and re-submit.
>
>
>> Remove the definition and declaration.
>>
>> Signed-off-by: Wei Yang <address@hidden>
>> ---
>> hw/i386/pc.c | 27 ---------------------------
>> include/hw/i386/pc.h | 1 -
>> 2 files changed, 28 deletions(-)
>>
>> diff --git a/hw/i386/pc.c b/hw/i386/pc.c
>> index 5317e08f60..734d3268fa 100644
>> --- a/hw/i386/pc.c
>> +++ b/hw/i386/pc.c
>> @@ -1280,33 +1280,6 @@ void pc_pci_as_mapping_init(Object *owner,
>> MemoryRegion *system_memory,
>> pci_address_space, -1);
>> }
>>
>> -void pc_acpi_init(const char *default_dsdt)
>> -{
>> - char *filename;
>> -
>> - if (acpi_tables != NULL) {
>> - /* manually set via -acpitable, leave it alone */
>> - return;
>> - }
>> -
>> - filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, default_dsdt);
>> - if (filename == NULL) {
>> - warn_report("failed to find %s", default_dsdt);
>> - } else {
>> - QemuOpts *opts = qemu_opts_create(qemu_find_opts("acpi"), NULL, 0,
>> - &error_abort);
>> - Error *err = NULL;
>> -
>> - qemu_opt_set(opts, "file", filename, &error_abort);
>> -
>> - acpi_table_add_builtin(opts, &err);
>> - if (err) {
>> - warn_reportf_err(err, "failed to load %s: ", filename);
>> - }
>> - g_free(filename);
>> - }
>> -}
>> -
>> void xen_load_linux(PCMachineState *pcms)
>> {
>> int i;
>> diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h
>> index 9d29c4b1df..541124ba6d 100644
>> --- a/include/hw/i386/pc.h
>> +++ b/include/hw/i386/pc.h
>> @@ -187,7 +187,6 @@ void pc_acpi_smi_interrupt(void *opaque, int irq, int
>> level);
>>
>> void pc_cpus_init(PCMachineState *pcms);
>> void pc_hot_add_cpu(const int64_t id, Error **errp);
>> -void pc_acpi_init(const char *default_dsdt);
>>
>> void pc_guest_info_init(PCMachineState *pcms);
>>
--
Wei Yang
Help you, Help me