From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id DFA28C07E9D for ; Mon, 26 Sep 2022 11:37:17 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 0695784DA9; Mon, 26 Sep 2022 13:37:16 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=fail (p=none dis=none) header.from=arm.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Received: by phobos.denx.de (Postfix, from userid 109) id BDC4984DB5; Mon, 26 Sep 2022 13:37:13 +0200 (CEST) Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by phobos.denx.de (Postfix) with ESMTP id 9B70184C98 for ; Mon, 26 Sep 2022 13:37:10 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=abdellatif.elkhlifi@arm.com Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id A3045106F; Mon, 26 Sep 2022 04:37:16 -0700 (PDT) Received: from e121910.cambridge.arm.com (unknown [10.57.32.60]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id EDBB63F66F; Mon, 26 Sep 2022 04:37:08 -0700 (PDT) Date: Mon, 26 Sep 2022 12:37:06 +0100 From: Abdellatif El Khlifi To: Simon Glass Cc: u-boot@lists.denx.de, nd@arm.com Subject: Re: [PATCH v3 3/4] arm_ffa: introduce Arm FF-A low-level driver Message-ID: <20220926113705.GA12866@e121910.cambridge.arm.com> References: <20220801172053.20163-1-abdellatif.elkhlifi@arm.com> <20220801172053.20163-4-abdellatif.elkhlifi@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.6 at phobos.denx.de X-Virus-Status: Clean On Fri, Aug 12, 2022 at 08:21:02PM -0600, Simon Glass wrote: > Hi Abdellatif, > > On Mon, 1 Aug 2022 at 11:21, Abdellatif El Khlifi > wrote: > > > > Add the driver implementing Arm Firmware Framework for Armv8-A v1.0 > > > > The Firmware Framework for Arm A-profile processors (FF-A) > > describes interfaces (ABIs) that standardize communication > > between the Secure World and Normal World leveraging TrustZone > > technology. > > > > This driver uses 64-bit registers as per SMCCCv1.2 spec and comes > > on top of the SMCCC layer. The driver provides the FF-A ABIs needed for > > querying the FF-A framework from the secure world. > > > > 32-bit version of the ABIs is supported and 64-bit version of FFA_RXTX_MAP > > and FFA_MSG_SEND_DIRECT_{REQ, RESP}. > > > > In u-boot FF-A design, FF-A is considered as a discoverable bus. > > The Secure World is considered as one entity to communicate with > > using the FF-A bus. FF-A communication is handled by one device and > > one instance (the bus). This FF-A driver takes care of all the > > interactions between Normal world and Secure World. > > > > The driver exports its operations to be used by upper layers. > > > > Exported operations: > > > > - partition_info_get > > - sync_send_receive > > - rxtx_unmap > > > > This implementation provides an optional feature to copy the driver data > > to EFI runtime area. > > This is v2 but I don't see a change list? Sorry for not including that in the cover letter. Here is the changelog for v2 [1]. >From v3 upwards the changelog is included in the cover letter. [1] https://lore.kernel.org/all/20220509105514.GA4201@e121910.cambridge.arm.com/ > > > > > Signed-off-by: Abdellatif El Khlifi > > Cc: Tom Rini > > Cc: Ilias Apalodimas > > Cc: Jens Wiklander > > --- > > MAINTAINERS | 6 + > > common/board_r.c | 7 + > > drivers/Kconfig | 2 + > > drivers/Makefile | 1 + > > drivers/arm-ffa/Kconfig | 33 + > > drivers/arm-ffa/Makefile | 7 + > > drivers/arm-ffa/arm-ffa-uclass.c | 16 + > > drivers/arm-ffa/arm_ffa_prv.h | 219 ++++ > > drivers/arm-ffa/core.c | 1338 ++++++++++++++++++++ > > drivers/arm-ffa/efi_ffa_runtime_data_mgr.c | 94 ++ > > include/arm_ffa.h | 132 ++ > > include/dm/uclass-id.h | 1 + > > include/uuid.h | 8 + > > lib/efi_loader/efi_boottime.c | 17 + > > lib/uuid.c | 65 + > > 15 files changed, 1946 insertions(+) > > create mode 100644 drivers/arm-ffa/Kconfig > > create mode 100644 drivers/arm-ffa/Makefile > > create mode 100644 drivers/arm-ffa/arm-ffa-uclass.c > > create mode 100644 drivers/arm-ffa/arm_ffa_prv.h > > create mode 100644 drivers/arm-ffa/core.c > > create mode 100644 drivers/arm-ffa/efi_ffa_runtime_data_mgr.c > > create mode 100644 include/arm_ffa.h > > > > I think this should be in an existing directory, or perhaps drivers/firmware. Done in v4 (under drivers/firmware). > > Regards, > Simon