Thanks Ashraf > -----Original Message----- > From: Ni, Ray <ray...@intel.com> > Sent: Wednesday, May 13, 2020 12:19 PM > To: Javeed, Ashraf <ashraf.jav...@intel.com>; devel@edk2.groups.io > Cc: Wang, Jian J <jian.j.w...@intel.com>; Wu, Hao A > <hao.a...@intel.com> > Subject: RE: [edk2-staging/UEFI_PCI_ENHANCE-2 PATCH 12/15] > MdeModulePkg/PciBusDxe: Enable LTR feature > > > +EFI_STATUS > > +LtrScan ( > > + IN PCI_IO_DEVICE *PciIoDevice, > > + IN UINTN Level, > > + IN VOID **Context > > + ) > > +{ > > + BOOLEAN *Ltr; > > + ASSERT (Level <= PCI_MAX_BUS); > > + //ASSERT (Context != NULL); > > Why the "ASSERT (Context != NULL)" is commented out? > I thought this check is not necessary for LTR alone, wanted to remove this later but missed. What is your suggestion? If it is used for LTR Scan than it has to also used for other features.
> Thanks, > Ray > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#59415): https://edk2.groups.io/g/devel/message/59415 Mute This Topic: https://groups.io/mt/74118523/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-