From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Tue, 3 May 2016 14:33:59 +0200 Subject: [U-Boot] [PATCH 11/26] x86: acpi: Remove unneeded codes In-Reply-To: <1462174426-3470-12-git-send-email-bmeng.cn@gmail.com> References: <1462174426-3470-1-git-send-email-bmeng.cn@gmail.com> <1462174426-3470-12-git-send-email-bmeng.cn@gmail.com> Message-ID: <57289AB7.80307@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Bin, only a minor nitpicking comment about the patch subject. Perhaps its better to say: x86: acpi: Remove unused code On 02.05.2016 09:33, Bin Meng wrote: > - Remove #include <> header files. > - Remove APM_CNT register defines, which should not be here as > they are SMI related. > - Remove MP_IRQ_ defines as they are duplicates of the same ones > in asm/mpspec.h. > - Remove ACTL register defines, which should not be here as they > are chipset specific. > - Remove functional fixed hardware defines, which are not used. > - Remove dev_scope related defines, which are not used. > > Signed-off-by: Bin Meng > --- > > arch/x86/include/asm/acpi_table.h | 68 --------------------------------------- > 1 file changed, 68 deletions(-) > > diff --git a/arch/x86/include/asm/acpi_table.h b/arch/x86/include/asm/acpi_table.h > index 9856fa6..e82752a 100644 > --- a/arch/x86/include/asm/acpi_table.h > +++ b/arch/x86/include/asm/acpi_table.h > @@ -6,11 +6,6 @@ > * SPDX-License-Identifier: GPL-2.0+ > */ > > -#include > -#include > -#include > -#include > - > #define RSDP_SIG "RSD PTR " /* RSDT pointer signature */ > #define ACPI_TABLE_CREATOR "UBOOT " /* Must be 8 bytes long! */ > #define OEM_ID "UBOOT " /* Must be 6 bytes long! */ > @@ -19,42 +14,6 @@ > #define OEM_REVISION 42 > #define ASL_COMPILER_REVISION 42 > > -/* IO ports to generate SMIs */ > -#define APM_CNT 0xb2 > -#define APM_CNT_CST_CONTROL 0x85 > -#define APM_CNT_PST_CONTROL 0x80 > -#define APM_CNT_ACPI_DISABLE 0x1e > -#define APM_CNT_ACPI_ENABLE 0xe1 > -#define APM_CNT_MBI_UPDATE 0xeb > -#define APM_CNT_GNVS_UPDATE 0xea > -#define APM_CNT_FINALIZE 0xcb > -#define APM_CNT_LEGACY 0xcc > -#define APM_ST 0xb3 > - > -/* Multiple Processor Interrupts */ > -#define MP_IRQ_POLARITY_DEFAULT 0x0 > -#define MP_IRQ_POLARITY_HIGH 0x1 > -#define MP_IRQ_POLARITY_LOW 0x3 > -#define MP_IRQ_POLARITY_MASK 0x3 > -#define MP_IRQ_TRIGGER_DEFAULT 0x0 > -#define MP_IRQ_TRIGGER_EDGE 0x4 > -#define MP_IRQ_TRIGGER_LEVEL 0xc > -#define MP_IRQ_TRIGGER_MASK 0xc > - > -/* > - * Interrupt assigned for SCI in order to > - * create the ACPI MADT IRQ override entry > - */ > -#define ACTL 0x00 > -#define SCIS_MASK 0x07 > -#define SCIS_IRQ9 0x00 > -#define SCIS_IRQ10 0x01 > -#define SCIS_IRQ11 0x02 > -#define SCIS_IRQ20 0x04 > -#define SCIS_IRQ21 0x05 > -#define SCIS_IRQ22 0x06 > -#define SCIS_IRQ23 0x07 > - > #define ACPI_REV_ACPI_1_0 1 > #define ACPI_REV_ACPI_2_0 1 > #define ACPI_REV_ACPI_3_0 2 > @@ -105,14 +64,6 @@ enum acpi_address_space_type { > ACPI_ADDRESS_SPACE_FIXED = 0x7f /* Functional fixed hardware */ > }; > > -/* functional fixed hardware */ > -#define ACPI_FFIXEDHW_VENDOR_INTEL 1 /* Intel */ > -#define ACPI_FFIXEDHW_CLASS_HLT 0 /* C1 Halt */ > -#define ACPI_FFIXEDHW_CLASS_IO_HLT 1 /* C1 I/O then Halt */ > -#define ACPI_FFIXEDHW_CLASS_MWAIT 2 /* MWAIT Native C-state */ > -#define ACPI_FFIXEDHW_FLAG_HW_COORD 1 /* Hardware Coordination bit */ > -#define ACPI_FFIXEDHW_FLAG_BM_STS 2 /* BM_STS avoidance bit */ > - > /* Access size definitions for Generic address structure */ > enum acpi_address_space_size { > ACPI_ACCESS_SIZE_UNDEFINED = 0, /* Undefined (legacy reasons) */ > @@ -172,25 +123,6 @@ struct acpi_madt { > u32 flags; /* Multiple APIC flags */ > } acpi_madt_t; > > -enum dev_scope_type { > - SCOPE_PCI_ENDPOINT = 1, > - SCOPE_PCI_SUB = 2, > - SCOPE_IOAPIC = 3, > - SCOPE_MSI_HPET = 4 > -}; > - > -typedef struct dev_scope { > - u8 type; > - u8 length; > - u8 reserved[2]; > - u8 enumeration; > - u8 start_bus; > - struct { > - u8 dev; > - u8 fn; > - } path[0]; > -} __packed dev_scope_t; > - > /* MADT: APIC Structure Type*/ > enum acpi_apic_types { > LOCALAPIC = 0, /* Processor local APIC */ > Thanks, Stefan