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

[edn/sw] Return the whole sts code in dif_edn_get_status #22465

Open
h-filali opened this issue Apr 8, 2024 · 3 comments
Open

[edn/sw] Return the whole sts code in dif_edn_get_status #22465

h-filali opened this issue Apr 8, 2024 · 3 comments
Assignees
Labels
Component:Software Issue related to Software IP:edn Subsystem:Entropy entropy_src, csrng, or edn related issues

Comments

@h-filali
Copy link

h-filali commented Apr 8, 2024

Description

Currently dif_edn_get_status sets the *set argument to 1 in case of an error and to 0 in case of success.
It would be better if the set bit could be replaced by the whole sts response signal, i.e. the whole status code (which is 0 in case of success and a positive value representing the status in the error case).

@h-filali h-filali added Component:DV DV issue: testbench, test case, etc. Component:Software Issue related to Software Subsystem:Entropy entropy_src, csrng, or edn related issues IP:edn labels Apr 8, 2024
@h-filali h-filali self-assigned this Apr 8, 2024
@vogelpi vogelpi added this to the Earlgrey-PROD.ROM milestone Apr 8, 2024
@vogelpi vogelpi removed the Component:DV DV issue: testbench, test case, etc. label Apr 8, 2024
@vogelpi
Copy link
Contributor

vogelpi commented Apr 8, 2024

Thanks for creating this issue @h-filali . This is not critical for the current hardware milestones. We need to check if this is relevant for ROM. I am now moving to that milestone.

@vogelpi vogelpi changed the title [csrng/sw] Return the whole sts code in dif_edn_get_status [edn/sw] Return the whole sts code in dif_edn_get_status Apr 8, 2024
@moidx
Copy link
Contributor

moidx commented Jun 1, 2024

Hi @vogelpi, currently the ROM only uses boot mode entropy. Should we move this to the ROM_EXT? That is the first time we do a full entropy complex configuration in continuous mode. If you agree, please move to the ROM_EXT milestone. Thanks!

@vogelpi
Copy link
Contributor

vogelpi commented Jun 7, 2024

Yes, that sounds fair to me @moidx . In ROM, the whole entropy complex runs in hardware defined mode. Software cannot operate it in the wrong way, so there is no strict need to infer status flags.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component:Software Issue related to Software IP:edn Subsystem:Entropy entropy_src, csrng, or edn related issues
Projects
None yet
Development

No branches or pull requests

3 participants