From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44248) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yfv6Y-0007HN-5R for qemu-devel@nongnu.org; Wed, 08 Apr 2015 14:53:39 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yfv6T-0001L7-4i for qemu-devel@nongnu.org; Wed, 08 Apr 2015 14:53:38 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54446) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yfv6S-0001Kw-RZ for qemu-devel@nongnu.org; Wed, 08 Apr 2015 14:53:33 -0400 Date: Wed, 8 Apr 2015 20:53:26 +0200 From: "Michael S. Tsirkin" Message-ID: <20150408205117-mutt-send-email-mst@redhat.com> References: <1428055432-12120-1-git-send-email-zhaoshenglong@huawei.com> <1428055432-12120-8-git-send-email-zhaoshenglong@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1428055432-12120-8-git-send-email-zhaoshenglong@huawei.com> Subject: Re: [Qemu-devel] [PATCH v4 07/20] hw/arm/virt-acpi-build: Generate FADT table and update ACPI headers List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Shannon Zhao Cc: peter.maydell@linaro.org, hangaohuai@huawei.com, a.spyridakis@virtualopensystems.com, msalter@redhat.com, claudio.fontana@huawei.com, qemu-devel@nongnu.org, peter.huangpeng@huawei.com, hanjun.guo@linaro.org, imammedo@redhat.com, pbonzini@redhat.com, lersek@redhat.com, christoffer.dall@linaro.org, shannon.zhao@linaro.org On Fri, Apr 03, 2015 at 06:03:39PM +0800, Shannon Zhao wrote: > @@ -135,6 +138,43 @@ struct AcpiFadtDescriptorRev1 > } QEMU_PACKED; > typedef struct AcpiFadtDescriptorRev1 AcpiFadtDescriptorRev1; > > +struct acpi_generic_address { > + uint8_t space_id; /* Address space where struct or register exists */ > + uint8_t bit_width; /* Size in bits of given register */ > + uint8_t bit_offset; /* Bit offset within the register */ > + uint8_t access_width; /* Minimum Access size (ACPI 3.0) */ > + uint64_t address; /* 64-bit address of struct or register */ > +} QEMU_PACKED; Pls use standard QEMU style for structs. There are more like this in the patchset, pls find and fix them. > + > +struct AcpiFadtDescriptorRev5_1 { > + ACPI_FADT_COMMON_DEF > + uint16_t boot_flags; /* IA-PC Boot Architecture Flags (see below for individual flags) */ > + uint8_t reserved; /* Reserved, must be zero */ > + uint32_t flags; /* Miscellaneous flag bits (see below for individual flags) */ > + struct acpi_generic_address reset_register; /* 64-bit address of the Reset register */ > + uint8_t reset_value; /* Value to write to the reset_register port to reset the system */ > + uint16_t arm_boot_flags; /* ARM-Specific Boot Flags (see below for individual flags) (ACPI 5.1) */ > + uint8_t minor_revision; /* FADT Minor Revision (ACPI 5.1) */ > + uint64_t Xfacs; /* 64-bit physical address of FACS */ > + uint64_t Xdsdt; /* 64-bit physical address of DSDT */ > + struct acpi_generic_address xpm1a_event_block; /* 64-bit Extended Power Mgt 1a Event Reg Blk address */ > + struct acpi_generic_address xpm1b_event_block; /* 64-bit Extended Power Mgt 1b Event Reg Blk address */ > + struct acpi_generic_address xpm1a_control_block; /* 64-bit Extended Power Mgt 1a Control Reg Blk address */ > + struct acpi_generic_address xpm1b_control_block; /* 64-bit Extended Power Mgt 1b Control Reg Blk address */ > + struct acpi_generic_address xpm2_control_block; /* 64-bit Extended Power Mgt 2 Control Reg Blk address */ > + struct acpi_generic_address xpm_timer_block; /* 64-bit Extended Power Mgt Timer Ctrl Reg Blk address */ > + struct acpi_generic_address xgpe0_block; /* 64-bit Extended General Purpose Event 0 Reg Blk address */ > + struct acpi_generic_address xgpe1_block; /* 64-bit Extended General Purpose Event 1 Reg Blk address */ > + struct acpi_generic_address sleep_control; /* 64-bit Sleep Control register (ACPI 5.0) */ > + struct acpi_generic_address sleep_status; /* 64-bit Sleep Status register (ACPI 5.0) */ > +} QEMU_PACKED; empty line missing. > +typedef struct AcpiFadtDescriptorRev5_1 AcpiFadtDescriptorRev5_1; > + > +enum { > + ACPI_FADT_ARM_USE_PSCI_G_0_2, > + ACPI_FADT_ARM_PSCI_USE_HVC, > +}; These are part of tables, are they not? Pls add = 0, = 1, so we don't change them by mistake. > + > /* > * ACPI 1.0 Root System Description Table (RSDT) > */ > -- > 2.0.4 >