* [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token
@ 2017-08-01 20:55 Felix Poludov
2017-08-02 2:20 ` Gao, Liming
0 siblings, 1 reply; 5+ messages in thread
From: Felix Poludov @ 2017-08-01 20:55 UTC (permalink / raw)
To: edk2-devel@lists.01.org
Add PcdPciExpressRegionLength PCD Token to MdePkg.
The new token can be used in conjunction with PcdPciExpressBaseAddress PCD token to describe PCI Express MMIO region.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Felix Polyudov <felixp@ami.com>
---
Please consider the environment before printing this email.
The information contained in this message may be confidential and proprietary to American Megatrends, Inc. This communication is intended to be read only by the individual or entity to whom it is addressed or by their designee. If the reader of this message is not the intended recipient, you are on notice that any distribution of this message, in any form, is strictly prohibited. Please promptly notify the sender by reply e-mail or by telephone at 770-246-8600, and then delete or destroy all copies of the transmission.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token
2017-08-01 20:55 [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token Felix Poludov
@ 2017-08-02 2:20 ` Gao, Liming
0 siblings, 0 replies; 5+ messages in thread
From: Gao, Liming @ 2017-08-02 2:20 UTC (permalink / raw)
To: Felix Poludov, edk2-devel@lists.01.org
Felix:
Could you attach your patch?
>-----Original Message-----
>From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
>Felix Poludov
>Sent: Wednesday, August 02, 2017 4:56 AM
>To: edk2-devel@lists.01.org
>Subject: [edk2] [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token
>
>Add PcdPciExpressRegionLength PCD Token to MdePkg.
>The new token can be used in conjunction with PcdPciExpressBaseAddress
>PCD token to describe PCI Express MMIO region.
>
>Contributed-under: TianoCore Contribution Agreement 1.0
>Signed-off-by: Felix Polyudov <felixp@ami.com>
>---
>
>
>Please consider the environment before printing this email.
>
>The information contained in this message may be confidential and
>proprietary to American Megatrends, Inc. This communication is intended to
>be read only by the individual or entity to whom it is addressed or by their
>designee. If the reader of this message is not the intended recipient, you are
>on notice that any distribution of this message, in any form, is strictly
>prohibited. Please promptly notify the sender by reply e-mail or by telephone
>at 770-246-8600, and then delete or destroy all copies of the transmission.
>_______________________________________________
>edk2-devel mailing list
>edk2-devel@lists.01.org
>https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token
@ 2017-08-02 13:48 Felix Poludov
2017-08-02 13:56 ` Marvin H?user
0 siblings, 1 reply; 5+ messages in thread
From: Felix Poludov @ 2017-08-02 13:48 UTC (permalink / raw)
To: edk2-devel@lists.01.org
Add PcdPciExpressRegionLength PCD Token to MdePkg.
The new token can be used in conjunction with PcdPciExpressBaseAddress PCD token to describe PCI Express MMIO region.
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Felix Polyudov <felixp@ami.com>
---
Resending with inline patch.
--
diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec
index d6928b3..7e55019 100644
--- a/MdePkg/MdePkg.dec
+++ b/MdePkg/MdePkg.dec
@@ -2089,6 +2089,10 @@
# @Prompt PCI Express Base Address.
gEfiMdePkgTokenSpaceGuid.PcdPciExpressBaseAddress|0xE0000000|UINT64|0x0000000a
+ ## Length of the PCI express region.
+ # @Prompt PCI Express Region Length.
+ gEfiMdePkgTokenSpaceGuid.PcdPciExpressRegionLength|0x10000000|UINT32|0x00000031
+
## Default current ISO 639-2 language: English & French.
# @Prompt Default Value of LangCodes Variable.
gEfiMdePkgTokenSpaceGuid.PcdUefiVariableDefaultLangCodes|"engfraengfra"|VOID*|0x0000001c
diff --git a/MdePkg/MdePkg.uni b/MdePkg/MdePkg.uni
index a110e45..62df5dc 100644
--- a/MdePkg/MdePkg.uni
+++ b/MdePkg/MdePkg.uni
@@ -276,6 +276,10 @@
#string STR_gEfiMdePkgTokenSpaceGuid_PcdPciExpressBaseAddress_HELP #language en-US "This value is used to set the base address of PCI express hierarchy."
+#string STR_gEfiMdePkgTokenSpaceGuid_PcdPciExpressRegionLength_PROMPT #language en-US "PCI Express Region Length"
+
+#string STR_gEfiMdePkgTokenSpaceGuid_PcdPciExpressRegionLength_HELP #language en-US "Length of the PCI express region."
+
#string STR_gEfiMdePkgTokenSpaceGuid_PcdUefiVariableDefaultLangCodes_PROMPT #language en-US "Default Value of LangCodes Variable"
#string STR_gEfiMdePkgTokenSpaceGuid_PcdUefiVariableDefaultLangCodes_HELP #language en-US "Default current ISO 639-2 language: English & French."
--
Please consider the environment before printing this email.
The information contained in this message may be confidential and proprietary to American Megatrends, Inc. This communication is intended to be read only by the individual or entity to whom it is addressed or by their designee. If the reader of this message is not the intended recipient, you are on notice that any distribution of this message, in any form, is strictly prohibited. Please promptly notify the sender by reply e-mail or by telephone at 770-246-8600, and then delete or destroy all copies of the transmission.
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token
2017-08-02 13:48 Felix Poludov
@ 2017-08-02 13:56 ` Marvin H?user
2017-08-02 16:30 ` Felix Poludov
0 siblings, 1 reply; 5+ messages in thread
From: Marvin H?user @ 2017-08-02 13:56 UTC (permalink / raw)
To: edk2-devel@lists.01.org
Cc: Felix Poludov, michael.d.kinney@intel.com, Gao, Liming
A bunch of platforms, such as QuarkSocPkg, already declare such a PCD named PcdPciExpressSize of type UINT64.
Maybe PcdPciExpressSize|UINT64 should be 'promoted' to a MdePkg PCD and the platform-defined PCDs replaced?
Regards,
Marvin.
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Felix Poludov
> Sent: Wednesday, August 2, 2017 3:48 PM
> To: edk2-devel@lists.01.org
> Subject: [edk2] [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD
> Token
>
> Add PcdPciExpressRegionLength PCD Token to MdePkg.
> The new token can be used in conjunction with PcdPciExpressBaseAddress
> PCD token to describe PCI Express MMIO region.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Felix Polyudov <felixp@ami.com>
> ---
>
> Resending with inline patch.
>
> --
> diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec index
> d6928b3..7e55019 100644
> --- a/MdePkg/MdePkg.dec
> +++ b/MdePkg/MdePkg.dec
> @@ -2089,6 +2089,10 @@
> # @Prompt PCI Express Base Address.
>
> gEfiMdePkgTokenSpaceGuid.PcdPciExpressBaseAddress|0xE0000000|UINT6
> 4|0x0000000a
> + ## Length of the PCI express region.
> + # @Prompt PCI Express Region Length.
> +
> +
> gEfiMdePkgTokenSpaceGuid.PcdPciExpressRegionLength|0x10000000|UINT
> 32|0
> + x00000031
> +
> ## Default current ISO 639-2 language: English & French.
> # @Prompt Default Value of LangCodes Variable.
>
> gEfiMdePkgTokenSpaceGuid.PcdUefiVariableDefaultLangCodes|"engfraengf
> ra"|VOID*|0x0000001c
> diff --git a/MdePkg/MdePkg.uni b/MdePkg/MdePkg.uni index
> a110e45..62df5dc 100644
> --- a/MdePkg/MdePkg.uni
> +++ b/MdePkg/MdePkg.uni
> @@ -276,6 +276,10 @@
> #string STR_gEfiMdePkgTokenSpaceGuid_PcdPciExpressBaseAddress_HELP
> #language en-US "This value is used to set the base address of PCI express
> hierarchy."
> +#string
> STR_gEfiMdePkgTokenSpaceGuid_PcdPciExpressRegionLength_PROMPT
> #language en-US "PCI Express Region Length"
> +
> +#string
> STR_gEfiMdePkgTokenSpaceGuid_PcdPciExpressRegionLength_HELP
> #language en-US "Length of the PCI express region."
> +
> #string
> STR_gEfiMdePkgTokenSpaceGuid_PcdUefiVariableDefaultLangCodes_PROM
> PT #language en-US "Default Value of LangCodes Variable"
> #string
> STR_gEfiMdePkgTokenSpaceGuid_PcdUefiVariableDefaultLangCodes_HELP
> #language en-US "Default current ISO 639-2 language: English & French."
> --
>
> Please consider the environment before printing this email.
>
> The information contained in this message may be confidential and
> proprietary to American Megatrends, Inc. This communication is intended to
> be read only by the individual or entity to whom it is addressed or by their
> designee. If the reader of this message is not the intended recipient, you are
> on notice that any distribution of this message, in any form, is strictly
> prohibited. Please promptly notify the sender by reply e-mail or by
> telephone at 770-246-8600, and then delete or destroy all copies of the
> transmission.
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token
2017-08-02 13:56 ` Marvin H?user
@ 2017-08-02 16:30 ` Felix Poludov
0 siblings, 0 replies; 5+ messages in thread
From: Felix Poludov @ 2017-08-02 16:30 UTC (permalink / raw)
To: Marvin H?user, edk2-devel@lists.01.org
Cc: michael.d.kinney@intel.com, Gao, Liming
I find that many platform packages have some kind of PCD token to represent size of the PCI express region.
That is the reason I think it should be standardized.
As far as PCD token name and type (UINT32 vs. UINT64), I'm seeking community feedback.
Both PcdPciExpressSize and PcdPciExpressRegionLength are names used in different platform packages.
-----Original Message-----
From: Marvin H?user [mailto:Marvin.Haeuser@outlook.com]
Sent: Wednesday, August 02, 2017 9:57 AM
To: edk2-devel@lists.01.org
Cc: Felix Poludov; michael.d.kinney@intel.com; Gao, Liming
Subject: RE: [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token
A bunch of platforms, such as QuarkSocPkg, already declare such a PCD named PcdPciExpressSize of type UINT64.
Maybe PcdPciExpressSize|UINT64 should be 'promoted' to a MdePkg PCD and the platform-defined PCDs replaced?
Regards,
Marvin.
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Felix Poludov
> Sent: Wednesday, August 2, 2017 3:48 PM
> To: edk2-devel@lists.01.org
> Subject: [edk2] [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD
> Token
>
> Add PcdPciExpressRegionLength PCD Token to MdePkg.
> The new token can be used in conjunction with PcdPciExpressBaseAddress
> PCD token to describe PCI Express MMIO region.
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Felix Polyudov <felixp@ami.com>
> ---
>
> Resending with inline patch.
>
> --
> diff --git a/MdePkg/MdePkg.dec b/MdePkg/MdePkg.dec index
> d6928b3..7e55019 100644
> --- a/MdePkg/MdePkg.dec
> +++ b/MdePkg/MdePkg.dec
> @@ -2089,6 +2089,10 @@
> # @Prompt PCI Express Base Address.
>
> gEfiMdePkgTokenSpaceGuid.PcdPciExpressBaseAddress|0xE0000000|UINT6
> 4|0x0000000a
> + ## Length of the PCI express region.
> + # @Prompt PCI Express Region Length.
> +
> +
> gEfiMdePkgTokenSpaceGuid.PcdPciExpressRegionLength|0x10000000|UINT
> 32|0
> + x00000031
> +
> ## Default current ISO 639-2 language: English & French.
> # @Prompt Default Value of LangCodes Variable.
>
> gEfiMdePkgTokenSpaceGuid.PcdUefiVariableDefaultLangCodes|"engfraengf
> ra"|VOID*|0x0000001c
> diff --git a/MdePkg/MdePkg.uni b/MdePkg/MdePkg.uni index
> a110e45..62df5dc 100644
> --- a/MdePkg/MdePkg.uni
> +++ b/MdePkg/MdePkg.uni
> @@ -276,6 +276,10 @@
> #string STR_gEfiMdePkgTokenSpaceGuid_PcdPciExpressBaseAddress_HELP
> #language en-US "This value is used to set the base address of PCI
> express hierarchy."
> +#string
> STR_gEfiMdePkgTokenSpaceGuid_PcdPciExpressRegionLength_PROMPT
> #language en-US "PCI Express Region Length"
> +
> +#string
> STR_gEfiMdePkgTokenSpaceGuid_PcdPciExpressRegionLength_HELP
> #language en-US "Length of the PCI express region."
> +
> #string
> STR_gEfiMdePkgTokenSpaceGuid_PcdUefiVariableDefaultLangCodes_PROM
> PT #language en-US "Default Value of LangCodes Variable"
> #string
> STR_gEfiMdePkgTokenSpaceGuid_PcdUefiVariableDefaultLangCodes_HELP
> #language en-US "Default current ISO 639-2 language: English & French."
> --
>
> Please consider the environment before printing this email.
>
> The information contained in this message may be confidential and
> proprietary to American Megatrends, Inc. This communication is
> intended to be read only by the individual or entity to whom it is
> addressed or by their designee. If the reader of this message is not
> the intended recipient, you are on notice that any distribution of
> this message, in any form, is strictly prohibited. Please promptly
> notify the sender by reply e-mail or by telephone at 770-246-8600, and
> then delete or destroy all copies of the transmission.
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel
Please consider the environment before printing this email.
The information contained in this message may be confidential and proprietary to American Megatrends, Inc. This communication is intended to be read only by the individual or entity to whom it is addressed or by their designee. If the reader of this message is not the intended recipient, you are on notice that any distribution of this message, in any form, is strictly prohibited. Please promptly notify the sender by reply e-mail or by telephone at 770-246-8600, and then delete or destroy all copies of the transmission.
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2017-08-02 16:28 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-08-01 20:55 [PATCH] MdePkg: Add PcdPciExpressRegionLength PCD Token Felix Poludov
2017-08-02 2:20 ` Gao, Liming
-- strict thread matches above, loose matches on Subject: below --
2017-08-02 13:48 Felix Poludov
2017-08-02 13:56 ` Marvin H?user
2017-08-02 16:30 ` Felix Poludov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox