From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qt1-f196.google.com (mail-qt1-f196.google.com [209.85.160.196]) (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 98A602E1EF4 for ; Sun, 28 Dec 2025 23:50:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.196 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766965832; cv=none; b=quiVAONDQKWRBx03clrCPuRRUNZtj1Cra0MHLnMZoQkmWHLjFCa7l0ce5Cd8NHPMeijbEdKhIJ6sd8XWG2CUOMsRgM1PZNasN4ZuNZ1HGTMtM0G+sJaPbhaVysVPF/doodbEGIQtnPp+BqwLLA2+XWwmz/2163OslcjYPFkxKxo= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766965832; c=relaxed/simple; bh=FMGcKOOH2QAAM035N5VPwjsf6iLF3bsS2NHda98XWxQ=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=nO6owRH/qJp6lFFzXA8Abx6yeuATUhGUCuPSCMvGtwwY78iir/eZ+zNi697ddpgwTOnG0gvoMtDOd3PoJQcAPuNRRdxtRRpC+BDLmw/x5CnKRIJ0wJulvbar9R8rf9aDbRgSSJ2ZFkIC9Mo3IE256xCWQoah2z4yxhwha+wrNiI= 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=dU2x0BLr; arc=none smtp.client-ip=209.85.160.196 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="dU2x0BLr" Received: by mail-qt1-f196.google.com with SMTP id d75a77b69052e-4ed9c19248bso74319861cf.1 for ; Sun, 28 Dec 2025 15:50:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=riscstar-com.20230601.gappssmtp.com; s=20230601; t=1766965828; x=1767570628; 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=0iJY5kULQZeqgTosQ9scmyKuRaTaMg4i5yh/l64mgrA=; b=dU2x0BLrAW7aIWrVxChAmVhfNNqzEuTc2uSvxqkXU8zZTd+mzKWPcRqNXMtJS2oldL sILAiOlbaQUoMjgFqCI9YJwRYAsnel5zH+u1R/5c65oSapoJUy7eVxZvStk/9xxr4Lrm CYPBcsvZanVVbxXTqurivV1TEQd693CUojZG+YCB/2U1m5YO2qkSmEgBx7MlinMLHgbZ OFhGQBvPVlHnDdyXoGN8DCjdU0gCTN3SJHTWi36DL9fQ3wIJLOeKrKkuvKPzRNDHQlIr zXor/8M1Q0rL33yPWrXIS2OIlKIjW8119JDq/faMg8XObNodGXe7YCSJNHUvyTnlGLSq 6A6g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1766965828; x=1767570628; 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=0iJY5kULQZeqgTosQ9scmyKuRaTaMg4i5yh/l64mgrA=; b=wV1x2+0e2Ri9gE19nOH5dol/IqD8hepL/HIDhTUuGgU6nD2+jfZXteQmP2/AKO1tAE M6ggk+dkUVRYaxTgdCPq1IN1yiDCkHob7/qnVeU61a+mzHBTGS/EGDNbXgeIHPfbEVmX JaHByzPU64LEW4QGXEfb+WoKSEGZcnQ5qVetZKK94W4wH95cp1PdXxWY3qjqbRR7Ul1M EHq1rIdVM8FatTYAjmlCuh3KgtHamrsIhBVO9asParXWPHPB6ItDSrN8McqQR7VSVa04 Up8BpLGC6ewPcfGo3TeLG4c6nzB+zrJ8AFtwwJ16BIR6m5m0EBcU0W7wGxGEX3XVHgjv D2MQ== X-Forwarded-Encrypted: i=1; AJvYcCXyBCCyzP4cw48cEN1uvjQhs6S2zJl4C24nEyg0YHeuciUlem2ZbPwazc7JL/xAWCv/YE525q0PNJwS@vger.kernel.org X-Gm-Message-State: AOJu0Yxp/pWPTMjtvneVMgKDnCxKf4Hbz/hklPc8ofO/JkjgmqP/tof2 eLBL6dLdNAmgzBWIeX2gvuOViBAH5V2rThzDrRw2MpPBFwWTYnofI4Kveoo1OvXMrnk= X-Gm-Gg: AY/fxX4vZvvNWu+04xkYWm+uyJIk8qi079dWyWz9i2t7RExCVPCflH/QaL9FNzgwvv7 EE5+3nxQdBuEpX30ttS4fMVBcJNnTJO7mhzEwcje0dqgs2baIuwKe0Tyn44TYAvxfisqX4g5K9F RmLgMXMnSX1PPx/1Hq8UlkNB/VCWT+6Z2lE2EtJzV95rYL6F74YOiyAm1ExeWiqtYY+l+cV7EDz 9/72yMbACDG277DQWUCwIhcS0etbKCIWizMRC4uthtKmOyoYEKAB4sFaxZrciqa4H7PNvCL0+9k 7xQ4x5htb0ARCmZSlP1p0VHwU7kxYRhSZpq4ruv2eSXZEsBR9MoD4Q9DoR6zrho5scOcJ9Max8B QfCH6ZLVBcrRi0FOKsKHmOLKsDvQjPbP9SathJ2oE+8l3CbinNyfPYi9J5rMrcOnlpfueWjBKy6 KsbpccHDREjCfi5sz5X+NYfJudrtEv6/fnSru7L2GeHIwAmdF5vrw= X-Google-Smtp-Source: AGHT+IHMoTvSCcsoBat644ksem8ZqP4Cs7kB7zIlLcWG+j/tQjQblk+rAjCWaI4lEYjd6IAfo873cw== X-Received: by 2002:ac8:6f07:0:b0:4f3:4fe8:b5ca with SMTP id d75a77b69052e-4f4abcf6a40mr349244621cf.10.1766965828133; Sun, 28 Dec 2025 15:50:28 -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-8c0973f28e3sm2406423185a.45.2025.12.28.15.50.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 28 Dec 2025 15:50:27 -0800 (PST) Message-ID: Date: Sun, 28 Dec 2025 17:50:25 -0600 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 09/13] dt-bindings: riscv: Add Ssccptr, Sscounterenw, Sstvala, Sstvecd, Ssu64xl To: Guodong Xu Cc: 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 , 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-9-3af3f3cd0f8a@riscstar.com> <6925aa57-874c-479f-9f5b-f0fd6ede685b@riscstar.com> Content-Language: en-US From: Alex Elder In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 12/28/25 6:31 AM, Guodong Xu wrote: > Hi, Alex > > On Sat, Dec 27, 2025 at 5:28 AM Alex Elder wrote: >> >> On 12/22/25 7:04 AM, Guodong Xu wrote: >>> Add descriptions for five new extensions: Ssccptr, Sscounterenw, Sstvala, >>> Sstvecd, and Ssu64xl. These extensions are ratified in RISC-V Profiles >>> Version 1.0 (commit b1d806605f87 "Updated to ratified state."). >>> >>> They are introduced as new extension names for existing features and >>> regulate implementation details for RISC-V Profile compliance. According >>> to RISC-V Profiles Version 1.0 and RVA23 Profiles Version 1.0, their >>> requirement status are: >>> >>> - Ssccptr: Mandatory in RVA20S64, RVA22S64, RVA23S64 >>> - Sscounterenw: Mandatory in RVA22S64, RVA23S64 >>> - Sstvala: Mandatory in RVA20S64, RVA22S64, RVA23S64 >>> - Sstvecd: Mandatory in RVA20S64, RVA22S64, RVA23S64 >>> - Ssu64xl: Optional in RVA20S64, RVA22S64; Mandatory in RVA23S64 >> >> Again, I did not verify your statements about where these are >> optional and mandatory, but I assume they're correct. > > Thanks for the review. > > As mentioned in my response to patch 8, the requirement status for these > extensions is clearly defined in the RISC-V Profiles specification(s) which > I mentioned above. > > I have verified these details against the official document to ensure > accuracy. Right, my comment here was just to be transparent that *I* did not check this, and am trusting that what you said is right. Thanks for confirming. -Alex > The extension descriptions are included in the commit message to provide > necessary background information. > >> >>> Signed-off-by: Guodong Xu >>> --- >>> v2: New patch. >>> --- >>> .../devicetree/bindings/riscv/extensions.yaml | 32 ++++++++++++++++++++++ >>> 1 file changed, 32 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml >>> index a6b9d7e3edf86ecfb117ba72e295ef097bdc9831..ed7a88c0ab3b7dc7ad4a4d2fd300d6fb33ef050c 100644 >>> --- a/Documentation/devicetree/bindings/riscv/extensions.yaml >>> +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml >>> @@ -160,12 +160,26 @@ properties: >>> behavioural changes to interrupts as frozen at commit ccbddab >>> ("Merge pull request #42 from riscv/jhauser-2023-RC4") of riscv-aia. >>> >>> + - const: ssccptr >>> + description: | >>> + The standard Ssccptr extension for main memory (cacheability and >>> + coherence) hardware page-table reads, as ratified in RISC-V >>> + Profiles Version 1.0, with commit b1d806605f87 ("Updated to >>> + ratified state.") >>> + >>> - const: sscofpmf >>> description: | >>> The standard Sscofpmf supervisor-level extension for count overflow >>> and mode-based filtering as ratified at commit 01d1df0 ("Add ability >>> to manually trigger workflow. (#2)") of riscv-count-overflow. >>> >>> + - const: sscounterenw >>> + description: | >>> + The standard Sscounterenw extension for support writable enables >>> + in scounteren for any supported counter, as ratified in RISC-V > > Here, I used the abbreviated version. See below. > >>> + Profiles Version 1.0, with commit b1d806605f87 ("Updated to >>> + ratified state.") >> >> Maybe you should just copy the text from the RVA23 specification >> for all of these. So something like: >> >> For any hpmcounter that is not read-only zero, the corresponding >> bit in scounteren must be writable. This was ratified in the >> RISC-V Profiles specification version 1.0, with commit ... >> > > As you noticed, the RVA23 Profile v1.0 provides two versions of explanation > for Sscounterenw: > "Sscounterenw: For any hpmcounter that is not read-only zero, the corresponding > bit in scounteren must be writable" > "Sscounterenw: Support writeable enables for any supported counter" > > BR, > Guodong Xu > >>> + >>> - const: ssnpm >>> description: | >>> The standard Ssnpm extension for next-mode pointer masking as >>> @@ -178,6 +192,24 @@ properties: >>> ratified at commit 3f9ed34 ("Add ability to manually trigger >>> workflow. (#2)") of riscv-time-compare. >>> >>> + - const: sstvala >>> + description: | >>> + The standard Sstvala extension for stval provides all needed values >>> + as ratified in RISC-V Profiles Version 1.0, with commit b1d806605f87 >>> + ("Updated to ratified state.") >> >> Or alternative to the full text in the spec, you could copy over >> the abbreviated description listed in the glossary at the end >> instead. Something like: >> >> stval provides all needed values. This was ratified in... >> >> -Alex >> >>> + >>> + - const: sstvecd >>> + description: | >>> + The standard Sstvecd extension for stvec supports Direct mode as >>> + ratified in RISC-V Profiles Version 1.0, with commit b1d806605f87 >>> + ("Updated to ratified state.") >>> + >>> + - const: ssu64xl >>> + description: | >>> + The standard Ssu64xl extension for UXLEN=64 must be supported, as >>> + ratified in RISC-V Profiles Version 1.0, with commit b1d806605f87 >>> + ("Updated to ratified state.") >>> + >>> - const: svade >>> description: | >>> The standard Svade supervisor-level extension for SW-managed PTE A/D >>> >>