diff options
| author | davidcunado-arm <david.cunado@arm.com> | 2018-01-29 09:54:52 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-01-29 09:54:52 +0000 |
| commit | 1b56ed666c95e20e5d85e7cd4ed2c65f8b3e072e (patch) | |
| tree | 7470688a74c37c47d2c3af11900c7b91cc361876 | |
| parent | 5df277800f89754a3ee103d944ddb1490bf1b36d (diff) | |
| parent | d5d5595ab16d54d779108922d3b74f24c94c7eb7 (diff) | |
Merge pull request #1241 from vchong/fixmemprot
hikey: fix assert in sec_protect()
| -rw-r--r-- | plat/hisilicon/hikey/hikey_security.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/plat/hisilicon/hikey/hikey_security.c b/plat/hisilicon/hikey/hikey_security.c index be8c39f4..863ad2b4 100644 --- a/plat/hisilicon/hikey/hikey_security.c +++ b/plat/hisilicon/hikey/hikey_security.c @@ -71,10 +71,12 @@ static void sec_protect(uint32_t region_base, uint32_t region_size, volatile struct rgn_attr_reg *rgn_attr; uint32_t i = 0; - assert(region < 1 || region > 15); - assert(!IS_POWER_OF_TWO(region_size) || region_size < 0x10000); - /* ensure secure region_base is aligned to region_size */ - assert((region_base & (region_size - 1))); + /* ensure secure region number is between 1-15 */ + assert(region > 0 && region < 16); + /* ensure secure region size is a power of 2 >= 64KB */ + assert(IS_POWER_OF_TWO(region_size) && region_size >= 0x10000); + /* ensure secure region address is aligned to region size */ + assert(!(region_base & (region_size - 1))); INFO("BL2: TrustZone: protecting %u bytes of memory at 0x%x\n", region_size, region_base); |