Re: [edk2] Is this a right place to discuss EDK2's USB IF implementation?

2016-08-16 Thread Andrew Fish
> On Aug 16, 2016, at 11:42 PM, Yosuke Katayama1 wrote: > > Hello, > > We have a USB vendor who has a different understanding on USB IF > specification from the EDK2's understanding(implementation). > > This mismatch was causing a trouble on our products in the past. There is no > issue with

[edk2] Is this a right place to discuss EDK2's USB IF implementation?

2016-08-16 Thread Yosuke Katayama1
Hello, We have a USB vendor who has a different understanding on USB IF specification from the EDK2's understanding(implementation). This mismatch was causing a trouble on our products in the past. There is no issue with our ongoing products as we applied a workaround to our BIOS code but we w

Re: [edk2] USB 3.1 Support in UEFI

2016-08-16 Thread GN Keshava
Thanks, Feng! J On Wed, 17 Aug 2016 at 11:25 Tian, Feng wrote: > DP support depends on the graphic card you are using. You need consult IHV > rather than usJ > > > > Thanks > > Feng > > > > *From:* GN Keshava [mailto:keshava...@gmail.com] > *Sent:* Tuesday, August 16, 2016 12:37 PM > *To:* Tian,

Re: [edk2] USB 3.1 Support in UEFI

2016-08-16 Thread Tian, Feng
DP support depends on the graphic card you are using. You need consult IHV rather than us☺ Thanks Feng From: GN Keshava [mailto:keshava...@gmail.com] Sent: Tuesday, August 16, 2016 12:37 PM To: Tian, Feng ; edk2-devel@lists.01.org Subject: Re: [edk2] USB 3.1 Support in UEFI Hello Feng, Thank y

Re: [edk2] DHCP Automatic Configure at Driver Connect

2016-08-16 Thread Wu, Jiaxin
Eugene, > > I mean I didn't know whether the *un-configured* status you want > > contain *no policy setting * or not. But it doesn't matter now, from > > your statement here, I know you don't care the policy setting. > > Correct the static vs dhcp is irrelevant - it's the fact that the IP layer

Re: [edk2] [PATCH] UefiCpuPkg: MTRR_PHYSMASK.Valid should be one bit instead of 8 bits

2016-08-16 Thread Tian, Feng
Reviewed-by: Feng Tian Thanks Feng -Original Message- From: Ni, Ruiyu Sent: Monday, August 15, 2016 4:41 PM To: edk2-devel@lists.01.org Cc: Tian, Feng Subject: [PATCH] UefiCpuPkg: MTRR_PHYSMASK.Valid should be one bit instead of 8 bits Contributed-under: TianoCore Contribution Agreem

[edk2] [PATCH] PcAtChipsetPkg AcpiTimerLib: Wait 363 ACPI timer counts to get TSC Freq

2016-08-16 Thread Star Zeng
Compute the number of ticks to wait to measure TSC frequency. Instead of (ACPI_TIMER_FREQUENCY / 1) = 357 and 357 * 1 = 357, use 363 * 9861 = 3579543 Hz which is within 2 Hz of ACPI_TIMER_FREQUENCY. 363 counts is a calibration time of 101.4 uS. The idea comes from Michael and Paolo. C

Re: [edk2] IP4 Config Troubles with DHCP

2016-08-16 Thread Wu, Jiaxin
Eugene, > > Yeah. When I was drafting the UDP APP to test the new fix, I found the > > same case you mentioned. We must issue another UDP Configure () to > > clean the previous state once the Ip4Mode.IsConfigured is TRUE. So, > > the above example is not accurate with my the current > > implement

[edk2] Do BSPs only have to be logical thread 0 ?

2016-08-16 Thread Shubha Ramani
In other words, suppose a BSP  is running in hyperthreaded ON mode and it has 2 threads per physical core. Can the BSP be either thread 0 or thread 1 ?Is this documented somewhere ? Thanks, Shubha Shubha D. ramanishubharam...@gmail.com shubharam...@yahoo.com __

Re: [edk2] IP4 Config Troubles with DHCP

