Hi Ard, Leif, śr., 22 wrz 2021 o 14:20 Ard Biesheuvel <a...@kernel.org> napisał(a): > > On Wed, 22 Sept 2021 at 14:14, Leif Lindholm <l...@nuviainc.com> wrote: > > > > On Wed, Sep 22, 2021 at 13:24:07 +0200, Ard Biesheuvel wrote: > > > On Tue, 21 Sept 2021 at 13:02, Marcin Wojtas <m...@semihalf.com> wrote: > > > > > > > > Hi, > > > > > > > > This short patchset adds README files for the platforms > > > > based on Marvell SoC's, which have already been > > > > SystemReady ES certified. It also bumps the firmware > > > > revision after the PCI / IcuLib rework and addition of > > > > the CN913x CEx7 Evaluation Board support. > > > > > > > > The patches are available on a public branch: > > > > https://github.com/semihalf-wojtas-marcin/edk2-platforms/commits/marvell-howtos-upstream-r20210921 > > > > > > > > > > Thanks > > > > > > Pushed as 64394fd2b48f..d1d4dd2c39e7 > > > > Ugh, wish I'd taken the time to respond yesterday. > > > > This is more proliferation of "include all very specific build > > steps for a very specific target and build environment" documentation. > > > > I don't see this as an improvement. > > > > Platform docs should document what one needs to know specific to the > > platform. If a vendor wishes to provide full recipes, they should host > > that on a wiki somewhere. > > > > Fair point. > > Marcin, when you find the time, could you please do a pass over these > files with Leif's critique in mind?
For all 3 platforms, how about the following update: - extend the "Summary" section with supported features or interfaces (or add an extra heading for that). - "Build" section -> add link to external wiki/.md file - "ARM System Ready certification." - leave intact Please let me know if that will work for you. Best regards1, Marcin -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#80963): https://edk2.groups.io/g/devel/message/80963 Mute This Topic: https://groups.io/mt/85762836/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-