Hi Suhaas On 12/01/26 15:06, Suhaas Joshi wrote: > Instead of hard-coding ATF and OPTEE addresses in firewall configuration > templates, use K3_*_LOAD_ADDR. Doing so ensures that if someone moves > ATF/OPTEE regions, the change gets picked up by binman without > explicitly having to modify dts files. > > Signed-off-by: Suhaas Joshi <[email protected]> > --- > arch/arm/dts/k3-binman.dtsi | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/arch/arm/dts/k3-binman.dtsi b/arch/arm/dts/k3-binman.dtsi > index 761b1730464..0fd93f9536a 100644 > --- a/arch/arm/dts/k3-binman.dtsi > +++ b/arch/arm/dts/k3-binman.dtsi > @@ -476,8 +476,8 @@ > permissions = <((FWPRIVID_ARMV8 << FWPRIVID_SHIFT) | > FWPERM_SECURE_PRIV_RWCD | > FWPERM_SECURE_USER_RWCD)>; > - start_address = <0x0 0x70000000>; > - end_address = <0x0 0x7001ffff>; > + start_address = <0x0 CONFIG_K3_ATF_LOAD_ADDR>; > + end_address = <0x0 (CONFIG_K3_ATF_LOAD_ADDR + 0x1ffff)>; > }; > firewall_armv8_optee_fg: template-8 { > control = <(FWCTRL_EN | FWCTRL_LOCK | > @@ -485,8 +485,8 @@ > permissions = <((FWPRIVID_ARMV8 << FWPRIVID_SHIFT) | > FWPERM_SECURE_PRIV_RWCD | > FWPERM_SECURE_USER_RWCD)>; > - start_address = <0x0 0x9e800000>; > - end_address = <0x0 0x9fffffff>; > + start_address = <0x0 CONFIG_K3_OPTEE_LOAD_ADDR>; > + end_address = <0x0 (CONFIG_K3_OPTEE_LOAD_ADDR + 0x17fffff)>; > }; > > ti_falcon_template: template-9 {
Reviewed-by: Neha Malcom Francis <[email protected]> -- Thanking You Neha Malcom Francis
