Re: [edk2-devel] [PATCH 2/4] ShellPkg: Update the comments of ReadKeyStroke and ReadKeyStrokeEx

["Chao Li" <lichao@loongson.cn>]

[-- Attachment #1: Type: text/plain, Size: 1999 bytes --]

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<gaoliming@byosoft.com.cn>
> Cc: Zhichao Gao<zhichao.gao@intel.com>
> Cc: Chao Li<lichao@loongson.cn>
> Signed-off-by: Qingyu<qingyu.shang@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 (#116703): https://edk2.groups.io/g/devel/message/116703
Mute This Topic: https://groups.io/mt/104886872/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



[-- Attachment #2: Type: text/html, Size: 3628 bytes --]