-
Notifications
You must be signed in to change notification settings - Fork 10
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
Support of wire declaration for the instance ports. #6
base: master
Are you sure you want to change the base?
Conversation
fix(help): remove duplicated tags
Could you please apply the modifications i required in the issue? |
Here is what needs to be done:
|
OK. How about this way. I will start to modify verilog_instance.py to support wire declaration with the assumption that there could be argv[2]. If argv[2] ==1, wire declaration., else port mapping. "ggb" is OK to me. After that, we can discuss the details about README and help. |
I finished the merge (removing verilog_wire.py), if !hasmapto('VerilogInstance') && maparg('gb','n') ==# '' if !hasmapto('VerilogWire') && maparg('ggb','n') ==# '' |
@KyleJeong , I will take a look at it in a few days, don't be surprised if my responses are delayed. |
Oops, sorry, i completely forgot about this PR. |
before (assume that user copied the port declaration to top)
input clk,
input [2:0] in_a,
input [2:0] in_b,
output reg [3:0] out_sum
module module_name (
input clk,
input [2:0] in_a,
input [2:0] in_b,
output reg [3:0] out_sum
);
...
end module
after (assume the user execute the wire declaration on top and execute the instantiation on buttom)
wire clk;
wire [2:0] in_a;
wire [2:0] in_b;
wire [3:0] out_sum;
module module_name (
.clk(clk),
.in_a(in_a),
.in_b(in_b),
.out_sum(out_sum)
);
...
end module
If we can do that in one command, it will be better definetely.