Re: [PATCH] BaseTools: Allow multiple PciDeviceId in Fdf OptionROM override

[Laszlo Ersek <lersek@redhat.com>]

+Yonghong, +Liming

On 10/02/18 16:46, Tomas Pilar (tpilar) wrote:
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Tomas Pilar <tpilar@solarflare.com>
> ---
>  BaseTools/Source/Python/GenFds/FdfParser.py | 11 ++++++++---
>  1 file changed, 8 insertions(+), 3 deletions(-)
> 
> diff --git a/BaseTools/Source/Python/GenFds/FdfParser.py b/BaseTools/Source/Python/GenFds/FdfParser.py
> index 63687e98bb..a65f2cfd2d 100644
> --- a/BaseTools/Source/Python/GenFds/FdfParser.py
> +++ b/BaseTools/Source/Python/GenFds/FdfParser.py
> @@ -4469,10 +4469,15 @@ class FdfParser:
>                  if self.__IsKeyword( "PCI_DEVICE_ID"):
>                      if not self.__IsToken( "="):
>                          raise Warning("expected '='", self.FileName, self.CurrentLineNumber)
> -                    if not self.__GetNextHexNumber():
> -                        raise Warning("expected Hex device id", self.FileName, self.CurrentLineNumber)
>  
> -                    Overrides.PciDeviceId = self.__Token
> +                    # Get a list of PCI IDs
> +                    Overrides.PciDeviceId = ""
> +
> +                    while self.__GetNextHexNumber():
> +                        Overrides.PciDeviceId += " " + self.__Token
> +
> +                    if not Overrides.PciDeviceId:
> +                        raise Warning("expected one or more Hex device ids", self.FileName, self.CurrentLineNumber)
>                      continue
>  
>                  if self.__IsKeyword( "PCI_REVISION"):
>