Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-10-12 Thread Yao, Jiewen
Sounds good! Thanks to confirm that. From: Carsey, Jaben Sent: Thursday, October 13, 2016 5:04 AM To: Yao, Jiewen ; Ni, Ruiyu ; Rothman, Michael A ; Kinney, Michael D ; 'edk2-devel@lists.01.org' Cc: Tian, Feng ; Zhang, Chao B ; Gao, Liming ; Zeng, Star ; Carsey, Jaben Subject: RE: [PATCH 17

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-10-12 Thread Carsey, Jaben
I agree. We should move the 3 protocols from the UEFI Shell Specification from ShellPkg to MdePkg. I think that this is a non-backwards-compatible-change and will require a new GUID in ShellPkg. -Jaben From: Yao, Jiewen Sent: Tuesday, October 11, 2016 7:45 AM To: Ni, Ruiyu ; Carsey, Jaben ;

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-10-11 Thread Yao, Jiewen
vel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Yao, Jiewen Sent: Thursday, September 29, 2016 10:37 AM To: Kinney, Michael D mailto:michael.d.kin...@intel.com>>; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> Cc: Tian, Feng mailto:feng.t...@intel.com>>; Zhang, Cha

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-10-11 Thread Ni, Ruiyu
om>>; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> Cc: Tian, Feng mailto:feng.t...@intel.com>>; Zhang, Chao B mailto:chao.b.zh...@intel.com>>; Gao, Liming mailto:liming@intel.com>>; Zeng, Star mailto:star.z...@intel.com>> Subject: Re: [edk2] [PATC

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-10-11 Thread Ni, Ruiyu
is not described in the Shell spec. Maybe we can put the CapsuleApp in ShellPkg, then let it directly use ShellParameter protocol to get the ARGV. Copying Jaben. Regards, Ray From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Yao, Jiewen Sent: Thursday, September 29, 2016 10:37

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-10-11 Thread Yao, Jiewen
Yes, I think that is a very good idea. Shell spec defines EFI_SHELL_PROTOCOL, EFI_SHELL_DYNAMIC_COMMAND_PROTOCOL, EFI_SHELL_PARAMETERS_PROTOCOL. These should be treated as industry stand and we can move these 3 to MdePkg. SHELL_ENVIRONMENT_2_PROTOCOL and SHELL_INTERFACE_PROTOCOL should be still

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-10-03 Thread Carsey, Jaben
rsday, September 29, 2016 10:37 AM To: Kinney, Michael D mailto:michael.d.kin...@intel.com>>; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> Cc: Tian, Feng mailto:feng.t...@intel.com>>; Zhang, Chao B mailto:chao.b.zh...@intel.com>>; Gao, Liming mailto:liming@intel.co

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-10-03 Thread Rothman, Michael A
;; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> Cc: Tian, Feng mailto:feng.t...@intel.com>>; Zhang, Chao B mailto:chao.b.zh...@intel.com>>; Gao, Liming mailto:liming@intel.com>>; Zeng, Star mailto:star.z...@intel.com>> Subject: Re: [edk2] [PATCH 17/45] Md

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-09-30 Thread Carsey, Jaben
Of Yao, Jiewen Sent: Thursday, September 29, 2016 10:37 AM To: Kinney, Michael D mailto:michael.d.kin...@intel.com>>; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> Cc: Tian, Feng mailto:feng.t...@intel.com>>; Zhang, Chao B mailto:chao.b.zh...@intel.com>>; Gao, Lim

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-09-29 Thread Ni, Ruiyu
-devel@lists.01.org> Cc: Tian, Feng mailto:feng.t...@intel.com>>; Zhang, Chao B mailto:chao.b.zh...@intel.com>>; Gao, Liming mailto:liming....@intel.com>>; Zeng, Star mailto:star.z...@intel.com>> Subject: Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp app

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-09-29 Thread Carsey, Jaben
..@intel.com>>; Zhang, Chao B mailto:chao.b.zh...@intel.com>>; Gao, Liming mailto:liming@intel.com>>; Zeng, Star mailto:star.z...@intel.com>> Subject: Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application. Right. CapsuleOnDisk is not suppor

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-09-29 Thread Ni, Ruiyu
Jaben. Regards, Ray From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Yao, Jiewen Sent: Thursday, September 29, 2016 10:37 AM To: Kinney, Michael D ; edk2-devel@lists.01.org Cc: Tian, Feng ; Zhang, Chao B ; Gao, Liming ; Zeng, Star Subject: Re: [edk2] [PATCH 17/45

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-09-28 Thread Yao, Jiewen
Right. CapsuleOnDisk is not supported in BIOS yet. I will remove it in CapsuleApp. CapsuleApp just works as an agent to pass capsule to CapsuleDxeRuntime driver. So it is checked by CapsuleDxeRuntime, not CapsuleApp. Thank you Yao Jiewen From: Kinney, Michael D Sent: Thursday, September 29, 2016

Re: [edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-09-28 Thread Kinney, Michael D
Jiewen, Does the capsule on disk feature provided by this app work? I tried it with the QuarkPlatformPkg capsule and I can get The app to copy the capsule to an EFI System Partition, but The logic after that point looks incomplete. Also, if I clear the Persist and Reset flags on the capsule, the

[edk2] [PATCH 17/45] MdeModulePkg/CapsuleApp: Add CapsuleApp application.

2016-09-20 Thread Jiewen Yao
This CapsuleApp can help perform capsule update in UEFI shell environment. It can also dump capsule information, capsule status variable, ESRT and FMP. Cc: Feng Tian Cc: Star Zeng Cc: Michael D Kinney Cc: Liming Gao Cc: Chao Zhang Contributed-under: TianoCore Contribution Agreement 1.0 Signed