diff mbox series

[2/2] dt-bindings: riscv: Add Zicclsm ISA extension description.

Message ID 20240809162240.1842373-3-jesse@rivosinc.com
State Not Applicable
Headers show
Series RISC-V: Add Zicclsm extension support | expand

Checks

Context Check Description
robh/checkpatch success
robh/patch-applied success
robh/dtbs-check warning build log
robh/dt-meta-schema success

Commit Message

Jesse Taube Aug. 9, 2024, 4:22 p.m. UTC
Add description for Zicclsm ISA extension.

Signed-off-by: Jesse Taube <jesse@rivosinc.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
---
V1 -> V2:
 - New patch
V2 -> V3:
 - No changes
V3 -> V4:
 - No changes
V4 -> V5:
 - No changes
V5 -> V6:
 - No changes
V6 -> V7:
 - No changes
V7 -> V8:
 - Rebase onto 2d1f51d8a4b0 (palmer/for-next)
---
 Documentation/devicetree/bindings/riscv/extensions.yaml | 7 +++++++
 1 file changed, 7 insertions(+)

Comments

Rob Herring Aug. 9, 2024, 6:15 p.m. UTC | #1
On Fri, Aug 09, 2024 at 12:22:40PM -0400, Jesse Taube wrote:
> Add description for Zicclsm ISA extension.
> 
> Signed-off-by: Jesse Taube <jesse@rivosinc.com>
> Acked-by: Conor Dooley <conor.dooley@microchip.com>
> ---
> V1 -> V2:
>  - New patch
> V2 -> V3:
>  - No changes
> V3 -> V4:
>  - No changes
> V4 -> V5:
>  - No changes
> V5 -> V6:
>  - No changes
> V6 -> V7:
>  - No changes
> V7 -> V8:
>  - Rebase onto 2d1f51d8a4b0 (palmer/for-next)

Please also put the version in the subject. '-vN' is the git-send-email 
option to do it for you.

Rob
Charlie Jenkins Aug. 9, 2024, 6:56 p.m. UTC | #2
On Fri, Aug 09, 2024 at 12:15:36PM -0600, Rob Herring wrote:
> On Fri, Aug 09, 2024 at 12:22:40PM -0400, Jesse Taube wrote:
> > Add description for Zicclsm ISA extension.
> > 
> > Signed-off-by: Jesse Taube <jesse@rivosinc.com>
> > Acked-by: Conor Dooley <conor.dooley@microchip.com>
Reviewed-by: Charlie Jenkins <charlie@rivosinc.com>
> > ---
> > V1 -> V2:
> >  - New patch
> > V2 -> V3:
> >  - No changes
> > V3 -> V4:
> >  - No changes
> > V4 -> V5:
> >  - No changes
> > V5 -> V6:
> >  - No changes
> > V6 -> V7:
> >  - No changes
> > V7 -> V8:
> >  - Rebase onto 2d1f51d8a4b0 (palmer/for-next)
> 
> Please also put the version in the subject. '-vN' is the git-send-email 
> option to do it for you.
> 
> Rob
> 

These patches were originally part of a different series [1] but are no
longer related to that series so I had asked Jesse to spin these off into a
different series. These version tags probably should not have been left
on here when made into this new series though.

- Charlie

Link:
https://lore.kernel.org/lkml/20240726163719.1667923-1-jesse@rivosinc.com/
[1]
Conor Dooley Aug. 12, 2024, 3:56 p.m. UTC | #3
On Fri, Aug 09, 2024 at 11:56:14AM -0700, Charlie Jenkins wrote:
> On Fri, Aug 09, 2024 at 12:15:36PM -0600, Rob Herring wrote:
> > On Fri, Aug 09, 2024 at 12:22:40PM -0400, Jesse Taube wrote:
> > > Add description for Zicclsm ISA extension.
> > > 
> > > Signed-off-by: Jesse Taube <jesse@rivosinc.com>
> > > Acked-by: Conor Dooley <conor.dooley@microchip.com>
> Reviewed-by: Charlie Jenkins <charlie@rivosinc.com>
> > > ---
> > > V1 -> V2:
> > >  - New patch
> > > V2 -> V3:
> > >  - No changes
> > > V3 -> V4:
> > >  - No changes
> > > V4 -> V5:
> > >  - No changes
> > > V5 -> V6:
> > >  - No changes
> > > V6 -> V7:
> > >  - No changes
> > > V7 -> V8:
> > >  - Rebase onto 2d1f51d8a4b0 (palmer/for-next)
> > 
> > Please also put the version in the subject. '-vN' is the git-send-email 
> > option to do it for you.
> > 
> > Rob
> > 
> 
> These patches were originally part of a different series [1] but are no
> longer related to that series so I had asked Jesse to spin these off into a
> different series. These version tags probably should not have been left
> on here when made into this new series though.

I dunno, I disagree. I think the versioning should continue on being
split - especially when there's been tags provided on earlier versions.
Jesse Taube Aug. 12, 2024, 4:47 p.m. UTC | #4
On 8/12/24 11:56, Conor Dooley wrote:
> On Fri, Aug 09, 2024 at 11:56:14AM -0700, Charlie Jenkins wrote:
>> On Fri, Aug 09, 2024 at 12:15:36PM -0600, Rob Herring wrote:
>>> On Fri, Aug 09, 2024 at 12:22:40PM -0400, Jesse Taube wrote:
>>>> Add description for Zicclsm ISA extension.
>>>>
>>>> Signed-off-by: Jesse Taube <jesse@rivosinc.com>
>>>> Acked-by: Conor Dooley <conor.dooley@microchip.com>
>> Reviewed-by: Charlie Jenkins <charlie@rivosinc.com>
>>>> ---
>>>> V1 -> V2:
>>>>   - New patch
>>>> V2 -> V3:
>>>>   - No changes
>>>> V3 -> V4:
>>>>   - No changes
>>>> V4 -> V5:
>>>>   - No changes
>>>> V5 -> V6:
>>>>   - No changes
>>>> V6 -> V7:
>>>>   - No changes
>>>> V7 -> V8:
>>>>   - Rebase onto 2d1f51d8a4b0 (palmer/for-next)
>>>
>>> Please also put the version in the subject. '-vN' is the git-send-email
>>> option to do it for you.
>>>
>>> Rob
>>>
>>
>> These patches were originally part of a different series [1] but are no
>> longer related to that series so I had asked Jesse to spin these off into a
>> different series. These version tags probably should not have been left
>> on here when made into this new series though.
> 
> I dunno, I disagree. I think the versioning should continue on being
> split - especially when there's been tags provided on earlier versions.

Good to know I should keep the version number when splinting a set.

Thanks,
Jesse
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml
index a06dbc6b4928..d06ee82abcc1 100644
--- a/Documentation/devicetree/bindings/riscv/extensions.yaml
+++ b/Documentation/devicetree/bindings/riscv/extensions.yaml
@@ -361,6 +361,13 @@  properties:
             The standard Zicboz extension for cache-block zeroing as ratified
             in commit 3dd606f ("Create cmobase-v1.0.pdf") of riscv-CMOs.
 
+        - const: zicclsm
+          description:
+            The standard Zicclsm extension for misaligned support for all regular
+            load and store instructions (including scalar and vector) but not AMOs
+            or other specialized forms of memory access. Defined in the
+            RISC-V RVA Profiles Specification.
+
         - const: zicntr
           description:
             The standard Zicntr extension for base counters and timers, as