Ni, Ruiyu
2013-Dec-03 09:30 UTC
Re: [edk2] [PATCH v4 1/7] MdeModulePkg: introduce PcdPciDisableBusEnumeration
For your information, I tried to replace the DUET PciBus driver using MdeModulePkg one. It works well after a small fix to the DUET PciRootBridgeNoEnumerationDxe driver. Thanks, Ray -----Original Message----- From: Kinney, Michael D [mailto:michael.d.kinney@intel.com] Sent: Tuesday, December 03, 2013 3:44 AM To: Jordan Justen; Wei Liu Cc: edk2-devel@lists.sourceforge.net; xen-devel Subject: Re: [edk2] [PATCH v4 1/7] MdeModulePkg: introduce PcdPciDisableBusEnumeration Jordan, Only rule is that TokenNumber must be unique within the TokenSpaceGuid. Reviewed-by: Michael Kinney <<michael.d.kinney@intel.com>> Mike -----Original Message----- From: Jordan Justen [mailto:jljusten@gmail.com] Sent: Saturday, November 30, 2013 3:56 PM To: Kinney, Michael D; Wei Liu Cc: xen-devel; edk2-devel@lists.sourceforge.net Subject: Re: [edk2] [PATCH v4 1/7] MdeModulePkg: introduce PcdPciDisableBusEnumeration On Fri, Nov 29, 2013 at 6:13 AM, Wei Liu <wei.liu2@citrix.com> wrote:> diff --git a/MdeModulePkg/Bus/Pci/PciBusDxe/PciBus.c b/MdeModulePkg/Bus/Pci/PciBusDxe/PciBus.c > index 5afbb82..cc6be8b 100644 > --- a/MdeModulePkg/Bus/Pci/PciBusDxe/PciBus.c > +++ b/MdeModulePkg/Bus/Pci/PciBusDxe/PciBus.c > @@ -284,7 +284,10 @@ PciBusDriverBindingStart ( > ); > } > > - gFullEnumeration = (BOOLEAN) ((SearchHostBridgeHandle (Controller) ? FALSE : TRUE)); > + if (PcdGetBool (PcdPciDisableBusEnumeration)) > + gFullEnumeration = FALSE; > + else > + gFullEnumeration = (BOOLEAN) ((SearchHostBridgeHandle (Controller) ? FALSE : TRUE));Code style { } I think this could be fixed at commit time.> diff --git a/MdeModulePkg/MdeModulePkg.dec b/MdeModulePkg/MdeModulePkg.dec > index b627eb1..5198451 100644 > --- a/MdeModulePkg/MdeModulePkg.dec > +++ b/MdeModulePkg/MdeModulePkg.dec > @@ -878,6 +878,9 @@ > ## This PCD specified whether the S.M.A.R.T feature of attached ATA hard disks are enabled. > gEfiMdeModulePkgTokenSpaceGuid.PcdAtaSmartEnable|TRUE|BOOLEAN|0x00010065 > > + ## This PCD specifies whether full PCI enumeration is disabled. > + gEfiMdeModulePkgTokenSpaceGuid.PcdPciDisableBusEnumeration|FALSE|BOOLEAN|0x10000048Mike, Any preference on token number other than don''t clash? Do you give your Reviewed-by for this patch? If so, I could take care of committing it. -Jordan ------------------------------------------------------------------------------ Rapidly troubleshoot problems before they affect your business. Most IT organizations don''t have a clear picture of how application performance affects their revenue. With AppDynamics, you get 100% visibility into your Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro! http://pubads.g.doubleclick.net/gampad/clk?id=84349351&iu=/4140/ostg.clktrk _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel