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::242; helo=mail-pl0-x242.google.com; envelope-from=ming.huang@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-pl0-x242.google.com (mail-pl0-x242.google.com [IPv6:2607:f8b0:400e:c01::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 06AC721BADAB2 for ; Wed, 15 Aug 2018 19:08:45 -0700 (PDT) Received: by mail-pl0-x242.google.com with SMTP id u11-v6so1290286plq.5 for ; Wed, 15 Aug 2018 19:08:45 -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=ZGJ7mOhTWlW9jfn0iprKqwV/vIsZzVnYVz7fBRjjw3I=; b=Fu7ysaiHd3IKWUccnEBqpv6PpmxJWAIoEFYnAbci2YMwlq2MCkV7GfUNleZqcYEI1Y kd491A82ZMs/fJXtK6yfmz+dOrvK/wuAyFi8tGfBfua8Q9HjnjBBf0YqlAACP/gDEI7g wbMraW7m2ZflzKG+vWAWO590MEO7/pVuqIL+w= 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=ZGJ7mOhTWlW9jfn0iprKqwV/vIsZzVnYVz7fBRjjw3I=; b=NTACfauqICP51BChoD++hZnvc9Ymw/F/kJMXEOhbzX8COfke8nBti3U4V0jRFTZ5HD JEMS+kJu2BBuVi0IgYKqD5nK6V0VVrSaMkbevYy1fPakvnO0GPHZv3M1HiwSoK2apQPK 5HxXLtarK9QQ7H6upjsfxU0I6GFJZcqPL+qOCGAOaLnUx7uQgC7I4vwV8cT/5y5DYpHf NBj4RDXuvqyBAxMAG6I1Uh2QGAhvfLleeGC4sRZTJjLIR5W61Oq6KUkskwWHrl3gRN/z mNTI50n0xvH4ENhGhzVw3fiwG5RXjZ7qzaWzwpvnNteEuMq3oWs42pGpuyW+NBMlKsIf IeAg== X-Gm-Message-State: AOUpUlEhCrzkLj5GEFcRXP1JPHJuZDJUHfUEI0LzxEph7uQZGQxt0Vh3 cI1y3gn3KST3I+LYe/noFozvmg== X-Google-Smtp-Source: AA+uWPwyWlZjF7st/gdKNDgXGQ+tz23sbdSG5L3lidHpDkRW15MH04auSq3TBxW29Mmp3THi8mxHjg== X-Received: by 2002:a17:902:8d8c:: with SMTP id v12-v6mr26511964plo.94.1534385325465; Wed, 15 Aug 2018 19:08:45 -0700 (PDT) Received: from [10.48.0.18] ([64.64.108.64]) by smtp.gmail.com with ESMTPSA id x66-v6sm42123003pff.123.2018.08.15.19.08.37 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 15 Aug 2018 19:08:44 -0700 (PDT) To: Leif Lindholm Cc: linaro-uefi@lists.linaro.org, edk2-devel@lists.01.org, graeme.gregory@linaro.org, ard.biesheuvel@linaro.org, michael.d.kinney@intel.com, lersek@redhat.com, guoheyi@huawei.com, wanghuiqiang@huawei.com, huangming23@huawei.com, zhangjinsong2@huawei.com, huangdaode@hisilicon.com, john.garry@huawei.com, xinliang.liu@linaro.org References: <20180814080451.48845-1-ming.huang@linaro.org> <20180814080451.48845-5-ming.huang@linaro.org> <20180815152218.ov5vq6csikzvfuir@bivouac.eciton.net> From: Ming Message-ID: <3e83faff-a32e-d8c1-b4df-f7f7df5fcf5e@linaro.org> Date: Thu, 16 Aug 2018 10:08:33 +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: <20180815152218.ov5vq6csikzvfuir@bivouac.eciton.net> Subject: Re: [PATCH edk2-non-osi v2 4/4] Silicon/Hisilicon/D06: Add Oem Shell libraries 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: Thu, 16 Aug 2018 02:08:46 -0000 Content-Type: text/plain; charset=gbk Content-Transfer-Encoding: 8bit ÔÚ 8/15/2018 11:22 PM, Leif Lindholm дµÀ: > On Tue, Aug 14, 2018 at 04:04:51PM +0800, Ming Huang wrote: >> Add Hisilicon Oem Shell libraries for some commands,such as >> biosupdate, getmac, setmac, provision, acpistartos, brdinfo, >> sfpinfo. >> >> biosupdate example: >> ifconfig -s eth2 static 192.168.2.68 255.255.255.0 192.168.2.1 >> biosupdate serverIp -u user -p password -f D06.fd -m master > > Is this expected to be used in reality? > There is capsule update and there is a BMC - why do you need a magic > platform-specific command in the Shell? > Is it a debug feature? Yes, it is a debug feature. Think about this case, we don't have a hpm file, and only have fd file or .cap file. for capsule update in uefi shell, ramdisk or USB disk is needed, or we must boot to OS to update .cap. It is convenient to update fd file in uefi shell. > >> acpistartos example: >> ifconfig -s eth2 static 192.168.2.69 255.255.255.0 192.168.2.1 >> provision serverIp -u user -p password -f os\Image -a 0x80000 >> 2 >> provision serverIp -u user -p password -f os\minifs.cpio.gz -a >> 0x07000000 >> 2 >> acpistartos > > Why is there a magic Shell comand to load a Linux kernel and > initramfs? This is the opposite of what UEFI is for. This is also for debug. It is convenient to boot OS while we don't have disk in board and PXE setting. > > / > Leif > >> Contributed-under: TianoCore Contribution Agreement 1.1 >> Signed-off-by: Ming Huang >> --- >> Silicon/Hisilicon/Hi1620/Library/HisiD06ShellCmdLib/HisiD06ShellCmdLib.inf | 41 ++++++++++++++ >> Silicon/Hisilicon/Library/HisiShellCmdLib/HisiShellCmdLib.inf | 59 ++++++++++++++++++++ >> Silicon/Hisilicon/Hi1620/Library/HisiD06ShellCmdLib/HisiD06ShellCmdLib.lib | Bin 0 -> 144880 bytes >> Silicon/Hisilicon/Library/HisiShellCmdLib/HisiShellCmdLib.lib | Bin 0 -> 1109382 bytes >> 4 files changed, 100 insertions(+) >> >> diff --git a/Silicon/Hisilicon/Hi1620/Library/HisiD06ShellCmdLib/HisiD06ShellCmdLib.inf b/Silicon/Hisilicon/Hi1620/Library/HisiD06ShellCmdLib/HisiD06ShellCmdLib.inf >> new file mode 100644 >> index 0000000..3dea3fb >> --- /dev/null >> +++ b/Silicon/Hisilicon/Hi1620/Library/HisiD06ShellCmdLib/HisiD06ShellCmdLib.inf >> @@ -0,0 +1,41 @@ >> +/** @file >> +* >> +* Copyright (c) 2018, Hisilicon Limited. All rights reserved. >> +* Copyright (c) 2018, Linaro Limited. All rights reserved. >> +* >> +* This program and the accompanying materials >> +* are licensed and made available under the terms and conditions of the BSD License >> +* which accompanies this distribution. The full text of the license may be found at >> +* http://opensource.org/licenses/bsd-license.php >> +* >> +* THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, >> +* WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. >> +* >> +**/ >> + >> + >> + >> +[Defines] >> + INF_VERSION = 0x0001001A >> + BASE_NAME = HisiD06ShellCmdLib >> + FILE_GUID = 41878264-1668-460f-b0bf-26bdc89d464c >> + MODULE_TYPE = UEFI_APPLICATION >> + VERSION_STRING = 1.0 >> + LIBRARY_CLASS = NULL|UEFI_APPLICATION UEFI_DRIVER >> + CONSTRUCTOR = HisiD06ShellCmdLibConstructor >> + DESTRUCTOR = HisiD06ShellCmdLibDestructor >> + >> + >> +[Binaries.AARCH64] >> + LIB|HisiD06ShellCmdLib.lib|* >> + >> +[Packages] >> + ShellPkg/ShellPkg.dec >> + Silicon/Hisilicon/HisiPkg.dec >> + >> +[LibraryClasses] >> + I2CLib >> + ShellLib >> + >> +[Guids] >> + HisiPlatformShellSampleHiiGuid ## CONSUMES ## HII >> diff --git a/Silicon/Hisilicon/Library/HisiShellCmdLib/HisiShellCmdLib.inf b/Silicon/Hisilicon/Library/HisiShellCmdLib/HisiShellCmdLib.inf >> new file mode 100644 >> index 0000000..a81af92 >> --- /dev/null >> +++ b/Silicon/Hisilicon/Library/HisiShellCmdLib/HisiShellCmdLib.inf >> @@ -0,0 +1,59 @@ >> +/** @file >> +* >> +* Copyright (c) 2018, Hisilicon Limited. All rights reserved. >> +* Copyright (c) 2018, Linaro Limited. All rights reserved. >> +* >> +* This program and the accompanying materials >> +* are licensed and made available under the terms and conditions of the BSD License >> +* which accompanies this distribution. The full text of the license may be found at >> +* http://opensource.org/licenses/bsd-license.php >> +* >> +* THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, >> +* WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. >> +* >> +**/ >> + >> + >> +[Defines] >> + INF_VERSION = 0x0001001A >> + BASE_NAME = HisiShellCmdLib >> + FILE_GUID = 88324505-C49D-4eac-8253-75D324457B88 >> + MODULE_TYPE = UEFI_APPLICATION >> + VERSION_STRING = 1.0 >> + LIBRARY_CLASS = NULL|UEFI_APPLICATION UEFI_DRIVER >> + CONSTRUCTOR = HisiShellCmdLibConstructor >> + DESTRUCTOR = HisiShellCmdLibDestructor >> + >> +[Binaries.AARCH64] >> + LIB|HisiShellCmdLib.lib|* >> + >> +[Packages] >> + ArmPkg/ArmPkg.dec >> + ShellPkg/ShellPkg.dec >> + Silicon/Hisilicon/HisiPkg.dec >> + >> +[LibraryClasses] >> + ArmSmcLib >> + IpmiCmdLib >> + PcdLib >> + ShellCommandLib >> + ShellLib >> + >> +[Pcd] >> + gEfiShellPkgTokenSpaceGuid.PcdShellProfileMask ## CONSUMES >> + gHisiTokenSpaceGuid.PcdMdioSubctrlAddress ## CONSUMES >> + gHisiTokenSpaceGuid.PcdStartOsParameter ## CONSUMES >> + >> +[FixedPcd.common] >> + gArmTokenSpaceGuid.PcdFdSize ## CONSUMES >> + gHisiTokenSpaceGuid.PcdBaseBoardProductName ## CONSUMES >> + gHisiTokenSpaceGuid.PcdTrustedFirmwareEnable ## CONSUMES >> + >> +[Protocols] >> + gHisiSpiFlashProtocolGuid ## CONSUMES >> + gOemNicProtocolGuid ## CONSUMES >> + gUniNorFlashProtocolGuid ## CONSUMES >> + >> +[Guids] >> + gVersionInfoHobGuid ## CONSUMES >> + HisiShellSampleHiiGuid ## CONSUMES ## HII >> diff --git a/Silicon/Hisilicon/Hi1620/Library/HisiD06ShellCmdLib/HisiD06ShellCmdLib.lib b/Silicon/Hisilicon/Hi1620/Library/HisiD06ShellCmdLib/HisiD06ShellCmdLib.lib >> new file mode 100644 >> index 0000000..9ba14a2 >> Binary files /dev/null and b/Silicon/Hisilicon/Hi1620/Library/HisiD06ShellCmdLib/HisiD06ShellCmdLib.lib differ >> diff --git a/Silicon/Hisilicon/Library/HisiShellCmdLib/HisiShellCmdLib.lib b/Silicon/Hisilicon/Library/HisiShellCmdLib/HisiShellCmdLib.lib >> new file mode 100644 >> index 0000000..d3ca398 >> Binary files /dev/null and b/Silicon/Hisilicon/Library/HisiShellCmdLib/HisiShellCmdLib.lib differ >> -- >> 2.17.0 >>