From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f47.google.com (mail-wm1-f47.google.com [209.85.128.47]) by mx.groups.io with SMTP id smtpd.web10.10605.1601553165166985241 for ; Thu, 01 Oct 2020 04:52:45 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@nuviainc-com.20150623.gappssmtp.com header.s=20150623 header.b=GpXh9DKi; spf=pass (domain: nuviainc.com, ip: 209.85.128.47, mailfrom: leif@nuviainc.com) Received: by mail-wm1-f47.google.com with SMTP id v12so2719484wmh.3 for ; Thu, 01 Oct 2020 04:52:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nuviainc-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=tebE/7/Q2wzv68drS4ZWl7HzgKLxsPTw0m3/qB2u1QU=; b=GpXh9DKiU5aSm8P87f7ch3GBYHeHFA58OM3SOUNTZZ8bh185km3HwLmcmPyTetlxIQ 17QX+9FTLtKoThVkgUa4Kunvaf8C74WvJ2bQhxi3036bJ7udlrDoaDJT9Got64gwStpc UlLY+Bd6HOYzyNkxcPXFpFqjdzM+rCcns2elEnLkPF8qLS+thJHODH1I7bt0mg2taNwY 3W/kN3JapCGLbuD1a3TY/P45m3E8tr/f0Tcq4V9g8VDSba8tjShyi3fZYomvS9S36InU /X2b8OEQUev6X5n3x4QHbfeKfWXXSZHU4+4KTSMSlH9QRGAdORACO9qJWiJ2STmUrBns 4SPQ== 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=tebE/7/Q2wzv68drS4ZWl7HzgKLxsPTw0m3/qB2u1QU=; b=eYRN6XRG6xjZwFt90VZeRyvSPtcSN6QA+2gOnfb0GcokzlSGH3X5ogf/TxKYEZGkZM rASOTK8qFlxEgkPjt0LUHTdF0Q4yPomnSryIry1bzwd4pnmhsxeKAw1wtwv9GmNn/c5R nxEygNcmXZbMBSHQlOPhPumYkWTsQOAZ+rS3YHfMiqfFfHUHXeMNzIR5kS7xSfwAEqRp UtjzCkl8FkozMCWhPzhEOWGng5LC2Eb6ti8IrBYI9zBJyGebnhuZDcBgs/MFBiWKmUia j5vqZNR5V7ilrwyEq6eQlKto03Hjr2nq8lR4CyBIFsIyeR78N86JoTD9RU5bRU37RLYz c4Rg== X-Gm-Message-State: AOAM531Y355b2cqJa6N/vrNH+w5pa6dIEkulPAF9taRfiOX5gjwKw6XA 0yrOMBM7jCi33JX3XwsZTn1qSw== X-Google-Smtp-Source: ABdhPJy7pvCUWXLX0VDIsm9IgNkDikcfuHuOoNMhZB9mgvQ43yUkTXqCDAlBxH/+3GrWK85S70hs6g== X-Received: by 2002:a05:600c:2207:: with SMTP id z7mr8215836wml.42.1601553163055; Thu, 01 Oct 2020 04:52:43 -0700 (PDT) Return-Path: Received: from vanye ([2001:470:1f09:12f0:b26e:bfff:fea9:f1b8]) by smtp.gmail.com with ESMTPSA id 18sm7967376wmj.28.2020.10.01.04.52.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 01 Oct 2020 04:52:42 -0700 (PDT) Date: Thu, 1 Oct 2020 12:52:35 +0100 From: "Leif Lindholm" To: "Desimone, Nathaniel L" Cc: "devel@edk2.groups.io" , "rfc@edk2.groups.io" , Laszlo Ersek , Ard Biesheuvel , "Kirkendall, Garrett" , "Lendacky, Thomas" , "thomas.abraham@arm.com" , "Frank.Orr@dell.com" , Jim Dailey , Abner Chang , "Wang, Sunny (HPS SW)" , Daniel Schaefer , Liming Gao , Tim Lewis , "Kevin@Insyde" , Felix Polyudov , "Kinney, Michael D" , "Oram, Isaac W" , "Ni, Ray" Subject: Re: [RFC] Request to move MinPlatformPkg out of the Intel folder Message-ID: <20201001115235.GI5623@vanye> References: MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi Nate, On Wed, Sep 30, 2020 at 23:17:15 +0000, Desimone, Nathaniel L wrote: > This has come up informally a couple of times already, but I do > think that it would be beneficial to move MinPlatformPkg out of the > Platform/Intel folder and into a vendor neutral folder. MinPlatform > really isn't Intel specific, it's just "middleware" that is intended > to make it easier and faster to build EDK II based firmware. We have > always intended MinPlatform to be vendor neutral, and it would be > great to see MinPlatform based implementations for non-Intel > silicon. Anyone working with EDK II should feel welcome to work with > us on MinPlatform, regardless of the SoC in use. Very positive to this on the whole. I have only been following MinPlatformPkg peripherally - at the time it was merged, I did not have the bandwidth to look into it and help make it cross-architecture from the start, so I mostly refrained from comment. One thing I think is needed for this to move out of Platform/Intel would be a ReadMe.rst describing the intended real-world use. Some of this lives in Platform/Intel/Readme.md today. Could we start with a set moving this text into a separate file under MinPlatformPkg and rework that into something that stands on its own? > To that end, I propose that we move MinPlatformPkg out of > edk2-platforms/Platform/Intel to > edk2-platforms/Platform/MinPlatformPkg. If I was to bikeshed, I'd probably suggest Platform/TianoCore/MinPlatformPkg. Oops, guess I did. > It might also make sense to > move edk2-platforms/Platform/Intel/Tools and > edk2-platforms/Platform/Intel/build_bios.py to > edk2-platforms/Platform as well since those are all useful for > building MinPlatform based firmware, but I could also see that being > something for another time. Let me know what you think! I think we should see this as a separate step, regardless. I do think we should be able to do better on sharing some of this platform image tooling cross-architecture (even moreso now Risc-V is involved), but I would prefer for us to spend the effort of identifying the intersections upfront before we start uploading tools to places where they look generic and may confuse people. Maybe we should raise this part as a BZ? Best Regards, Leif