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 01741C00A5A for ; Thu, 19 Jan 2023 17:21:58 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 80B1D85677; Thu, 19 Jan 2023 18:21:56 +0100 (CET) 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="nXNri3p6"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 1433685677; Thu, 19 Jan 2023 18:21:55 +0100 (CET) Received: from mail-qt1-x834.google.com (mail-qt1-x834.google.com [IPv6:2607:f8b0:4864:20::834]) (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 F3BED85676 for ; Thu, 19 Jan 2023 18:21:51 +0100 (CET) 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-qt1-x834.google.com with SMTP id a25so2065111qto.10 for ; Thu, 19 Jan 2023 09:21:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=konsulko.com; s=google; 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=BhXkLsYRnHnh58pknvfinHkCQkeQ5rTn+iAegHOCBMo=; b=nXNri3p6kUiz+R75uIdtZEoLoOTvdcBHaAA1Niyg3ugqMmVMQxnE1yFAwYn5pI2ieX /GuWKepZjuWTj9ORnPYuz6lOmqGkwR+MPerTv/C17YdLaOOb4GeC9j1kNqzXtNCnheEJ uF6skLjKwskJPk5gbBEffX8WhprxBIoq8v2Yo= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; 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=BhXkLsYRnHnh58pknvfinHkCQkeQ5rTn+iAegHOCBMo=; b=Esj2pOCXSB4NL7poEPS4ZWOH2gUAR1/zrMYyjDnIeniCvlJxl3UjqRTPNybOlPz9Bp zRWR0RUHjwm5ysyqADA0eRnOGNNT6rT17WqW5X279bwQZoUIeeysXCAHabZhLvk2D6Td BCaUGet+KXRgskYhP+WIVpMz92fmh052doTlgZXrG/sK2uVdGGlm7DUc4FUwQ0ftmpXg YwfSsX20IcKz5fHEYb2JNow2qQY4fcpp04oQBcUxyyQTr0sm6wpoDVvOlRUV3+voQw7L JvtiMqc4aLZ8g/re0YYdpLSLWw9OVdAKSOYwFJvUYt1YbZNPdiCXXr3EMEeXGcYIb58z kkMA== X-Gm-Message-State: AFqh2kpLfHk3Y3WZIHaVqanC+9yOdmBO80tpy4+wd8xDEy5B4WOj1AvQ 5z5KK1w4t7SoWWbJFRnL26DVtA== X-Google-Smtp-Source: AMrXdXtxHamZWK9MCKUtvcPVvafAKq9nj9EVGBTD8rFM92tIn1PMj3mhv38j3XMUco9EX3LqYpJFHQ== X-Received: by 2002:a05:622a:1dc4:b0:3b0:12fc:ff96 with SMTP id bn4-20020a05622a1dc400b003b012fcff96mr14737082qtb.23.1674148910772; Thu, 19 Jan 2023 09:21:50 -0800 (PST) Received: from bill-the-cat (2603-6081-7b00-6400-99bd-1ec8-6b02-61e8.res6.spectrum.com. [2603:6081:7b00:6400:99bd:1ec8:6b02:61e8]) by smtp.gmail.com with ESMTPSA id t1-20020ac86a01000000b003a7e4129f83sm19012290qtr.85.2023.01.19.09.21.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 19 Jan 2023 09:21:50 -0800 (PST) Date: Thu, 19 Jan 2023 12:21:48 -0500 From: Tom Rini To: Sudeep Holla Cc: Simon Glass , Abdellatif El Khlifi , achin.gupta@arm.com, xueliang.zhong@arm.com, Drew.Reed@arm.com, robh@kernel.org, jens.wiklander@linaro.org, ilias.apalodimas@linaro.org, u-boot@lists.denx.de Subject: Re: [PATCH v8 03/10] arm_ffa: introduce Arm FF-A low-level driver Message-ID: <20230119172148.GH631605@bill-the-cat> References: <20230118124923.GB631605@bill-the-cat> <20230118135932.GC631605@bill-the-cat> <20230119163157.GA18384@e121910.cambridge.arm.com> <20230119164652.llh66rapyqqej63f@bogus> <20230119165744.GG631605@bill-the-cat> <20230119170945.cppg6irr7a26sv7n@bogus> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="iBipwI8N6cjWJAiJ" Content-Disposition: inline In-Reply-To: <20230119170945.cppg6irr7a26sv7n@bogus> 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.6 at phobos.denx.de X-Virus-Status: Clean --iBipwI8N6cjWJAiJ Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jan 19, 2023 at 05:09:45PM +0000, Sudeep Holla wrote: > On Thu, Jan 19, 2023 at 11:57:44AM -0500, Tom Rini wrote: > > > > But it's also true that at run-time, within U-Boot, we can modify the > > device tree we have, with live tree yes? So, the whole series in > > question here can be done without modifying the base DT and getting in > > to the further discussions that doing so entails. The assertion is that > > the software discoverable bus here is sufficient to not need DT, so, OK, > > lets go. >=20 > OK, may be I am not up-to-date on the U-Boot. IIUC, the modifications > done in the DT by U-Boot is mostly for consumption by the next stage > loader/OS and not for self-consumption. But if it is for self consumption, > then good. It helps especially for the subnodes(as Simon referred) or the > partitions that can be discovered at run-time using FF-A interface. >=20 > As mentioned I am not again DT, it is just not needed and especially > for subnodes it could result in inconsistency b/w what is in DT and > what the firmware provides. As mentioned in previous response, having a > simple node that Simon provided as example earlier is fine by me if that > is the only option to make progress as I just feel it is redundant and > one can say not scalable(but that is debatable again =F0=9F=98=84). >=20 > In short, I am not concerned about having simple node, just don't like > to see entire FF-A bus enumerated in DT as subnodes for reasons mentioned > already. So there's two parts to this discussion. The first of which has been, do Simon and I agree with the direction of defining a software discoverable bus rather than using device tree to describe it. To which the answer is no, and also neither camp seems likely to convince the other. The second part of the discussion is what can we practically do about it. To which the answer is that I believe the live tree support (see https://u-boot.readthedocs.io/en/latest/develop/driver-model/livetree.html) should allow for the FF-A bus support to be integrated with U-Boot, without having to add something to the base device tree. And we can evaluate down the line if our fears (or at least mine) were unfounded or not. --=20 Tom --iBipwI8N6cjWJAiJ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQGzBAABCgAdFiEEGjx/cOCPqxcHgJu/FHw5/5Y0tywFAmPJfCkACgkQFHw5/5Y0 tyxp1wv9ENw7IFWPjC5VROP6K1RGLvEUR0zzdlEGJXNg/sShKGswE0AtqvxDXrzp B0Tx54wt/jAPguB4PD7Ey98lT4Vp+CMzgvL9sVmlmqAU6KKvwNWMsTLZRV2hJj9Q s7tJbMvub9ySCgd0yX6mvITVympPwhiYTlQfGA4th2Uq1NGphbtXlXjcVkx/x2SB flGo2znjKGSbvrUExOCAsdge/z12HWiE7Q+xvvHPF35VI7cbAHlCh2DxFPQZm/Hf smE5osOwdopi/hOkFrP3ne7GJsKnugce+qRI3GcNj94z0y7g04bPV9+V/874VT5a RGBhd3UhwRUQe965CeLm4snUz8A+e7Cw0O6l8oOCni4+lXB4RmdBu9bqLMCtt4J4 gTLM2X3aohtio2Ch6CkYxBYLl3v4wm/ZVMw/nYgvumiYd+4rBpXoY6ZTjCrMihAs 5fAVBHBsS44rLJNW61u8X8PZuMdLoBmVD26/VWnXx3cTBi/R0jAGvboNXYj+eBnr gNPlDeVl =zhi7 -----END PGP SIGNATURE----- --iBipwI8N6cjWJAiJ--