From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:4864:20::d41; helo=mail-io1-xd41.google.com; envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-io1-xd41.google.com (mail-io1-xd41.google.com [IPv6:2607:f8b0:4864:20::d41]) (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 E5A3821A07A92 for ; Mon, 26 Nov 2018 08:50:33 -0800 (PST) Received: by mail-io1-xd41.google.com with SMTP id v10so8839153ios.13 for ; Mon, 26 Nov 2018 08:50:33 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=Ydd4aq+fXH2sNgrrdu7/RXclnhmViqxi8vXB1i3Bam0=; b=TdE0QPByP+smxaLLT4wqwFlpzcmsQqGgmYjRcT2g1qB31hB2OUFlMX9m3N0e8jFAPF h0r5Uag6kW3qLAn0oHSAx8mIDdGy+m0DhCt63R5wPX3XOEgHvyZZ6l/oyIRJTKTp+ep0 B8DGIHquK+Gb06prL4G7IJ/MWfIrv9ZXhlZoc= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Ydd4aq+fXH2sNgrrdu7/RXclnhmViqxi8vXB1i3Bam0=; b=MbY1Zqnh+6EhGumDitbibfv/APdfCgo9kh3SwUd9rI9R5ea5UFpIKp5XXRjbCd9wiL XiHsOuP0esmv7GMp8LS4sDkqq+lxq8ActTxwSKY+d7N33S1vrGcRbi1Owl7zZzbIt3XX tXAlXJiTU53Vd8QjYKJb//RggXgRhDHXxga0nTAzEk3jxOhZqnNYy0jDIqh+hIHeBRg6 rOHR32ejhaxjT3lInTKI1AFcUG7/A+/D2wsAFbVKMJ78XuuThaNvbmaO4yv7M15ASLbz VtWE1vL/6dq6I3Qjo2IdXrPCL3vvIzEmjkHdHe0sRUpbfcM5elcOS32okr+ct76vI/hW fHaA== X-Gm-Message-State: AA+aEWbf710RoZNFNC7eGNoI5h+++E2dMV0gfJFknVN1Skl/l8IMwwlp TuRPDbDfX320oNjGFlszv7v/OrGODq8+9aVEwP3JGg== X-Google-Smtp-Source: AFSGD/VIQaKO7nVWMfDTi7/8rvZqKGSstA7tD27LHohU2Ad6Q/iJm/edBm5z1M50OXXqegx9zijL8OXKCnVi/7ms76w= X-Received: by 2002:a6b:7a46:: with SMTP id k6mr22681159iop.60.1543251032704; Mon, 26 Nov 2018 08:50:32 -0800 (PST) MIME-Version: 1.0 References: <20181123185459.2870-1-ard.biesheuvel@linaro.org> <20181126151241.aup4brnxmbqytnty@bivouac.eciton.net> In-Reply-To: <20181126151241.aup4brnxmbqytnty@bivouac.eciton.net> From: Ard Biesheuvel Date: Mon, 26 Nov 2018 17:50:20 +0100 Message-ID: To: Leif Lindholm Cc: "edk2-devel@lists.01.org" Subject: Re: [PATCH] ArmPkg: remove now unused BsdLib.h X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 26 Nov 2018 16:50:34 -0000 Content-Type: text/plain; charset="UTF-8" On Mon, 26 Nov 2018 at 16:12, Leif Lindholm wrote: > > On Fri, Nov 23, 2018 at 07:54:59PM +0100, Ard Biesheuvel wrote: > > The last remaining users of the BdsLib.h header reside in the > > edk2-platforms tree, and so it has been copied there. This > > allows us to remove the original from ArmPkg. > > > > Contributed-under: TianoCore Contribution Agreement 1.1 > > Signed-off-by: Ard Biesheuvel > > Reviewed-by: Leif Lindholm > Thanks. This series (and the Platform/Comcast patch) pushed as 397bbafdbff3..f98fb46d3a3d > > --- > > ArmPkg/Include/Library/BdsLib.h | 212 -------------------- > > 1 file changed, 212 deletions(-) > > > > diff --git a/ArmPkg/Include/Library/BdsLib.h b/ArmPkg/Include/Library/BdsLib.h > > deleted file mode 100644 > > index 4528c2e8739b..000000000000 > > --- a/ArmPkg/Include/Library/BdsLib.h > > +++ /dev/null > > @@ -1,212 +0,0 @@ > > -/** @file > > -* > > -* Copyright (c) 2013-2015, ARM 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. > > -* > > -**/ > > - > > -#ifndef __BDS_ENTRY_H__ > > -#define __BDS_ENTRY_H__ > > - > > -#define IS_DEVICE_PATH_NODE(node,type,subtype) \ > > - (((node)->Type == (type)) && ((node)->SubType == (subtype))) > > - > > -/** > > - This is defined by the UEFI specs, don't change it > > -**/ > > -typedef struct { > > - UINT16 LoadOptionIndex; > > - EFI_LOAD_OPTION *LoadOption; > > - UINTN LoadOptionSize; > > - > > - UINT32 Attributes; > > - UINT16 FilePathListLength; > > - CHAR16 *Description; > > - EFI_DEVICE_PATH_PROTOCOL *FilePathList; > > - > > - VOID* OptionalData; > > - UINTN OptionalDataSize; > > -} BDS_LOAD_OPTION; > > - > > -/** > > - Connect a Device Path and return the handle of the driver that support this DevicePath > > - > > - @param DevicePath Device Path of the File to connect > > - @param Handle Handle of the driver that support this DevicePath > > - @param RemainingDevicePath Remaining DevicePath nodes that do not match the driver DevicePath > > - > > - @retval EFI_SUCCESS A driver that matches the Device Path has been found > > - @retval EFI_NOT_FOUND No handles match the search. > > - @retval EFI_INVALID_PARAMETER DevicePath or Handle is NULL > > - > > -**/ > > -EFI_STATUS > > -BdsConnectDevicePath ( > > - IN EFI_DEVICE_PATH_PROTOCOL* DevicePath, > > - OUT EFI_HANDLE *Handle, > > - OUT EFI_DEVICE_PATH_PROTOCOL **RemainingDevicePath > > - ); > > - > > -/** > > - Connect all DXE drivers > > - > > - @retval EFI_SUCCESS All drivers have been connected > > - @retval EFI_NOT_FOUND No handles match the search. > > - @retval EFI_OUT_OF_RESOURCES There is not resource pool memory to store the matching results. > > - > > -**/ > > -EFI_STATUS > > -BdsConnectAllDrivers ( > > - VOID > > - ); > > - > > -/** > > - Return the value of a global variable defined by its VariableName. > > - The variable must be defined with the VendorGuid gEfiGlobalVariableGuid. > > - > > - @param VariableName A Null-terminated string that is the name of the vendor's > > - variable. > > - @param DefaultValue Value returned by the function if the variable does not exist > > - @param DataSize On input, the size in bytes of the return Data buffer. > > - On output the size of data returned in Data. > > - @param Value Value read from the UEFI Variable or copy of the default value > > - if the UEFI Variable does not exist > > - > > - @retval EFI_SUCCESS All drivers have been connected > > - @retval EFI_NOT_FOUND No handles match the search. > > - @retval EFI_OUT_OF_RESOURCES There is not resource pool memory to store the matching results. > > - > > -**/ > > -EFI_STATUS > > -GetGlobalEnvironmentVariable ( > > - IN CONST CHAR16* VariableName, > > - IN VOID* DefaultValue, > > - IN OUT UINTN* Size, > > - OUT VOID** Value > > - ); > > - > > -/** > > - Return the value of the variable defined by its VariableName and VendorGuid > > - > > - @param VariableName A Null-terminated string that is the name of the vendor's > > - variable. > > - @param VendorGuid A unique identifier for the vendor. > > - @param DefaultValue Value returned by the function if the variable does not exist > > - @param DataSize On input, the size in bytes of the return Data buffer. > > - On output the size of data returned in Data. > > - @param Value Value read from the UEFI Variable or copy of the default value > > - if the UEFI Variable does not exist > > - > > - @retval EFI_SUCCESS All drivers have been connected > > - @retval EFI_NOT_FOUND No handles match the search. > > - @retval EFI_OUT_OF_RESOURCES There is not resource pool memory to store the matching results. > > - > > -**/ > > -EFI_STATUS > > -GetEnvironmentVariable ( > > - IN CONST CHAR16* VariableName, > > - IN EFI_GUID* VendorGuid, > > - IN VOID* DefaultValue, > > - IN OUT UINTN* Size, > > - OUT VOID** Value > > - ); > > - > > -EFI_STATUS > > -BootOptionFromLoadOptionIndex ( > > - IN UINT16 LoadOptionIndex, > > - OUT BDS_LOAD_OPTION** BdsLoadOption > > - ); > > - > > -EFI_STATUS > > -BootOptionFromLoadOptionVariable ( > > - IN CHAR16* BootVariableName, > > - OUT BDS_LOAD_OPTION** BdsLoadOption > > - ); > > - > > -EFI_STATUS > > -BootOptionToLoadOptionVariable ( > > - IN BDS_LOAD_OPTION* BdsLoadOption > > - ); > > - > > -UINT16 > > -BootOptionAllocateBootIndex ( > > - VOID > > - ); > > - > > -/** > > - Start an EFI Application from a Device Path > > - > > - @param ParentImageHandle Handle of the calling image > > - @param DevicePath Location of the EFI Application > > - > > - @retval EFI_SUCCESS All drivers have been connected > > - @retval EFI_NOT_FOUND The Linux kernel Device Path has not been found > > - @retval EFI_OUT_OF_RESOURCES There is not enough resource memory to store the matching results. > > - > > -**/ > > -EFI_STATUS > > -BdsStartEfiApplication ( > > - IN EFI_HANDLE ParentImageHandle, > > - IN EFI_DEVICE_PATH_PROTOCOL *DevicePath, > > - IN UINTN LoadOptionsSize, > > - IN VOID* LoadOptions > > - ); > > - > > -EFI_STATUS > > -BdsLoadImage ( > > - IN EFI_DEVICE_PATH *DevicePath, > > - IN EFI_ALLOCATE_TYPE Type, > > - IN OUT EFI_PHYSICAL_ADDRESS* Image, > > - OUT UINTN *FileSize > > - ); > > - > > -/** > > - * Call BS.ExitBootServices with the appropriate Memory Map information > > - */ > > -EFI_STATUS > > -ShutdownUefiBootServices ( > > - VOID > > - ); > > - > > -/** > > - Locate an EFI application in a the Firmware Volumes by its name > > - > > - @param EfiAppGuid Guid of the EFI Application into the Firmware Volume > > - @param DevicePath EFI Device Path of the EFI application > > - > > - @return EFI_SUCCESS The function completed successfully. > > - @return EFI_NOT_FOUND The protocol could not be located. > > - @return EFI_OUT_OF_RESOURCES There are not enough resources to find the protocol. > > - > > -**/ > > -EFI_STATUS > > -LocateEfiApplicationInFvByName ( > > - IN CONST CHAR16* EfiAppName, > > - OUT EFI_DEVICE_PATH **DevicePath > > - ); > > - > > -/** > > - Locate an EFI application in a the Firmware Volumes by its GUID > > - > > - @param EfiAppGuid Guid of the EFI Application into the Firmware Volume > > - @param DevicePath EFI Device Path of the EFI application > > - > > - @return EFI_SUCCESS The function completed successfully. > > - @return EFI_NOT_FOUND The protocol could not be located. > > - @return EFI_OUT_OF_RESOURCES There are not enough resources to find the protocol. > > - > > -**/ > > -EFI_STATUS > > -LocateEfiApplicationInFvByGuid ( > > - IN CONST EFI_GUID *EfiAppGuid, > > - OUT EFI_DEVICE_PATH **DevicePath > > - ); > > - > > -#endif > > -- > > 2.19.1 > >