Linaro-acpi July 2014

linaro-acpi@lists.linaro.org

12 participants
13 discussions

[PATCH 0/4 v2] Add tool to enable APEI tables test, other minor changes.

by Tomasz Nowicki

Patch set implement as follows: 1. Minor changes around existing tools - reorganize building process, - move common code to separate direction, - no functionality changes. 2. Add BFAPEI (Blob For APEI) tool which can create blobs for APEI testing. - see commit log for more details Changes for v2: - compilation regardless of whether APEI is used or not, - correct file name in comment headers, - move ACPI structures, defines, etc. to acpi.h headers, - suppress compiler warnings. Tomasz Nowicki (4): map, cab: Reorganize building process and tree hierarchy. bfapei: Add tool that can create blobs for APEI table testing. bfapei: Move generic ACPI structures to separated file, remove duplicated defines. Suppress compiler warnings, file comment headers cleanup. Makefile | 28 +- platforms/Makefile | 9 +- tools/bfapei/Makefile | 11 + tools/bfapei/bfapei.c | 271 +++++++++++++++++++ tools/bfapei/bfapei.h | 49 ++++ tools/cab/Makefile | 12 +- tools/cab/cab.c | 535 +------------------------------------- tools/cab/cab.h | 149 ----------- tools/common/Makefile | 4 + tools/common/build_aml.c | 455 ++++++++++++++++++++++++++++++++ tools/common/check_aml.c | 512 ++++++++++++++++++++++++++++++++++++ tools/common/include/acpi.h | 204 +++++++++++++++ tools/common/include/build_aml.h | 74 ++++++ tools/common/include/check_aml.h | 92 +++++++ tools/mab/Makefile | 13 +- tools/mab/mab.c | 437 +------------------------------ tools/mab/mab.h | 77 ------ 17 files changed, 1734 insertions(+), 1198 deletions(-) create mode 100644 tools/bfapei/Makefile create mode 100644 tools/bfapei/bfapei.c create mode 100644 tools/bfapei/bfapei.h create mode 100644 tools/common/Makefile create mode 100644 tools/common/build_aml.c create mode 100644 tools/common/check_aml.c create mode 100644 tools/common/include/acpi.h create mode 100644 tools/common/include/build_aml.h create mode 100644 tools/common/include/check_aml.h -- 1.7.9.5

7 years, 11 months

[RFC] ACPI: Add GPIO-signaled event simulator.

by Tomasz Nowicki

GPIO signaled events is quite new thing in Linux kernel. AFAIK, there are not many board which can take advantage of it. However, GPIO events are very useful feature during work on ACPI subsystems. This commit emulates GPIO h/w behaviour and consists on read/write operation to debugfs file. GPIO device instance is still required in DSDT table along with _AEI resources and event methods. Reading from file provides pin to GPIO device map e.g. : $ cat /sys/kernel/debug/acpi/gpio_event GPIO device name: /__SB.GPI0 Available GPIO pin map: /__SB.GPI0 <-> pin 0x100 Based on that, user can trigger method corresponding to device pin number: $ echo "/__SB.GPI0 0x100" > /sys/kernel/debug/acpi/gpio_event Please, see Kconfig help and driver head section for more details regarding tool usage. Signed-off-by: Tomasz Nowicki <tomasz.nowicki(a)linaro.org> --- drivers/acpi/Kconfig | 11 +++ drivers/acpi/Makefile | 1 + drivers/acpi/gpio_evt_emu.c | 207 ++++++++++++++++++++++++++++++++++++++++++++ 3 files changed, 219 insertions(+) create mode 100644 drivers/acpi/gpio_evt_emu.c diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig index fd54a74..59e4c67 100644 --- a/drivers/acpi/Kconfig +++ b/drivers/acpi/Kconfig @@ -122,6 +122,17 @@ config ACPI_BUTTON To compile this driver as a module, choose M here: the module will be called button. +config ACPI_GPIO_EVT_EMULATE + bool "ACPI GPIO-signaled Events Emulation Support" + depends on DEBUG_FS + default n + help + This will enable your system to emulate GPIO-signaled event through + proc file system /sys/kernel/debug/acpi/gpio_event. User needs to print + available GPIO devices and pin map (cat path_to_proc_file) so that he + knows how to trigger given event by echo "XXX YYY" > path_to_proc_file + (where, XXX is a path to GPIO device and YYY is a pin number). + config ACPI_VIDEO tristate "Video" depends on X86 && BACKLIGHT_CLASS_DEVICE diff --git a/drivers/acpi/Makefile b/drivers/acpi/Makefile index 9fa20ff..fb3c335 100644 --- a/drivers/acpi/Makefile +++ b/drivers/acpi/Makefile @@ -55,6 +55,7 @@ acpi-$(CONFIG_ACPI_PROCFS_POWER) += cm_sbs.o ifdef CONFIG_ACPI_VIDEO acpi-y += video_detect.o endif +acpi-$(CONFIG_ACPI_GPIO_EVT_EMULATE) += gpio_evt_emu.o # These are (potentially) separate modules diff --git a/drivers/acpi/gpio_evt_emu.c b/drivers/acpi/gpio_evt_emu.c new file mode 100644 index 0000000..a2f762c --- /dev/null +++ b/drivers/acpi/gpio_evt_emu.c @@ -0,0 +1,207 @@ +/* + * Code to emulate GPIO-signaled events. + * + * The sole purpose of this module is to help with GPIO event triggering. + * Suggested way of using: + * 1. Perform walk of the namespac device tree looking for GPIO devices + * with associated _AEI resources e.g.: + * $ cat /sys/kernel/debug/acpi/gpio_event + * GPIO device name: /__SB.GPI0 + * Available GPIO pin map: + * /__SB.GPI0 <-> pin 0x100 + * + * 2. Trigger method corresponding to device pin number: + * $ echo "/__SB.GPI0 0x100" > /sys/kernel/debug/acpi/gpio_event + */ + +/* + * Copyright (C) 2014, Linaro Ltd. + * Author: Tomasz Nowicki <tomasz.nowicki(a)linaro.org> + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License version 2 as + * published by the Free Software Foundation. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + */ + +#include <linux/module.h> +#include <linux/uaccess.h> +#include <linux/debugfs.h> +#include <linux/acpi.h> + +#include "acpica/accommon.h" +#include "acpica/acnamesp.h" + +#include "internal.h" + +static struct dentry *gpio_evt_dentry; + +static void gpio_trigger_event(char *gpio_path, u32 pin) +{ + acpi_handle gpio_handle, evt_handle; + char ev_name[5]; + + if (ACPI_FAILURE(acpi_get_handle(NULL, gpio_path, &gpio_handle))) { + pr_err(PREFIX "getting handle to <%s> failed\n", gpio_path); + return; + } + + if (pin <= 255) + sprintf(ev_name, "_L%02X", pin); + else + sprintf(ev_name, "_EVT"); + + + if (ACPI_FAILURE(acpi_get_handle(gpio_handle, ev_name, &evt_handle))) { + pr_err(PREFIX "getting handle to <%s.%s> failed, there is no method related to 0x%02X pin\n", + gpio_path, ev_name, pin); + return; + } + + if (ACPI_FAILURE(acpi_execute_simple_method(evt_handle, NULL, + pin <= 255 ? 0 : pin))) + pr_err(PREFIX "evaluating <%s._EVT for pin %d> failed\n", + gpio_path, pin); + + return; +} + +static ssize_t gpio_evt_write(struct file *file, const char __user *user_buf, + size_t count, loff_t *ppos) +{ + u32 pin_number; + char *gpio_path = NULL; + char *pin_str = NULL; + const char *delim = " "; + char *temp_buf = NULL; + char *temp_buf_addr = NULL; + + temp_buf = kmalloc(count+1, GFP_ATOMIC); + if (!temp_buf) { + pr_warn(PREFIX "%s: Memory allocation failed\n", __func__); + return count; + } + temp_buf[count] = '\0'; + temp_buf_addr = temp_buf; + if (copy_from_user(temp_buf, user_buf, count)) + goto out; + + gpio_path = strsep(&temp_buf, delim); + pin_str = strsep(&temp_buf, delim); + + if (gpio_path && pin_str) { + ssize_t ret; + unsigned long val; + + ret = kstrtoul(pin_str, 10, &val); + if (ret) { + pr_warn(PREFIX "unknown event\n"); + goto out; + } + + pin_number = (u32)val; + } else { + pr_warn(PREFIX "unknown device\n"); + goto out; + } + + pr_info(PREFIX "ACPI device name is <%s>, pin number is <%d>\n", + gpio_path, pin_number); + + gpio_trigger_event(gpio_path, pin_number); + +out: + kfree(temp_buf_addr); + return count; +} + +static acpi_status gpio_list_resource(struct acpi_resource *ares, + void *context) +{ + struct acpi_resource_gpio *agpio; + struct seq_file *m = context; + int pin; + + if (!m) + return AE_OK; + + if (ares->type != ACPI_RESOURCE_TYPE_GPIO) + return AE_OK; + + agpio = &ares->data.gpio; + if (agpio->connection_type != ACPI_RESOURCE_GPIO_TYPE_INT) + return AE_OK; + + pin = agpio->pin_table[0]; + seq_printf(m, "%s <-> pin %d\n", agpio->resource_source.string_ptr, + pin); + + return AE_OK; +} + +static acpi_status gpio_find_resource(acpi_handle handle, u32 lvl, + void *context, void **rv) +{ + struct acpi_namespace_node *node; + struct seq_file *m = context; + + if (ACPI_FAILURE(acpi_walk_resources(handle, METHOD_NAME__AEI, + gpio_list_resource, NULL))) + return AE_OK; + + node = acpi_ns_validate_handle(handle); + if (!node) { + pr_err(PREFIX "Mapping GPIO handle to node failed\n"); + return AE_OK; + } + + seq_printf(m, "GPIO device name: %4.4s\nAvailable GPIO pin map:\n", + ACPI_CAST_PTR(char, &node->name)); + acpi_walk_resources(handle, METHOD_NAME__AEI, gpio_list_resource, + context); + return AE_OK; +} + +static int gpio_evt_show(struct seq_file *m, void *v) +{ + acpi_get_devices(NULL, gpio_find_resource, m, NULL); + return 0; +} + +static int gpio_evt_open(struct inode *inode, struct file *file) +{ + return single_open(file, gpio_evt_show, NULL); +} + +static const struct file_operations gpio_evt_emu_fops = { + .open = gpio_evt_open, + .write = gpio_evt_write, + .read = seq_read, + .llseek = seq_lseek, + .release = single_release, +}; + +static int __init gpio_evt_emu_init(void) +{ + if (acpi_debugfs_dir == NULL) + return -ENOENT; + + gpio_evt_dentry = debugfs_create_file("gpio_event", S_IWUSR, + acpi_debugfs_dir, NULL, &gpio_evt_emu_fops); + if (gpio_evt_dentry == NULL) + return -ENODEV; + + return 0; +} + +device_initcall(gpio_evt_emu_init); +ACPI_MODULE_NAME("acpi_evt_emu"); +MODULE_LICENSE("GPL"); -- 1.9.1

10 years, 6 months

leg-kernel tag leg-20140730.0 released

by G Gregory

The leg-kernel release has been made and tagged as leg-20140730.0 This is based on mainline kernel v3.16-rc7 Repository : http://git.linaro.org/leg/acpi/leg-kernel.git Direct Link: http://git.linaro.org/leg/acpi/leg-kernel.git/commit/2189d8891fd5ae91eeaa56… Notes :- 1) This is the first version of leg-kernel to be based on ACPI 5.1 release.

10 years, 7 months

[PATCH] ACPI:GPIO: Add GPIO description on JUNO

by Yi Li

Add the general description for GPIO on JUNO board. Define pin number 256 as power button function (for demo) in this patch. However,JUNO has no GPIO controller in hardware, Using GPIO event emulation which can trigger the GPIO action, then test the GPIO-signal event model for ACPI. Signed-off-by: Yi Li <yi.li(a)linaro.org> --- ArmPlatformPkg/ArmJunoPkg/AcpiTables/juno/dsdt.asl | 42 ++++++++++++++++++++ 1 file changed, 42 insertions(+) diff --git a/ArmPlatformPkg/ArmJunoPkg/AcpiTables/juno/dsdt.asl b/ArmPlatformPkg/ArmJunoPkg/AcpiTables/juno/dsdt.asl index 5bfd521..e4e9cad 100644 --- a/ArmPlatformPkg/ArmJunoPkg/AcpiTables/juno/dsdt.asl +++ b/ArmPlatformPkg/ArmJunoPkg/AcpiTables/juno/dsdt.asl @@ -58,6 +58,48 @@ DefinitionBlock ( } } + //GPIO controller description + Device (GPO0) + { + Name (_HID, "LNRO0009") + Name (_ADR, 0) + Name (_UID, 0) + + Name (_CRS, ResourceTemplate () { + //GPIO Register Interface(demo,need to modify by the real h/w setting) + Memory32Fixed (ReadWrite, 0x1c010000, 0x1000) + + //Interrupt vector 0x80 for GPIO controller(demo,need to modify by the real h/w setting) + Interrupt (ResourceConsumer, Edge, ActiveLow, Exclusive) {0x80} + }) + + Name (_AEI,ResourceTemplate () { + //Assume GPIO256 as PowerButton + GpioInt(Edge, ActiveLow, ExclusiveAndWake, PullUp, , " \\_SB.GPO0") {0x100} + }) + + //Handle all ACPI Events signaled by GPIO Controller GPO0 + Method (_EVT, 0x1, Serialized) + { + //Arg0 - EventNumber. An Integer indicating the event number. + //(Controller-relative zero-based GPIO pin number) of the current event. + //Must be in the range 0x0000 - 0xffff. + Switch (ToInteger(Arg0)) + { + //Pin number 256 + Case (0x100) + { + //Notify OSPM the power button is pressed + Notify (PWRB, 0x80) + } + Default + { + + } + } + } + } + //Power button device description Device (PWRB) { -- 1.7.9.5

