From 737eeb4a73fb153247bc1b0030407e7b2a2addad Mon Sep 17 00:00:00 2001 From: Pirmin Vogel Date: Mon, 21 Feb 2022 09:18:25 +0100 Subject: [PATCH] [csrng] Update AscentLint waiver 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/OpenTitan#10844. Signed-off-by: Pirmin Vogel --- hw/ip/csrng/lint/csrng.waiver | 2 ++ 1 file changed, 2 insertions(+) diff --git a/hw/ip/csrng/lint/csrng.waiver b/hw/ip/csrng/lint/csrng.waiver index d839b49acc32b..75a4e51a72567 100644 --- a/hw/ip/csrng/lint/csrng.waiver +++ b/hw/ip/csrng/lint/csrng.waiver @@ -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."