From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752118AbcIML31 (ORCPT ); Tue, 13 Sep 2016 07:29:27 -0400 Received: from mail-wm0-f43.google.com ([74.125.82.43]:36958 "EHLO mail-wm0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750951AbcIML3Z (ORCPT ); Tue, 13 Sep 2016 07:29:25 -0400 Date: Tue, 13 Sep 2016 12:29:21 +0100 From: Matt Fleming To: Lukas Wunner Cc: linux-efi@vger.kernel.org, linux-kernel@vger.kernel.org, Andreas Noever , grub-devel@gnu.org, x86@kernel.org, Ard Biesheuvel Subject: Re: [PATCH v2 0/4] Apple device properties Message-ID: <20160913112921.GH3872@codeblueprint.co.uk> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.24+41 (02bc14ed1569) (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 07 Sep, at 12:49:18PM, Lukas Wunner wrote: > Retrieve device properties from EFI on Macs before ExitBootServices is > called and assign them to devices (patch [3/4]). The devices that > properties pertain to are encoded as EFI Device Paths, so add a parser > for these (patch [2/4]). As a first use case, amend the Thunderbolt driver > to take advantage of the Device ROM supplied by EFI (patch [4/4]). > > Patch [1/4] is already queued in Rafael J. Wysocki's tree for 4.9 and is > included here only because patch [2/4] wouldn't compile without it. > > The series also depends on these two patches which are already queued > in Matt's tree for 4.9: > > - x86/efi: Optimize away setup_gop32/64 if unused > https://patchwork.kernel.org/patch/9315763/ > > - x86/efi: Allow invocation of arbitrary boot services > https://patchwork.kernel.org/patch/9293371/ I had a quick look over these series and nothing looks too crazy, but I still need to do an in-depth review. Given the patch dependencies you outlined above, could you resubmit this after the v4.9 merge window closes? That way it won't be forgotten about.