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 CB1CAC7EE29 for ; Thu, 8 Jun 2023 19:30:43 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 6AEFA8471B; Thu, 8 Jun 2023 21:30:41 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=kernel.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=kernel.org header.i=@kernel.org header.b="lfdr8/t7"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id C8B978471B; Thu, 8 Jun 2023 21:30:39 +0200 (CEST) Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id CAA4D84706 for ; Thu, 8 Jun 2023 21:30:36 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=kernel.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=conor@kernel.org Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 0CAC464B27; Thu, 8 Jun 2023 19:30:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id DC148C4339B; Thu, 8 Jun 2023 19:30:30 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1686252634; bh=Sn9hAi2Kyf0okzCneZ3amJPSCNVbUHG48YWRl4CGl/M=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=lfdr8/t7jAH4glSpjV2KYZiJ23pFC02NUTTTkCYlLGOvVOrmDCcRICN1I7ckFGeAQ TPGCO5Yx6pvnH5XLP1XTniYRePwwraYyUDpYJqCXzS6uNl5YuDSEtEqzaa6vast+yd AMjS+CzLVMzQGuWTQNRSM0Ufd/re+R18iMYOl+Biy/Afeke4T74nR7ID5P7Lx82b2L 1/KMCBpD/NbyjEYVOdgT3myEqZ9cUkkHXTTjYCo/FHA0ZWQmct3tB4tNdElz8hzlKD dMXMAC24mzOI1YV/emxTct+WnhjV22drqsWa8YBbOCsThta5By4dLh6opE8JHBZsev ojK/xAa+VBS6A== Date: Thu, 8 Jun 2023 20:30:28 +0100 From: Conor Dooley To: Rob Herring Cc: Conor Dooley , Sean Anderson , Anup Patel , Andrew Jones , palmer@dabbelt.com, Paul Walmsley , Krzysztof Kozlowski , Alistair Francis , Anup Patel , Atish Patra , Jessica Clarke , Rick Chen , Leo , linux-riscv@lists.infradead.org, qemu-riscv@nongnu.org, u-boot@lists.denx.de, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v1] dt-bindings: riscv: deprecate riscv,isa Message-ID: <20230608-cobbler-giving-e0fac2185e11@spud> References: <20230518-thermos-sanitary-cf3fbc777ea1@wendy> <20230518-4050231ca8dbe93c08cf9c9a@orel> <20230518-hammock-doornail-478e8ea8e6a7@wendy> <20230518-monkhood-dispersal-6749b1228b0d@spud> <20230530-duller-reset-a34ae111f207@wendy> <20230608191537.GA3233857-robh@kernel.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="J8Z0GTvlCQGm2lQj" Content-Disposition: inline In-Reply-To: <20230608191537.GA3233857-robh@kernel.org> 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 --J8Z0GTvlCQGm2lQj Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Jun 08, 2023 at 01:15:37PM -0600, Rob Herring wrote: > On Tue, May 30, 2023 at 03:12:12PM +0100, Conor Dooley wrote: > > On Thu, May 18, 2023 at 10:42:34PM +0100, Conor Dooley wrote: > > > On Thu, May 18, 2023 at 02:30:53PM -0400, Sean Anderson wrote: > >=20 > > > >=20 > > > > Why not just have something like > > > >=20 > > > > mycpu { > > > > ... > > > > riscv,isa { > > > > i; > > > > m; > > > > a; > > > > zicsr; > > > > ... >=20 > I prefer property names be globally unique. The tools are geared towards= =20 > that too. That's largely a symptom of having 0 type information in the=20 > DT. >=20 > For example if you had an extension called 'reg', it would be a problem. Per the current ISA rules, that'd not be valid. But then again, I do have trust issues & it's not like "reg" is the only property name in DT land. > > > Naming of the node aside (perhaps that could be riscv,isa-extensions) > > > there's not something hitting me immediately as to why that is a no-n= o. > > > If the size is a concern, this would certainly be more efficient & not > > > like the probing would be anything other than trivial more difficult > > > what I have in my proposal. > >=20 > > Having started messing around with this, one of the main advantages, to > > me, of this approach is proper validation. > > cpus.yaml has additionalProperties: true in it, which would have had to > > be sorted out, or worked around, but creating a child-node with the > > properties in it allows setting additionalProperties: false. >=20 > That's an issue on my radar to fix. I started that for the Arm cpus.yaml= =20 > a while back. Sadly it involves adding all the misc properties vendors=20 > added. It's not a lot, but still better to get in front of that for=20 > Risc-V. Yeah, guess I can append that to my todo list. > > > Rob's AFK at the moment, and I was hoping that he would take a look at > > > the idea, so I won't respin til he is back, but I'll give this a go in > > > the interim. > >=20 > > Mechanically, the conversion of the patch isn't difficult, but I'll sti= ll > > wait for Rob to come back before sending a v2. But that v2 will more > > than likely implement your suggestion. >=20 > I haven't read the whole thread, but the initial proposal looks okay to= =20 > me. >=20 > Another way you could do this is a list of strings: >=20 > riscv,isa-ext =3D "i", "m", "zicsr"; >=20 > I think we have a helper to test is a string in the list. Perhaps I should've waited a bit longer than a month for a v2 - I sent one this afternoon doing what Sean suggested: https://lore.kernel.org/all/20230608-sitting-bath-31eddc03c5a5@spud/ I'll do some poking at the strings and see what I think of it. Cheers, Conor. --J8Z0GTvlCQGm2lQj Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZIIsVAAKCRB4tDGHoIJi 0g5YAQC9F/U/qWJxcOhejKFcF7fDmwOe+gCq0sgnacuq/Vu+ZAEAwAZ9EAwNs4oF aCnC2JOfp//gPzZTwmsYjjFeeLhAkwA= =KmSP -----END PGP SIGNATURE----- --J8Z0GTvlCQGm2lQj--