From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755469AbcG0LUs (ORCPT ); Wed, 27 Jul 2016 07:20:48 -0400 Received: from mailout2.hostsharing.net ([83.223.90.233]:41077 "EHLO mailout2.hostsharing.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753636AbcG0LUr (ORCPT ); Wed, 27 Jul 2016 07:20:47 -0400 X-Mailbox-Line: From 8743c01825c3d16f812acec7525e5c87f4eed37a Mon Sep 17 00:00:00 2001 Message-Id: From: Lukas Wunner Date: Wed, 27 Jul 2016 13:20:41 +0200 Subject: [PATCH 0/6] Apple device properties To: linux-efi@vger.kernel.org, Matt Fleming , linux-kernel@vger.kernel.org Cc: Andreas Noever , Pierre Moreau , reverser@put.as, grub-devel@gnu.org, x86@kernel.org, Aleksey Makarov , "Rafael J. Wysocki" , Mika Westerberg , Andy Shevchenko , Greg Kroah-Hartman Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Apple EFI drivers supply device properties which are needed to support Macs optimally. This series extends the efistub to retrieve the device properties before ExitBootServices is called (patch [1/6]). They are assigned to devices in an fs_initcall (patch [5/6]). As a first use case, the Thunderbolt driver is amended to take advantage of the Device ROM supplied by EFI (patch [6/6]). A by-product is a parser for EFI Device Paths which finds the struct device corresponding to a given path. This is needed to assign properties to their devices (patch [3/6]). I've pushed these patches to GitHub where they can be reviewed more comfortably with green/red highlighting: https://github.com/l1k/linux/commits/apple_properties_v1 It would be good if one of the resident EFI experts could look over patch [1/6] to see if I've made any mistakes that might prevent this from working on 32 bit. It was only tested on 64 bit, I don't know anyone with an older Mac who could test this. Specifically, is the following okay: efi_early->call((unsigned long)sys_table->boottime->locate_protocol, ...) It would be convenient to have LocateProtocol or LocateHandleBuffer in struct efi_config so that they can be called with efi_call_early(). Would a patch to add those be entertained? Right now we only offer LocateHandle and HandleProtocol, which is somewhat cumbersome and needs more code as the setup_pci() functions show. Thanks, Lukas Lukas Wunner (6): efi: Retrieve Apple device properties ACPI / bus: Make acpi_get_first_physical_node() public efi: Add device path parser driver core: Don't leak secondary fwnode on device removal efi: Assign Apple device properties thunderbolt: Use Device ROM retrieved from EFI Documentation/kernel-parameters.txt | 5 + arch/x86/boot/compressed/eboot.c | 55 ++++++++ arch/x86/include/uapi/asm/bootparam.h | 1 + drivers/acpi/internal.h | 1 - drivers/base/core.c | 1 + drivers/firmware/efi/Kconfig | 16 +++ drivers/firmware/efi/Makefile | 2 + drivers/firmware/efi/apple-properties.c | 219 ++++++++++++++++++++++++++++++++ drivers/firmware/efi/dev-path-parser.c | 186 +++++++++++++++++++++++++++ drivers/thunderbolt/Kconfig | 1 + drivers/thunderbolt/eeprom.c | 42 ++++++ drivers/thunderbolt/switch.c | 2 +- include/linux/acpi.h | 7 + include/linux/efi.h | 38 ++++++ 14 files changed, 574 insertions(+), 2 deletions(-) create mode 100644 drivers/firmware/efi/apple-properties.c create mode 100644 drivers/firmware/efi/dev-path-parser.c -- 2.8.1