Reviewed-by: Jordan Justen <jordan.l.jus...@intel.com>

On 2015-04-15 19:40:45, Qiu, Shumin wrote:
>    Hot,
> 
>    Yes, you are right. Patch updated.
> 
>     
> 
>    Thanks,
> 
>    Shumin
> 
>     
> 
>    From: Tian, Hot [mailto:hot.t...@intel.com]
>    Sent: Thursday, April 16, 2015 10:31 AM
>    To: edk2-devel@lists.sourceforge.net; Fan, Jeff
>    Subject: Re: [edk2] [PATCH]IntelFrameworkPkg: Fix typo in LegacyBios.h.
> 
>     
> 
>    This is a non ASCII char. From CSM 0.98 spec, it should be UmaAddress - 1.
> 
>     
> 
>    UmaAddress
> 
>    Start Address of Upper Memory Area (UMA) to be set as Read/Write. If
>    UmaAddress is a valid address in the shadow RAM, it also indicates that
>    the region from 0xC0000 to (UmaAddress - 1) can be used for Option ROM.
> 
>     
> 
>    Thanks,
> 
>    Hot
> 
>     
> 
>    From: Qiu, Shumin [mailto:shumin....@intel.com]
>    Sent: Thursday, April 16, 2015 10:21
>    To: Fan, Jeff
>    Cc: edk2-devel@lists.sourceforge.net
>    Subject: [edk2] [PATCH]IntelFrameworkPkg: Fix typo in LegacyBios.h.
> 
>     
> 
>    Hi Jeff,
> 
>    Could you help review the patch? It fixed the typo in LegacyBios.h
> 
>     
> 
>    Contributed-under: TianoCore Contribution Agreement 1.0
> 
>    Signed-off-by: Qiu Shumin <shumin....@intel.com>
> 
>     
> 
>    Thanks,
> 
>    Shumin

------------------------------------------------------------------------------
BPM Camp - Free Virtual Workshop May 6th at 10am PDT/1PM EDT
Develop your own process in accordance with the BPMN 2 standard
Learn Process modeling best practices with Bonita BPM through live exercises
http://www.bonitasoft.com/be-part-of-it/events/bpm-camp-virtual- event?utm_
source=Sourceforge_BPM_Camp_5_6_15&utm_medium=email&utm_campaign=VA_SF
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to