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

Add aditional uint32_t object to ulp_insn_t to get the encoded instruction from bit-field structs (IDFGH-1492) #3759

Closed
wants to merge 1 commit into from

Conversation

hex007
Copy link
Contributor

@hex007 hex007 commented Jul 10, 2019

This allows users to quickly get the encoded instruction that can be assigned rather than needing memcpy to assign.

Current way of assignment looks bad :

ulp_insn_t wr_inst = I_WR_REG(DAC_REG, 19, 26, i);
*inst_p = *(uint32_t *)&wr_inst

@CLAassistant
Copy link

CLAassistant commented Jul 10, 2019

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot changed the title Add aditional uint32_t object to ulp_insn_t to get the encoded instruction from bit-field structs Add aditional uint32_t object to ulp_insn_t to get the encoded instruction from bit-field structs (IDFGH-1492) Jul 10, 2019
igrr pushed a commit that referenced this pull request Aug 21, 2019
Used to get the encoded instruction from bit-field structs.

Merges #3759
@hex007
Copy link
Contributor Author

hex007 commented Aug 24, 2019

@igrr Shall I close the PR considering that it has been merged ?

@Alvin1Zhang
Copy link
Collaborator

@hex007 Thanks for the contribution, you could close this PR, it has been merged. Thanks.

@hex007 hex007 closed this Sep 13, 2019
@hex007 hex007 deleted the add-ulp-decode-instruction branch September 13, 2019 03:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants