[edk2-devel] [edk2-platforms] [PATCH] Platform/Intel: Update Readme.md to point to edk2-non-osi master branch
Leif Lindholm
leif.lindholm at linaro.org
Fri Aug 9 10:32:32 UTC 2019
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 <afish at apple.com>
> Cc: Laszlo Ersek <lersek at redhat.com>
> Cc: Leif Lindholm <leif.lindholm at linaro.org>
> Cc: Michael D Kinney <michael.d.kinney at intel.com>
> Cc: Michael Kubacki <michael.a.kubacki at intel.com>
> Cc: Ankit Sinha <ankit.sinha at intel.com>
> Cc: Chasel Chiu <chasel.chiu at intel.com>
> Signed-off-by: Nate DeSimone <nathaniel.l.desimone at 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/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list