[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] acpi: Remove duplicated code handling OEM ID and OEM table I
From: |
Marian Postevca |
Subject: |
Re: [PATCH] acpi: Remove duplicated code handling OEM ID and OEM table ID fields |
Date: |
Mon, 15 Feb 2021 20:26:10 +0200 |
Igor Mammedov <imammedo@redhat.com> writes:
> hmm, looks like adding instead of removing
>
Do you mean that the commit message does not describe the change
correctly, or that my refactoring is too extreme?
If it is the latter, I think I tried to simplify things, by creating
macros to be used in multiple places where this structure is created.
And passing the structure around instead of two parameters seemed simpler.
> have you considered, putting this field into X86MachineState?
> (that way you will be able to handle both PC and microvm in one place,
> without duplication io init/property setters)
>
I did not, will try this approach.