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

rimage: Set of small improvements #8532

Merged
merged 3 commits into from
Dec 11, 2023
Merged

Conversation

softwarecki
Copy link
Collaborator

Added limit of a number of input modules provided in a command line to the maximum number of supported modules (size of module array / MAX_MODULES)

Created ALIGN_UP macro to simplify calculations.
Used DIV_ROUND_UP macro to calculate page count.

Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

tools/rimage/src/rimage.c Show resolved Hide resolved
@softwarecki softwarecki force-pushed the rimage-fix branch 2 times, most recently from e53e25b to 9bbda46 Compare November 28, 2023 10:11
@lgirdwood
Copy link
Member

@softwarecki any ETA, these look nice.

Added limit of a number of input modules provided in a command line to the
maximum number of supported modules (size of module array / MAX_MODULES)

Signed-off-by: Adrian Warecki <adrian.warecki@intel.com>
Used DIV_ROUND_UP macro to calculate page count.

Signed-off-by: Adrian Warecki <adrian.warecki@intel.com>
Created ALIGN_UP macro to simplify calculations.

Signed-off-by: Adrian Warecki <adrian.warecki@intel.com>
@softwarecki
Copy link
Collaborator Author

rebased

@softwarecki softwarecki marked this pull request as ready for review December 7, 2023 14:30
@kv2019i kv2019i merged commit 7a63f66 into thesofproject:main Dec 11, 2023
43 of 46 checks passed
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.

5 participants