* [edk2-platforms] [PATCH v1 0/1] DisplayLink GOP driver USB msg initialisation
@ 2019-09-10 14:49 Andy Hayes
2019-09-10 14:49 ` [PATCH v1 1/1] Drivers/DisplayLink/DisplayLinkPkg DisplayLinkGop Andy Hayes
0 siblings, 1 reply; 4+ messages in thread
From: Andy Hayes @ 2019-09-10 14:49 UTC (permalink / raw)
To: devel@edk2.groups.io; +Cc: Andy Hayes, Leif Lindholm, Ard Biesheuvel
Corrected initialisation of one of data structures used to transmit USB
control messages. Mistake had no practical effects but fixing to be on safe
side.
https://github.com/andy-hayes/edk2-platforms
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Andy Hayes (1):
Drivers/DisplayLink/DisplayLinkPkg DisplayLinkGop
Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--
1.8.3.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v1 1/1] Drivers/DisplayLink/DisplayLinkPkg DisplayLinkGop
2019-09-10 14:49 [edk2-platforms] [PATCH v1 0/1] DisplayLink GOP driver USB msg initialisation Andy Hayes
@ 2019-09-10 14:49 ` Andy Hayes
0 siblings, 0 replies; 4+ messages in thread
From: Andy Hayes @ 2019-09-10 14:49 UTC (permalink / raw)
To: devel@edk2.groups.io; +Cc: Andy Hayes, Leif Lindholm, Ard Biesheuvel
Corrected initialisation of one of data structures used to transmit USB
control messages. Mistake had no practical effects but fixing to be on safe
side.
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Signed-off-by: Andy Hayes <andy.hayes@displaylink.com>
---
Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c b/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c
index 252293da39d4..9871ab0378ce 100644
--- a/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c
+++ b/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c
@@ -107,7 +107,7 @@ DlUsbSendControlWriteMessage (
UINT32 UsbStatus;
EFI_USB_DEVICE_REQUEST UsbRequest;
- ZeroMem (&Request, sizeof (Request));
+ ZeroMem (&UsbRequest, sizeof (UsbRequest));
UsbRequest.RequestType = USB_REQ_TYPE_VENDOR | USB_TARGET_INTERFACE;
UsbRequest.Index = Device->InterfaceDescriptor.InterfaceNumber;
UsbRequest.Request = Request;
--
1.8.3.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH v1 0/1] DisplayLink GOP driver USB msg initialisation
@ 2019-09-11 7:42 Andy Hayes
2019-09-11 7:42 ` [PATCH v1 1/1] Drivers/DisplayLink/DisplayLinkPkg DisplayLinkGop Andy Hayes
0 siblings, 1 reply; 4+ messages in thread
From: Andy Hayes @ 2019-09-11 7:42 UTC (permalink / raw)
To: devel@edk2.groups.io; +Cc: Andy Hayes, Leif Lindholm, Ard Biesheuvel
Corrected initialisation of one of data structures used to transmit USB
control messages. Mistake had no practical effects but fixing to be on safe
side.
https://github.com/andy-hayes/edk2-platforms
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Andy Hayes (1):
Drivers/DisplayLink/DisplayLinkPkg DisplayLinkGop
Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
--
1.8.3.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v1 1/1] Drivers/DisplayLink/DisplayLinkPkg DisplayLinkGop
2019-09-11 7:42 [PATCH v1 0/1] DisplayLink GOP driver USB msg initialisation Andy Hayes
@ 2019-09-11 7:42 ` Andy Hayes
2019-09-17 15:27 ` Leif Lindholm
0 siblings, 1 reply; 4+ messages in thread
From: Andy Hayes @ 2019-09-11 7:42 UTC (permalink / raw)
To: devel@edk2.groups.io; +Cc: Andy Hayes, Leif Lindholm, Ard Biesheuvel
Corrected initialisation of one of data structures used to transmit USB
control messages. Mistake had no practical effects but fixing to be on safe
side.
Cc: Leif Lindholm <leif.lindholm@linaro.org>
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Signed-off-by: Andy Hayes <andy.hayes@displaylink.com>
---
Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c b/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c
index 252293da39d4..9871ab0378ce 100644
--- a/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c
+++ b/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c
@@ -107,7 +107,7 @@ DlUsbSendControlWriteMessage (
UINT32 UsbStatus;
EFI_USB_DEVICE_REQUEST UsbRequest;
- ZeroMem (&Request, sizeof (Request));
+ ZeroMem (&UsbRequest, sizeof (UsbRequest));
UsbRequest.RequestType = USB_REQ_TYPE_VENDOR | USB_TARGET_INTERFACE;
UsbRequest.Index = Device->InterfaceDescriptor.InterfaceNumber;
UsbRequest.Request = Request;
--
1.8.3.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v1 1/1] Drivers/DisplayLink/DisplayLinkPkg DisplayLinkGop
2019-09-11 7:42 ` [PATCH v1 1/1] Drivers/DisplayLink/DisplayLinkPkg DisplayLinkGop Andy Hayes
@ 2019-09-17 15:27 ` Leif Lindholm
0 siblings, 0 replies; 4+ messages in thread
From: Leif Lindholm @ 2019-09-17 15:27 UTC (permalink / raw)
To: Andy Hayes; +Cc: devel@edk2.groups.io, Ard Biesheuvel
On Wed, Sep 11, 2019 at 07:42:03AM +0000, Andy Hayes wrote:
> Corrected initialisation of one of data structures used to transmit USB
> control messages. Mistake had no practical effects but fixing to be on safe
> side.
So, was the only request used index 0? Or why didn't this cause an
issue? Nevertheless, a clear fix.
> Cc: Leif Lindholm <leif.lindholm@linaro.org>
> Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Signed-off-by: Andy Hayes <andy.hayes@displaylink.com>
Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
Pushed as 958aaf600728.
/
Leif
> ---
> Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c b/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c
> index 252293da39d4..9871ab0378ce 100644
> --- a/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c
> +++ b/Drivers/DisplayLink/DisplayLinkPkg/DisplayLinkGop/UsbTransfer.c
> @@ -107,7 +107,7 @@ DlUsbSendControlWriteMessage (
> UINT32 UsbStatus;
> EFI_USB_DEVICE_REQUEST UsbRequest;
>
> - ZeroMem (&Request, sizeof (Request));
> + ZeroMem (&UsbRequest, sizeof (UsbRequest));
> UsbRequest.RequestType = USB_REQ_TYPE_VENDOR | USB_TARGET_INTERFACE;
> UsbRequest.Index = Device->InterfaceDescriptor.InterfaceNumber;
> UsbRequest.Request = Request;
> --
> 1.8.3.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-09-17 15:27 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-09-10 14:49 [edk2-platforms] [PATCH v1 0/1] DisplayLink GOP driver USB msg initialisation Andy Hayes
2019-09-10 14:49 ` [PATCH v1 1/1] Drivers/DisplayLink/DisplayLinkPkg DisplayLinkGop Andy Hayes
-- strict thread matches above, loose matches on Subject: below --
2019-09-11 7:42 [PATCH v1 0/1] DisplayLink GOP driver USB msg initialisation Andy Hayes
2019-09-11 7:42 ` [PATCH v1 1/1] Drivers/DisplayLink/DisplayLinkPkg DisplayLinkGop Andy Hayes
2019-09-17 15:27 ` Leif Lindholm
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox