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 955D6C0015E for ; Tue, 25 Jul 2023 13:47:36 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 2A4F5862E8; Tue, 25 Jul 2023 15:47:34 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (1024-bit key; unprotected) header.d=konsulko.com header.i=@konsulko.com header.b="V2z50pNf"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id B3097854BD; Tue, 25 Jul 2023 15:47:32 +0200 (CEST) Received: from mail-vk1-xa2d.google.com (mail-vk1-xa2d.google.com [IPv6:2607:f8b0:4864:20::a2d]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id 4701F854BD for ; Tue, 25 Jul 2023 15:47:29 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=konsulko.com Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=trini@konsulko.com Received: by mail-vk1-xa2d.google.com with SMTP id 71dfb90a1353d-47ec8c9d7a0so2048668e0c.3 for ; Tue, 25 Jul 2023 06:47:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com; s=google; t=1690292848; x=1690897648; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=tc92jY/FR5EzS7PbLIhf29JUCuAwBUSF95FzFzLFW0Q=; b=V2z50pNfUkcTR0HgZx6osmV+jbY67uQQ0fTi9LDM9WY+s6S5wFh2Vs7BoVijulyDiQ LncJ4HH9sH8TnqcW3Dla8suobKmDLWl1XIo3JH5CHiO9KqM1X1YJDXyjQjAs0XBjPWyf 5sDqzz13NLYdcVN5rEqZqGlXaGDSEErPXzgzE= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1690292848; x=1690897648; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=tc92jY/FR5EzS7PbLIhf29JUCuAwBUSF95FzFzLFW0Q=; b=JULl1kmPHg1l+h/xSFqGMyeI2iTrkptfl/cIG03lnDOrBDDVk1RYAeoZhagh/BS1EG JqpaLXWU9WrdSnUhg4Tki5vNwsQj9HZh9E2ljZyYDcoYtgorTpUa6n3co42RDyWzgysp b5iIMASxzPVyzzFGJACtl5p5ihtrZfMJqzyMQ2+ReN6kGL7g4P29Mnegj46NnNga6CJY P67u3KzGv3r0hVUs5ypMhGhJyqEy2F0IQfUJYaCuzYsl0Mg/8IpH8LMvzo8Emn/ZYH+/ qz978U/Cp8nRcK5UtZei/GcHZzfF+nJnyjanMRONG096e5Te5a4gdGxDRSUZuVwguRbr MBjA== X-Gm-Message-State: ABy/qLY63c1nm6yThoT071Pe/DtflYg/FzdXbJUlARwu2QQnWjsZUUGc h0uhQYtLI5qoFt3rIzmylVM3GA== X-Google-Smtp-Source: APBJJlFAKwXMy54/Ktkl64/DEozb11LqnqK8MISjxNAzGQtuvWzrEIYYEMyRigTDqu6ZvsQPPNuzBg== X-Received: by 2002:a1f:edc2:0:b0:485:e984:6b31 with SMTP id l185-20020a1fedc2000000b00485e9846b31mr3602872vkh.6.1690292847854; Tue, 25 Jul 2023 06:47:27 -0700 (PDT) Received: from bill-the-cat (2603-6081-7b00-6400-b236-e3a5-57be-379d.res6.spectrum.com. [2603:6081:7b00:6400:b236:e3a5:57be:379d]) by smtp.gmail.com with ESMTPSA id l25-20020ac5c359000000b00481570fe645sm43082vkk.50.2023.07.25.06.47.27 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 25 Jul 2023 06:47:27 -0700 (PDT) Date: Tue, 25 Jul 2023 09:47:24 -0400 From: Tom Rini To: Abdellatif El Khlifi Cc: nd@arm.com, u-boot@lists.denx.de Subject: Re: [PATCH v15 00/10] introduce Arm FF-A support Message-ID: <20230725134724.GO3630934@bill-the-cat> References: <20230710161828.GS148062@bill-the-cat> <20230713132847.176000-1-abdellatif.elkhlifi@arm.com> <20230724235007.GA3188301@bill-the-cat> <20230725092616.GA100869@e130802.arm.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="cNbSXy8dcFfUt3Dn" Content-Disposition: inline In-Reply-To: <20230725092616.GA100869@e130802.arm.com> X-Clacks-Overhead: GNU Terry Pratchett 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.8 at phobos.denx.de X-Virus-Status: Clean --cNbSXy8dcFfUt3Dn Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jul 25, 2023 at 10:26:16AM +0100, Abdellatif El Khlifi wrote: > Hi Tom, >=20 > On Mon, Jul 24, 2023 at 07:50:07PM -0400, Tom Rini wrote: > > On Thu, Jul 13, 2023 at 02:28:37PM +0100, Abdellatif El Khlifi wrote: > > > Adding support for Arm FF-A v1.0 (Arm Firmware Framework for Armv8-A)= [A]. > > >=20 > > > FF-A specifies interfaces that enable a pair of software execution en= vironments aka partitions to > > > communicate with each other. A partition could be a VM in the Normal = or Secure world, an > > > application in S-EL0, or a Trusted OS in S-EL1. > > >=20 > > > FF-A is a discoverable bus and similar to architecture features. > > > FF-A bus is discovered using ARM_SMCCC_FEATURES mechanism performed > > > by the PSCI driver. > > >=20 > > > ... > > >=20 > > > Changelog of changes: > > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > > >=20 > > > v15: > >=20 > > So, this does not pass CI: > > https://source.denx.de/u-boot/u-boot/-/jobs/662303 > > https://source.denx.de/u-boot/u-boot/-/jobs/662304#L2234 > >=20 > > And it's on me for merging this before letting the CI run I kicked > > complete, sorry. But I'm reverting this now. > >=20 >=20 > The v15 patchset has been built successfully on next using this SHA: [1]. >=20 > The tested platforms are the following: [2]. >=20 > Is there any action from my side please ? >=20 > [1]: 56c7fac8ad89955d3e5d08864bbd1343a058bf4b > [2]: tested platforms >=20 > sandbox64 > sandbox > corstone1000-fvp > corstone1000-mps3 > qemu_arm64 Yes, you need to go and fix all of the problems that CI is showing: - On 64bit, espresso7420 is building cmd/armffa.c and pointing out an unused variable. - Maybe related to why that platform is building the code, a ton of 32bit platforms are building the code and showing warnings. And also that your Kconfig logic is wrong. - Finally, the MAINTAINERS file needs to be alphabetical. --=20 Tom --cNbSXy8dcFfUt3Dn Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmS/0mYACgkQFHw5/5Y0 tyzKMAv/V/aU68yRNLLUmYzFJfUV3xMjcr/DcFEFLTZUzYndHhwpvw42GtguaKmC 7mUHIK5GW83YSXIJZHWfnUOkK1cVUmWxNbExvfgsEi0uGe8v/gUkxcggsbtuSJVO tnaarx+9kCQ0vRBD+PpC9Yw/QYOFZT16b2+Q4JTJT3Vh8Gyfkkxsc2qPfbDjhUbC Q+N2fOiKPsxPAljy2o2ox2U3r4F98SfhqxWVKL7AbSFVLHSLT9OLJ8n4LrlH5vkB A/a4Tv/wHJppt1hwJK2McgQlsho5yNkFTJymAyFjL902pPANNJCI89mVbGRS8t93 dwB/6zI9M0lCYzk9R21kHdP9UR3cY7NxpH9u1eG0r9N/MOjXLbt9XUdHiH2GTXIc OvgHLjfmsDvgKxhaY3U42E/ETG0/Ahz98uqBnGETMaefDvm64GwJP//eIq9sP1pI 9nxRm8O30hLnA5tDsjNYgmXKC6/aOPBXcQZV10q28buSiOQnqyGuJ+5B3H48gpCV wJNROZDe =Wyuh -----END PGP SIGNATURE----- --cNbSXy8dcFfUt3Dn--