Hi, Olivier
You need tell me why you need expose this parameter? What's your intention &
use case?
Thanks
Feng
From: Olivier Martin [mailto:[email protected]]
Sent: Tuesday, February 11, 2014 22:50
To: [email protected]; Tian, Feng
Subject: RE: [edk2] [PATCH] MdeModulePkg/DiskIo: Introduced
'PcdDiskIoDataBufferBlockNum'
Thanks Mike for the feedback, I attached the new version.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Olivier Martin
<[email protected]<mailto:[email protected]>>
From: Kinney, Michael D [mailto:[email protected]]
Sent: 10 February 2014 21:22
To: [email protected]<mailto:[email protected]>;
Tian, Feng
Subject: Re: [edk2] [PATCH] MdeModulePkg/DiskIo: Introduced
'PcdDiskIoDataBufferBlockNum'
Olivier,
It would be better to remove the #define all together and use PcdGet32()
instead of the define name. This will allow the PCD to be more flexible and
support both FixedAtBuild and PatchableInModule types.
Thanks,
Mike
From: Olivier Martin [mailto:[email protected]]
Sent: Monday, February 10, 2014 9:18 AM
To: Tian, Feng
Cc: [email protected]<mailto:[email protected]>
Subject: [edk2] [PATCH] MdeModulePkg/DiskIo: Introduced
'PcdDiskIoDataBufferBlockNum'
Dear MdeModulePkg maintainer,
Please find the attached patch that introduced 'PcdDiskIoDataBufferBlockNum'.
PcdDiskIoDataBufferBlockNum replaced the hardcoded value into the Disk I/O
driver.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Olivier Martin
<[email protected]<mailto:[email protected]>>
Regards,
Olivier
------------------------------------------------------------------------------
Android apps run on BlackBerry 10
Introducing the new BlackBerry 10.2.1 Runtime for Android apps.
Now with support for Jelly Bean, Bluetooth, Mapview and more.
Get your Android app in front of a whole new audience. Start now.
http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-devel
