On Thu, Aug 08, 2019 at 02:30:48PM -0700, Nate DeSimone wrote: > Updated Platform/Intel/Readme.md to point to master branch on > edk2-non-osi instead of devel-MinPlatform > > Updated edk2-platforms/Readme.md to point to Platform/Intel/Readme.md > for build instructions on Intel based Minimum Platforms > > Added a link to the EDK II Minimum Platform Draft Specification > to Platform/Intel/Readme.md > > Cc: Andrew Fish <af...@apple.com> > Cc: Laszlo Ersek <ler...@redhat.com> > Cc: Leif Lindholm <leif.lindh...@linaro.org> > Cc: Michael D Kinney <michael.d.kin...@intel.com> > Cc: Michael Kubacki <michael.a.kuba...@intel.com> > Cc: Ankit Sinha <ankit.si...@intel.com> > Cc: Chasel Chiu <chasel.c...@intel.com> > Signed-off-by: Nate DeSimone <nathaniel.l.desim...@intel.com> > --- > Platform/Intel/Readme.md | 6 ++++-- > Readme.md | 8 +++++++- > 2 files changed, 11 insertions(+), 3 deletions(-) > > diff --git a/Platform/Intel/Readme.md b/Platform/Intel/Readme.md > index 0a0e8a8c80..00f42985a2 100644 > --- a/Platform/Intel/Readme.md > +++ b/Platform/Intel/Readme.md > @@ -1,7 +1,9 @@ > # **EDK II Minimum Platform Firmware for Intel(R) Platforms** > > The Minimum Platform is a software architecture that guides uniform delivery > of Intel platforms enabling firmware > -solutions for basic boot functionality with extensibility built-in. > +solutions for basic boot functionality with extensibility built-in. Please > see the > +[EDK II Minimum Platform Draft > Specification](https://edk2-docs.gitbooks.io/edk-ii-minimum-platform-specification/) > +for more details. > > Package maintainers for the Minimum Platform projects are listed in > Maintainers.txt. > > @@ -98,7 +100,7 @@ return back to the minimum platform caller. > * ``git clone https://github.com/tianocore/edk2-platforms.git`` > > * edk2-non-osi repository > - * ``git clone https://github.com/tianocore/edk2-non-osi.git -b > devel-MinPlatform`` > + * ``git clone https://github.com/tianocore/edk2-non-osi.git``
Really happy to see this change. > * FSP repository > * ``git clone https://github.com/IntelFsp/FSP.git`` > diff --git a/Readme.md b/Readme.md > index 63e59f60b4..28badd6b4b 100644 > --- a/Readme.md > +++ b/Readme.md > @@ -34,6 +34,9 @@ edk2-devel mailing list with a subject prefix of > `[platforms]`. See > on how to do this successfully. > > # How to build (Linux Environment) > +Note: Build instructions for Intel based Minimum Platforms differ from these > +instructions and are available at > +[`edk2-platforms/Platform/Intel/README.md`](https://github.com/tianocore/edk2-platforms/blob/master/Platform/Intel/Readme.md) Less happy to see this. I am well aware that there are platform-specific steps required to achieve the final firmware image for some systems. But we need to work to address that. There have been discussions about implementing support for hooking pre/post-build python scripts into the .dsc format - has there been any progress on that? For this patch, I would prefer if instead of adding these notes we changed the ## [Intel](Platform/Intel/Readme.md) ### Minimum Platforms * [Clevo](Platform/Intel/ClevoOpenBoardPkg) * [Kaby Lake](Platform/Intel/KabylakeOpenBoardPkg) * [Purley](Platform/Intel/PurleyOpenBoardPkg) links to all point to (Platform/Intel) instead. I realise that my intent with these platform links was unclear - for all of the ARM platforms, these point to directories that have their own Readme.md (meaning on github you see the documentation *and* the files when you click the link). Arguably, it would make sense to move all of the MinPlatform ... platforms ... and the Readme.md to a separate subdirectory and link to that directory. Not everything under Platform/Intel is a MinPlatform platform - soon there'll even be an ARM one. Best Regards, Leif -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#45263): https://edk2.groups.io/g/devel/message/45263 Mute This Topic: https://groups.io/mt/32803353/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
