Skip to content

Set All DUT Params

Eclipse Webmaster requested to merge github/fork/silabs-robin/xparams into main

Created by: silabs-robin

This PR explicitly sets all DUT parameters, and removes default values from "dut_wrap".

(Note: There are still defaults in the "constants" file, but the difference is that missing connections in the tb will be detected, and missing parameters in external "param_set" files will also be detected.)

Test status:

  • ci_check
    • Passes acceptably. (clic, debug_test2.)
  • Formal
    • Still runnable.

Background:

Today, I did a mistake and it got masked by the dut_wrap defaults. Yesterday, someone else did a mistake and it got masked by the dut_wrap defaults. In the past, several other mistakes have been masked by the dut_wrap defaults. This is a problem.

It might be convenient with defaults, when one has to type less when instantiating these modules. But I think it creates more problems than it solves. Therefore, I want to remove the defaults and make every instantiation explicit about its choices.

Merge request reports

Loading