From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qk1-f169.google.com (mail-qk1-f169.google.com [209.85.222.169]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0A36927F756 for ; Fri, 26 Dec 2025 21:28:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.169 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766784534; cv=none; b=lyBQOWp+1sgbjGUcBUf9jzRVHfe6PieYsx+fKxHNsMU5geZZRVIhfO8LG1bzPdlmkAJXdVhYCuxSWMQ0h7aGNumnoEa8jmH4/SQhcHXswfUGuPO9tIoa92l3Z7RQ5YbXNsdMZWz7gGuyN2euwW3hAHsEvWc29IFlf6tGlDekGzE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766784534; c=relaxed/simple; bh=NspC7S3ikTMUwNzaq52BA472WHff+d7fgODKBGfPMtk=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=iO7Zo20Cirj+DOW7ukYBl0veGoXX5BZyDzn2IC1cxjwjXmLYCJ6Q4MswMPo7TEYxBFQpMbhmlBc69zo1A0nDYUWJdm+Dm0cN+ubME7LWMIYxgh3pwZ+XgE7nqihbCzSxI7M2pnOmQ0mZUaytwhnb1miOpzKDiup7H+KPl4YafOc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=riscstar.com; spf=pass smtp.mailfrom=riscstar.com; dkim=pass (2048-bit key) header.d=riscstar-com.20230601.gappssmtp.com header.i=@riscstar-com.20230601.gappssmtp.com header.b=Tq97YNnN; arc=none smtp.client-ip=209.85.222.169 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=riscstar.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=riscstar.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=riscstar-com.20230601.gappssmtp.com header.i=@riscstar-com.20230601.gappssmtp.com header.b="Tq97YNnN" Received: by mail-qk1-f169.google.com with SMTP id af79cd13be357-8b220ddc189so1023519585a.0 for ; Fri, 26 Dec 2025 13:28:51 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=riscstar-com.20230601.gappssmtp.com; s=20230601; t=1766784531; x=1767389331; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=nntctK0gEdJs8MBuE2OMh+NNa2bU4c08j9CTg1079CE=; b=Tq97YNnNwqXEjYB22akOkiNSUBqFu2zAbhh8GFPHNGeJb9qIZgxaN9nwUxrGV83eWk Ug0/KSQwi3zp0qvgAdKshfnWLH1rXzOL1V+Mp5e3pBs0yJiF5M9REjMDQiMGtAHJ1Jq+ yTT8HdMIbbFnrvwA5qKkT3J7jPwZo41jBp6nIJDg8l1L8SiOzIWIIJ0RwNP5nZfkuuw5 xTkdGKfXVjquky0lLXsqMBXHyF0VJyAd65PpJxZUsMvvKISAK26rctCMWgyjJEjQYhj9 bpU/uYu+LgxMaLN3F3gAEiKDS26nGvMV5Q4svqtF2nTry84cRHkgqSklLo5+youVzU0V 67Zg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1766784531; x=1767389331; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-gg:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=nntctK0gEdJs8MBuE2OMh+NNa2bU4c08j9CTg1079CE=; b=c3zbr5mtjz10+hlxA0d+QvtHQ2yv/zmMEp55dFz91wfbC7TThmdtC5ph5dcn/XNON4 tiwx8NnqBhlbWqyW8YTdh4u94GAjGAf4Rj5kNHrTfRkO8g9HaKSMHDAN65coyfk2BAMH s2WrpFUKRKaiODQM5uAWkmpVpqH/vEb3zD9FuIsTukXDgQ2/igcywd0lBvv51AJhiNqI Y8Nf4k+NiW2a0gLGsDV09N9QOcL24vZSXuU4066H4YzpM/FUq9EzEPF9zmokDZshCgOx YYoamwb25e0aBzIkoxPlHSO21xuDedkJUTujlZQ2yk3zaacqsycHOj3xHxY/cQgQ/VHF rOtA== X-Forwarded-Encrypted: i=1; AJvYcCXhr8mq7ddHUP61O0y6+KyLToCn2q/ihamC0UR9rKt3Ww/vFV/HLfeQrSHmWZSKoi1BqxHNj8UW/q2TPzw=@vger.kernel.org X-Gm-Message-State: AOJu0YyExWjmfqAOQnl0NfEa/q046/nn393oSZ0hAvwO5FNfiTGjdGas Jl6O/A5BrxI4030nSa7GTTcfmEej1ZzxWAB67Pk85xKoPz+JZ/9uR+t2UxhJ197ZMIM= X-Gm-Gg: AY/fxX76SMDD9lcuHprZ2Ahc16i09BoqUjzr+6RygyLlemKFFns8LgbNBn7WBL9OfuP AD7NHeRxoS5JImeD+h7cQ8BnfFCDDkILDSz1obtKRyyquMGE+RRHvtx3YguVrlVwTsWP+rbVF2h eQlcZqK8ljkBPOupMzL/iSDexagfITRs2jGclyZ0NW5uz55JhRw8wCr4X0ehn2LpSzWvAKZRasY Sz702S2uNV76k8iRbk352XeW0vqD3edPoiRQ53z43ICHei3a15Ikdp0xPuzN7UCwcwnGlJqBHLY 1AMOoZNSyBK6JlFxpE5v0+tC0Uo2RR6x8fpovzJDPebomFNP4GNFM6d+LEyRnWrt7ECOcxxvyPw LMzFlHsd9DGt2LMYgSv6IE5iDDq005ooaPXIY8oOeJ2Y5HGoyeEu+HEUimhJvk5l7rEVt7KMnGg dwilpBf/6wZpcAex/k9z0dtXXhIztk5DoQZt46DDFMq1AVfa0nFK0= X-Google-Smtp-Source: AGHT+IEy/QOrxe2SQiRFQjJ/+nrpNVPGTV1wSqBMtHRM+j197OQHW+EwApB9rAJn4Nlwz0TrLac2kQ== X-Received: by 2002:a05:620a:454d:b0:8b1:ed55:e4f0 with SMTP id af79cd13be357-8c08fbc6077mr3782302885a.39.1766784530950; Fri, 26 Dec 2025 13:28:50 -0800 (PST) Received: from [172.22.22.28] (c-75-72-117-212.hsd1.mn.comcast.net. [75.72.117.212]) by smtp.gmail.com with ESMTPSA id af79cd13be357-8c0973ee08dsm1824350585a.36.2025.12.26.13.28.48 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 26 Dec 2025 13:28:50 -0800 (PST) Message-ID: Date: Fri, 26 Dec 2025 15:28:47 -0600 Precedence: bulk X-Mailing-List: linux-serial@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 11/13] dt-bindings: riscv: Add Supm extension description To: Guodong Xu , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Paul Walmsley , Palmer Dabbelt , Albert Ou , Alexandre Ghiti , Yixun Lan , Daniel Lezcano , Thomas Gleixner , Samuel Holland , Anup Patel , Greg Kroah-Hartman , Jiri Slaby , Lubomir Rintel , Yangyu Chen Cc: Paul Walmsley , Conor Dooley , Heinrich Schuchardt , Kevin Meng Zhang , Andrew Jones , devicetree@vger.kernel.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, spacemit@lists.linux.dev, linux-serial@vger.kernel.org References: <20251222-k3-basic-dt-v2-0-3af3f3cd0f8a@riscstar.com> <20251222-k3-basic-dt-v2-11-3af3f3cd0f8a@riscstar.com> Content-Language: en-US From: Alex Elder In-Reply-To: <20251222-k3-basic-dt-v2-11-3af3f3cd0f8a@riscstar.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 12/22/25 7:04 AM, Guodong Xu wrote: > Add description for the Supm extension. Supm indicates support for pointer > masking in user mode. Supm is mandatory for RVA23S64. > > The Supm extension is ratified in commit d70011dde6c2 ("Update to ratified > state") of riscv-j-extension. > > Supm depends on either Smnpm or Ssnpm, so add a schema check to enforce > this dependency. I have the same general question on this, about whether it's really necessary for the DT binding to enforce these requirements. The RISC-V specifications are what truly defines their meaning, so I don't really see why the DT framework should need to enforce them. (That said, I'm sure there are other cases where DT enforces things it shouldn't have to.) And now, having looked at these added binding definitions (in patches 07 through 11 in this series), I wonder what exactly is required for them to be accepted. For the most part these seem to just be defining how the extensions specified for RISC-V are to be expressed in DT files. It seems to be a fairly straightforward copy from the ratified specification(s) to the YAML format. Who need to sign off on it? Conor? Paul? DT maintainers? Thanks. -Alex > Signed-off-by: Guodong Xu > --- > v2: New patch. > --- > Documentation/devicetree/bindings/riscv/extensions.yaml | 16 ++++++++++++++++ > 1 file changed, 16 insertions(+) > > diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml > index 1066b7e65dab89704dbac449db4aa5605c95b9d3..4997f533b2c0defad88fd59413a6885b5b9e109a 100644 > --- a/Documentation/devicetree/bindings/riscv/extensions.yaml > +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml > @@ -267,6 +267,12 @@ properties: > ratified in RISC-V Profiles Version 1.0, with commit b1d806605f87 > ("Updated to ratified state.") > > + - const: supm > + description: | > + The standard Supm extension for pointer masking support in user > + mode as ratified at commit d70011dde6c2 ("Update to ratified state") > + of riscv-j-extension. > + > - const: svade > description: | > The standard Svade supervisor-level extension for SW-managed PTE A/D > @@ -892,6 +898,16 @@ properties: > const: shvstvecd > - contains: > const: ssstateen > + # Supm depends on either Smnpm or Ssnpm > + - if: > + contains: > + const: supm > + then: > + anyOf: > + - contains: > + const: smnpm > + - contains: > + const: ssnpm > # Zcb depends on Zca > - if: > contains: >