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

Generate domain information for every port in Manifest files #1711

Open
martijnbastiaan opened this issue Mar 18, 2021 · 0 comments
Open

Generate domain information for every port in Manifest files #1711

martijnbastiaan opened this issue Mar 18, 2021 · 0 comments

Comments

@martijnbastiaan
Copy link
Member

Clocks, resets, and enables currently include their synthesize domain in manifest files:

                {
                    "domain": "System",
                    "width": 1,
                    "name": "clock",
                    "is_clock": true,
                    "type_name": "clock.addBound_types.clk_System"
                },

However, this doesn't happen for "normal" signals:

                {
                    "width": 33,
                    "name": "i",
                    "is_clock": false,
                    "type_name": "i.addBound_types.Maybe"
                }

It's currently not included because HWType does not store domain information for types other than clocks, resets, and enables. I'm not 100% sure, but I'm guessing a proper solution would be to keep all casts, and cast Signal a ~ a where appropriate. There's probably a more hacky way of achieving the same.

Having this information present would help integration with external tooling.

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

No branches or pull requests

1 participant