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

core: mmu: fix collect_device_mem_ranges() to pass size #6811

Merged
merged 1 commit into from
Apr 30, 2024

Conversation

ysbnim
Copy link
Contributor

@ysbnim ysbnim commented Apr 30, 2024

The 6th parameter of add_phys_mem() has to be the size of memory region and not the end address of the region.

Copy link
Contributor

@etienne-lms etienne-lms left a comment

Choose a reason for hiding this comment

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

Could you mention in the header line it's FF-A/SPMC-EL2 related. Or maybe:

core: mmu: fix memory regions found from ff-a manifest

Fix the 6th parameter of add_phys_mem() in collect_device_mem_ranges()
that has to be the size of memory region and not the end address of the
region.

S-o-b: ...

Could you add a fix tag?
Fixes: 72a6827a6353 ("core: arm: SPMC obtain device memory info from DTB")
with that tag added:
Reviewed-by: Etienne Carriere <etienne.carriere@foss.st.com>

@ysbnim
Copy link
Contributor Author

ysbnim commented Apr 30, 2024

Thank you for your review. I changed the commit message and added fix and reviewed tag.

Copy link
Contributor

@jenswi-linaro jenswi-linaro left a comment

Choose a reason for hiding this comment

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

Acked-by: Jens Wiklander <jens.wiklander@linaro.org>

Fix the 6th parameter of add_phys_mem() in collect_device_mem_ranges()
that has to be the size of memory region and not the end address of the
region.

Fixes: 72a6827 ("core: arm: SPMC obtain device memory info from DTB")
Signed-off-by: Sungbae Yoo <sungbaey@nvidia.com>
Reviewed-by: Etienne Carriere <etienne.carriere@foss.st.com>
Acked-by: Jens Wiklander <jens.wiklander@linaro.org>
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

4 participants