• Palmer Dabbelt's avatar
    Merge patch series "ISA string parser cleanups" · 42b89447
    Palmer Dabbelt authored
    Conor Dooley <conor@kernel.org> says:
    
    From: Conor Dooley <conor.dooley@microchip.com>
    
    Here are some bits that were discussed with Drew on the "should we
    allow caps" threads that I have now created patches for:
    - splitting of riscv_of_processor_hartid() into two distinct functions,
      one for use purely during early boot, prior to the establishment of
      the possible-cpus mask & another to fit the other current use-cases
    - that then allows us to then completely skip some validation of the
      hartid in the parser
    - the biggest diff in the series is a rework of the comments in the
      parser, as I have mostly found the existing (sparse) ones to not be
      all that helpful whenever I have to go back and look at it
    - from writing the comments, I found a conditional doing a bit of a
      dance that I found counter-intuitive, so I've had a go at making that
      match what I would expect a little better
    - `i` implies 4 other extensions, so add them as extensions and set
      them for the craic. Sure why not like...
    
    * b4-shazam-merge:
      RISC-V: always report presence of extensions formerly part of the base ISA
      dt-bindings: riscv: explicitly mention assumption of Zicntr & Zihpm support
      RISC-V: remove decrement/increment dance in ISA string parser
      RISC-V: rework comments in ISA string parser
      RISC-V: validate riscv,isa at boot, not during ISA string parsing
      RISC-V: split early & late of_node to hartid mapping
      RISC-V: simplify register width check in ISA string parsing
    
    Link: https://lore.kernel.org/r/20230607-audacity-overhaul-82bb867a825f@spudSigned-off-by: default avatarPalmer Dabbelt <palmer@rivosinc.com>
    42b89447
cpu.c 9.43 KB