Good catch. Please update the commit message. Then take my R-B. Thanks, Zhichao
From: Chao Li <lic...@loongson.cn> Sent: Wednesday, March 13, 2024 10:13 AM To: devel@edk2.groups.io; Shang, Qingyu <qingyu.sh...@intel.com> Cc: Liming Gao <gaolim...@byosoft.com.cn>; Gao, Zhichao <zhichao....@intel.com> Subject: Re: [edk2-devel] [PATCH 2/4] ShellPkg: Update the comments of ReadKeyStroke and ReadKeyStrokeEx Hi Qingyu, It may not be my domain, I think what you are trying to say is "Refer to UEFI spec 2.10 section 12.3.3", not 13.3.3, because the 13.3.3 is "Number and Location of System Partitions", see me comment below. I recommend fixing all of the patch commit messages in this series. The changes looks good to me and would recommend Liming and Zhichao to check again. Thanks, Chao On 2024/3/11 16:41, Qingyu wrote: Refer to Uefi spec 2.10 section 13.3.3, Add a new retval It should be section 12.3.3, not 13.3.3. EFI_UNSUPPORTED to EFI_SIMPLE_TEXT_INPUT_EX_PROTOCOL.ReadKeyStrokeEx and EFI_SIMPLE_TEXT_INPUT_PROTOCOL.ReadKeyStroke(). Cc: Liming Gao <gaolim...@byosoft.com.cn><mailto:gaolim...@byosoft.com.cn> Cc: Zhichao Gao <zhichao....@intel.com><mailto:zhichao....@intel.com> Cc: Chao Li <lic...@loongson.cn><mailto:lic...@loongson.cn> Signed-off-by: Qingyu <qingyu.sh...@intel.com><mailto:qingyu.sh...@intel.com> --- ShellPkg/Application/Shell/ConsoleWrappers.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/ShellPkg/Application/Shell/ConsoleWrappers.c b/ShellPkg/Application/Shell/ConsoleWrappers.c index eae11370e4e0..dbffae30cdf5 100644 --- a/ShellPkg/Application/Shell/ConsoleWrappers.c +++ b/ShellPkg/Application/Shell/ConsoleWrappers.c @@ -67,7 +67,9 @@ FileBasedSimpleTextInReset ( @param[in] This A pointer to the SimpleTextIn structure. @param[in, out] Key A pointer to the Key structure to fill. - @retval EFI_SUCCESS The read was successful. + @retval EFI_SUCCESS The read was successful. + @retval EFI_UNSUPPORTED The device does not support the ability to read + keystroke data. **/ EFI_STATUS EFIAPI -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#117082): https://edk2.groups.io/g/devel/message/117082 Mute This Topic: https://groups.io/mt/104886872/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
