Skip to content

Commit

Permalink
[csrng] Update AscentLint waiver
Browse files Browse the repository at this point in the history
CSRNG uses an unmasked AES cipher core intentionally. We can thus waive
all the lint errors related to non-default values for the SecMasking
and SecSBoxImpl parameters.

This is related to lowRISC#10844.

Signed-off-by: Pirmin Vogel <[email protected]>
  • Loading branch information
vogelpi authored and MaayanKashani committed Feb 28, 2022
1 parent e267356 commit 737eeb4
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions hw/ip/csrng/lint/csrng.waiver
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,5 @@

waive -rules {ONE_BIT_MEM_WIDTH} -location {prim_arbiter_ppc.sv} -regexp {.*has word width which is single bit wide.*} \
-comment "Usage case specific to CSRNG and how the arbiter is used."
waive -rules {LHS_TOO_SHORT} -location {aes_cipher_control_fsm.sv aes_cipher_core.sv aes_key_expand.sv aes_sbox.sv} -regexp {Bitlength mismatch between 'unused_assert_static_lint_error' length 1 and.*} \
-comment "CSRNG intentionally uses an unmasked AES implementation."

0 comments on commit 737eeb4

Please sign in to comment.