-
Notifications
You must be signed in to change notification settings - Fork 895
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
Feature request: $display cell #2310
Comments
Could you tell us more about the cell you'd want? What would be the inputs and semantics? Which parts of yosys should support it? (I'd guess Verilog frontend, cxxrtl, Verilog backend?) |
@mwkmwkmwk Didn't we discuss it in a meeting a few weeks ago? |
I believe the semantics we worked out was along the lines of: module $display(...);
parameter CLK_ENABLE = 1'b0;
parameter CLK_POLARITY = 1'b0;
parameter FORMAT_WIDTH;
parameter ARGS_WIDTH;
parameter ARGS_LAYOUT;
input CLK;
input EN;
input [FORMAT_WIDTH:0] FORMAT;
input [ARGS_WIDTH:0] ARGS;
endmodule The idea is to have a primitive that maps well to Verilog backend, CXXRTL backend, and formal verification. The function of Synchronous display cells trigger on the clock. Asynchronous display cells trigger on any change of inputs. (This was Claire's suggestion.) Names subject to bikeshed, of course. |
Yeah, sounds good (I just haven't seen it written down before). |
If I understand correctly, this is handled in #2459. |
This feature has been added under the name of |
A few persons (including myself) have expressed the need for a $display cell that would allow to write things on a console from inside a module (like $display in Verilog). This would make debugging far easier.
Related issue in nMigen: amaranth-lang/amaranth#432
The text was updated successfully, but these errors were encountered: