search

Linaro / poplar-tools

4 stars 16 forks source link

build_instructions.md: Add notes to build and test OP-TEE #16

Closed vchong closed 6 years ago

vchong commented 6 years ago

Signed-off-by: Victor Chong victor.chong@linaro.org

vchong commented 6 years ago

[ 2.457573] sysfs: cannot create duplicate filename '/devices/virtual/misc/ion'

@shawnguo2 looks like you're right. Both Hisi and SWG ION patches cannot coexist so I'll remove the SWG ION patches from 4.9 kernel PR (https://github.com/Linaro/poplar-linux/pull/2).

vchong commented 6 years ago

The optee_os port by itself isn't that complicated to the point where we need to maintain a separate fork. optee_os PRs also move pretty fast usually, and patches for adding extra board/platform support can be merged in just a few days, so another fork is just extra maintenance we don't really need. It's also better to work directly upstream since that that means we will automatically be in sync with optee_client and optee_test repos.

shawnguo2 commented 6 years ago

In that case, we should probably get poplar support into optee_os upstream first, and then merge the document update in this PR.

vchong commented 6 years ago

Ok, agree. Let's hope the internal patch review clears asap so we can send the patch upstream.