2016-08-16 Thread Cohen, Eugene
Jiaxin, > Yeah. When I was drafting the UDP APP to test the new fix, I found the > same case you mentioned. We must issue another UDP Configure () to > clean the previous state once the Ip4Mode.IsConfigured is TRUE. So, > the above example is not accurate with my the current > implementation:(. B

Re: [edk2] DHCP Automatic Configure at Driver Connect

2016-08-16 Thread Cohen, Eugene
Jiaxin, > I mean I didn't know whether the *un-configured* status you want > contain *no policy setting * or not. But it doesn't matter now, from > your statement here, I know you don't care the policy setting. Correct the static vs dhcp is irrelevant - it's the fact that the IP layer comes up

Re: [edk2] DHCP Automatic Configure at Driver Connect

2016-08-16 Thread Cohen, Eugene
> Hi Eugene, > > I think we need fully understand your usage before analyzing the > problem. Could you please explain more details? You mentioned you > only wanted to enable the network interface when directed by an > application. If so, is it possible that you don't connect your network > devic

Re: [edk2] [staging/HTTPS-TLS][PATCH 0/2] Readme.MD and TlsAuthConfigDxe update

2016-08-16 Thread Samer El Haj Mahmoud
Series reviewed-by: Samer El-Haj-Mahmoud Samer El-Haj-Mahmoud SESM - OS / SW Architect Systems Management Development, Data Center Group Lenovo United States +1.919.908.5833 +1.512.659.1523 smahm...@lenovo.com   Lenovo.com /us  Twitter | Facebook | Instagram | Blogs | Forums -Origin

Re: [edk2] Setting BuildOptions by module type does not seem to work

2016-08-16 Thread Gao, Liming
Andrew: I got your point. Yes. The precedence rules can be updated to reflect their order. Thanks Liming From: af...@apple.com [mailto:af...@apple.com] Sent: Wednesday, August 17, 2016 12:28 AM To: Gao, Liming Cc: Kurt Kennett ; edk2-devel Subject: Re: [edk2] Setting BuildOptions by module t

Re: [edk2] [MdeModulePkg][PeiCore] I seemed to have crashed the PEI Core by grabbing memory from PeiTemporaryRamBase?

2016-08-16 Thread Gao, Liming
Andrew: PI spec has not defined such information. But, PPI implementation and PeiCore needs to align new heap and stack layout. The full PPI should include new heap base and new stack base. Current PPI has only one Base. Then, PPI implementation needs mach PeiCore implementation. Thanks Limin

Re: [edk2] [staging/HTTPS-TLS][PATCH 0/2] Readme.MD and TlsAuthConfigDxe update

2016-08-16 Thread Palmer, Thomas
Reviewed by Thomas Palmer -Original Message- From: Jiaxin Wu [mailto:jiaxin...@intel.com] Sent: Tuesday, August 16, 2016 2:20 AM To: edk2-devel@lists.01.org Cc: Palmer, Thomas ; Long Qin ; Ye Ting ; Fu Siyuan Subject: [staging/HTTPS-TLS][PATCH 0/2] Readme.MD and TlsAuthConfigDxe update

Re: [edk2] Setting BuildOptions by module type does not seem to work

2016-08-16 Thread Andrew Fish
> On Aug 16, 2016, at 9:21 AM, Gao, Liming wrote: > > Andrew: > arch value may be common. 3.3.1 Common Definitions, ::= {"IA32"} > {"X64"} {"IPF"} {"EBC"} {} {"common"}. > I agree [BuildOptions.AARCH64.common.DXE_RUNTIME_DRIVER should be supported. > We will look it. > I agree but precede

Re: [edk2] Setting BuildOptions by module type does not seem to work

2016-08-16 Thread Gao, Liming
Andrew: arch value may be common. 3.3.1 Common Definitions, ::= {"IA32"} {"X64"} {"IPF"} {"EBC"} {} {"common"}. I agree [BuildOptions.AARCH64.common.DXE_RUNTIME_DRIVER should be supported. We will look it. Thanks Liming From: af...@apple.com [mailto:af...@apple.com] Sent: Tuesday, August 16

Re: [edk2] Setting BuildOptions by module type does not seem to work

2016-08-16 Thread Andrew Fish
> On Aug 16, 2016, at 8:22 AM, Kurt Kennett wrote: > > Thanks for your help, Andre. I appreciate it. ☺ > > [BuildOptions.AARCH64.common.DXE_RUNTIME_DRIVER] > Does not work. > Can you file a bugzilla report on the tools, and another on the spec to mention BuildOptions.AARCH64.common.DXE_RUNTI

Re: [edk2] Setting BuildOptions by module type does not seem to work

2016-08-16 Thread Kurt Kennett
Thanks for your help, Andre. I appreciate it. ☺ [BuildOptions.AARCH64.common.DXE_RUNTIME_DRIVER] Does not work. [BuildOptions.AARCH64.common.EDKII.DXE_RUNTIME_DRIVER] Does not work. [BuildOptions.common.EDKII.DXE_RUNTIME_DRIVER] Works. K2 From: af...@apple.com [mailto:af...@apple.com] Sent: T

Re: [edk2] Setting BuildOptions by module type does not seem to work

2016-08-16 Thread Andrew Fish
> On Aug 16, 2016, at 7:49 AM, Kurt Kennett wrote: > > Hi Liming. I’m a bit confused by your email. > > I my original email below, I reference directly: > > >> DSC spec (January 2016 1.26) says I can do this: > >> > >> (Section 3.6 pp 76) > > In this section it says: > > … > Build optio

Re: [edk2] Setting BuildOptions by module type does not seem to work

2016-08-16 Thread Kurt Kennett
Hi Liming. I'm a bit confused by your email. I my original email below, I reference directly: >> DSC spec (January 2016 1.26) says I can do this: >> >> (Section 3.6 pp 76) In this section it says: ... Build options priority (appended from lowest to highest and/or highest replacement) is: * Hi

Re: [edk2] [Patch] MdeModulePkg: Fix potential failure if UseDefaultAddress configured

2016-08-16 Thread Ye, Ting
Looks good to me. Reviewed-by: Ye Ting -Original Message- From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jiaxin Wu Sent: Monday, August 15, 2016 10:47 AM To: edk2-devel@lists.01.org Cc: Ye, Ting ; Fu, Siyuan ; Cohen Eugene Subject: [edk2] [Patch] MdeModulePkg: F

Re: [edk2] [staging/HTTPS-TLS][PATCH 0/2] Readme.MD and TlsAuthConfigDxe update

2016-08-16 Thread Ye, Ting
Series reviewed by: Ye Ting -Original Message- From: Wu, Jiaxin Sent: Tuesday, August 16, 2016 3:20 PM To: edk2-devel@lists.01.org Cc: Palmer Thomas ; Long, Qin ; Ye, Ting ; Fu, Siyuan Subject: [staging/HTTPS-TLS][PATCH 0/2] Readme.MD and TlsAuthConfigDxe update Cc: Palmer Thomas Cc

Re: [edk2] [PATCH] ShellPkg/Ls: Handle the case when SearchString is NULL

2016-08-16 Thread Wu, Hao A
Reviewed-by: Hao Wu Best Regards, Hao Wu > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Ruiyu > Ni > Sent: Tuesday, August 16, 2016 3:33 PM > To: edk2-devel@lists.01.org > Cc: Wu, Hao A > Subject: [edk2] [PATCH] ShellPkg/Ls: Handle the case

[edk2] [PATCH] ShellPkg/Ls: Handle the case when SearchString is NULL

2016-08-16 Thread Ruiyu Ni
Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Ruiyu Ni Cc: Hao Wu --- ShellPkg/Library/UefiShellLevel2CommandsLib/Ls.c | 5 + 1 file changed, 5 insertions(+) diff --git a/ShellPkg/Library/UefiShellLevel2CommandsLib/Ls.c b/ShellPkg/Library/UefiShellLevel2CommandsLib

[edk2] [staging/HTTPS-TLS][PATCH 0/2] Readme.MD and TlsAuthConfigDxe update

2016-08-16 Thread Jiaxin Wu
Cc: Palmer Thomas Cc: Long Qin Cc: Ye Ting Cc: Fu Siyuan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiaxin Wu Jiaxin Wu (2): Readme.MD: Clarify the feature support scope NetworkPkg/TlsAuthConfigDxe: TlsCaCertificate variable attribute update NetworkPkg/TlsAut

[edk2] [staging/HTTPS-TLS][PATCH 1/2] Readme.MD: Clarify the feature support scope

2016-08-16 Thread Jiaxin Wu
Cc: Palmer Thomas Cc: Long Qin Cc: Ye Ting Cc: Fu Siyuan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiaxin Wu --- Readme.MD | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/Readme.MD b/Readme.MD index 2411042..389935e 100644 --- a/Readme.M

[edk2] [staging/HTTPS-TLS][PATCH 2/2] NetworkPkg/TlsAuthConfigDxe: TlsCaCertificate variable attribute update

2016-08-16 Thread Jiaxin Wu
Remove the RT attribute for TlsCaCertificate variable protecting. If so, we can treat this variable as security in certain case. Cc: Palmer Thomas Cc: Long Qin Cc: Ye Ting Cc: Fu Siyuan Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiaxin Wu --- NetworkPkg/TlsAuthCon

Re: [edk2] [Patch] MdePkg: Fix guid conflict.

2016-08-16 Thread Jordan Justen
Some notes for future reference, since this is already committed. I think subject should have been something like: MdePkg/ImageDecoder: Fix GUID conflict with PCI Root Bridge Protocol When browsing commits with just the subject line, you should have a good idea of what the commit is about. (For