[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [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@xxxxxxxxx] 
Sent: Tuesday, December 03, 2013 3:44 AM
To: Jordan Justen; Wei Liu
Cc: edk2-devel@xxxxxxxxxxxxxxxxxxxxx; 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@xxxxxxxxx>>

Mike

-----Original Message-----
From: Jordan Justen [mailto:jljusten@xxxxxxxxx] 
Sent: Saturday, November 30, 2013 3:56 PM
To: Kinney, Michael D; Wei Liu
Cc: xen-devel; edk2-devel@xxxxxxxxxxxxxxxxxxxxx
Subject: Re: [edk2] [PATCH v4 1/7] MdeModulePkg: introduce 
PcdPciDisableBusEnumeration

On Fri, Nov 29, 2013 at 6:13 AM, Wei Liu <wei.liu2@xxxxxxxxxx> 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|0x10000048

Mike,

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@xxxxxxxxxxxxxxxxxxxxx
https://lists.sourceforge.net/lists/listinfo/edk2-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.