10 years, 7 months

[PATCH] ACPI:PB: Add the power button device on JUNO

by Yi Li

Add the power button description on juno board, notify power button with 0x80 value, the OS will detect the power button is pressed, how to deal with the event which depends on OS's policy. Usually the acpid deamon will call the specific script to let OS shutdown. Signed-off-by: Yi Li <yi.li(a)linaro.org> --- ArmPlatformPkg/ArmJunoPkg/AcpiTables/juno/dsdt.asl | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/ArmPlatformPkg/ArmJunoPkg/AcpiTables/juno/dsdt.asl b/ArmPlatformPkg/ArmJunoPkg/AcpiTables/juno/dsdt.asl index bdee349..5bfd521 100644 --- a/ArmPlatformPkg/ArmJunoPkg/AcpiTables/juno/dsdt.asl +++ b/ArmPlatformPkg/ArmJunoPkg/AcpiTables/juno/dsdt.asl @@ -58,6 +58,19 @@ DefinitionBlock ( } } + //Power button device description + Device (PWRB) + { + Name (_HID, EISAID("PNP0C0C")) + Name (_ADR, 0) + Name (_UID, 0) + + Method (_STA, 0x0, Notserialized) + { + Return (0x0F) + } + } + // apb_pclk Device (CLK0) { Name (_HID, "LNRO0008") -- 1.7.9.5

10 years, 7 months

[PATCH 0/3] Update APM XGene tables to ACPI 5.1

by al.stone＠linaro.org

From: Al Stone <al.stone(a)linaro.org> These patches are my best guess at initial ACPI tables for XGene, in accordance with the new ACPI 5.1 specification. These have not been fully tested yet but that process has been started. Al Stone (3): APM / XGene: Minor cleanup of some of the basics APM / XGene: Update core tables to replace ACPI 5.1 changes APM / XGene: Add in empty SSDT for future use in developing complete tables platforms/APMXGene.acpi/APMXGene.manifest | 1 + platforms/APMXGene.acpi/Apic.asl | 262 ++-- platforms/APMXGene.acpi/Dsdt.asl | 2215 ++++++++++++++++------------- platforms/APMXGene.acpi/Facp.asl | 57 +- platforms/APMXGene.acpi/Gtdt.asl | 87 +- platforms/APMXGene.acpi/Rsdp.asl | 2 +- platforms/APMXGene.acpi/Ssdt.asl | 21 + platforms/APMXGene.acpi/Xsdt.asl | 10 +- 8 files changed, 1508 insertions(+), 1147 deletions(-) create mode 100644 platforms/APMXGene.acpi/Ssdt.asl -- 1.9.3

10 years, 7 months

libacpica for aarch64 - package/source?

by Claudio Fontana

Hello, what is the current state of libacpica for aarch64? Are packages available from Linaro (static libs and includes would be best)? Is it available on which linaro trees? Is anything already in upstream libacpica? Thank you, Claudio

10 years, 7 months

[PATCH v5 0/3] APEI: Make APEI architecture independent.

by Tomasz Nowicki

APEI is currently implemented so that it depends on x86 hardware. The primary dependency is that GHES uses the x86 NMI for hardware error notification and MCE for memory error handling. These patches remove that dependency. Other APEI features such as error reporting via external IRQ, error serialization, or error injection, do not require changes to use them on non-x86 architectures. The following patch set eliminates the APEI Kconfig x86 dependency by making these changes: - treat NMI notification as GHES architecture - HAVE_ACPI_APEI_NMI - group and wrap around #ifdef CONFIG_HAVE_ACPI_APEI_NMI code which is used only for NMI path - identify architectural boxes and abstract it accordingly (tlb flush and MCE) - rework ioremap for both IRQ and NMI context NMI code is kept in ghes.c file since NMI and IRQ context are tightly coupled. Note, these patches introduce no functional changes for x86. The NMI notification feature is hard selected for x86. Architectures that want to use this feature should also provide NMI code infrastructure. V1->V2 - address Borislav's comment - abstract arch-specific calls instead of wrapping into the #ifdef V2->V3 - address Robert's comment - disable ACPI_APEI_NMI selection so that it is hard selected by arch Kconfig - rename ACPI_APEI_NMI to ARCH_HAS_ACPI_APEI_NMI V3->V4 - do not abstract NMI calls for archs which do not support it - merge some of patches to make review process easier V4->V5 - markers for preprocessor statements (#else, #endif) around long code block Tomasz Nowicki (3): apei, mce: Factor out APEI architecture specific MCE calls. acpi, apei, ghes: Make NMI error notification to be GHES architecture extension. acpi, apei, ghes: Factor out ioremap virtual memory for IRQ and NMI context. arch/x86/Kconfig | 2 + arch/x86/kernel/acpi/Makefile | 1 + arch/x86/kernel/acpi/apei.c | 62 +++++++++++++++ drivers/acpi/apei/Kconfig | 8 +- drivers/acpi/apei/apei-base.c | 13 ++++ drivers/acpi/apei/ghes.c | 173 +++++++++++++++++++++++++++--------------- drivers/acpi/apei/hest.c | 29 +------ include/acpi/apei.h | 4 + include/linux/nmi.h | 4 + 9 files changed, 204 insertions(+), 92 deletions(-) create mode 100644 arch/x86/kernel/acpi/apei.c -- 1.9.1

10 years, 7 months

[PATCH v3 0/3] Clean up ACPI core to prepare for running ACPI on ARM64

by Hanjun Guo

