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 X-Spam-Level: X-Spam-Status: No, score=-8.2 required=3.0 tests=DKIM_ADSP_CUSTOM_MED, DKIM_SIGNED,DKIM_VALID,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 21DAFC10F14 for ; Wed, 10 Apr 2019 15:32:04 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id E665A20818 for ; Wed, 10 Apr 2019 15:32:03 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="eZEEQUOl"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="Pr4sT7WD" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org E665A20818 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=1mxPeMbh1NmdHfmxxL1hXPX7B8UF5iK3i6v5zGkCsA8=; b=eZEEQUOlD6io6J8z/4ShZg6oc ytdpgQZjQJe788idUZacKWdgm2Ti+P9syNfNBGRDoVd0IWtu73niZ+sw3IrMQhAfaU9QYWqw9Ev4i bVFrW7EXMWCepwtToipEU6DaI7dFmDZr1NqWllRCRoFUcCwo6lta9MrDczA/KhuCGhZV71HWy/kbW zBc5K1D369/H/Svsgu8s0tWdgfOm8K9bR6GLj7/hvmaUzlkzKRBeIUu4ZGxfLPqNQep3lXHBkDY3y KwiZmJzIvXnmraGAwi9bUpgLJHJUjmMFX7OoSclmfE9jS/e1bHs4ceTA1l19DcW5i1egNBoto8kqC 7ZgqKUhWg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1hEFCl-0006pD-30; Wed, 10 Apr 2019 15:32:03 +0000 Received: from mail-wm1-x344.google.com ([2a00:1450:4864:20::344]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1hEFCh-0006oQ-3y for linux-arm-kernel@lists.infradead.org; Wed, 10 Apr 2019 15:32:00 +0000 Received: by mail-wm1-x344.google.com with SMTP id o25so3092031wmf.5 for ; Wed, 10 Apr 2019 08:31:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=a70Ngf20cZ0gaEh6gWhiCfY1BMm7qQ8iXOYVdgnR8YQ=; b=Pr4sT7WDh87DkV8DZX7S/sUtMHizplRKMySkaecvDLwo5pu2RWmiVkgxjHWVWn4/eM kBabjGtleQN9bj4NRXdY5oOqxnLRbm5/VWCc312FkA7B+kZzZOSbLA8SpPiKwkt3xOKe KXCusm1S2XU8UBwidUmgDhXb1RPKC6sUE50szzLfmTiMsdSJkqWHbyGgda+QZ6+TVIzL 3Nx7WX55KMdLMMNkqrWjTklw1g+DzHHEOdB0alAAKc/E/kczWO6EgbHFC8gDh4YQD/Ir 3vPC08R/O+xIiNR/jfnl/oVapnfxke7MiPxbj11MJ6iD2jnzn81Xx63sLjb8X9pTJIqF BSVw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=a70Ngf20cZ0gaEh6gWhiCfY1BMm7qQ8iXOYVdgnR8YQ=; b=V7fMxLSdPsG4XIGN3sfEVOofK4Cz6fSgVY2r7yJcjTZVS9yPMFG7JO3b+/GYcslJAk 7UtP9+1G88G1bAkNhuU+zMTJJjuQLmlaJ6dYJyNhoGAZOi4qp/rZ6ryd3SXvuNCczamE 6Lf7YtdCC5Vc/PeV8C+2uxO+FADu5Col3xGAv1y6eqdS37SIzAVIVuJwNcj36qK4iLex 5Qsy2pFZw84sXC4JQREcF27dsG36SMfJ71dTQK3IVoD74O0J1M3W/B/jim/Il4lywkn8 N/r+vvvl4EiNnga2E9t/SITCwAUN5RcIhm/a24rf2OY/ThNxTOcqqbw3Z5Cqd6eu3WrF spSQ== X-Gm-Message-State: APjAAAV7P5c1cubf8NSb+bl5KGEBJjYeg+MQ/ThCPqw02t4n31PZ5qFU L7RcNUjXepb63+S9s9gAz5A= X-Google-Smtp-Source: APXvYqwBxFLVA5Q07/wrMZKyQaWU+l0Byo43jfdBpAmOInjUF2uSoMARRJPYhN4XutKgvEIwMCIY1A== X-Received: by 2002:a1c:cf83:: with SMTP id f125mr3254428wmg.96.1554910317048; Wed, 10 Apr 2019 08:31:57 -0700 (PDT) Received: from localhost (pD9E51B25.dip0.t-ipconnect.de. [217.229.27.37]) by smtp.gmail.com with ESMTPSA id s16sm29896483wrw.58.2019.04.10.08.31.55 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Wed, 10 Apr 2019 08:31:56 -0700 (PDT) Date: Wed, 10 Apr 2019 17:31:55 +0200 From: Thierry Reding To: Dmitry Osipenko Subject: Re: [PATCH 2/2] firmware: Move Trusted Foundations support Message-ID: <20190410153155.GB1460@ulmo> References: <20190410085708.10798-1-thierry.reding@gmail.com> <20190410085708.10798-2-thierry.reding@gmail.com> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.11.4 (2019-03-13) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190410_083159_168147_25321BA7 X-CRM114-Status: GOOD ( 31.24 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-tegra@vger.kernel.org, Russell King , linux-arm-kernel@lists.infradead.org, Jon Hunter Content-Type: multipart/mixed; boundary="===============5664218325825580995==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============5664218325825580995== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="8P1HSweYDcXXzwPJ" Content-Disposition: inline --8P1HSweYDcXXzwPJ Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Apr 10, 2019 at 01:36:35PM +0300, Dmitry Osipenko wrote: > 10.04.2019 11:57, Thierry Reding =D0=BF=D0=B8=D1=88=D0=B5=D1=82: > > From: Thierry Reding > >=20 > > Move the Trusted Foundations support out of arch/arm/firmware and into > > drivers/firmware where most other firmware support implementations are > > located. > >=20 > > Signed-off-by: Thierry Reding > > --- > > arch/arm/Kconfig | 2 -- > > arch/arm/Makefile | 1 - > > arch/arm/firmware/Kconfig | 29 ------------------- > > arch/arm/firmware/Makefile | 4 --- > > arch/arm/mach-tegra/Kconfig | 2 +- > > arch/arm/mach-tegra/cpuidle-tegra114.c | 3 +- > > arch/arm/mach-tegra/pm.c | 3 +- > > arch/arm/mach-tegra/reset.c | 3 +- > > arch/arm/mach-tegra/tegra.c | 3 +- > > drivers/firmware/Kconfig | 15 ++++++++++ > > drivers/firmware/Makefile | 1 + > > .../firmware/trusted_foundations.c | 4 ++- > > .../linux/firmware}/trusted_foundations.h | 4 +-- > > 13 files changed, 30 insertions(+), 44 deletions(-) > > delete mode 100644 arch/arm/firmware/Kconfig > > delete mode 100644 arch/arm/firmware/Makefile > > rename {arch/arm =3D> drivers}/firmware/trusted_foundations.c (98%) > > rename {arch/arm/include/asm =3D> include/linux/firmware}/trusted_foun= dations.h (97%) > >=20 > > diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig > > index 054ead960f98..f006b3c69247 100644 > > --- a/arch/arm/Kconfig > > +++ b/arch/arm/Kconfig > > @@ -899,8 +899,6 @@ config PLAT_PXA > > config PLAT_VERSATILE > > bool > > =20 > > -source "arch/arm/firmware/Kconfig" > > - > > source "arch/arm/mm/Kconfig" > > =20 > > config IWMMXT > > diff --git a/arch/arm/Makefile b/arch/arm/Makefile > > index 807a7d06c2a0..05ecc004de86 100644 > > --- a/arch/arm/Makefile > > +++ b/arch/arm/Makefile > > @@ -290,7 +290,6 @@ core-y +=3D arch/arm/kernel/ arch/arm/mm/ arch/a= rm/common/ > > core-y +=3D arch/arm/probes/ > > core-y +=3D arch/arm/net/ > > core-y +=3D arch/arm/crypto/ > > -core-y +=3D arch/arm/firmware/ > > core-y +=3D $(machdirs) $(platdirs) > > =20 > > drivers-$(CONFIG_OPROFILE) +=3D arch/arm/oprofile/ > > diff --git a/arch/arm/firmware/Kconfig b/arch/arm/firmware/Kconfig > > deleted file mode 100644 > > index ad396af68e47..000000000000 > > --- a/arch/arm/firmware/Kconfig > > +++ /dev/null > > @@ -1,29 +0,0 @@ > > -config ARCH_SUPPORTS_FIRMWARE > > - bool > > - > > -config ARCH_SUPPORTS_TRUSTED_FOUNDATIONS > > - bool > > - select ARCH_SUPPORTS_FIRMWARE > > - > > -menu "Firmware options" > > - depends on ARCH_SUPPORTS_FIRMWARE > > - > > -config TRUSTED_FOUNDATIONS > > - bool "Trusted Foundations secure monitor support" > > - depends on ARCH_SUPPORTS_TRUSTED_FOUNDATIONS > > - default y > > - help > > - Some devices (including most Tegra-based consumer devices on the > > - market) are booted with the Trusted Foundations secure monitor > > - active, requiring some core operations to be performed by the secure > > - monitor instead of the kernel. > > - > > - This option allows the kernel to invoke the secure monitor whenever > > - required on devices using Trusted Foundations. See > > - arch/arm/include/asm/trusted_foundations.h or the > > - tlm,trusted-foundations device tree binding documentation for detai= ls > > - on how to use it. > > - > > - Say n if you don't know what this is about. > > - > > -endmenu > > diff --git a/arch/arm/firmware/Makefile b/arch/arm/firmware/Makefile > > deleted file mode 100644 > > index 6e41336b0bc4..000000000000 > > --- a/arch/arm/firmware/Makefile > > +++ /dev/null > > @@ -1,4 +0,0 @@ > > -obj-$(CONFIG_TRUSTED_FOUNDATIONS) +=3D trusted_foundations.o > > - > > -# tf_generic_smc() fails to build with -fsanitize-coverage=3Dtrace-pc > > -KCOV_INSTRUMENT :=3D n > > diff --git a/arch/arm/mach-tegra/Kconfig b/arch/arm/mach-tegra/Kconfig > > index 7b3fd0995a16..cbad7823f602 100644 > > --- a/arch/arm/mach-tegra/Kconfig > > +++ b/arch/arm/mach-tegra/Kconfig > > @@ -3,7 +3,6 @@ menuconfig ARCH_TEGRA > > bool "NVIDIA Tegra" > > depends on ARCH_MULTI_V7 > > select ARCH_HAS_RESET_CONTROLLER > > - select ARCH_SUPPORTS_TRUSTED_FOUNDATIONS > > select ARM_AMBA > > select ARM_GIC > > select CLKSRC_MMIO > > @@ -14,6 +13,7 @@ menuconfig ARCH_TEGRA > > select PM_OPP > > select RESET_CONTROLLER > > select SOC_BUS > > + select TRUSTED_FOUNDATIONS >=20 > Do we really want to force-enable the driver? I think it's better to > have a choice and then instead enable the driver in the > tegra_defconfig. The driver is getting enabled by default prior to this patch, and given that it's fairly small, I thought it'd be okay to include it by default. But I guess there's no harm in enabling it in tegra_defconfig and multi_v7_defconfig, that way people do have the possibility to disable it if they really don't want it. > > select ZONE_DMA if ARM_LPAE > > help > > This enables support for NVIDIA Tegra based systems. >=20 >=20 > [snip] >=20 > > --- a/drivers/firmware/Kconfig > > +++ b/drivers/firmware/Kconfig > > @@ -267,6 +267,21 @@ config TI_SCI_PROTOCOL > > This protocol library is used by client drivers to use the features > > provided by the system controller. > > =20 > > +config TRUSTED_FOUNDATIONS > > + bool "Trusted Foundations secure monitor support" >=20 > Th driver shall depend at least on ARM because of the assembly. And > maybe even on ARCH_TEGRA. Indeed, I've added a "depends on ARM" in v2. > Do you know if there are any platforms other than Tegra that use that > firmware? Won't it make sense to move the driver to tegra/ ? I don't think there's any reason to make this Tegra specific, even if this was only ever deployed on Tegra. Thierry --8P1HSweYDcXXzwPJ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAlyuDGsACgkQ3SOs138+ s6HomhAAlIUtFHAokRU/WGn+FH70CVnn+zkSSUpO3NsdlXJB1zUSJGVq6u6xOqBW dyeoYnhHNdmTTa8Kbh9PYAkkfk/YlQIwZmKTqsUq5AC1RHW/4E+L2cSb37fUiq// 7Z42swN/A6++1DybHHvtOsmzbV8G8ZxSheHLPG0GRzntUuR9N7yzpXXgN0DyZHwj 7kg9iI3hVO+qx8hXNLemqRCbVgbFQx5H/vNVGi+mb0PiM1VAm3Pbu06D9aiPhPL0 rVw2SZRXphHZDg/aZEAh099n1/P8fA3FDlKQADW+n6aGYHjxT2cDPB/4KDcWv12Q LRMStmFyybFpvp1QVZKIeJuDmSzAVCPSwjQrtAKO97JPd+k4bv3TV5/fO7rlJKH8 YZmwwr8hdkNrAKtlunL/NLHYoFjYzp3eCmTWf+vBO6ljZe520dGbEaTSkaMC91N7 rG3MhxPTXRPdbT3ODwEe3Gb3tdFvIjaZwtkyl2OJw8XwmHI4w692jlrLHUNEcbwk W9bST+58eYHX0uw/tOQKXZaG0WF3Q53lAfcjzcDCIwY8b60S/0xv0/R+gIMY24OR Irny/wZFQH5QL3aspricSgYWHM6cmQ1MCoarpp9H52p8xZHMDVoTFzt0rL6Aj2T3 +7B13hES6qfMctFPqvIR6OzsIo8Yuy26AndLoWOSNuInp8/iCI4= =XWf3 -----END PGP SIGNATURE----- --8P1HSweYDcXXzwPJ-- --===============5664218325825580995== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============5664218325825580995==--