[lint] Waive Verilator UNUSED warnings for packages
By default, recent versions of Verilator complain if not every
parameter that was defined in a package gets used somewhere. This
might be reasonable if there's a single top-level, but it doesn't work
at all if you have lots of different top-levels (several chip
top-levels, then each IP block), all of whom use different
combinations of the parameters.
Signed-off-by: Rupert Swarbrick <rswarbrick@lowrisc.org>
diff --git a/hw/lint/tools/verilator/common.vlt b/hw/lint/tools/verilator/common.vlt
index 8f41dc8..625b46a 100644
--- a/hw/lint/tools/verilator/common.vlt
+++ b/hw/lint/tools/verilator/common.vlt
@@ -13,3 +13,8 @@
// This warning gives wrong results with blackboxed embedded modules, see
// https://github.com/verilator/verilator/issues/2430
lint_off -rule DECLFILENAME -file "*" -match "Filename '*' does not match NOTFOUNDMODULE name:*"
+
+// Don't generate lint errors for unused parameters in packages. The point is
+// that a user of a package might not want to use all of the parameters it
+// defines.
+lint_off -rule UNUSED -file "*_pkg.sv" -match "*Parameter is not used*"
