From 04d02a9c0b8598d3111e6a44a6f2aa1d872cc72f Mon Sep 17 00:00:00 2001 From: Xialin Liu <Xialin.Liu@ARM.com> Date: Thu, 13 Jun 2024 14:42:50 -0500 Subject: [PATCH] refactor(fvp): align the NV counter naming Align the naming of nv_counter to nv_ctr in the DTBs so that they match with the static C files. Update the binding documentation accordingly. This renaming is beneficial for the upcoming conversion tool that will convert CoT DT files to C files. Change-Id: If65d51ad9fc6445b1ae9937f1691becf8742cf01 Signed-off-by: Xialin Liu <Xialin.Liu@ARM.com> --- fdts/cca_cot_descriptors.dtsi | 20 +++++++-------- fdts/dualroot_cot_descriptors.dtsi | 26 ++++++++++---------- fdts/tbbr_cot_descriptors.dtsi | 26 ++++++++++---------- plat/arm/board/fvp/fdts/fvp_tb_fw_config.dts | 6 ++--- 4 files changed, 39 insertions(+), 39 deletions(-) diff --git a/fdts/cca_cot_descriptors.dtsi b/fdts/cca_cot_descriptors.dtsi index 821f60029..93d60ea3c 100644 --- a/fdts/cca_cot_descriptors.dtsi +++ b/fdts/cca_cot_descriptors.dtsi @@ -15,7 +15,7 @@ cot { cca_content_cert: cca_content_cert { root-certificate; image-id =<CCA_CONTENT_CERT_ID>; - antirollback-counter = <&cca_nv_counter>; + antirollback-counter = <&cca_nv_ctr>; tb_fw_hash: tb_fw_hash { oid = TRUSTED_BOOT_FW_HASH_OID; @@ -44,7 +44,7 @@ cot { root-certificate; image-id = <CORE_SWD_KEY_CERT_ID>; signing-key = <&swd_rot_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; core_swd_pk: core_swd_pk { oid = CORE_SWD_PK_OID; @@ -55,7 +55,7 @@ cot { image-id = <TRUSTED_OS_FW_CONTENT_CERT_ID>; parent = <&core_swd_key_cert>; signing-key = <&core_swd_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; tos_fw_hash: tos_fw_hash { oid = TRUSTED_OS_FW_HASH_OID; @@ -69,7 +69,7 @@ cot { root-certificate; image-id = <PLAT_KEY_CERT_ID>; signing-key = <&prot_pk>; - antirollback-counter = <&non_trusted_nv_counter>; + antirollback-counter = <&non_trusted_nv_ctr>; plat_pk: plat_pk { oid = PLAT_PK_OID; @@ -80,7 +80,7 @@ cot { image-id = <NON_TRUSTED_FW_CONTENT_CERT_ID>; parent = <&plat_key_cert>; signing-key = <&plat_pk>; - antirollback-counter = <&non_trusted_nv_counter>; + antirollback-counter = <&non_trusted_nv_ctr>; nt_world_bl_hash: nt_world_bl_hash { oid = NON_TRUSTED_WORLD_BOOTLOADER_HASH_OID; @@ -95,7 +95,7 @@ cot { image-id = <SIP_SP_CONTENT_CERT_ID>; parent = <&core_swd_key_cert>; signing-key = <&core_swd_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; sp_pkg1_hash: sp_pkg1_hash { oid = SP_PKG1_HASH_OID; @@ -115,7 +115,7 @@ cot { image-id = <PLAT_SP_CONTENT_CERT_ID>; parent = <&plat_key_cert>; signing-key = <&plat_pk>; - antirollback-counter = <&non_trusted_nv_counter>; + antirollback-counter = <&non_trusted_nv_ctr>; sp_pkg5_hash: sp_pkg5_hash { oid = SP_PKG5_HASH_OID; @@ -242,17 +242,17 @@ non_volatile_counters: non_volatile_counters { #address-cells = <1>; #size-cells = <0>; - cca_nv_counter: cca_nv_counter { + cca_nv_ctr: cca_nv_ctr { id = <TRUSTED_NV_CTR_ID>; oid = CCA_FW_NVCOUNTER_OID; }; - trusted_nv_counter: trusted_nv_counter { + trusted_nv_ctr: trusted_nv_ctr { id = <TRUSTED_NV_CTR_ID>; oid = TRUSTED_FW_NVCOUNTER_OID; }; - non_trusted_nv_counter: non_trusted_nv_counter { + non_trusted_nv_ctr: non_trusted_nv_ctr { id = <NON_TRUSTED_NV_CTR_ID>; oid = NON_TRUSTED_FW_NVCOUNTER_OID; }; diff --git a/fdts/dualroot_cot_descriptors.dtsi b/fdts/dualroot_cot_descriptors.dtsi index 459a1dda8..bea7af5b4 100644 --- a/fdts/dualroot_cot_descriptors.dtsi +++ b/fdts/dualroot_cot_descriptors.dtsi @@ -15,7 +15,7 @@ cot { trusted_boot_fw_cert: trusted_boot_fw_cert { root-certificate; image-id =<TRUSTED_BOOT_FW_CERT_ID>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; tb_fw_hash: tb_fw_hash { oid = TRUSTED_BOOT_FW_HASH_OID; @@ -34,7 +34,7 @@ cot { trusted_key_cert: trusted_key_cert { root-certificate; image-id = <TRUSTED_KEY_CERT_ID>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; trusted_world_pk: trusted_world_pk { oid = TRUSTED_WORLD_PK_OID; @@ -45,7 +45,7 @@ cot { image-id = <SCP_FW_KEY_CERT_ID>; parent = <&trusted_key_cert>; signing-key = <&trusted_world_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; scp_fw_content_pk: scp_fw_content_pk { oid = SCP_FW_CONTENT_CERT_PK_OID; @@ -56,7 +56,7 @@ cot { image-id = <SCP_FW_CONTENT_CERT_ID>; parent = <&scp_fw_key_cert>; signing-key = <&scp_fw_content_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; scp_fw_hash: scp_fw_hash { oid = SCP_FW_HASH_OID; @@ -67,7 +67,7 @@ cot { image-id = <SOC_FW_KEY_CERT_ID>; parent = <&trusted_key_cert>; signing-key = <&trusted_world_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; soc_fw_content_pk: soc_fw_content_pk { oid = SOC_FW_CONTENT_CERT_PK_OID; }; @@ -77,7 +77,7 @@ cot { image-id = <SOC_FW_CONTENT_CERT_ID>; parent = <&soc_fw_key_cert>; signing-key = <&soc_fw_content_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; soc_fw_hash: soc_fw_hash { oid = SOC_AP_FW_HASH_OID; @@ -91,7 +91,7 @@ cot { image-id = <TRUSTED_OS_FW_KEY_CERT_ID>; parent = <&trusted_key_cert>; signing-key = <&trusted_world_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; tos_fw_content_pk: tos_fw_content_pk { oid = TRUSTED_OS_FW_CONTENT_CERT_PK_OID; @@ -102,7 +102,7 @@ cot { image-id = <TRUSTED_OS_FW_CONTENT_CERT_ID>; parent = <&trusted_os_fw_key_cert>; signing-key = <&tos_fw_content_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; tos_fw_hash: tos_fw_hash { oid = TRUSTED_OS_FW_HASH_OID; @@ -122,7 +122,7 @@ cot { root-certificate; image-id = <NON_TRUSTED_FW_CONTENT_CERT_ID>; signing-key = <&prot_pk>; - antirollback-counter = <&non_trusted_nv_counter>; + antirollback-counter = <&non_trusted_nv_ctr>; nt_world_bl_hash: nt_world_bl_hash { oid = NON_TRUSTED_WORLD_BOOTLOADER_HASH_OID; @@ -137,7 +137,7 @@ cot { image-id = <SIP_SP_CONTENT_CERT_ID>; parent = <&trusted_key_cert>; signing-key = <&trusted_world_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; sp_pkg1_hash: sp_pkg1_hash { oid = SP_PKG1_HASH_OID; @@ -157,7 +157,7 @@ cot { root-certificate; image-id = <PLAT_SP_CONTENT_CERT_ID>; signing-key = <&prot_pk>; - antirollback-counter = <&non_trusted_nv_counter>; + antirollback-counter = <&non_trusted_nv_ctr>; sp_pkg5_hash: sp_pkg5_hash { oid = SP_PKG5_HASH_OID; @@ -296,12 +296,12 @@ non_volatile_counters: non_volatile_counters { #address-cells = <1>; #size-cells = <0>; - trusted_nv_counter: trusted_nv_counter { + trusted_nv_ctr: trusted_nv_ctr { id = <TRUSTED_NV_CTR_ID>; oid = TRUSTED_FW_NVCOUNTER_OID; }; - non_trusted_nv_counter: non_trusted_nv_counter { + non_trusted_nv_ctr: non_trusted_nv_ctr { id = <NON_TRUSTED_NV_CTR_ID>; oid = NON_TRUSTED_FW_NVCOUNTER_OID; }; diff --git a/fdts/tbbr_cot_descriptors.dtsi b/fdts/tbbr_cot_descriptors.dtsi index d11e2bef8..226915a9e 100644 --- a/fdts/tbbr_cot_descriptors.dtsi +++ b/fdts/tbbr_cot_descriptors.dtsi @@ -15,7 +15,7 @@ cot { trusted_boot_fw_cert: trusted_boot_fw_cert { root-certificate; image-id =<TRUSTED_BOOT_FW_CERT_ID>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; tb_fw_hash: tb_fw_hash { oid = TRUSTED_BOOT_FW_HASH_OID; @@ -34,7 +34,7 @@ cot { trusted_key_cert: trusted_key_cert { root-certificate; image-id = <TRUSTED_KEY_CERT_ID>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; trusted_world_pk: trusted_world_pk { oid = TRUSTED_WORLD_PK_OID; @@ -48,7 +48,7 @@ cot { image-id = <SCP_FW_KEY_CERT_ID>; parent = <&trusted_key_cert>; signing-key = <&trusted_world_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; scp_fw_content_pk: scp_fw_content_pk { oid = SCP_FW_CONTENT_CERT_PK_OID; @@ -59,7 +59,7 @@ cot { image-id = <SCP_FW_CONTENT_CERT_ID>; parent = <&scp_fw_key_cert>; signing-key = <&scp_fw_content_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; scp_fw_hash: scp_fw_hash { oid = SCP_FW_HASH_OID; @@ -70,7 +70,7 @@ cot { image-id = <SOC_FW_KEY_CERT_ID>; parent = <&trusted_key_cert>; signing-key = <&trusted_world_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; soc_fw_content_pk: soc_fw_content_pk { oid = SOC_FW_CONTENT_CERT_PK_OID; }; @@ -80,7 +80,7 @@ cot { image-id = <SOC_FW_CONTENT_CERT_ID>; parent = <&soc_fw_key_cert>; signing-key = <&soc_fw_content_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; soc_fw_hash: soc_fw_hash { oid = SOC_AP_FW_HASH_OID; @@ -94,7 +94,7 @@ cot { image-id = <TRUSTED_OS_FW_KEY_CERT_ID>; parent = <&trusted_key_cert>; signing-key = <&trusted_world_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; tos_fw_content_pk: tos_fw_content_pk { oid = TRUSTED_OS_FW_CONTENT_CERT_PK_OID; @@ -105,7 +105,7 @@ cot { image-id = <TRUSTED_OS_FW_CONTENT_CERT_ID>; parent = <&trusted_os_fw_key_cert>; signing-key = <&tos_fw_content_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; tos_fw_hash: tos_fw_hash { oid = TRUSTED_OS_FW_HASH_OID; @@ -125,7 +125,7 @@ cot { image-id = <NON_TRUSTED_FW_KEY_CERT_ID>; parent = <&trusted_key_cert>; signing-key = <&non_trusted_world_pk>; - antirollback-counter = <&non_trusted_nv_counter>; + antirollback-counter = <&non_trusted_nv_ctr>; nt_fw_content_pk: nt_fw_content_pk { oid = NON_TRUSTED_FW_CONTENT_CERT_PK_OID; @@ -136,7 +136,7 @@ cot { image-id = <NON_TRUSTED_FW_CONTENT_CERT_ID>; parent = <&non_trusted_fw_key_cert>; signing-key = <&nt_fw_content_pk>; - antirollback-counter = <&non_trusted_nv_counter>; + antirollback-counter = <&non_trusted_nv_ctr>; nt_world_bl_hash: nt_world_bl_hash { oid = NON_TRUSTED_WORLD_BOOTLOADER_HASH_OID; @@ -151,7 +151,7 @@ cot { image-id = <SIP_SP_CONTENT_CERT_ID>; parent = <&trusted_key_cert>; signing-key = <&trusted_world_pk>; - antirollback-counter = <&trusted_nv_counter>; + antirollback-counter = <&trusted_nv_ctr>; sp_pkg1_hash: sp_pkg1_hash { oid = SP_PKG1_HASH_OID; @@ -302,12 +302,12 @@ non_volatile_counters: non_volatile_counters { #address-cells = <1>; #size-cells = <0>; - trusted_nv_counter: trusted_nv_counter { + trusted_nv_ctr: trusted_nv_ctr { id = <TRUSTED_NV_CTR_ID>; oid = TRUSTED_FW_NVCOUNTER_OID; }; - non_trusted_nv_counter: non_trusted_nv_counter { + non_trusted_nv_ctr: non_trusted_nv_ctr { id = <NON_TRUSTED_NV_CTR_ID>; oid = NON_TRUSTED_FW_NVCOUNTER_OID; }; diff --git a/plat/arm/board/fvp/fdts/fvp_tb_fw_config.dts b/plat/arm/board/fvp/fdts/fvp_tb_fw_config.dts index b1d3bc185..f5d7f6586 100644 --- a/plat/arm/board/fvp/fdts/fvp_tb_fw_config.dts +++ b/plat/arm/board/fvp/fdts/fvp_tb_fw_config.dts @@ -117,16 +117,16 @@ #if defined(ARM_COT_cca) /* FVP does not support the CCA NV Counter so use the Trusted one. */ -&cca_nv_counter { +&cca_nv_ctr { reg = <TFW_NVCTR_BASE>; }; #endif -&trusted_nv_counter { +&trusted_nv_ctr { reg = <TFW_NVCTR_BASE>; }; -&non_trusted_nv_counter { +&non_trusted_nv_ctr { reg = <NTFW_CTR_BASE>; }; #endif