From nobody Wed May 8 09:04:33 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) client-ip=66.175.222.12; envelope-from=bounce+27952+39497+1787277+3901457@groups.io; helo=web01.groups.io; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+39497+1787277+3901457@groups.io; dmarc=fail(p=none dis=none) header.from=linaro.org ARC-Seal: i=1; a=rsa-sha256; t=1556112598; cv=none; d=zoho.com; s=zohoarc; b=KvjgWF0/9clE2PqruLkixjI0SwhMUF2+G1qHg774HLxrJA2JwLSyi7bqaS6aM8FDXA6F9mms9UP0mXFSIxFOrVlia0dPZerc+7zRmjpd5oNrz5r1/K5S+Nk1I67Xf1z+RlX8aDxcu7vvGlxjxI6hzfWhOVje+QqEMhmJdC8Is3g= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1556112598; h=Content-Transfer-Encoding:Cc:Date:From:List-Id:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Sender:Subject:To:ARC-Authentication-Results; bh=Iro2w1xbTCaUcI3/mvyT5u8I8cPjS6C2iKMvZbKTHfo=; b=IDOIQf203ofc3+rHtBtI9FhVfO9fj4XmQf0MpWB+rWW/JGTHObNQj3s3BbCpa/0RapGypHRIqE6vppY5mz16qel65c2lw6Zz2v9cVAjAEF21qVGoZAIPpSN5hEsSghSVqdPs3Zye8uQvauvYoxEsAkAIm+GKWr1yRKJATjfTygg= ARC-Authentication-Results: i=1; mx.zoho.com; dkim=pass; spf=pass (zoho.com: domain of groups.io designates 66.175.222.12 as permitted sender) smtp.mailfrom=bounce+27952+39497+1787277+3901457@groups.io; dmarc=fail header.from= (p=none dis=none) header.from= Received: from web01.groups.io (web01.groups.io [66.175.222.12]) by mx.zohomail.com with SMTPS id 15561125985212.6061424515013414; Wed, 24 Apr 2019 06:29:58 -0700 (PDT) Return-Path: X-Received: from mail-wm1-f66.google.com (mail-wm1-f66.google.com [209.85.128.66]) by groups.io with SMTP; Wed, 24 Apr 2019 06:29:57 -0700 X-Received: by mail-wm1-f66.google.com with SMTP id v14so5090454wmf.2 for ; Wed, 24 Apr 2019 06:29:57 -0700 (PDT) X-Gm-Message-State: APjAAAWtQ5SsQEl527Ijn9L5h+KLXKKgtPGJ1aGjcK0nwliYZcDQwclw TcpO2EOlY9m15oC3vFzmqeQ80TEWSDVEgw== X-Google-Smtp-Source: APXvYqxmSbrlN9ASSFd8ZhVffBPGSPAmpwn46F7BAZjxqKILxo9yLYYpzl0P3RjiuOV38EkSZ4kk2g== X-Received: by 2002:a1c:6587:: with SMTP id z129mr6394495wmb.84.1556112595256; Wed, 24 Apr 2019 06:29:55 -0700 (PDT) X-Received: from localhost.localdomain (91-167-84-221.subs.proxad.net. [91.167.84.221]) by smtp.gmail.com with ESMTPSA id s189sm36374996wmf.45.2019.04.24.06.29.54 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 24 Apr 2019 06:29:54 -0700 (PDT) From: "Ard Biesheuvel" To: devel@edk2.groups.io Cc: leif.lindholm@linaro.org, Ard Biesheuvel Subject: [edk2-devel] [PATCH edk2-platforms] Silicon/AMD/Styx: remove the non-capsule based flasher tool Date: Wed, 24 Apr 2019 15:29:52 +0200 Message-Id: <20190424132952.8883-1-ard.biesheuvel@linaro.org> MIME-Version: 1.0 Precedence: Bulk List-Unsubscribe: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Reply-To: devel@edk2.groups.io,ard.biesheuvel@linaro.org Content-Transfer-Encoding: quoted-printable DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=groups.io; q=dns/txt; s=20140610; t=1556112598; bh=fHSkBAp+KrYGY3P7nbbgIRTvVB4fu8Qh6cIBOyU2hBA=; h=Cc:Date:From:Reply-To:Subject:To; b=mtAexNIiCYCMQll4FZnRnMn57mOEjHORbI+URYHe7BKE+FxWYmAAW9LT8s8JKqtEbvk cSx6OpL4tjKFdcXQU1ec5oNtoVYvqStU0h91loWES9cXxiLbjwWeuS7tZ8lb5IYmDZGBl heYOPTqksZaFOeNN5TuqlO+JdGN/i9N1Z2I= X-ZohoMail-DKIM: pass (identity @groups.io) Content-Type: text/plain; charset="utf-8" We have had capsule support enabled on this platform for a while now, so let's drop the hacked up flasher tool that we no longer have a need for. Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Ard Biesheuvel Reviewed-by: Leif Lindholm --- Platform/AMD/OverdriveBoard/OverdriveBoard.dsc | 8 -- Platform/LeMaker/CelloBoard/CelloBoard.dsc | 8 -- Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc | 8 -- Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.inf | 53 -----= ------ Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.c | 96 -----= --------------- Silicon/AMD/Styx/Applications/StyxFlashUefi/Scripts/GccBase.lds | 86 -----= ------------- Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashImage.S | 25 ----- 7 files changed, 284 deletions(-) diff --git a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc b/Platform/AMD/= OverdriveBoard/OverdriveBoard.dsc index 0e1db0745d89..616553172bf0 100644 --- a/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc +++ b/Platform/AMD/OverdriveBoard/OverdriveBoard.dsc @@ -18,7 +18,6 @@ [Defines] =20 DEFINE NUM_CORES =3D 8 -DEFINE DO_FLASHER =3D FALSE DEFINE X64EMU_ENABLE =3D FALSE =20 PLATFORM_NAME =3D Overdrive @@ -747,13 +746,6 @@ DEFINE X64EMU_ENABLE =3D FALSE gEfiMdePkgTokenSpaceGuid.PcdUefiLibMaxPrintBufferSize|8000 } =20 -!if $(DO_FLASHER) =3D=3D TRUE - Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.inf { - - ShellCEntryLib|ShellPkg/Library/UefiShellCEntryLib/UefiShellCEntryLi= b.inf - } -!endif - # # Firmware update # diff --git a/Platform/LeMaker/CelloBoard/CelloBoard.dsc b/Platform/LeMaker/= CelloBoard/CelloBoard.dsc index 56a31a97a4ec..c26d38226cd2 100644 --- a/Platform/LeMaker/CelloBoard/CelloBoard.dsc +++ b/Platform/LeMaker/CelloBoard/CelloBoard.dsc @@ -18,7 +18,6 @@ [Defines] =20 DEFINE NUM_CORES =3D 4 -DEFINE DO_FLASHER =3D FALSE =20 PLATFORM_NAME =3D Cello PLATFORM_GUID =3D 77861b3e-74b0-4ff3-8d18-c5ba5803e1bf @@ -676,10 +675,3 @@ DEFINE DO_FLASHER =3D FALSE !ifdef $(RENESAS_XHCI_FW_DIR) OpenPlatformPkg/Drivers/Xhci/RenesasFirmwarePD720202/RenesasFirmwarePD72= 0202.inf !endif - -!if $(DO_FLASHER) =3D=3D TRUE - Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.inf { - - ShellCEntryLib|ShellPkg/Library/UefiShellCEntryLib/UefiShellCEntryLi= b.inf - } -!endif diff --git a/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc b/= Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc index 6ae0f2620c38..4fbc4201cf6b 100644 --- a/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc +++ b/Platform/SoftIron/Overdrive1000Board/Overdrive1000Board.dsc @@ -18,7 +18,6 @@ [Defines] =20 DEFINE NUM_CORES =3D 4 -DEFINE DO_FLASHER =3D FALSE =20 PLATFORM_NAME =3D Overdrive1000 PLATFORM_GUID =3D 36774DD7-20DE-4C5B-8722-f8861DFF1F16 @@ -668,10 +667,3 @@ DEFINE DO_FLASHER =3D FALSE gEfiShellPkgTokenSpaceGuid.PcdShellLibAutoInitialize|FALSE gEfiMdePkgTokenSpaceGuid.PcdUefiLibMaxPrintBufferSize|8000 } - -!if $(DO_FLASHER) =3D=3D TRUE - Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.inf { - - ShellCEntryLib|ShellPkg/Library/UefiShellCEntryLib/UefiShellCEntryLi= b.inf - } -!endif diff --git a/Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.inf = b/Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.inf deleted file mode 100644 index 0c289dcf95f4..000000000000 --- a/Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.inf +++ /dev/null @@ -1,53 +0,0 @@ -#/** @file -# -# Copyright (c) 2017, Linaro Ltd. All rights reserved.
-# -# This program and the accompanying materials -# are licensed and made available under the terms and conditions of the B= SD 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 IM= PLIED. -# -#**/ - -[Defines] - INF_VERSION =3D 0x00010019 - BASE_NAME =3D StyxFlashUefi - FILE_GUID =3D 07b65d9d-b1a2-416e-bd04-0b61b775f924 - MODULE_TYPE =3D UEFI_APPLICATION - VERSION_STRING =3D 0.1 - ENTRY_POINT =3D ShellCEntryLib - -# -# VALID_ARCHITECTURES =3D AARCH64 -# - -[Sources] - StyxFlashImage.S - StyxFlashUefi.c - -[Packages] - ArmPkg/ArmPkg.dec - MdePkg/MdePkg.dec - ShellPkg/ShellPkg.dec - Silicon/AMD/Styx/AmdModulePkg/AmdModulePkg.dec - -[LibraryClasses] - BaseMemoryLib - ShellCEntryLib - UefiBootServicesTableLib - UefiLib - -[Protocols] - gAmdIscpDxeProtocolGuid - -[FixedPcd] - gArmTokenSpaceGuid.PcdFdBaseAddress - gArmTokenSpaceGuid.PcdFvBaseAddress - -[BuildOptions] - *_*_*_CC_FLAGS =3D -mcmodel=3Dsmall - *_*_*_DLINK_FLAGS =3D -z common-page-size=3D0x1000 -Wl,-T,$(MODULE_DIR)/= Scripts/GccBase.lds - *_*_*_PLATFORM_FLAGS =3D -I$(BIN_DIR)/../FV diff --git a/Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.c b/= Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.c deleted file mode 100644 index 8f181bc2a54d..000000000000 --- a/Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashUefi.c +++ /dev/null @@ -1,96 +0,0 @@ -/** @file - - Copyright (c) 2017, Linaro Ltd. All rights reserved.
- - This program and the accompanying materials - are licensed and made available under the terms and conditions of the BS= D License - which accompanies this distribution. The full text of the license may b= e 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 IMP= LIED. - -**/ - -#include -#include -#include -#include -#include - -#include - -#define UEFI_IMAGE_OFFSET FixedPcdGet64 (PcdFvBaseAddress) - FixedPcdG= et64 (PcdFdBaseAddress) -#define BLOCK_SIZE SIZE_64KB - -STATIC AMD_ISCP_DXE_PROTOCOL *mIscpDxeProtocol; -STATIC UINT8 Buffer[BLOCK_SIZE]; - -extern CONST UINT8 StyxFlashImageStart[]; -extern CONST UINT32 StyxFlashImageSize; - -/*** - Main entrypoint - - Establishes the main structure of the application. - - @retval 0 The application exited normally. - @retval Other An error occurred. -***/ -INTN -EFIAPI -ShellAppMain ( - IN UINTN Argc, - IN CHAR16 **Argv - ) -{ - EFI_STATUS Status; - UINTN Index; - INTN Remaining; - - Print (L"StyxFlashUefi: firmware updater for AMD Seattle based boards.\n= "); - - Status =3D gBS->LocateProtocol (&gAmdIscpDxeProtocolGuid, NULL, - (VOID **)&mIscpDxeProtocol); - if (EFI_ERROR (Status)) { - Print (L"Failed to locate ISCP communication protocol, terminating...\= n"); - return (INTN)Status; - } - - Index =3D 0; - Remaining =3D StyxFlashImageSize; - do { - Status =3D mIscpDxeProtocol->AmdExecuteEraseFvBlockDxe ( - mIscpDxeProtocol, - UEFI_IMAGE_OFFSET + Index * BLOCK_SIZE, - BLOCK_SIZE); - if (EFI_ERROR (Status)) { - Print (L"Erase failed!\n"); - return (INTN)Status; - } - - CopyMem (Buffer, StyxFlashImageStart + Index * BLOCK_SIZE, - MIN (Remaining, BLOCK_SIZE)); - - Status =3D mIscpDxeProtocol->AmdExecuteUpdateFvBlockDxe ( - mIscpDxeProtocol, - UEFI_IMAGE_OFFSET + Index * BLOCK_SIZE, - Buffer, - MIN (Remaining, BLOCK_SIZE)); - - if (EFI_ERROR (Status)) { - Print (L"Update failed!\n"); - return (INTN)Status; - } - - Remaining -=3D BLOCK_SIZE; - Index++; - - Print (L"Block %d of %d updated\n", Index, StyxFlashImageSize / BLOCK_= SIZE); - - } while (Remaining > 0); - - Print (L"\nDone!\n"); - - return 0; -} diff --git a/Silicon/AMD/Styx/Applications/StyxFlashUefi/Scripts/GccBase.ld= s b/Silicon/AMD/Styx/Applications/StyxFlashUefi/Scripts/GccBase.lds deleted file mode 100644 index 7a0c87c6e32b..000000000000 --- a/Silicon/AMD/Styx/Applications/StyxFlashUefi/Scripts/GccBase.lds +++ /dev/null @@ -1,86 +0,0 @@ -/** @file - - Unified linker script for GCC based builds - - Copyright (c) 2010 - 2015, Intel Corporation. All rights reserved.
- Copyright (c) 2015 - 2017, Linaro Ltd. All rights reserved.
- (C) Copyright 2016 Hewlett Packard Enterprise Development LP
- - This program and the accompanying materials are licensed and made availa= ble under - the terms and conditions of the BSD License that accompanies this distri= bution. - 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 IMP= LIED. - -**/ - -SECTIONS { - - /* - * The PE/COFF binary consists of DOS and PE/COFF headers, and a sequenc= e of - * section headers adding up to PECOFF_HEADER_SIZE bytes (which differs - * between 32-bit and 64-bit builds). The actual start of the .text sect= ion - * will be rounded up based on its actual alignment. - */ - . =3D PECOFF_HEADER_SIZE; - - .text : ALIGN(CONSTANT(COMMONPAGESIZE)) { - *(.text .text.* .stub .gnu.linkonce.t.*) - *(.rodata .rodata.* .gnu.linkonce.r.*) - *(.got .got.*) - - /* - * The contents of AutoGen.c files are mostly constant from the POV of= the - * program, but most of it ends up in .data or .bss by default since f= ew of - * the variable definitions that get emitted are declared as CONST. - * Unfortunately, we cannot pull it into the .text section entirely, s= ince - * patchable PCDs are also emitted here, but we can at least move all = of the - * emitted GUIDs here. - */ - *:AutoGen.obj(.data.g*Guid) - } - - /* - * The alignment of the .data section should be less than or equal to the - * alignment of the .text section. This ensures that the relative offset - * between these sections is the same in the ELF and the PE/COFF version= s of - * this binary. - */ - .data ALIGN(ALIGNOF(.text)) : ALIGN(CONSTANT(COMMONPAGESIZE)) { - *(.data .data.* .gnu.linkonce.d.*) - *(.bss .bss.*) - *(.payload) - } - - .eh_frame ALIGN(CONSTANT(COMMONPAGESIZE)) : { - KEEP (*(.eh_frame)) - } - - .rela (INFO) : { - *(.rela .rela.*) - } - - .hii : ALIGN(CONSTANT(COMMONPAGESIZE)) { - KEEP (*(.hii)) - } - - /* - * Retain the GNU build id but in a non-allocatable section so GenFw - * does not copy it into the PE/COFF image. - */ - .build-id (INFO) : { *(.note.gnu.build-id) } - - /DISCARD/ : { - *(.note.GNU-stack) - *(.gnu_debuglink) - *(.interp) - *(.dynsym) - *(.dynstr) - *(.dynamic) - *(.hash .gnu.hash) - *(.comment) - *(COMMON) - } -} diff --git a/Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashImage.S b= /Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashImage.S deleted file mode 100644 index 041339ee9b47..000000000000 --- a/Silicon/AMD/Styx/Applications/StyxFlashUefi/StyxFlashImage.S +++ /dev/null @@ -1,25 +0,0 @@ -/** @file - - Copyright (c) 2017, Linaro Ltd. All rights reserved.
- - This program and the accompanying materials - are licensed and made available under the terms and conditions of the BS= D License - which accompanies this distribution. The full text of the license may b= e 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 IMP= LIED. - -**/ - - .section ".payload" - .align 12 - -ASM_GLOBAL ASM_PFX(StyxFlashImageStart) -ASM_PFX(StyxFlashImageStart): - .incbin "STYX_EFI.Fv" - - .align 2 -ASM_GLOBAL ASM_PFX(StyxFlashImageSize) -ASM_PFX(StyxFlashImageSize): - .long . - ASM_PFX(StyxFlashImageStart) --=20 2.20.1 -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#39497): https://edk2.groups.io/g/devel/message/39497 Mute This Topic: https://groups.io/mt/31321859/1787277 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-