]> git.proxmox.com Git - mirror_edk2.git/blobdiff - ArmPkg/Library/BdsLib/BdsLinuxFdt.c
ArmPkg/BdsLib: Removed PSCI discoverability from the Linux loader
[mirror_edk2.git] / ArmPkg / Library / BdsLib / BdsLinuxFdt.c
index 5ca24d7a7df8e34790c9294588986ae5c0b1f5e9..fb7bd5646e874155244b1c31b1e595f01d87c0d6 100644 (file)
@@ -1,6 +1,6 @@
 /** @file\r
 *\r
-*  Copyright (c) 2011-2013, ARM Limited. All rights reserved.\r
+*  Copyright (c) 2011-2014, ARM Limited. All rights reserved.\r
 *\r
 *  This program and the accompanying materials\r
 *  are licensed and made available under the terms and conditions of the BSD License\r
@@ -16,8 +16,6 @@
 #include <Library/PcdLib.h>\r
 #include <libfdt.h>\r
 \r
-#include <IndustryStandard/ArmSmc.h>\r
-\r
 #include "BdsInternal.h"\r
 #include "BdsLinuxLoader.h"\r
 \r
@@ -214,38 +212,6 @@ IsLinuxReservedRegion (
   }\r
 }\r
 \r
-\r
-STATIC\r
-BOOLEAN\r
-IsPsciSmcSupported (\r
-  VOID\r
-  )\r
-{\r
-  BOOLEAN               PsciSmcSupported;\r
-  UINTN                 Rx;\r
-\r
-  PsciSmcSupported = FALSE;\r
-\r
-  // Check the SMC response to the Presence SMC\r
-  Rx = ARM_SMC_ID_PRESENCE;\r
-  ArmCallSmc (&Rx);\r
-  if (Rx == 1) {\r
-    // Check the SMC UID\r
-    Rx = ARM_SMC_ID_UID;\r
-    ArmCallSmc (&Rx);\r
-    if (Rx == ARM_TRUSTZONE_UID_4LETTERID) {\r
-      Rx = ARM_SMC_ID_UID + 1;\r
-      ArmCallSmc (&Rx);\r
-      if (Rx == ARM_TRUSTZONE_ARM_UID) {\r
-        PsciSmcSupported = TRUE;\r
-      }\r
-    }\r
-  }\r
-\r
-  return PsciSmcSupported;\r
-}\r
-\r
-\r
 /**\r
 ** Relocate the FDT blob to a more appropriate location for the Linux kernel.\r
 ** This function will allocate memory for the relocated FDT blob.\r
@@ -360,11 +326,9 @@ PrepareFdt (
   UINTN                 DescriptorSize;\r
   UINT32                DescriptorVersion;\r
   UINTN                 Pages;\r
-  BOOLEAN               PsciSmcSupported;\r
   UINTN                 OriginalFdtSize;\r
   BOOLEAN               CpusNodeExist;\r
   UINTN                 CoreMpId;\r
-  UINTN                 Smc;\r
 \r
   NewFdtBlobAllocation = 0;\r
 \r
@@ -395,17 +359,6 @@ PrepareFdt (
     goto FAIL_RELOCATE_FDT;\r
   }\r
 \r
-  //\r
-  // Ensure the Power State Coordination Interface (PSCI) SMCs are there if supported\r
-  //\r
-  PsciSmcSupported = FALSE;\r
-  if (FeaturePcdGet (PcdArmPsciSupport) == TRUE) {\r
-    PsciSmcSupported = IsPsciSmcSupported();\r
-    if (PsciSmcSupported == FALSE) {\r
-      DEBUG ((EFI_D_ERROR, "Warning: The Power State Coordination Interface (PSCI) is not supported by your platform Trusted Firmware.\n"));\r
-    }\r
-  }\r
-\r
   fdt = (VOID*)(UINTN)NewFdtBlobBase;\r
 \r
   node = fdt_subnode_offset (fdt, 0, "chosen");\r
@@ -564,31 +517,29 @@ PrepareFdt (
 \r
           CoreMpId = cpu_to_fdtn (CoreMpId);\r
           fdt_setprop (fdt, cpu_node, "reg", &CoreMpId, sizeof (CoreMpId));\r
-          if (PsciSmcSupported) {\r
-            fdt_setprop_string (fdt, cpu_node, "enable-method", "psci");\r
-          }\r
         } else {\r
           cpu_node = fdt_subnode_offset(fdt, node, Name);\r
         }\r
 \r
-        // If Power State Coordination Interface (PSCI) is not supported then it is expected the secondary\r
-        // cores are spinning waiting for the Operating System to release them\r
-        if ((PsciSmcSupported == FALSE) && (cpu_node >= 0)) {\r
-          // We as the bootloader are responsible for either creating or updating\r
-          // these entries. Do not trust the entries in the DT. We only know about\r
-          // 'spin-table' type. Do not try to update other types if defined.\r
-          Method = fdt_getprop(fdt, cpu_node, "enable-method", &lenp);\r
-          if ( (Method == NULL) || (!AsciiStrCmp((CHAR8 *)Method, "spin-table")) ) {\r
-            fdt_setprop_string(fdt, cpu_node, "enable-method", "spin-table");\r
-            CpuReleaseAddr = cpu_to_fdt64(ArmCoreInfoTable[Index].MailboxSetAddress);\r
-            fdt_setprop(fdt, cpu_node, "cpu-release-addr", &CpuReleaseAddr, sizeof(CpuReleaseAddr));\r
-\r
-            // If it is not the primary core than the cpu should be disabled\r
-            if (((ArmCoreInfoTable[Index].ClusterId != ClusterId) || (ArmCoreInfoTable[Index].CoreId != CoreId))) {\r
-              fdt_setprop_string(fdt, cpu_node, "status", "disabled");\r
+        if (cpu_node >= 0) {\r
+          Method = fdt_getprop (fdt, cpu_node, "enable-method", &lenp);\r
+          // We only care when 'enable-method' == 'spin-table'. If the enable-method is not defined\r
+          // or defined as 'psci' then we ignore its properties.\r
+          if ((Method != NULL) && (AsciiStrCmp ((CHAR8 *)Method, "spin-table") == 0)) {\r
+            // There are two cases;\r
+            //  - UEFI firmware parked the secondary cores and/or UEFI firmware is aware of the CPU\r
+            //    release addresses (PcdArmLinuxSpinTable == TRUE)\r
+            //  - the parking of the secondary cores has been managed before starting UEFI and/or UEFI\r
+            //    does not anything about the CPU release addresses - in this case we do nothing\r
+            if (FeaturePcdGet (PcdArmLinuxSpinTable)) {\r
+              CpuReleaseAddr = cpu_to_fdt64 (ArmCoreInfoTable[Index].MailboxSetAddress);\r
+              fdt_setprop (fdt, cpu_node, "cpu-release-addr", &CpuReleaseAddr, sizeof(CpuReleaseAddr));\r
+\r
+              // If it is not the primary core than the cpu should be disabled\r
+              if (((ArmCoreInfoTable[Index].ClusterId != ClusterId) || (ArmCoreInfoTable[Index].CoreId != CoreId))) {\r
+                fdt_setprop_string(fdt, cpu_node, "status", "disabled");\r
+              }\r
             }\r
-          } else {\r
-            Print(L"Warning: Unsupported enable-method type for CPU[%d] : %a\n", Index, (CHAR8 *)Method);\r
           }\r
         }\r
       }\r
@@ -596,36 +547,6 @@ PrepareFdt (
     }\r
   }\r
 \r
-  // If the Power State Coordination Interface is supported then we signal it in the Device Tree\r
-  if (PsciSmcSupported == TRUE) {\r
-    // Before to create it we check if the node is not already defined in the Device Tree\r
-    node = fdt_subnode_offset(fdt, 0, "psci");\r
-    if (node < 0) {\r
-      // The 'psci' node does not exist, create it\r
-      node = fdt_add_subnode(fdt, 0, "psci");\r
-      if (node < 0) {\r
-        DEBUG((EFI_D_ERROR,"Error on creating 'psci' node\n"));\r
-        Status = EFI_INVALID_PARAMETER;\r
-        goto FAIL_COMPLETE_FDT;\r
-      } else {\r
-        fdt_setprop_string (fdt, node, "compatible", "arm,psci");\r
-        fdt_setprop_string (fdt, node, "method", "smc");\r
-\r
-        Smc = cpu_to_fdtn (ARM_SMC_ARM_CPU_SUSPEND);\r
-        fdt_setprop (fdt, node, "cpu_suspend", &Smc, sizeof (Smc));\r
-\r
-        Smc = cpu_to_fdtn (ARM_SMC_ARM_CPU_OFF);\r
-        fdt_setprop (fdt, node, "cpu_off", &Smc, sizeof (Smc));\r
-\r
-        Smc = cpu_to_fdtn (ARM_SMC_ARM_CPU_ON);\r
-        fdt_setprop (fdt, node, "cpu_on", &Smc, sizeof (Smc));\r
-\r
-        Smc = cpu_to_fdtn (ARM_SMC_ARM_MIGRATE);\r
-        fdt_setprop (fdt, node, "migrate", &Smc, sizeof (Smc));\r
-      }\r
-    }\r
-  }\r
-\r
   DEBUG_CODE_BEGIN();\r
     //DebugDumpFdt (fdt);\r
   DEBUG_CODE_END();\r