Merge pull request #245 from danh-arm/sm/psci_version

Increment the PSCI VERSION to 1.0 (PR v2)

docs/firmware-design.md

@@ -734,32 +734,43 @@ restoring the stack and CPU state and returning from the original SMC.
 
 TODO: Provide design walkthrough of PSCI implementation.
 
-The complete PSCI API is not yet implemented. The following functions are
+The PSCI v1.0 specification categorizes APIs as optional and mandatory. All the
-currently implemented:
+mandatory APIs in PSCI v1.0 and all the APIs in PSCI v0.2 draft specification
+[Power State Coordination Interface PDD] [PSCI] are implemented. The table lists
+the PSCI v1.0 APIs and their support in generic code.
 
--   `PSCI_VERSION`
+An API implementation might have a dependency on platform code e.g. CPU_SUSPEND
--   `CPU_OFF`
+requires the platform to export a part of the implementation. Hence the level
--   `CPU_ON`
+of support of the mandatory APIs depends upon the support exported by the
--   `CPU_SUSPEND`
+platform port as well. The Juno and FVP (all variants) platforms export all the
--   `AFFINITY_INFO`
+required support.
--   `SYSTEM_OFF`
--   `SYSTEM_RESET`
 
-The `CPU_ON`, `CPU_OFF` and `CPU_SUSPEND` functions implement the warm boot
+| PSCI v1.0 API         |Supported| Comments                                  |
-path in ARM Trusted Firmware. `CPU_ON` and `CPU_OFF` have undergone testing
+|:----------------------|:--------|:------------------------------------------|
-on all the supported FVPs. `CPU_SUSPEND` & `AFFINITY_INFO` have undergone
+|`PSCI_VERSION`         | Yes     | The version returned is 1.0               |
-testing only on the AEM v8 Base FVP. Support for `AFFINITY_INFO` is still
+|`CPU_SUSPEND`          | Yes*    | The original `power_state` format is used |
-experimental. Support for `CPU_SUSPEND` is stable for entry into power down
+|`CPU_OFF`              | Yes*    |                                           |
-states. Standby states are currently not supported. `PSCI_VERSION` is
+|`CPU_ON`               | Yes*    |                                           |
-present but completely untested in this version of the software.
+|`AFFINITY_INFO`        | Yes     |                                           |
+|`MIGRATE`              | Yes**   |                                           |
+|`MIGRATE_INFO_TYPE`    | Yes**   |                                           |
+|`MIGRATE_INFO_CPU`     | Yes**   |                                           |
+|`SYSTEM_OFF`           | Yes*    |                                           |
+|`SYSTEM_RESET`         | Yes*    |                                           |
+|`PSCI_FEATURES`        | Yes     |                                           |
+|`CPU_FREEZE`           | No      |                                           |
+|`CPU_DEFAULT_SUSPEND`  | No      |                                           |
+|`CPU_HW_STATE`         | No      |                                           |
+|`SYSTEM_SUSPEND`       | No      |                                           |
+|`PSCI_SET_SUSPEND_MODE`| No      |                                           |
+|`PSCI_STAT_RESIDENCY`  | No      |                                           |
+|`PSCI_STAT_COUNT`      | No      |                                           |
 
-The following unsupported functions return with a error code as documented in
+*Note : These PSCI APIs require platform power management hooks to be
-the [Power State Coordination Interface PDD] [PSCI].
+registered with the generic PSCI code to be supported.
 
--   `MIGRATE` : -1 (NOT_SUPPORTED)
+**Note : These PSCI APIs require appropriate Secure Payload Dispatcher
--   `MIGRATE_INFO_TYPE` : 2 (Trusted OS is either not present or does not
+hooks to be registered with the generic PSCI code to be supported.
-     require migration)
--   `MIGRATE_INFO_UP_CPU` : 0 (Return value is UNDEFINED)
 
 
 5.  Secure-EL1 Payloads and Dispatchers

fdts/fvp-base-gicv2-psci.dts

@@ -52,7 +52,7 @@
 	};
 
 	psci {
-		compatible = "arm,psci";
+		compatible = "arm,psci-1.0", "arm,psci-0.2", "arm,psci";
 		method = "smc";
 		cpu_suspend = <0xc4000001>;
 		cpu_off = <0x84000002>;

fdts/fvp-base-gicv2legacy-psci.dts

@@ -52,7 +52,7 @@
 	};
 
 	psci {
-		compatible = "arm,psci";
+		compatible = "arm,psci-1.0", "arm,psci-0.2", "arm,psci";
 		method = "smc";
 		cpu_suspend = <0xc4000001>;
 		cpu_off = <0x84000002>;

fdts/fvp-base-gicv3-psci.dts

@@ -52,7 +52,7 @@
 	};
 
 	psci {
-		compatible = "arm,psci";
+		compatible = "arm,psci-1.0", "arm,psci-0.2", "arm,psci";
 		method = "smc";
 		cpu_suspend = <0xc4000001>;
 		cpu_off = <0x84000002>;

fdts/fvp-foundation-gicv2-psci.dts

@@ -52,7 +52,7 @@
 	};
 
 	psci {
-		compatible = "arm,psci";
+		compatible = "arm,psci-1.0", "arm,psci-0.2", "arm,psci";
 		method = "smc";
 		cpu_suspend = <0xc4000001>;
 		cpu_off = <0x84000002>;

fdts/fvp-foundation-gicv2legacy-psci.dts

@@ -83,7 +83,7 @@
 		};
 
 		idle-states {
-			entry-method = "arm,psci";
+			entry-method = "arm,psci-1.0", "arm,psci-0.2", "arm,psci";
 
 			CPU_SLEEP_0: cpu-sleep-0 {
 				compatible = "arm,idle-state";

fdts/fvp-foundation-gicv3-psci.dts

@@ -52,7 +52,7 @@
 	};
 
 	psci {
-		compatible = "arm,psci";
+		compatible = "arm,psci-1.0", "arm,psci-0.2", "arm,psci";
 		method = "smc";
 		cpu_suspend = <0xc4000001>;
 		cpu_off = <0x84000002>;

include/bl31/services/psci.h

@@ -115,8 +115,8 @@
 /*******************************************************************************
  * PSCI version
  ******************************************************************************/
-#define PSCI_MAJOR_VER		(0 << 16)
+#define PSCI_MAJOR_VER		(1 << 16)
-#define PSCI_MINOR_VER		0x2
+#define PSCI_MINOR_VER		0x0
 
 /*******************************************************************************
  * PSCI error codes