From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:4001:c0b::242; helo=mail-it0-x242.google.com; envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-it0-x242.google.com (mail-it0-x242.google.com [IPv6:2607:f8b0:4001:c0b::242]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 9837F210C1ED8 for ; Wed, 25 Jul 2018 04:41:29 -0700 (PDT) Received: by mail-it0-x242.google.com with SMTP id g14-v6so8220384iti.1 for ; Wed, 25 Jul 2018 04:41:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=UVGcdysVGiGQpvkkO7B0+vtvE3BnJ0q8wOBvY00CIck=; b=QbIrsUe3oyXLvF4YYaTxx/xMwHBUKQA2YWkeEpSApnAlD3rM8sWnbGo1fUCEKgEO0M bnB6kZtr8nzO/2fGSRNDi48956gk7I3uENvSdjLdp3uSTCLLsM/hIkbV0tYSfjOM8Nva pUBJK0rIhO0GYzw9b0eWhTKzPespFagFrybyI= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=UVGcdysVGiGQpvkkO7B0+vtvE3BnJ0q8wOBvY00CIck=; b=RaVjwCaawDIzt2VEWVFgyKcKJdKJVOm3sqVQ0RAyr7XhgM+I3sZQPc8z4msj5JE9G5 9VfEve3jCsB/go4ttWkqlN0K0GaxDaucjsa5PUfOKpypsqp4JWs2RG2mFt/CegGx9Mtl ZlJMfq7D4hZv+0mwQRCdOsScGaAHxb2AdvmbcdYgCYRIWtPkYfGoxL3PkiE+KJirdMCq slgiuj8Khffsnp1/dvGDd3RC/8pyPGwcnVfYxTdm1DmW62rigdDEuqEKXym3qzpZkbvf 2V9zBILHH8RYcN98IsWDkjJR97fGiinaXQiq0lswjkX8+8R1O8+HPKJW3JLf/mvKklb3 q/0g== X-Gm-Message-State: AOUpUlE0jSFZLYnbyNNJp8c0CmVw1kv4h527GeWlFTO+RX8mDp1fwMNV mr1sGZh/CisOyIbH5GBYpDjEaHhaWWkvqV7nlOE+uw== X-Google-Smtp-Source: AAOMgpeELoAWPqjiiCttuJusSJM4dXTXJkxMJXm7R8QVLCiiOz3TUD2queXaoSpGdF+CbRCO65fFj+IIR3LYYbzpV98= X-Received: by 2002:a24:610d:: with SMTP id s13-v6mr5961462itc.68.1532518888752; Wed, 25 Jul 2018 04:41:28 -0700 (PDT) MIME-Version: 1.0 Received: by 2002:a6b:ac05:0:0:0:0:0 with HTTP; Wed, 25 Jul 2018 04:41:28 -0700 (PDT) In-Reply-To: <20180724063220.61679-1-ming.huang@linaro.org> References: <20180724063220.61679-1-ming.huang@linaro.org> From: Ard Biesheuvel Date: Wed, 25 Jul 2018 13:41:28 +0200 Message-ID: To: Ming Huang Cc: Leif Lindholm , linaro-uefi , "edk2-devel@lists.01.org" , Graeme Gregory , guoheyi@huawei.com, wanghuiqiang , huangming , Jason Zhang , huangdaode@hisilicon.com, John Garry , Xinliang Liu Subject: Re: [PATCH edk2-platforms v2 00/12] Switching to generic PciHostBridge driver X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.27 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 25 Jul 2018 11:41:29 -0000 Content-Type: text/plain; charset="UTF-8" On 24 July 2018 at 08:32, Ming Huang wrote: > The major features of this patchset include: > 1 switching to generic PciHostBridge driver; > 2 Change DEBUG log level; > > Code can also be found in github: https://github.com/hisilicon/OpenPlatformPkg.git > branch: pcihostbridge-v2 > > > Heyi Guo (9): > Hisilicon: Enable WARN and INFO debug message > Hisilicon/D05/PlatformPciLib: fix misuse of macro > Hisilicon/Pci: Move PciPlatform to common directory > Hisilicon/PlatformPciLib: add segment for each root bridge > Hisilicon: add PciHostBridgeLib > Hisilicon: add PciSegmentLib for Hi161x > Hisilicon/D0x: Switch to generic PciHostBridge driver > Hisilicon: remove platform specific PciHostBridge > Hisilicon/PlatformPciLib: clear redundant felds in RESOURCE_APPETURE > > Ming Huang (3): > Hisilicon/Pci: Add two api for PciPlatform driver > Hisilicon/Pci: move ATU configuration to PciPlatformLib > Hisilicon/Pci: move EnlargeAtuConfig0() to PciPlatformLib > For the series Reviewed-by: Ard Biesheuvel Pushed as a34ea15dbf31..016d55843a01 > Platform/Hisilicon/D03/D03.dsc | 14 +- > Platform/Hisilicon/D03/D03.fdf | 5 +- > .../D03/Drivers/PciPlatform/PciPlatform.h | 180 -- > .../Library/PlatformPciLib/PlatformPciLib.c | 24 +- > Platform/Hisilicon/D05/D05.dsc | 14 +- > Platform/Hisilicon/D05/D05.fdf | 5 +- > .../Library/PlatformPciLib/PlatformPciLib.c | 66 +- > .../PciHostBridgeLib/PciHostBridgeLib.c | 304 +++ > .../PciHostBridgeLib/PciHostBridgeLib.inf | 51 + > .../Drivers/PciHostBridgeDxe/PciHostBridge.c | 1659 ------------ > .../Drivers/PciHostBridgeDxe/PciHostBridge.h | 528 ---- > .../PciHostBridgeDxe/PciHostBridgeDxe.inf | 74 - > .../PciHostBridgeDxe/PciRootBridgeIo.c | 2405 ----------------- > .../Drivers/PciPlatform/PciPlatform.c | 45 + > .../Drivers/PciPlatform/PciPlatform.inf | 1 + > .../Hi161xPciPlatformLib.c | 384 +++ > .../Hi161xPciPlatformLib.inf | 42 + > .../Hi161xPciSegmentLib.inf | 36 + > .../Hi161xPciSegmentLib/PciSegmentLib.c | 1503 ++++++++++ > Silicon/Hisilicon/Hisilicon.dsc.inc | 8 +- > .../Include/Library/PlatformPciLib.h | 3 +- > 21 files changed, 2434 insertions(+), 4917 deletions(-) > delete mode 100644 Platform/Hisilicon/D03/Drivers/PciPlatform/PciPlatform.h > create mode 100644 Platform/Hisilicon/Library/PciHostBridgeLib/PciHostBridgeLib.c > create mode 100644 Platform/Hisilicon/Library/PciHostBridgeLib/PciHostBridgeLib.inf > delete mode 100644 Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciHostBridge.c > delete mode 100644 Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciHostBridge.h > delete mode 100644 Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciHostBridgeDxe.inf > delete mode 100644 Silicon/Hisilicon/Drivers/PciHostBridgeDxe/PciRootBridgeIo.c > rename {Platform/Hisilicon/D03 => Silicon/Hisilicon}/Drivers/PciPlatform/PciPlatform.c (89%) > rename {Platform/Hisilicon/D03 => Silicon/Hisilicon}/Drivers/PciPlatform/PciPlatform.inf (94%) > create mode 100644 Silicon/Hisilicon/Hi1610/Library/Hi161xPciPlatformLib/Hi161xPciPlatformLib.c > create mode 100644 Silicon/Hisilicon/Hi1610/Library/Hi161xPciPlatformLib/Hi161xPciPlatformLib.inf > create mode 100644 Silicon/Hisilicon/Hi1610/Library/Hi161xPciSegmentLib/Hi161xPciSegmentLib.inf > create mode 100644 Silicon/Hisilicon/Hi1610/Library/Hi161xPciSegmentLib/PciSegmentLib.c > > -- > 2.17.0 >