Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

asap7: dff.v was missing internal signals #2677

Conversation

oharboe
Copy link
Collaborator

@oharboe oharboe commented Jan 13, 2025

This caused .vcd files in mock-array not to have pin values for all pins.

This caused .vcd files in mock-array not to have pin values for all pins

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
@oharboe
Copy link
Collaborator Author

oharboe commented Jan 13, 2025

@maliberty Shooting from the hip here. I don't find IQNN, nor IQN exactly in the original behavioral Verilog(which Verilator can't support), but they do exist in the .lib file:

  cell (DFFASRHQNx1_ASAP7_75t_R) {
    [deleted]
    pin (QN) {
      direction : output;
      function : "IQN";
      power_down_function : "(!VDD) + (VSS)";
      related_ground_pin : VSS;
    [deleted]
    ff (IQN,IQNN) {
      clear : "!SETN";
      clear_preset_var1 : L;
      clear_preset_var2 : L;
      clocked_on : "CLK";
      next_state : "!D";
      power_down_function : "(!VDD) + (VSS)";
      preset : "!RESETN";
    }
  }

@maliberty
Copy link
Member

I disagree with this change. There is no need for these in Verilog. The state nodes are internal to Liberty. Perhaps you can expand on what the problem you are trying to solve is.

@oharboe
Copy link
Collaborator Author

oharboe commented Jan 13, 2025

I disagree with this change. There is no need for these in Verilog. The state nodes are internal to Liberty. Perhaps you can expand on what the problem you are trying to solve is.

I'm finding pins that are not annotated with activities by the .vcd file

@maliberty
Copy link
Member

We seem to be having this conversation in two PRs. Let's close this and discuss in the other.

@maliberty maliberty closed this Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants