From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f67.google.com (mail-wm1-f67.google.com [209.85.128.67]) by mx.groups.io with SMTP id smtpd.web12.140.1593787488269003912 for ; Fri, 03 Jul 2020 07:44:48 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@nuviainc-com.20150623.gappssmtp.com header.s=20150623 header.b=vXmNZUrc; spf=pass (domain: nuviainc.com, ip: 209.85.128.67, mailfrom: leif@nuviainc.com) Received: by mail-wm1-f67.google.com with SMTP id f139so34340814wmf.5 for ; Fri, 03 Jul 2020 07:44:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nuviainc-com.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=TEAvWnNSlEN8f37RPeGfJ4KNhQN31V1eJ7vnWdLCeFQ=; b=vXmNZUrcmIlbmuZxkwP9WSTELrcDnuocsFDzdv0Bx/APfH57Jz1WZ3EtK9wKG6PKs2 1IhQfQq5+WkL2V1aQkdJtk9b61IaDtwTQqdM+Gv6iFGkfu6ywjqkjM1SvY1hiPuqNPCq 7z37SXnUP8n5FFdGajkzIPwURjWJICo0BJ8Dwz+724X2wm0srP/uG4Qj20iK6KjlMG29 +kjaYc6ZlN/mRaUcfdUNfb/3B+klFC0AT/NHZ2/DdTp2+ZOKJcYRaTe7FE2Mu6ZcjhMX l3DgqYkbtO3s5zZH3fIBQETl+JX9n1Cvl3qHFfOW7eoPD0gLGOioKqVty3nBPxwYy+m9 y9Ng== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=TEAvWnNSlEN8f37RPeGfJ4KNhQN31V1eJ7vnWdLCeFQ=; b=I/CVGkJTRGIGe6Er5a6xC5n2wAhOZq3zQBaT741nJSp8OOkyjANm23LA0uYWz9p47S KsTELbsHq92eF99hw7qjKb2Z17oiNDTF5R8nzQxhQ0PllYOgganFTD53m5QbETKrheN6 tCBJKwVb/wu+VZU76b1vueYJgtVWqJ1JwDZavXMnVFmpTY1Gpwh/k2Virj8STR+KWTnD 6DFy2NjMuFkGivf1GumEAHzWBcyyu5QkM1O/W1GBY+YFx5cwvEEEKlek0lG4At8r4Wt7 XXNW1A/l5fbqAOOSqeeaoDj60Ctgznkwl6iVzjHq+KtnH76HS+TnF2C7rn3jDxNuVtH0 s6QQ== X-Gm-Message-State: AOAM530VK5QQC5C6Cq/BiQCOuqqqhvdZoW/F2Hh69Be+Bhn6Y3mNls/O IvCbFs+mT4ifREM8vFjbGUsg1w== X-Google-Smtp-Source: ABdhPJy31mIMaVZWL/dbEx8Xjopzy7pYAfd1qGeQN+dwoopzyL75bQfFKwrjrvSJciIZ1sLCPlTo2Q== X-Received: by 2002:a7b:c041:: with SMTP id u1mr39162741wmc.56.1593787486794; Fri, 03 Jul 2020 07:44:46 -0700 (PDT) Return-Path: Received: from vanye ([2001:470:1f09:12f0:b26e:bfff:fea9:f1b8]) by smtp.gmail.com with ESMTPSA id y77sm3674765wmd.36.2020.07.03.07.44.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 03 Jul 2020 07:44:46 -0700 (PDT) Date: Fri, 3 Jul 2020 15:44:43 +0100 From: "Leif Lindholm" To: Wasim Khan Cc: devel@edk2.groups.io, meenakshi.aggarwal@nxp.com, V.Sethi@nxp.com, ard.biesheuvel@arm.com, Wasim Khan Subject: Re: [PATCH edk2-platforms v2 0/7] NXP: Add PCIe Support for LX2160aRdbPkg Message-ID: <20200703144443.GW6739@vanye> References: <1593413106-1672-1-git-send-email-wasim.khan@oss.nxp.com> MIME-Version: 1.0 In-Reply-To: <1593413106-1672-1-git-send-email-wasim.khan@oss.nxp.com> User-Agent: Mutt/1.10.1 (2018-07-13) Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Mon, Jun 29, 2020 at 12:14:59 +0530, Wasim Khan wrote: > From: Wasim Khan > > LX2160-Rev1 and LX2160-Rev2 has different PCIe controller. > This patch series adds PCIe support for LX2160aRdbPkg which includes > - Add PCIe space in VirtualMemoryMap > - Platform driver to check SoC version and sets PCDs for PCIe controller, which > are used by PciHostBridgeLib and PciSegmentLib. > PciHostBridgeLib and PciSegmentLib already has support for both PCIe controllers. > - Enable NetworkPkg for LX2160aRdbPkg Platform. > > > Changes in V2: > - Addressed review comments on V1 Thanks! For the series: Reviewed-by: Leif Lindholm Pushed as b716f363e752..b9fba411178f. > V1 series can be referred here: > https://edk2.groups.io/g/devel/message/61062 > > Wasim Khan (7): > Silicon/NXP: LX2160A: Define PCIe related PCDs > Platform/NXP: LX2160aRdbPkg: Add PCIe space in VirtualMemoryMap > Platform/NXP: LX2160aRdbPkg: Add PlatformDxe driver > Platform/NXP: LX2160aRdbPkg: Enable PlatformDxe driver > Platform/NXP: LX2160aRdbPkg: Hide Root Port for LX2160A-Rev2 > Platform/NXP: LX2160aRdbPkg: Enable NetworkPkg > Platform/NXP: LX2160aRdbPkg: Enable PCIE support > > Silicon/NXP/NxpQoriqLs.dec | 1 + > Silicon/NXP/LX2160A/LX2160A.dsc.inc | 5 + > Platform/NXP/LX2160aRdbPkg/LX2160aRdbPkg.dsc | 22 +++++ > Platform/NXP/LX2160aRdbPkg/LX2160aRdbPkg.fdf | 15 +++ > Platform/NXP/LX2160aRdbPkg/Drivers/PlatformDxe/PlatformDxe.inf | 36 ++++++++ > Platform/NXP/LX2160aRdbPkg/Library/ArmPlatformLib/ArmPlatformLib.inf | 2 + > Silicon/NXP/Library/PciHostBridgeLib/PciHostBridgeLib.inf | 1 + > Silicon/NXP/Include/Pcie.h | 1 + > Silicon/NXP/LX2160A/Include/Soc.h | 3 + > Platform/NXP/LX2160aRdbPkg/Drivers/PlatformDxe/PlatformDxe.c | 97 ++++++++++++++++++++ > Platform/NXP/LX2160aRdbPkg/Library/ArmPlatformLib/ArmPlatformLibMem.c | 11 ++- > Silicon/NXP/Library/PciHostBridgeLib/PciHostBridgeLib.c | 6 +- > 12 files changed, 198 insertions(+), 2 deletions(-) > create mode 100644 Platform/NXP/LX2160aRdbPkg/Drivers/PlatformDxe/PlatformDxe.inf > create mode 100644 Platform/NXP/LX2160aRdbPkg/Drivers/PlatformDxe/PlatformDxe.c > > -- > 2.7.4 >