qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] pc: acpi: get rid of IASL warnings in MCRS method
@ 2015-02-19 13:49 Igor Mammedov
  0 siblings, 0 replies; only message in thread
From: Igor Mammedov @ 2015-02-19 13:49 UTC (permalink / raw)
  To: qemu-devel; +Cc: mst

Newer IASL produces a bunch of false warnings like:
        ,, MW32, AddressRangeMemory, TypeStatic)
            Object is not referenced ^  (Name is within method [MCRS])

Caused by local integer constant name DescriptorName
in DWordMemory resource macro, it appears that compiler creates
a name for each of the fields of DWordMemory resource macro
if optional DescriptorName is provided and the complains
about not used ones.

So silence compiler by getting rid of DescriptorName in
MW[64|23] resource macros and use numeric field offsets
for referencing fields in MW32 buffer as it's already done
for MW64.

Signed-off-by: Igor Mammedov <imammedo@redhat.com>
---
 hw/i386/acpi-dsdt-mem-hotplug.dsl | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/hw/i386/acpi-dsdt-mem-hotplug.dsl b/hw/i386/acpi-dsdt-mem-hotplug.dsl
index 2a36c47..4d693e3 100644
--- a/hw/i386/acpi-dsdt-mem-hotplug.dsl
+++ b/hw/i386/acpi-dsdt-mem-hotplug.dsl
@@ -105,7 +105,7 @@
                     0xFFFFFFFFFFFFFFFE,        // Address Range Maximum
                     0x0000000000000000,        // Address Translation Offset
                     0xFFFFFFFFFFFFFFFF,        // Address Length
-                    ,, MW64, AddressRangeMemory, TypeStatic)
+                    ,,, AddressRangeMemory, TypeStatic)
                 })
 
                 CreateDWordField(MR64, 14, MINL)
@@ -140,11 +140,11 @@
                         0xFFFFFFFE,        // Address Range Maximum
                         0x00000000,        // Address Translation Offset
                         0xFFFFFFFF,        // Address Length
-                        ,, MW32, AddressRangeMemory, TypeStatic)
+                        ,,, AddressRangeMemory, TypeStatic)
                     })
-                    CreateDWordField(MR32, MW32._MIN, MIN)
-                    CreateDWordField(MR32, MW32._MAX, MAX)
-                    CreateDWordField(MR32, MW32._LEN, LEN)
+                    CreateDWordField(MR32, 10, MIN)
+                    CreateDWordField(MR32, 14, MAX)
+                    CreateDWordField(MR32, 22, LEN)
                     Store(MINL, MIN)
                     Store(MAXL, MAX)
                     Store(LENL, LEN)
-- 
1.8.3.1

^ permalink raw reply related	[flat|nested] only message in thread

only message in thread, other threads:[~2015-02-19 13:49 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-02-19 13:49 [Qemu-devel] [PATCH] pc: acpi: get rid of IASL warnings in MCRS method Igor Mammedov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).