[edk2-devel] [PATCH 1/2] MdeModulePkg/Usb/Keyboard.c: remove Get/SetConfig calls
Wu, Hao A
hao.a.wu at intel.com
Fri Jan 22 01:17:09 UTC 2021
Sorry, please grant me some time for this patch series. I will try to provide feedbacks before end of next week.
Meanwhile, could you help to provide the information on what kind of tests have been performed for these 2 patches? Thanks in advance.
Best Regards,
Hao Wu
> -----Original Message-----
> From: devel at edk2.groups.io <devel at edk2.groups.io> On Behalf Of Patrick
> Rudolph
> Sent: Wednesday, January 20, 2021 11:59 PM
> To: devel at edk2.groups.io
> Cc: Wang, Jian J <jian.j.wang at intel.com>; Wu, Hao A <hao.a.wu at intel.com>;
> Ni, Ray <ray.ni at intel.com>
> Subject: [edk2-devel] [PATCH 1/2] MdeModulePkg/Usb/Keyboard.c:
> remove Get/SetConfig calls
>
> From: Matt DeVillier <matt.devillier at gmail.com>
>
> SetConfig is already called during device enumeration,
> no need to do it again here.
>
> Signed-off-by: Matt DeVillier <matt.devillier at gmail.com>
> ---
> MdeModulePkg/Bus/Usb/UsbKbDxe/KeyBoard.c | 37 --------------------
> 1 file changed, 37 deletions(-)
>
> diff --git a/MdeModulePkg/Bus/Usb/UsbKbDxe/KeyBoard.c
> b/MdeModulePkg/Bus/Usb/UsbKbDxe/KeyBoard.c
> index 5faf82ea57..77e20b203f 100644
> --- a/MdeModulePkg/Bus/Usb/UsbKbDxe/KeyBoard.c
> +++ b/MdeModulePkg/Bus/Usb/UsbKbDxe/KeyBoard.c
> @@ -801,10 +801,8 @@ InitUSBKeyboard (
> IN OUT USB_KB_DEV *UsbKeyboardDevice
>
> )
>
> {
>
> - UINT16 ConfigValue;
>
> UINT8 Protocol;
>
> EFI_STATUS Status;
>
> - UINT32 TransferResult;
>
>
>
> REPORT_STATUS_CODE_WITH_DEVICE_PATH (
>
> EFI_PROGRESS_CODE,
>
> @@ -816,41 +814,6 @@ InitUSBKeyboard (
> InitQueue (&UsbKeyboardDevice->EfiKeyQueue, sizeof (EFI_KEY_DATA));
>
> InitQueue (&UsbKeyboardDevice->EfiKeyQueueForNotify, sizeof
> (EFI_KEY_DATA));
>
>
>
> - //
>
> - // Use the config out of the descriptor
>
> - // Assumed the first config is the correct one and this is not always the case
>
> - //
>
> - Status = UsbGetConfiguration (
>
> - UsbKeyboardDevice->UsbIo,
>
> - &ConfigValue,
>
> - &TransferResult
>
> - );
>
> - if (EFI_ERROR (Status)) {
>
> - ConfigValue = 0x01;
>
> - //
>
> - // Uses default configuration to configure the USB Keyboard device.
>
> - //
>
> - Status = UsbSetConfiguration (
>
> - UsbKeyboardDevice->UsbIo,
>
> - ConfigValue,
>
> - &TransferResult
>
> - );
>
> - if (EFI_ERROR (Status)) {
>
> - //
>
> - // If configuration could not be set here, it means
>
> - // the keyboard interface has some errors and could
>
> - // not be initialized
>
> - //
>
> - REPORT_STATUS_CODE_WITH_DEVICE_PATH (
>
> - EFI_ERROR_CODE | EFI_ERROR_MINOR,
>
> - (EFI_PERIPHERAL_KEYBOARD | EFI_P_EC_INTERFACE_ERROR),
>
> - UsbKeyboardDevice->DevicePath
>
> - );
>
> -
>
> - return EFI_DEVICE_ERROR;
>
> - }
>
> - }
>
> -
>
> UsbGetProtocolRequest (
>
> UsbKeyboardDevice->UsbIo,
>
> UsbKeyboardDevice->InterfaceDescriptor.InterfaceNumber,
>
> --
> 2.26.2
>
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#70585): https://edk2.groups.io/g/devel/message/70585
> Mute This Topic: https://groups.io/mt/79981643/1768737
> Group Owner: devel+owner at edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [hao.a.wu at intel.com]
> -=-=-=-=-=-=
>
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#70658): https://edk2.groups.io/g/devel/message/70658
Mute This Topic: https://groups.io/mt/79981643/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list