[edk2-devel] [PATCH V3] MdeModulePkg/Xhci: Initial XHCI DCI slot's Context value
Chang, Abner via groups.io
abner.chang=amd.com at groups.io
Wed Dec 21 15:44:09 UTC 2022
[AMD Official Use Only - General]
Hi Hao,
I just sent out three patches and below is PR which passed the CI.
https://github.com/tianocore/edk2/pull/3794
Thanks
Abner
> -----Original Message-----
> From: Wu, Hao A <hao.a.wu at intel.com>
> Sent: Tuesday, December 20, 2022 11:03 AM
> To: Chang, Abner <Abner.Chang at amd.com>; devel at edk2.groups.io
> Cc: Lin, Kuei-Hung (Timothy) <Kuei-Hung.Lin at amd.com>; Ni, Ray
> <ray.ni at intel.com>; Kirkendall, Garrett <Garrett.Kirkendall at amd.com>
> Subject: RE: [PATCH V3] MdeModulePkg/Xhci: Initial XHCI DCI slot's Context
> value
>
> Caution: This message originated from an External Source. Use proper caution
> when opening attachments, clicking links, or responding.
>
>
> Thanks,
>
> For this patch "MdeModulePkg/XhciPei: Unlinked XhciPei memory block", my
> take is that there will also be an updated version, right?
> If this is the case, could you help to send out a single 3-patch series for them
> instead of sending 3 separated patches? Thanks in advance.
>
> Best Regards,
> Hao Wu
>
> > -----Original Message-----
> > From: Chang, Abner <Abner.Chang at amd.com>
> > Sent: Tuesday, December 20, 2022 10:45 AM
> > To: Wu, Hao A <hao.a.wu at intel.com>; devel at edk2.groups.io
> > Cc: Lin, Kuei-Hung (Timothy) <Kuei-Hung.Lin at amd.com>; Ni, Ray
> > <ray.ni at intel.com>; Kirkendall, Garrett <Garrett.Kirkendall at amd.com>
> > Subject: RE: [PATCH V3] MdeModulePkg/Xhci: Initial XHCI DCI slot's
> > Context value
> >
> > [AMD Official Use Only - General]
> >
> > Hi Hao,
> > That is the uncrustify check error. I already fixed it at "[PATCH V3]
> > MdeModulePkg/Usb: Read a large number of blocks."
> >
> > Also, these three patches are passed the CI test.
> > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
> >
> ub.com%2Ftianocore%2Fedk2%2Fpull%2F3794&data=05%7C01%7CAbner.Chan
> g%40a
> >
> md.com%7C187600a1cd0942afb7b708dae236cbf5%7C3dd8961fe4884e608e11
> a82d99
> >
> 4e183d%7C0%7C0%7C638071022231599779%7CUnknown%7CTWFpbGZsb3d8e
> yJWIjoiMC
> >
> 4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C
> %7C%
> >
> 7C&sdata=AqFmywwFq9wJmix13UNfBDdT84wtFK%2BH8THRa7iYcm4%3D&rese
> rved=0
> >
> > Thanks
> > Abner
> >
> > > -----Original Message-----
> > > From: Wu, Hao A <hao.a.wu at intel.com>
> > > Sent: Monday, December 12, 2022 11:27 AM
> > > To: Chang, Abner <Abner.Chang at amd.com>; devel at edk2.groups.io
> > > Cc: Lin, Kuei-Hung (Timothy) <Kuei-Hung.Lin at amd.com>; Ni, Ray
> > > <ray.ni at intel.com>; Kirkendall, Garrett <Garrett.Kirkendall at amd.com>
> > > Subject: RE: [PATCH V3] MdeModulePkg/Xhci: Initial XHCI DCI slot's
> > > Context value
> > >
> > > Caution: This message originated from an External Source. Use proper
> > > caution when opening attachments, clicking links, or responding.
> > >
> > >
> > > For the 3 patches:
> > > [PATCH V2] MdeModulePkg/XhciPei: Unlinked XhciPei memory block
> > [PATCH
> > > V2] MdeModulePkg/Usb: Read a large number of blocks [PATCH V3]
> > > MdeModulePkg/Xhci: Initial XHCI DCI slot's Context value
> > >
> > > I found that they will cause CI failures:
> > >
> > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgith
> > >
> > ub.com%2Ftianocore%2Fedk2%2Fpull%2F3749&data=05%7C01%7Cabn
> > >
> > er.chang%40amd.com%7Cfe921dc0e08b4cdf30a208dadbf0cb8c%7C3dd8961
> > f
> > >
> > e4884e608e11a82d994e183d%7C0%7C0%7C638064124508360013%7CUnkno
> > w
> > >
> > n%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1ha
> > > WwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=0iQKJ3X3IJv1CSqe
> > > mjTWyT1hlYvqGZsYPAdwXEi%2B%2FZ4%3D&reserved=0
> > > Could you help to update them and help to ensure they can pass the CI test?
> > > Thanks in advance.
> > >
> > > Best Regards,
> > > Hao Wu
> > >
> > > > -----Original Message-----
> > > > From: abner.chang at amd.com <abner.chang at amd.com>
> > > > Sent: Saturday, December 10, 2022 11:08 PM
> > > > To: devel at edk2.groups.io
> > > > Cc: Kuei-Hung.Lin at amd.com; Wu, Hao A <hao.a.wu at intel.com>; Ni, Ray
> > > > <ray.ni at intel.com>; Garrett Kirkendall
> > > > <garrett.kirkendall at amd.com>; Abner Chang <abner.chang at amd.com>
> > > > Subject: [PATCH V3] MdeModulePkg/Xhci: Initial XHCI DCI slot's
> > > > Context value
> > > >
> > > > From: Abner Chang <abner.chang at amd.com>
> > > >
> > > > In V3: Add AMD copyright.
> > > > In V2: Remove the signed-off-by: Abner Chang.
> > > >
> > > > Initialize XHCI DCI slot's context entries value.
> > > >
> > > > Signed-off-by: Kuei-Hung Lin <Kuei-Hung.Lin at amd.com>
> > > > Cc: Hao A Wu <hao.a.wu at intel.com>
> > > > Cc: Ray Ni <ray.ni at intel.com>
> > > > Cc: Garrett Kirkendall <garrett.kirkendall at amd.com>
> > > > Cc: Abner Chang <abner.chang at amd.com>
> > > > ---
> > > > MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c | 7 +++++++
> > > > MdeModulePkg/Bus/Pci/XhciPei/XhciSched.c | 7 +++++++
> > > > 2 files changed, 14 insertions(+)
> > > >
> > > > diff --git a/MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c
> > > > b/MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c
> > > > index 13b0400e83d..f08722905f3 100644
> > > > --- a/MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c
> > > > +++ b/MdeModulePkg/Bus/Pci/XhciDxe/XhciSched.c
> > > > @@ -4,6 +4,7 @@
> > > >
> > > > Copyright (c) 2011 - 2020, Intel Corporation. All rights
> > > > reserved.<BR> Copyright (c) Microsoft Corporation.<BR>
> > > > +Copyright (C) 2022 Advanced Micro Devices, Inc. All rights
> > > > +reserved.<BR>
> > > > SPDX-License-Identifier: BSD-2-Clause-Patent
> > > >
> > > > **/
> > > > @@ -2807,6 +2808,9 @@ XhcInitializeEndpointContext (
> > > > MaxDci = 0;
> > > >
> > > > NumEp = IfDesc->NumEndpoints;
> > > > + if (NumEp == 0) {
> > > > + MaxDci = 1;
> > > > + }
> > > >
> > > > EpDesc = (USB_ENDPOINT_DESCRIPTOR *)(IfDesc + 1);
> > > > for (EpIndex = 0; EpIndex < NumEp; EpIndex++) { @@ -3006,6
> > > > +3010,9 @@ XhcInitializeEndpointContext64 (
> > > > MaxDci = 0;
> > > >
> > > > NumEp = IfDesc->NumEndpoints;
> > > > + if (NumEp == 0) {
> > > > + MaxDci = 1;
> > > > + }
> > > >
> > > > EpDesc = (USB_ENDPOINT_DESCRIPTOR *)(IfDesc + 1);
> > > > for (EpIndex = 0; EpIndex < NumEp; EpIndex++) { diff --git
> > > > a/MdeModulePkg/Bus/Pci/XhciPei/XhciSched.c
> > > > b/MdeModulePkg/Bus/Pci/XhciPei/XhciSched.c
> > > > index cc597a43717..8400c90f7ad 100644
> > > > --- a/MdeModulePkg/Bus/Pci/XhciPei/XhciSched.c
> > > > +++ b/MdeModulePkg/Bus/Pci/XhciPei/XhciSched.c
> > > > @@ -3,6 +3,7 @@ PEIM to produce gPeiUsb2HostControllerPpiGuid
> > based
> > > on
> > > > gPeiUsbControllerPpiGuid which is used to enable recovery
> > > > function from USB Drivers.
> > > >
> > > > Copyright (c) 2014 - 2017, Intel Corporation. All rights
> > > > reserved.<BR>
> > > > +Copyright (C) 2022 Advanced Micro Devices, Inc. All rights
> > > > +reserved.<BR>
> > > >
> > > > SPDX-License-Identifier: BSD-2-Clause-Patent
> > > >
> > > > @@ -1752,6 +1753,9 @@ XhcPeiSetConfigCmd (
> > > > }
> > > >
> > > > NumEp = IfDesc->NumEndpoints;
> > > > + if ((NumEp == 0) && (MaxDci == 0)) {
> > > > + MaxDci = 1;
> > > > + }
> > > >
> > > > EpDesc = (USB_ENDPOINT_DESCRIPTOR *)(IfDesc + 1);
> > > > for (EpIndex = 0; EpIndex < NumEp; EpIndex++) { @@ -1974,6
> > > > +1978,9 @@ XhcPeiSetConfigCmd64 (
> > > > }
> > > >
> > > > NumEp = IfDesc->NumEndpoints;
> > > > + if ((NumEp == 0) && (MaxDci == 0)) {
> > > > + MaxDci = 1;
> > > > + }
> > > >
> > > > EpDesc = (USB_ENDPOINT_DESCRIPTOR *)(IfDesc + 1);
> > > > for (EpIndex = 0; EpIndex < NumEp; EpIndex++) {
> > > > --
> > > > 2.37.1.windows.1
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#97691): https://edk2.groups.io/g/devel/message/97691
Mute This Topic: https://groups.io/mt/95582668/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
-------------- next part --------------
An embedded message was scrubbed...
From: "Chang, Abner via groups.io" <abner.chang=amd.com at groups.io>
Subject: [edk2-devel] [PATCH 3/3] MdeModulePkg/XhciPei: Unlinked XhciPei memory block
Date: Wed, 21 Dec 2022 15:42:17 +0000
Size: 13983
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20221221/14c71e21/attachment-0003.eml>
-------------- next part --------------
An embedded message was scrubbed...
From: "Chang, Abner via groups.io" <abner.chang=amd.com at groups.io>
Subject: [edk2-devel] [PATCH 2/3] MdeModulePkg/Usb: Read a large number of blocks
Date: Wed, 21 Dec 2022 15:41:44 +0000
Size: 14912
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20221221/14c71e21/attachment-0004.eml>
-------------- next part --------------
An embedded message was scrubbed...
From: "Chang, Abner via groups.io" <abner.chang=amd.com at groups.io>
Subject: [edk2-devel] [PATCH 1/3] MdeModulePkg/Xhci: Initial XHCI DCI slot's Context value
Date: Wed, 21 Dec 2022 15:41:11 +0000
Size: 14722
URL: <http://listman.redhat.com/archives/edk2-devel-archive/attachments/20221221/14c71e21/attachment-0005.eml>
More information about the edk2-devel-archive
mailing list