forked from Juniper/open-register-design-tool
-
Notifications
You must be signed in to change notification settings - Fork 3
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
XML Data Dictionary #3
Comments
Understand request - it has some benefits.
|
FYI @sdnellen, I have also noticed this limitation in RDL's semantics for type names & dynamic assigns. If you're interested, I'm putting together a proposed behavior to uniquify type names here: SystemRDL/systemrdl-compiler#29 ps: congrats on the new gig! |
Thanks, will take a look! |
Would probably shy away from modifying the type name in favor of a
hash-based approach. Then provide users with a method/property indicating
if the given type is overridden and methods to test for type equivalence
based on appropriate set of component attributes (and special bonus points
for allowing adjustment of equivalence based on output being generated ;) -
this has proved useful). Issue with just using the name is that hierarchy
types are broken, eg if a register contains multiple modified fields, each
of these would need to be conveyed via a new register type and likewise
with regfile/addrmap types, as a single change in a sub-component really
means that the hierarchy component isn't identical to one without the child
change. With hash/equivalence check, each level can verify it's own
properties are same and also that it has equivalent children. Maybe you
have a solution for this, but didn't see in brief description.
…On Fri, Sep 20, 2019 at 1:34 AM Alex Mykyta ***@***.***> wrote:
FYI @sdnellen <https://github.com/sdnellen>, I have also noticed this
limitation in RDL's semantics for type names & dynamic assigns. If you're
interested, I'm putting together a proposed behavior to uniquify type names
here: SystemRDL/systemrdl-compiler#29
<SystemRDL/systemrdl-compiler#29>
ps: congrats on the new gig!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#3?email_source=notifications&email_token=AADJ4OKFOYUGRWXPHFUWKY3QKROFLA5CNFSM4IG4V3M2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7FTV7Y#issuecomment-533412607>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AADJ4OORAJINOQVTF2YAG5TQKROFLANCNFSM4IG4V3MQ>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
From open-register-design-tool created by WebMonkey007: Juniper#52
The current XML output includes the Instance view but does not include the "data dictionary "
ie
addrmap counters {
counter_t transaction_cnt[10];
counter_t error_cnt[5];
};
so the xml will list tranaction_cnt and error_cnt but there is no indication that register def is the same.
I would be useful to include a "type" tag in the instances and a "data type" tree in the xml or a parallel xml file.
The text was updated successfully, but these errors were encountered: