Richard Henderson <[email protected]> writes:
> Remove the use of regime_is_secure from get_phys_addr_pmsav8. > Since we already had a local variable named secure, use that. > > Signed-off-by: Richard Henderson <[email protected]> > --- > target/arm/ptw.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/target/arm/ptw.c b/target/arm/ptw.c > index d424dec729..f7892a0c48 100644 > --- a/target/arm/ptw.c > +++ b/target/arm/ptw.c > @@ -1960,10 +1960,9 @@ void v8m_security_lookup(CPUARMState *env, uint32_t > address, > > static bool get_phys_addr_pmsav8(CPUARMState *env, uint32_t address, > MMUAccessType access_type, ARMMMUIdx > mmu_idx, > - GetPhysAddrResult *result, > + bool secure, GetPhysAddrResult > *result, NIT: why not use is_secure like all the other functions (and reformat the commit subject to match too). Otherwise: Reviewed-by: Alex Bennée <[email protected]> -- Alex Bennée
