-
Notifications
You must be signed in to change notification settings - Fork 6.9k
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
dsp: utils: Added macros to shift and convert float/fixed types #82439
dsp: utils: Added macros to shift and convert float/fixed types #82439
Conversation
Hello @mrcamilletti, and thank you very much for your first pull request to the Zephyr project! |
f96ef15
to
e3d9bb1
Compare
Hi @yperess, @stephanosio. Is there any possibility to review this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor nit on the documentation of the macros. But overall LGTM
e3d9bb1
to
5bc1e24
Compare
The DSP library does not provide functions to convert fixed representations (Q7,Q15,Q31) to float or double and viceversa. This commit implements those functions using a shift argument to indicate the number of decimal places. Signed-off-by: Matías Román Camilletti <[email protected]>
5bc1e24
to
1a94ce4
Compare
Hi @mrcamilletti! To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge. Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁 |
The DSP library does not provide functions to shift and convert fixed representations in Q7, Q15, and Q31 formats to float or double and vice versa. This commit implements those macros using a shift argument to indicate the number of decimal places.