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]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to