Jordan, The new library only exposes two APIs: FrameBufferBltConfigure and FrameBufferBlt. I like the new design very much! It's very clean now!
I also like the FrameBufferBltConfigure() to require caller supply the buffer so that we do not need to have an additional API like FrameBufferFreeConfigure(). I also like the idea to merge all the Blt operation to a single FrameBufferBlt(). I also like the idea to remove the GOP protocol version of BltLib. Two small comments: Can you change the parameter name for FrameBufferBltConfigure() from Buffer/BufferSize to Configure/ConfigureSize? And change the parameter name for FrameBufferBlt() from Config to Configure? So that we have consistent name for the frame buffer configuration block. Can you remove the EFIAPI for internal functions? such as FrameBufferBltVideoFill. Thanks, Ray -----Original Message----- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jordan Justen Sent: Wednesday, October 21, 2015 8:57 AM To: Ni, Ruiyu <ruiyu...@intel.com>; edk2-devel@lists.01.org Subject: Re: [edk2] [Patch 6/8] OptionRomPkg/OvmfPkg: Remove BltLib::BltConfigure API Ray, I've been thinking about this a bit. For the BltLib, I think you mainly like the FrameBufferBltLib library instance. So, how about just design the library interface just for this purpose, and call the interface FrameBufferBltLib rather than BltLib? What do you think of these 3 patches? https://github.com/jljusten/edk2/commits/framebuffer-blt-lib -Jordan On 2015-08-23 23:15:34, Ni, Ruiyu wrote: > Jordan, > The mail is too long so let me summarize in below: (I changed all > API name to remove "Lib") > 1. Have four or six BLT APIs? > four: BltVideoFill BltVideoToVideo BltVideoToBuffer BltBufferToVideo > six: BltVideoFill BltVideoToVideo BltVideoToBuffer BltBufferToVideo > BltVideoToBufferEx BltBufferToVideoEx > I prefer 4 because it aligns to the GOP protocol 4 BLT operations > and the parameters are also aligned. > You prefer 6 for easy calling. > > Laszlo and Mike, > any ideas? > > > > 2. BltConfigure/BltFreeConfiguration > 2.a. I agree to use the handle to hold the shift/mask. But I need to > see whether there is any concerns from Mike. > 2.b. I suggest to use VOID * instead of EFI_HANDLE. Let's wait for > Andrew/Mike's input. > > 3. VideoModeSetLib for more abstraction > I agree to either change the driver name as you suggested, either in > the other way to have a more generic GOP driver. Do you know any > graphics driver developers? Can you ask them if the more generic GOP > driver will satisfy their needs? > > Thanks, > Ray > > _______________________________________________ > 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 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel