From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qk1-f174.google.com (mail-qk1-f174.google.com [209.85.222.174]) (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 2916B278E63 for ; Fri, 26 Dec 2025 21:28:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766784534; cv=none; b=L38F5DxcePGDgvnX5a4S/ah7nHTBEd28Iy/HUWQVLLp3i5NwJ+MS3/uLfZXsy9PwsEk1sdLlFvMvL/BTGM78NNWMgyoDS1ubLjs4ZDIavm4KO3pQ0kqyctzixqaEUuAxqjKg5cEDvV1ZRrlrMIt5bbqY1Pcy+HGsdOblrULpZdA= 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.174 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-f174.google.com with SMTP id af79cd13be357-8b2148ca40eso1170404285a.1 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=VTubqMClFMLydSwXzcadbXwHmF9B7MLjL+cbrHBeCqJ/KKvD8UveNux8sMkaT49ajN WDtw+fhvKhzpMSwMtrDvhdlZfL6HFbOXC3TUMMB3cGImKd0L46NAV+Xdsun7pT1m+Io8 x5sptJ4Os0+WFnbnMnIB+vyYV0ewZP4y7pPSlWxJ77oKpvY2rcaCy2xsiHt4fQV+0K+I CQpCx7rFAIK7vrN9vtzTjG4+KqVxu3fqOzNd+wLIIW9SSQ+pUMrkhGbvEgwi5Ox/RB52 Y28gHN0T5QUnBYUZ1zNmsCtzoFAK7zQDZdwncvnvuIzE0LL3fLJQC2k3ZXjf6fFi4vW2 3caA== X-Forwarded-Encrypted: i=1; AJvYcCWGDAxVuLRrg0Mxj5ov6WsIsHnGDxTu4l2R/0EGiC6PkjtWCdCbWKkQoTIgyVPQ141lebqvr7TC9kH6Iks=@vger.kernel.org X-Gm-Message-State: AOJu0YxGlLI/rMoNCzM8tEUn1naLN115YZCZlm4Op0pObTNtSvlbPsvc Fob/kIE+8OJpcg1AwCI9KnONmq3Q6NZmBVxkytTrtkMNZbNV3dHd5r1DCPSwWBwGLqE= X-Gm-Gg: AY/fxX7MgDaGXyaZ20Wbf3U9bn/ejtTh85kwTnyM+1joTEe3FyU1HLs8NvyQ+bSnDlp e8wDX4h6AMJQemOJibe/BSP/w+UFaYlyKi6U2cUhnP1RdwjQ5/4CL6coEu/4mi5774NjUcAfrQ+ v63a4WAR0BiPKqYyY5K4rzFGRAy3oBpU8W2CY84u3TsnRoOt2hJQ3oecOLLBfQ546sRHXOzbJhH 4nccFChbJQ8vZ/TbPRPl0re4hgSJvZjgE3RMfTYDXiQxBsK8oDwsLvnvsqnSDfhRbCoXDF0LKas TQYUCriyXV+RfNSoFSa9FHI98REo7gz3SmImpFDtFuqfE913n2xPdJfv/SHSeB0RR0q7cIGk4aQ vqjHr9dbqttntEaeidWDTb+P1Gt0YobzX6aodNi0knQMCumxPwG2eppYuAVeMu8UdpA44OZNa06 OBiF4V0dz4GUUsArPXdsU/5WSdjHQZUnF/HLxghiDv0LqIvSdhYaE= 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-kernel@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: >