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

IPXACT parse hdl paths #14

Open
markcohen32 opened this issue Jan 15, 2023 · 2 comments
Open

IPXACT parse hdl paths #14

markcohen32 opened this issue Jan 15, 2023 · 2 comments

Comments

@markcohen32
Copy link

Hi,

What IPXACT xml tag is mapped to the hdl_path of a component? For example, I have an IPXACT file that one of its field supposed to have hdl_path, which is described in "pathSements" field, but the compiler doesn't seem to support it.

@amykyta3 amykyta3 transferred this issue from SystemRDL/systemrdl-compiler Jan 18, 2023
@amykyta3
Copy link
Member

amykyta3 commented Jan 18, 2023

The IP-XACT importer does not currently support ipxact:pathSegments tags, nor does it import any information into RDL's hdl_path

After reviewing the IP-XACT spec in a bit more detail, I think it may be difficult to add meaningful importer support for this. IP-XACT seems to do a much better job in capturing the semantics of HDL paths with its variety of accessHandle variants (support for indexing, arbitrary number of path views, etc). When importing, I would have to transcribe to RDL semantics which are pretty poorly designed when it comes to HDL paths.

SystemRDL spec authors did a really poor job of defining how HDL paths work to the point of them being almost useless. There is no sensible way to do path indexing, they limit to only two path views (RTL and gates), and the specification for the hdl_path_slice property is outright broken.

This means that if I were to add importer support, I suspect it would only work properly for a very narrow subset of IP-XACT features, rendering it pretty crippled. I suppose I could add support for simple cases, but it would be difficult to support the indexedAccessHandle tag which is basically essential any time arrays are used.

@MarinaQT
Copy link

Hello All

supporting the basic hdl_path specification from IP-XACT into RDL would already be very helpful!
Cause currently adding this translation with additional scripting on the top of peakrdl commands. And believe we are not the only ones that could use this basic translation.

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

No branches or pull requests

3 participants