From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:400e:c01::244; helo=mail-pl0-x244.google.com; envelope-from=ming.huang@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-pl0-x244.google.com (mail-pl0-x244.google.com [IPv6:2607:f8b0:400e:c01::244]) (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 41102210D97B3 for ; Tue, 7 Aug 2018 08:04:40 -0700 (PDT) Received: by mail-pl0-x244.google.com with SMTP id u11-v6so7246025plq.5 for ; Tue, 07 Aug 2018 08:04:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=s5A7KJWa7Kn/OuYHnL/0qDkLhywfWtM2G214c2UE24Y=; b=afXBH5tjhyHJr5repikKzy+xVfuPj0QK21CmWexhdvbO+krzWrwiOlc6SUumFPq1p4 2gBBpxhX3FL48YR6JKg1gARpKcaCtjGMjwWhOjFBfmjWpYL6qMaf4E59NzGPd1isz/E1 hWASZgd1einHeJfh6FkNVHlpYRWybMo9zFvFc= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=s5A7KJWa7Kn/OuYHnL/0qDkLhywfWtM2G214c2UE24Y=; b=Q0ky4huMri85Wwd4Rb223MxhKc+x4/wk248HavmApPmlF1uxHvjf4zRZsG56W+RsM2 UwcxKtybWqOC1VWe8tTut4tjqsj4xwaOGP40BPfvAA/41CtpuAnA9RM+4DQ2kFEZEF9n tXEs6+TRTkuq4kpYJA3G53p+hW0HmxDbjbczkQYuGo2XHvfEzOcWxahp5lrgmMYCCzPd jRAh6XXMhNr5Jyo006M+6EoEN9Fj3oL0yV9llNA49zOyrVFGuRSFqwbXKqC0P6ftqN7x ohAOLv33pPZ/hh7fmcxtGtoUsMh85GyJVDCd6cG8T03bC1ERwqrMheTD44fHPrXi93Ds R/iQ== X-Gm-Message-State: AOUpUlFpXugPYC0Priybgijg7MH94lu9/TQH10dc1G4OfleOUmuX8VIh tK41OSsXZ+j64cch0Puz0mKHLQ== X-Google-Smtp-Source: AAOMgpcdXhcZNSkPCwjVwCFKB8hsLda7mZ3Tu0yqvgJsTvzh5cHURGvNYBQ/yV9za7l8smCtczJeZw== X-Received: by 2002:a17:902:6b0b:: with SMTP id o11-v6mr18094103plk.214.1533654280697; Tue, 07 Aug 2018 08:04:40 -0700 (PDT) Received: from [10.199.0.182] ([64.64.108.224]) by smtp.gmail.com with ESMTPSA id n24-v6sm3946264pfi.161.2018.08.07.08.04.30 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Tue, 07 Aug 2018 08:04:39 -0700 (PDT) To: Leif Lindholm Cc: linaro-uefi@lists.linaro.org, edk2-devel@lists.01.org, graeme.gregory@linaro.org, ard.biesheuvel@linaro.org, guoheyi@huawei.com, wanghuiqiang@huawei.com, huangming23@huawei.com, zhangjinsong2@huawei.com, huangdaode@hisilicon.com, john.garry@huawei.com, xinliang.liu@linaro.org, Heyi Guo References: <20180724070922.63362-1-ming.huang@linaro.org> <20180724070922.63362-6-ming.huang@linaro.org> <20180802170556.aofxspo4mctd6a3b@bivouac.eciton.net> From: Ming Message-ID: <28cb204b-92af-d862-8af7-969bc0e9de72@linaro.org> Date: Tue, 7 Aug 2018 23:04:24 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: <20180802170556.aofxspo4mctd6a3b@bivouac.eciton.net> Subject: Re: [PATCH edk2-platforms v1 05/38] Platform/Hisilicon/D06: Add binary file for D06 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: Tue, 07 Aug 2018 15:04:41 -0000 Content-Type: text/plain; charset=gbk Content-Transfer-Encoding: 8bit ÔÚ 8/3/2018 1:05 AM, Leif Lindholm дµÀ: > Could the subject be changed to "add edk2-non-osi components for D06"? > OK > I should point out that I really like how this is done as a separate > patch. > The binary libraries are need for following patch, so this patch is added here. > On Tue, Jul 24, 2018 at 03:08:49PM +0800, Ming Huang wrote: >> Contributed-under: TianoCore Contribution Agreement 1.1 >> Signed-off-by: Ming Huang >> Signed-off-by: Heyi Guo >> --- >> Platform/Hisilicon/D06/D06.dsc | 7 +++++++ >> Platform/Hisilicon/D06/D06.fdf | 17 +++++++++++++++++ >> 2 files changed, 24 insertions(+) >> >> diff --git a/Platform/Hisilicon/D06/D06.dsc b/Platform/Hisilicon/D06/D06.dsc >> index e05c97e1c6..88869ba26e 100644 >> --- a/Platform/Hisilicon/D06/D06.dsc >> +++ b/Platform/Hisilicon/D06/D06.dsc >> @@ -42,6 +42,8 @@ >> >> I2CLib|Silicon/Hisilicon/Library/I2CLib/I2CLib.inf >> TimerLib|ArmPkg/Library/ArmArchTimerLib/ArmArchTimerLib.inf >> + IpmiCmdLib|Silicon/Hisilicon/Hi1610/Library/IpmiCmdLib/IpmiCmdLib.inf >> + > > Hmm, does this suggest that the IpmiCmdLib should move out of Hi1610 > to Silicon/Hisilicon/Library? > This is now the fourth platform making use of the Hi1610 variant. > > / > Leif > It is better to move out of Hi1610. I will add a patch to this patch set for D0x and add another patch for edk2-non-osi. Ming >> NetLib|MdeModulePkg/Library/DxeNetLib/DxeNetLib.inf >> DpcLib|MdeModulePkg/Library/DxeDpcLib/DxeDpcLib.inf >> HiiLib|MdeModulePkg/Library/UefiHiiLib/UefiHiiLib.inf >> @@ -65,8 +67,12 @@ >> >> CpldIoLib|Silicon/Hisilicon/Library/CpldIoLib/CpldIoLib.inf >> >> + SerdesLib|Silicon/Hisilicon/Hi1620/Library/Hi1620Serdes/Hi1620SerdesLib.inf >> + >> TimeBaseLib|EmbeddedPkg/Library/TimeBaseLib/TimeBaseLib.inf >> RealTimeClockLib|Platform/Hisilicon/D06/Library/M41T83RealTimeClockLib/M41T83RealTimeClockLib.inf >> + OemAddressMapLib|Platform/Hisilicon/D06/Library/OemAddressMapD06/OemAddressMapD06.inf >> + PlatformSysCtrlLib|Silicon/Hisilicon/Hi1620/Library/PlatformSysCtrlLibHi1620/PlatformSysCtrlLibHi1620.inf >> >> CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf >> GenericBdsLib|IntelFrameworkModulePkg/Library/GenericBdsLib/GenericBdsLib.inf >> @@ -82,6 +88,7 @@ >> # USB Requirements >> UefiUsbLib|MdePkg/Library/UefiUsbLib/UefiUsbLib.inf >> >> + LpcLib|Silicon/Hisilicon/Hi1620/Library/LpcLibHi1620/LpcLib.inf >> SerialPortLib|ArmPlatformPkg/Library/PL011SerialPortLib/PL011SerialPortLib.inf >> !if $(SECURE_BOOT_ENABLE) == TRUE >> FileExplorerLib|MdeModulePkg/Library/FileExplorerLib/FileExplorerLib.inf >> diff --git a/Platform/Hisilicon/D06/D06.fdf b/Platform/Hisilicon/D06/D06.fdf >> index 93c464c9f7..2730eb42a9 100644 >> --- a/Platform/Hisilicon/D06/D06.fdf >> +++ b/Platform/Hisilicon/D06/D06.fdf >> @@ -56,6 +56,7 @@ NumBlocks = 0x40 >> >> 0x00000000|0x00100000 >> gArmTokenSpaceGuid.PcdSecureFvBaseAddress|gArmTokenSpaceGuid.PcdSecureFvSize >> +FILE = Platform/Hisilicon/D06/Sec/FVMAIN_SEC.Fv >> >> 0x00100000|0x00280000 >> gArmTokenSpaceGuid.PcdFvBaseAddress|gArmTokenSpaceGuid.PcdFvSize >> @@ -163,6 +164,7 @@ READ_LOCK_STATUS = TRUE >> INF MdeModulePkg/Core/Dxe/DxeMain.inf >> INF MdeModulePkg/Universal/PCD/Dxe/Pcd.inf >> >> + INF Platform/Hisilicon/D06/Drivers/IoInitDxe/IoInitDxe.inf >> # >> # PI DXE Drivers producing Architectural Protocols (EFI Services) >> # >> @@ -170,6 +172,7 @@ READ_LOCK_STATUS = TRUE >> INF MdeModulePkg/Core/RuntimeDxe/RuntimeDxe.inf >> >> INF MdeModulePkg/Universal/SecurityStubDxe/SecurityStubDxe.inf >> + INF Platform/Hisilicon/D06/Drivers/SFC/SfcDxeDriver.inf >> >> >> INF Silicon/Hisilicon/Drivers/FlashFvbDxe/FlashFvbDxe.inf >> @@ -225,10 +228,15 @@ READ_LOCK_STATUS = TRUE >> INF MdeModulePkg/Bus/Usb/UsbMouseDxe/UsbMouseDxe.inf >> INF MdeModulePkg/Bus/Usb/UsbMassStorageDxe/UsbMassStorageDxe.inf >> >> + INF Platform/Hisilicon/D06/Drivers/Ipmi/IpmiInterfaceDxe/IpmiInterfaceDxe.inf >> + INF Platform/Hisilicon/D06/Drivers/GetInfoFromBmc/GetInfoFromBmc.inf >> INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf >> INF Silicon/Hisilicon/Drivers/Smbios/SmbiosMiscDxe/SmbiosMiscDxe.inf >> INF Silicon/Hisilicon/Drivers/Smbios/AddSmbiosType9/AddSmbiosType9.inf >> + INF Platform/Hisilicon/D06/Drivers/TransferSmbiosInfo/TransSmbiosInfo.inf >> + INF Platform/Hisilicon/D06/Drivers/IpmiMiscOpDxe/IpmiMiscOpDxe.inf >> >> + INF Platform/Hisilicon/D06/Drivers/IpmiWatchdogDxe/IpmiWatchdogDxe.inf >> >> >> INF Silicon/Hisilicon/Drivers/Smbios/MemorySubClassDxe/MemorySubClassDxe.inf >> @@ -246,6 +254,7 @@ READ_LOCK_STATUS = TRUE >> # >> #Network >> # >> + INF Platform/Hisilicon/D06/Drivers/Net/SnpHi1620NewDxe/SnpDxe.inf >> >> INF MdeModulePkg/Universal/Network/SnpDxe/SnpDxe.inf >> INF MdeModulePkg/Universal/Network/ArpDxe/ArpDxe.inf >> @@ -282,8 +291,14 @@ READ_LOCK_STATUS = TRUE >> INF ArmPkg/Drivers/ArmPciCpuIo2Dxe/ArmPciCpuIo2Dxe.inf >> INF MdeModulePkg/Bus/Pci/PciHostBridgeDxe/PciHostBridgeDxe.inf >> INF MdeModulePkg/Bus/Pci/PciBusDxe/PciBusDxe.inf >> + INF Platform/Hisilicon/D06/Drivers/PcieRasInitDxe/PcieRasInitDxe.inf >> + INF Platform/Hisilicon/D06/Drivers/RasInitDxe/RasInitDxe.inf >> >> + # VGA Driver >> + # >> + INF Platform/Hisilicon/D06/Drivers/Sm750Dxe/UefiSmi.inf >> INF MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpressDxe.inf >> + INF Platform/Hisilicon/D06/Drivers/Sas/SasDxeDriver.inf >> INF MdeModulePkg/Bus/Pci/SataControllerDxe/SataControllerDxe.inf >> INF MdeModulePkg/Bus/Ata/AtaAtapiPassThru/AtaAtapiPassThru.inf >> INF MdeModulePkg/Bus/Ata/AtaBusDxe/AtaBusDxe.inf >> @@ -335,6 +350,8 @@ READ_LOCK_STATUS = TRUE >> >> INF Silicon/Hisilicon/Drivers/VersionInfoPeim/VersionInfoPeim.inf >> >> + INF Platform/Hisilicon/D06/Drivers/Ipmi/IpmiInterfacePei/IpmiInterfacePei.inf >> + INF Platform/Hisilicon/D06/MemoryInitPei/MemoryInitPeim.inf >> INF ArmPkg/Drivers/CpuPei/CpuPei.inf >> INF MdeModulePkg/Universal/PCD/Pei/Pcd.inf >> INF IntelFrameworkModulePkg/Universal/StatusCode/Pei/StatusCodePei.inf >> -- >> 2.17.0 >>