This patch set have no function change for x86 and ia64 and just do some clean up to prepare for running ACPI on ARM64. This patch set is splited out from the patch set [1] "[PATCH v4 00/13] Enable ACPI on ARM64 in Kconfig" and hope it can be merged first before ARM64 ACPI core patches. [1]: https://lkml.org/lkml/2014/6/26/627 update from v2: Don not select ACPI_LEGACY_TABLES_LOOKUP on IA64 which is catched by Peter. update from v1: 1. Drop "Make EC debugfs depend on X86 || IA64 in Kconfig"; 2. Rename ACPI_SCAN_BIOS_NOT_EFI to ACPI_LEGACY_TABLES_LOOKUP suggested by Rafael; 3. Rename ARCH_HAS_ACPI_PDC to ARCH_MIGHT_HAVE_ACPI_PDC suggested by Rafael; 4. Remove the help for ARCH_MIGHT_HAVE_ACPI_PDC because it can't be selected; 5. Rename acpi_arch_is_smp() to acpi_has_cpu_in_madt() to be more explicit and easy understanding. Graeme Gregory (2): ACPI: ARM64 does not have a BIOS add config for BIOS table scan. ACPI: Don't use acpi_lapic in ACPI core code Hanjun Guo (1): ACPI / processor: Introduce ARCH_HAS_ACPI_PDC arch/ia64/Kconfig | 1 + arch/ia64/include/asm/acpi.h | 5 + arch/x86/Kconfig | 2 + arch/x86/include/asm/acpi.h | 5 + drivers/acpi/Kconfig | 6 ++ drivers/acpi/Makefile | 1 + drivers/acpi/acpi_processor.c | 2 +- drivers/acpi/internal.h | 5 + drivers/acpi/osl.c | 4 +- drivers/acpi/processor_core.c | 198 --------------------------------------- drivers/acpi/processor_pdc.c | 206 +++++++++++++++++++++++++++++++++++++++++ 11 files changed, 235 insertions(+), 200 deletions(-) create mode 100644 drivers/acpi/processor_pdc.c -- 1.7.9.5

10 years, 7 months

[PATCH v2 0/3] Clean up ACPI core to prepare for running ACPI on ARM64

by Hanjun Guo

This patch set have no function change for x86 and ia64 and just do some clean up to prepare for running ACPI on ARM64. This patch set is splited out from the patch set [1] "[PATCH v4 00/13] Enable ACPI on ARM64 in Kconfig" and hope it can be merged first before ARM64 ACPI core patches. [1]: https://lkml.org/lkml/2014/6/26/627 update from v1: 1. Drop "Make EC debugfs depend on X86 || IA64 in Kconfig"; 2. Rename ACPI_SCAN_BIOS_NOT_EFI to ACPI_LEGACY_TABLES_LOOKUP suggested by Rafael; 3. Rename ARCH_HAS_ACPI_PDC to ARCH_MIGHT_HAVE_ACPI_PDC suggested by Rafael; 4. Remove the help for ARCH_MIGHT_HAVE_ACPI_PDC because it can't be selected; 5. Rename acpi_arch_is_smp() to acpi_has_cpu_in_madt() to be more explicit and easy understanding. Graeme Gregory (2): ACPI: ARM64 does not have a BIOS add config for BIOS table scan. ACPI: Don't use acpi_lapic in ACPI core code Hanjun Guo (1): ACPI / processor: Introduce ARCH_HAS_ACPI_PDC arch/ia64/Kconfig | 2 + arch/ia64/include/asm/acpi.h | 5 + arch/x86/Kconfig | 2 + arch/x86/include/asm/acpi.h | 5 + drivers/acpi/Kconfig | 6 ++ drivers/acpi/Makefile | 1 + drivers/acpi/acpi_processor.c | 2 +- drivers/acpi/internal.h | 5 + drivers/acpi/osl.c | 4 +- drivers/acpi/processor_core.c | 198 --------------------------------------- drivers/acpi/processor_pdc.c | 206 +++++++++++++++++++++++++++++++++++++++++ 11 files changed, 236 insertions(+), 200 deletions(-) create mode 100644 drivers/acpi/processor_pdc.c -- 1.7.9.5

10 years, 7 months

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

Linaro-acpi July 2014