Re: [Patch v3 0/3] Add DisplayUpdateProgressLib for capsules

["Kinney, Michael D" <michael.d.kinney@intel.com>]

Ard,

I broke the series up into multiple series.

I can send out the 4th one this morning so
you can see the one that uses it.  It is a
repeat of content that was shared in a single
series earlier.  I deferred it so we can get
all the platform updates done first so there
are no build breaks.

Mike

> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-
> bounces@lists.01.org] On Behalf Of Ard Biesheuvel
> Sent: Tuesday, May 29, 2018 6:08 AM
> To: Gao, Liming <liming.gao@intel.com>
> Cc: Dong, Eric <eric.dong@intel.com>; edk2-
> devel@lists.01.org; Yao, Jiewen <jiewen.yao@intel.com>;
> Kinney, Michael D <michael.d.kinney@intel.com>; Zeng,
> Star <star.zeng@intel.com>; Wei, David
> <david.wei@intel.com>
> Subject: Re: [edk2] [Patch v3 0/3] Add
> DisplayUpdateProgressLib for capsules
> 
> Hello Liming,
> 
> On 29 May 2018 at 15:05, Gao, Liming
> <liming.gao@intel.com> wrote:
> > Do you try to find this one [edk2] [edk2-platforms
> Patch v4 0/6] Add DisplayUpdateProgressLib to platforms?
> >
> 
> No, I am trying to locate the patch that actually
> invokes the new
> PerformFlashWriteWithProgress() function when processing
> the capsule.
> 
> >> -----Original Message-----
> >> From: edk2-devel [mailto:edk2-devel-
> bounces@lists.01.org] On Behalf Of Ard Biesheuvel
> >> Sent: Tuesday, May 29, 2018 8:43 PM
> >> To: Kinney, Michael D <michael.d.kinney@intel.com>
> >> Cc: edk2-devel@lists.01.org; Yao, Jiewen
> <jiewen.yao@intel.com>; Dong, Eric
> <eric.dong@intel.com>; Zeng, Star
> >> <star.zeng@intel.com>; Wei, David
> <david.wei@intel.com>
> >> Subject: Re: [edk2] [Patch v3 0/3] Add
> DisplayUpdateProgressLib for capsules
> >>
> >> On 29 May 2018 at 11:51, Ard Biesheuvel
> <ard.biesheuvel@linaro.org> wrote:
> >> > On 29 May 2018 at 11:51, Ard Biesheuvel
> <ard.biesheuvel@linaro.org> wrote:
> >> >> On 25 May 2018 at 08:15, Michael D Kinney
> <michael.d.kinney@intel.com> wrote:
> >> >>>
> https://bugzilla.tianocore.org/show_bug.cgi?id=801
> >> >>>
> >> >>> Based on content from:
> >> >>>
> >> >>>
> >>
> https://github.com/Microsoft/MS_UEFI/blob/share/MsCapsul
> eSupport/MsCapsuleUpdatePkg/Include/Library/DisplayUpdat
> eProgress
> >> Lib.h
> >> >>>
> >>
> https://github.com/Microsoft/MS_UEFI/tree/share/MsCapsul
> eSupport/MsCapsuleUpdatePkg/Library/DisplayUpdateProgres
> sGraphics
> >> Lib
> >> >>>
> >>
> https://github.com/Microsoft/MS_UEFI/tree/share/MsCapsul
> eSupport/MsCapsuleUpdatePkg/Library/DisplayUpdateProgres
> sTextLib
> >> >>>
> >> >>> Updates for V3
> >> >>> ==============
> >> >>> * Add Version field to
> EDKII_FIRMWARE_MANAGEMENT_PROGRESS_PROTOCOL
> >> >>> * Break up patch series into 4 smaller patch
> series to handle dependencies
> >> >>>   between the edk2 repository and the edk2-
> platforms repository.
> >> >>>   + Patch series for edk2 repo that adds
> DisplayUpdateProgressLib class and
> >> >>>     instances.  Defines the
> EDKII_FIRMWARE_MANAGEMENT_PROGRESS_PROTOCOL.
> >> >>>     Adds PerformFlashWriteWithProgress() API to
> the PlatformFlashAccessLib.
> >> >>>   + Patch series for platforms in edk2-platforms
> that use capsules to add the
> >> >>>     DisplayUpdateProgressLib mapping to the DSC
> files and add the
> >> >>>     PerformFlashWriteWithProgress() API
> implementation to the
> >> >>>     PlatformFlashAccessLib implementations.
> >> >>>   + Patch series for platforms in edk2 that use
> capsules to add the
> >> >>>     DisplayUpdateProgressLib mapping to the DSC
> files and add the
> >> >>>     PerformFlashWriteWithProgress() API
> implementation to the
> >> >>>     PlatformFlashAccessLib implementations.
> >> >>>   + Patch for edk2 that adds the use of the
> DisplayUpateProgressLib and the
> >> >>>     PerformFlashWriteWithProgress() API .
> >> >>>
> >> >>> Updates for V2
> >> >>> ==============
> >> >>> * Change DisplayUpdateProgressGraphicsLib to
> DisplayUpdateProgressLibGraphics
> >> >>> * Change DisplayUpdateProgressTextLib to
> DisplayUpdateProgressLibText
> >> >>> * Clarify that color in Firmware Management
> Progress Protocol is the foreground color
> >> >>> * Add missing parameters to
> PerformFlashWriteWithProgress() function header.
> >> >>> * Update PerformFlashWriteWithProgress() function
> header describing the use of
> >> >>>   the start and end percentage values.
> >> >>> * Update QuarkPlatformPkg
> PerformFlashWriteWithProgress() to call Progress() for
> >> >>>   the end precentage.
> >> >>> * Update Vlv2Tbl2DevicePkg
> PerformFlashWriteWithProgress() to call Progress()
> >> >>>   for the end precentage.
> >> >>>
> >> >>> Add DisplayUpdateProgressLib class along
> implementations for both graphical
> >> >>> (Graphics Output Protocol based) and text (Simple
> Text Output Protocol based)
> >> >>> consoles.  Also add the EDK II Firmware
> Management Progress Protocol that is an
> >> >>> optional protocol that provides the progress bar
> color and a watchdog timeout
> >> >>> value thaty can be used when a firmware image is
> updated in a firmware device.
> >> >>>
> >> >>> * Add progress support to DxeCapsuleLibFmp
> >> >>> * Add progress support to SystemFirmwareUpdateDxe
> >> >>> * Add progress support to PlatformFlashAccessLib
> class and instances.
> >> >>> * Reduce Print() calls during a firmware update.
> >> >>>
> >> >>> Cc: Sean Brogan <sean.brogan@microsoft.com>
> >> >>> Cc: Star Zeng <star.zeng@intel.com>
> >> >>> Cc: Eric Dong <eric.dong@intel.com>
> >> >>> Cc: Jiewen Yao <jiewen.yao@intel.com>
> >> >>> Cc: David Wei <david.wei@intel.com>
> >> >>> Cc: Mang Guo <mang.guo@intel.com>
> >> >>> Cc: Kelly Steele <kelly.steele@intel.com>
> >> >>>
> >> >>> Signed-off-by: Michael D Kinney
> <michael.d.kinney@intel.com>
> >> >>> Contributed-under: TianoCore Contribution
> Agreement 1.1
> >> >>>
> >> >>> Michael D Kinney (3):
> >> >>>   MdeModulePkg: Add DisplayUpdateProgressLib
> class
> >> >>>   MdeModulePkg: Add DisplayUpdateProgressLib
> instances
> >> >>>   SignedCapsulePkg/PlatformFlashAccessLib: Add
> progress API
> >> >>>
> >> >>
> >> >> This message says 0/3, but there appear to be 4
> patches in the series???
> >> >>
> >> >
> >> > Never mind, the threading in my email client
> confused me.
> >>
> >> OK I am still confused
> >>
> >> Where is this patch:
> >>
> >>   + Patch for edk2 that adds the use of the
> DisplayUpateProgressLib and the
> >>     PerformFlashWriteWithProgress() API .
> >>
> >> ??
> >> _______________________________________________
> >> edk2-devel mailing list
> >> edk2-devel@lists.01.org
> >> https://lists.01.org/mailman/listinfo/edk2-devel
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel