From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:47479) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZbVCV-0007oD-Vc for qemu-devel@nongnu.org; Mon, 14 Sep 2015 10:57:49 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZbVCR-0007mI-SJ for qemu-devel@nongnu.org; Mon, 14 Sep 2015 10:57:47 -0400 Received: from relay-05.andrew.cmu.edu ([128.2.157.12]:37836 helo=relay.andrew.cmu.edu) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZbVCR-0007lJ-Oh for qemu-devel@nongnu.org; Mon, 14 Sep 2015 10:57:43 -0400 From: "Gabriel L. Somlo" Date: Mon, 14 Sep 2015 10:57:28 -0400 Message-Id: <1442242653-23284-1-git-send-email-somlo@cmu.edu> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: [Qemu-devel] [PATCH v2 0/5] add ACPI node for fw_cfg on pc and arm List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: peter.maydell@linaro.org, drjones@redhat.com, matt.fleming@intel.com, ehabkost@redhat.com, mst@redhat.com, zhaoshenglong@huawei.com, leif.lindholm@linaro.org, ard.biesheuvel@linaro.org, kevin@koconnor.net, kraxel@redhat.com, pbonzini@redhat.com, imammedo@redhat.com, markmb@redhat.com, lersek@redhat.com, rth@twiddle.net New since v1: - expose control register size (suggested by Marc Mar=C3=AD) - leaving out _UID and _STA fields (thanks Shannon & Igor) - using "QEMU0002" as the value of _HID (thanks Michael) - added documentation blurb to docs/specs/fw_cfg.txt (mainly to record usage of the "QEMU0002" string with fw_cfg). Thanks, --Gabriel > This series adds a fw_cfg device node to the SSDT (on pc), or to the > DSDT (on arm). > > - Patch 1/3 moves (and renames) the BIOS_CFG_IOPORT (0x510) > define from pc.c to pc.h, so that it could be used from > acpi-build.c in patch 2/3. >=20 > - Patch 2/3 adds a fw_cfg node to the pc SSDT. >=20 > - Patch 3/3 adds a fw_cfg node to the arm DSDT. > > I made up some names - "FWCF" for the node name, and "FWCF0001" > for _HID; no idea whether that's appropriate, or how else I should > figure out what to use instead... > > Also, using scope "\\_SB", based on where fw_cfg shows up in the > output of "info qtree". Again, if that's wrong, please point me in > the right direction. > > Re. 3/3 (also mentioned after the commit blurb in the patch itself), > I noticed none of the other DSDT entries contain a _STA field, wonderin= g > why it would (not) make sense to include that, same as on the PC. Gabriel L. Somlo (5): fw_cfg: expose control register size in fw_cfg.h pc: fw_cfg: move ioport base constant to pc.h acpi: pc: add fw_cfg device node to ssdt acpi: arm: add fw_cfg device node to dsdt fw_cfg: document ACPI device node information docs/specs/fw_cfg.txt | 9 +++++++++ hw/arm/virt-acpi-build.c | 13 +++++++++++++ hw/i386/acpi-build.c | 20 ++++++++++++++++++++ hw/i386/pc.c | 5 ++--- hw/nvram/fw_cfg.c | 8 +++++--- include/hw/i386/pc.h | 2 ++ include/hw/nvram/fw_cfg.h | 3 +++ 7 files changed, 54 insertions(+), 6 deletions(-) --=20 2.4.3