From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:4864:20::444; helo=mail-wr1-x444.google.com; envelope-from=leif.lindholm@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wr1-x444.google.com (mail-wr1-x444.google.com [IPv6:2a00:1450:4864:20::444]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 32E5521945DC7 for ; Tue, 26 Mar 2019 12:17:10 -0700 (PDT) Received: by mail-wr1-x444.google.com with SMTP id g3so12237282wrx.9 for ; Tue, 26 Mar 2019 12:17:09 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=JSxDsDAk6OL7nYWDCDC2TsP1A+ZpzdscgbiWmmMlDGI=; b=XppeOAyQdrxhGQWOyfH8OagccvXMcJgOdLkl1PrRn3YUi9NPI/7szlhakhaKYt2BdR UcAXL5m2QLyd2EFI8h2vjbt4WAbKXiQG9jAAXW4Ef4wu+JyFdpQLwx+Y9qjIyt+Ig16v 9RBkIlK+8jsKrBuzgUKJDnKdRD6MMnAFrChI/U945hak6O9fR62/PxdaV7F0EFYHJXUj BfG8HMK/Lpq+dWfCZN47NTRIIpAUzMe3lXcXBBEATe+eCKUhF3+DFI3K1Mbc/MS2aqQe MgX/ls323Ilm1LL9iy0uGG+6zRwseQK+hqJgplhRrhOZpmtAKvErfRXKKCgT9eVhqDjx ZeTA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=JSxDsDAk6OL7nYWDCDC2TsP1A+ZpzdscgbiWmmMlDGI=; b=FE6xMwNafHqyUsi/t+5bJH0sThPnb/gT6mJx0M2NQUSTUbQWBqSsFqV/gI+68FzosQ quAY7ItcCUgKm1CoBMJOyyhXtjRJtNFDJ0gp6op3lW6NtRWRPikQSR9PYcQsss5vi+2A xiUODp++E2+qwL8HwxA0kzt1YIqQPPsGm3pP/5V8t6ryyTYXh8CeKdxm3KkUKK8omj/1 YB6bn5J5ozu9sQEfwGmZrFQYAGr4hsK8Qlypa4LyKeh6hjFJMecbIeHAWuZAGQgkF6fP iuVbZ7YfgAhZ84aloORUys33xVlKRKKNuGscX+HL43SYqpEMFNg//otuCIv3fVquRn/i b9dw== X-Gm-Message-State: APjAAAWb1W9KBDQEaHLMTfh+uXpakfWnhwXszQ8cb+2nB1kKNzBnFln8 YGfmaHxgX1C3TLLWjbh2yxcMKg== X-Google-Smtp-Source: APXvYqxQAL+YkqiTHIu9RYnLOfElc2vhwszpkkXZcieI7NNgheQY/xTI8kNL2opOBcy7E3kE3afk2w== X-Received: by 2002:adf:f488:: with SMTP id l8mr20384175wro.213.1553627828170; Tue, 26 Mar 2019 12:17:08 -0700 (PDT) Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id w21sm22018398wmc.4.2019.03.26.12.17.07 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 26 Mar 2019 12:17:07 -0700 (PDT) Date: Tue, 26 Mar 2019 19:17:05 +0000 From: Leif Lindholm To: Sami Mujawar Cc: edk2-devel@lists.01.org, ard.biesheuvel@linaro.org, michael.d.kinney@intel.com, alexei.fedorov@arm.com, Matteo.Carlini@arm.com, Stephanie.Hughes-Fitt@arm.com, nd@arm.com Message-ID: <20190326191705.uasrrylwjyw2ia6a@bivouac.eciton.net> References: <20190326152319.15036-1-sami.mujawar@arm.com> MIME-Version: 1.0 In-Reply-To: <20190326152319.15036-1-sami.mujawar@arm.com> User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [PATCH edk2-platforms v4 0/4] Platform/ARM: Platform support for Dynamic Tables Framework X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 26 Mar 2019 19:17:10 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Tue, Mar 26, 2019 at 03:23:15PM +0000, Sami Mujawar wrote: > Dynamic Tables Framework aims to reduce the amount of effort > required for porting firmware to new platforms by simplifying > the generation of firmware tables based on hardware description > provided by a platform specific component. > > The Dynamic Tables Framework core queries the platform specific > component to retrieve the required hardware information for > generating standardised firmware tables at run-time. > > The platform specific component responsible for collating the > hardware information is called the Configuration Manager. > > This patch series introduce the Configuration Manager that > provides the hardware description to Dynamic Tables Framework. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Sami Mujawar > > The changes can be seen at: > https://github.com/samimujawar/edk2-platforms/tree/365_dynamic_tables_framework_v4 For the series: Reviewed-by: Leif Lindholm Pushed as 89f6901df0..8cb8080438. / Leif > This v4 patch series incorporates: > * Updates based on review comments to sort #include files and .inf sections > in alphabetical order. > > Patches updated in this series are: > - Platform/ARM: Configuration Manager for Juno > - Platform/ARM: Configuration Manager for FVP > > The corresponding edk2 code changes can be seen at: > https://github.com/samimujawar/edk2/tree/365_dynamic_tables_framework_v2 > > Sami Mujawar (4): > Platform/ARM: Configuration Manager for Juno > Platform/ARM: Dynamic Tables support for Juno > Platform/ARM: Configuration Manager for FVP > Platform/ARM: Dynamic Tables support for FVP > > Platform/ARM/JunoPkg/ArmJuno.dsc | 12 +- > Platform/ARM/JunoPkg/ArmJuno.fdf | 12 + > Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManager.dsc.inc | 29 + > Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c | 750 ++++++++++++++++++++ > Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h | 179 +++++ > Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf | 87 +++ > Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/Platform.h | 99 +++ > Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl | 276 +++++++ > Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/PlatformASLTablesLib.inf | 45 ++ > Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtJunoUsb.asl | 123 ++++ > Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtPci.asl | 201 ++++++ > Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtUart.asl | 48 ++ > Platform/ARM/JunoPkg/Drivers/ArmJunoDxe/ArmJunoDxe.c | 9 +- > Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.dsc | 15 + > Platform/ARM/VExpressPkg/ArmVExpress-FVP-AArch64.fdf | 16 +- > Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManager.dsc.inc | 31 + > Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c | 682 ++++++++++++++++++ > Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h | 181 +++++ > Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf | 80 +++ > Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/Platform.h | 99 +++ > Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl | 73 ++ > Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/PlatformASLTablesLib.inf | 35 + > 22 files changed, 3077 insertions(+), 5 deletions(-) > create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManager.dsc.inc > create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c > create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h > create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf > create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/ConfigurationManagerDxe/Platform.h > create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl > create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/PlatformASLTablesLib.inf > create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtJunoUsb.asl > create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtPci.asl > create mode 100644 Platform/ARM/JunoPkg/ConfigurationManager/PlatformASLTablesLib/SsdtUart.asl > create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManager.dsc.inc > create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.c > create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManager.h > create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/ConfigurationManagerDxe.inf > create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/ConfigurationManagerDxe/Platform.h > create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/Dsdt.asl > create mode 100644 Platform/ARM/VExpressPkg/ConfigurationManager/PlatformASLTablesLib/PlatformASLTablesLib.inf > > -- > 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' > >