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

Interface port connection using .* is wrong (?) #320

Closed
veripoolbot opened this issue Jan 27, 2011 · 2 comments
Closed

Interface port connection using .* is wrong (?) #320

veripoolbot opened this issue Jan 27, 2011 · 2 comments
Assignees
Labels

Comments

@veripoolbot
Copy link
Collaborator


Author Name: Pierre-David Pfister
Original Redmine Issue: 320 from https://www.veripool.org
Original Date: 2011-01-27
Original Assignee: Wilson Snyder (@wsnyder)


Hi,

It seems to me there are at least 2 problems with the .* for interfaces. E.g.

interface intf1;
  logic a;
  logic b;
  modport mp1 (input a, output b);
endinterface

interface intf2 (intf1.mp1 c);
  logic e;
endinterface

module top;
  intf1 c(.*);
  intf2 f(.*);
endmodule

First, some comments are left out when saving the file after C-c C-a (this is cosmetics, but is a little annoying).

Second, I'm not sure the internal declarations (i.e. not the port declarations) should be in the interface connection. I think it should not since these are not interface ports. When I try to compile, it produces compilation error. E.g. Signals 'a' and 'b' for c interface instantiation in 'top' should not be in the interface connection, same for signal 'e' for 'f' interface instantiation in 'top'.

Thanks,

Pierre

@veripoolbot
Copy link
Collaborator Author


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2011-01-27T21:56:58Z


#2 needs more thought, right now it's a side effect of how AUTOINST works, which simply isn't right.

I just fixed 1, I can make a release now if you want it.

@veripoolbot
Copy link
Collaborator Author


Original Redmine Comment
Author Name: Wilson Snyder (@wsnyder)
Original Date: 2015-03-13T22:26:51Z


The interface issue discusses was fixed somewhere along the line - a and b no longer appear.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants