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

plat-vexpress: juno: add missing DRAM1 #1874

Merged
merged 1 commit into from
Oct 12, 2017

Conversation

jenswi-linaro
Copy link
Contributor

Defines missing DRAM1 base 0x880000000 size 0x180000000 for Juno.

Tested-by: Jens Wiklander jens.wiklander@linaro.org (Juno)
Signed-off-by: Jens Wiklander jens.wiklander@linaro.org

Depends on #1860

@kenlsoft
Copy link

Hi Jens,
after register_nsec_ddr() is applied, adding SDP with CFG_TEE_SDP_MEM_BASE and the panic would appear on vexpress; and we have to do some hard code to skip SDP area in NSDDR to fix it.

I may comment this problem in other places so please just reply one is OK, sorry.

-Ken.

@jenswi-linaro
Copy link
Contributor Author

Hi Ken,

I'll address that problem in a separate pull request.

Thanks,
Jens

@jenswi-linaro
Copy link
Contributor Author

@ken08liu14, it seems the problem only occurs if you make out of tree changes to OP-TEE. So the problem isn't directly related to this pull request.

If you'd like to discuss this further please create an issue at https://github.com/OP-TEE/optee_os/issues

@jforissier
Copy link
Contributor

Acked-by: Jerome Forissier <jerome.forissier@linaro.org>

Defines missing DRAM1 base 0x880000000 size 0x180000000 for Juno.

Acked-by: Jerome Forissier <jerome.forissier@linaro.org>
Tested-by: Jens Wiklander <jens.wiklander@linaro.org> (Juno)
Signed-off-by: Jens Wiklander <jens.wiklander@linaro.org>
@jenswi-linaro
Copy link
Contributor Author

Rebased, tag applied

@jforissier jforissier merged commit 785be2e into OP-TEE:master Oct 12, 2017
@jenswi-linaro jenswi-linaro deleted the juno_dram1 branch October 12, 2017 09:55
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.

3 participants