refactor(fvp): add comment to RME run config

Add implication of RME options to following run configs:
fvp-tftf.rme
fvp-spm-tftf.rme.bmcov

Explain the effect of bp.secure_memory=0 and bp.has_rme=1

Align the ordering of options in the latter config to the
former for cosmetics.

Signed-off-by: Olivier Deprez <olivier.deprez@arm.com>
Change-Id: Ia43af1456fd9a38e790c6be069beaadc2dd70f43
diff --git a/run_config/fvp-spm-tftf.rme.bmcov b/run_config/fvp-spm-tftf.rme.bmcov
index 466ac1f..2c8f6c3 100644
--- a/run_config/fvp-spm-tftf.rme.bmcov
+++ b/run_config/fvp-spm-tftf.rme.bmcov
@@ -24,6 +24,19 @@
 	uart="2" file="hold_uart.exp" track_expect
 	uart="3" file="hold_uart.exp" track_expect
 
+	# For RME test configs, bp.secure_memory=0 and bp.has_rme=1 imply:
+	# TZ access controls disabled.
+	# RME access controls enabled.
+	# Only Root access to Trusted ROM and SRAM.
+	# SECURE/ROOT access only for below peripherals:
+	# -Trusted DRAM
+	# -REFCLK CNTControl (Generic Timer)
+	# -Trusted Watchdog SP805, Trustzone address space controller
+	# -AP_REFCLK CNTBase0 (Generic Timer)
+	# -Trusted RNG, Non-Volatile Counter and Root-Key Storage
+	# -Flash0, Flash1
+	# -uart0, uart1, uart2, uart3
+
 	if ! is_arm_jenkins_env && not_upon "$local_ci"; then
 		bmcov_plugin_path="${coverage_trace_plugin}"
 		bmcov_plugin="1"
@@ -31,17 +44,17 @@
 
 	model="$model" \
 		secure_memory="0" \
+		has_rme="1" \
 		amu_present="1" \
 		arch_version="8.6" \
 		has_branch_target_exception="1" \
-		has_rme="1" \
 		has_smmuv3_params="1" \
+		memory_tagging_support_level="2" \
+		restriction_on_speculative_execution="2" \
 		gicd_are_fixed_one="1" \
 		gicv3_ext_interrupt_range="1" \
 		gicd_ext_ppi_count="64" \
 		gicd_ext_spi_count="1024" \
-		memory_tagging_support_level="2" \
-		restriction_on_speculative_execution="2" \
 		gen_model_params
 
 	model="$model" gen_fvp_yaml
diff --git a/run_config/fvp-tftf.rme b/run_config/fvp-tftf.rme
index 45782e3..6822d9b 100644
--- a/run_config/fvp-tftf.rme
+++ b/run_config/fvp-tftf.rme
@@ -24,12 +24,25 @@
 	uart="2" file="hold_uart.exp" track_expect
 	uart="3" file="hold_uart.exp" track_expect
 
+	# For RME test configs, bp.secure_memory=0 and bp.has_rme=1 imply:
+	# TZ access controls disabled.
+	# RME access controls enabled.
+	# Only Root access to Trusted ROM and SRAM.
+	# SECURE/ROOT access only for below peripherals:
+	# -Trusted DRAM
+	# -REFCLK CNTControl (Generic Timer)
+	# -Trusted Watchdog SP805, Trustzone address space controller
+	# -AP_REFCLK CNTBase0 (Generic Timer)
+	# -Trusted RNG, Non-Volatile Counter and Root-Key Storage
+	# -Flash0, Flash1
+	# -uart0, uart1, uart2, uart3
+
 	model="$model" \
 		secure_memory="0" \
+		has_rme="1" \
 		amu_present="1" \
 		arch_version="8.6" \
 		has_branch_target_exception="1" \
-		has_rme="1" \
 		has_smmuv3_params="1" \
 		memory_tagging_support_level="2" \
 		restriction_on_speculative_execution="2" \