Re: [PATCH v2 staging][BaseToolsOpt 0/4] Enable multiple driver combination

[Laszlo Ersek <lersek@redhat.com>]

On 06/15/17 11:46, Liming Gao wrote:
> In V2, 
> 1) Add SamplePkg to include the example to show the combined drivers in DSC.
> 2) Update Read.MD to describe the Depex will be AND together in the combined driver.
> 
> Combine more drivers into the single one can reduce the image size and 
> compile link time. This patch adds this support in BaseTools.
> 
> Liming Gao (4):
>   BaseTools: Merge multiple drivers into one for size and link
>     performance
>   SamplePkg: Add it to show the edk2 usage case.
>   SamplePkg: Combine two drivers into one
>   Update Readme.MD to include multiple driver combination.
> 
>  BaseTools/Source/Python/AutoGen/GenC.py |  24 ++-
>  Readme.MD                               |   3 +
>  SamplePkg/SamplePkg.dec                 |  21 +++
>  SamplePkg/SamplePkg.dsc                 | 264 ++++++++++++++++++++++++++++++++
>  4 files changed, 307 insertions(+), 5 deletions(-)
>  create mode 100644 SamplePkg/SamplePkg.dec
>  create mode 100644 SamplePkg/SamplePkg.dsc
> 

Thank you Liming, this looks good to me.

Acked-by: Laszlo Ersek <lersek@redhat.com>

Cheers
Laszlo