Re: [edk2] [Patch] NetworkPkg: Add RAM disk boot support to HTTP Boot driver.

2016-04-06 Thread Ye, Ting
Siyuan, Some minor comments to the patch: 1. Add perform parameter check for ImageType in HttpBootGetFileFromCache(). 2. Update function description of HttpBootStart/HttpBootLoadFile for EFI_INVALID_PARAMETER since we now add further check. 3. Add EFI_INVALID_PARAMETER to function description

Re: [edk2] [Patch] NetworkPkg: Add RAM disk boot support to HTTP Boot driver.

2016-04-01 Thread El-Haj-Mahmoud, Samer
ch 31, 2016 7:46 PM To: El-Haj-Mahmoud, Samer <samer.el-haj-mahm...@hpe.com>; edk2-devel@lists.01.org Cc: Ye, Ting <ting...@intel.com>; Wu, Jiaxin <jiaxin...@intel.com> Subject: RE: [edk2] [Patch] NetworkPkg: Add RAM disk boot support to HTTP Boot driver. Hi, Samer Does th

Re: [edk2] [Patch] NetworkPkg: Add RAM disk boot support to HTTP Boot driver.

2016-03-31 Thread Fu, Siyuan
intel.com>; El-Haj- > Mahmoud, Samer <samer.el-haj-mahm...@hpe.com> > Subject: RE: [edk2] [Patch] NetworkPkg: Add RAM disk boot support to HTTP > Boot driver. > > Since this is industry standard, it should be moved to > MdePkg/IndustryStandard/Http11.h, n

Re: [edk2] [Patch] NetworkPkg: Add RAM disk boot support to HTTP Boot driver.

2016-03-31 Thread El-Haj-Mahmoud, Samer
Since this is industry standard, it should be moved to MdePkg/IndustryStandard/Http11.h, next to the similarly defined content types, like HTTP_CONTENT_TYPE_APP_JSON, HTTP_CONTENT_TYPE_APP_OCTET_STREAM, HTTP_CONTENT_TYPE_IMAGE_JPEG, etc... // +// Provisional Standard Media Types defined in //