From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f68.google.com (mail-wm1-f68.google.com [209.85.128.68]) by mx.groups.io with SMTP id smtpd.web12.136.1601594791415454600 for ; Thu, 01 Oct 2020 16:26:31 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@nuviainc-com.20150623.gappssmtp.com header.s=20150623 header.b=nkQD+Hwo; spf=pass (domain: nuviainc.com, ip: 209.85.128.68, mailfrom: leif@nuviainc.com) Received: by mail-wm1-f68.google.com with SMTP id e2so378557wme.1 for ; Thu, 01 Oct 2020 16:26:31 -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:content-transfer-encoding:in-reply-to :user-agent; bh=zWnkMUy92DuS8Zk2T9QgB/c1jcHTya4aCd2Zwzl39+E=; b=nkQD+HwoaY+K0Zq+VSiA9P8oXHXdkvpdhrbHjkmll8lSGP134UWcn3oOYOZb4kcquZ iFlpJk8msfiHcJRZkzx6Cg9S5Ywh7CHo4CgBD2SCpW5JIXaSZIVjRVCM1MSjNpLz8nz2 4KhqbKqkCZNCBw3cWahhAUmOQZe9jw+psFiWoMudCKzzeDB8aG8cvTE5bC9onQVdw5DS wj6+NJGoRQ2gAHGxOjkKoyY8b3pt+TS2sAHiRsbwGkKdIZOi+PeD2RYHVXEu6r/5etKn ojcAz52a+ITz5/JxuoGqaF+JM7RneGYzJZHcB1Yx+h2LMgqTV6G66JCKDXVuL4VDAhKC hlKQ== 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:content-transfer-encoding :in-reply-to:user-agent; bh=zWnkMUy92DuS8Zk2T9QgB/c1jcHTya4aCd2Zwzl39+E=; b=dX76AwIiZU4UjFCuRZoy1O7JwdWXB/G0xrf26v314Xt248OAHBh95O0qZSWz4zU0Fy rzH77GKEwPfryQl8mGFyXpdDeNb2pSkLgvYS8jrfU5DLqwjQCVUkMd70c0XXzdsUKs9o iYFwPR4TmjBAlJnxnyGruu8cvRE4UplbV5/ssO8wGHhH8rsOIvk9kCEyG5I4DSuFqFJa E6HsG0/GlBM2j+ofcjCQuhsdpbV8Au6ORTsQSbwfVU27daXNvcHYVfenLiR4GHCjK+3D LacM5ymxNDC5KP24YuoL+D5KXzELdWAcjhctZd2mNhUi7KFTemkUMO9kt7HXaDAXime4 NmIw== X-Gm-Message-State: AOAM532EWDxs3R0i92hfomaPxgPJUG/Ra76N0IBmZ/rbfetFsLiCDDUI IBYuskBC1BF6vEniyrIVoynyyw== X-Google-Smtp-Source: ABdhPJzEGXlw+2JE+4RENIC0snVoCVLNamHQJJe/8xIXSaWN3T630iyEgeGMHG/lZq2tm72YMah4Lg== X-Received: by 2002:a1c:4d12:: with SMTP id o18mr2452140wmh.177.1601594789632; Thu, 01 Oct 2020 16:26:29 -0700 (PDT) Return-Path: Received: from vanye ([2001:470:1f09:12f0:b26e:bfff:fea9:f1b8]) by smtp.gmail.com with ESMTPSA id y6sm11104083wrt.80.2020.10.01.16.26.28 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 01 Oct 2020 16:26:29 -0700 (PDT) Date: Fri, 2 Oct 2020 00:26:21 +0100 From: "Leif Lindholm" To: "Desimone, Nathaniel L" Cc: "rfc@edk2.groups.io" , "devel@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: [edk2-rfc] [RFC] Request to move MinPlatformPkg out of the Intel folder Message-ID: <20201001232621.GL5623@vanye> References: <20201001115235.GI5623@vanye> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit On Thu, Oct 01, 2020 at 23:20:01 +0000, Desimone, Nathaniel L wrote: > > 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? > > Yeah agreed, given that the feedback on this RFC has been pretty > positive thus far, I'm planning on making a patch series for > this. I'll make these changes part of said patch series. Sounds good. > > If I was to bikeshed, I'd probably suggest > > Platform/TianoCore/MinPlatformPkg. Oops, guess I did. > > Seems a little superfluous to me... isn't edk2-platforms part of TianoCore by definition? 😊 It is. It's just the pattern for everything else in edk2-platforms (apart from OptionRomPkg, which was just transplanted from edk2) exists in the form of
/$VENDOR/. Platform/MinPlatformPkg would be the only bare package in Platform/. > > 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? > > Yup, sounds like good next steps to me. I've filed the BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2992 Top man. Best Regards, Leif