commit db68441c979d4f3623a0fb6b906683955ff95c66
author Nico Huber <nico.huber@secunet.com> Mon Jun 04 22:44:58 2018 +0200
committer Nico Huber <nico.h@gmx.de> Wed Jun 13 08:59:35 2018 +0000
tree c186ebb4b913c5a1fe1036aa19d28795123b074a
parent 57bebc7110c4f097135f523e8e68af056248e4a7

gma skylake power/clocks: Refactor to allow proof without inlining

Change-Id: Ie660c69377d8624a3a6662c6ccb7b5e4efcd4629
Signed-off-by: Nico Huber <nico.huber@secunet.com>
Reviewed-on: https://review.coreboot.org/26847
Tested-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>

common/skylake/hw-gfx-gma-power_and_clocks_skylake.adb

diff --git a/common/skylake/hw-gfx-gma-power_and_clocks_skylake.adb b/common/skylake/hw-gfx-gma-power_and_clocks_skylake.adb
index ea84992..c6eb2cf 100644
--- a/common/skylake/hw-gfx-gma-power_and_clocks_skylake.adb
+++ b/common/skylake/hw-gfx-gma-power_and_clocks_skylake.adb
@@ -89,11 +89,11 @@
    GT_MAILBOX_READY                    : constant := 1 * 2 ** 31;
 
    function CDCLK_CTL_CD_FREQ_DECIMAL
-     (Freq        : Positive;
+     (Freq        : Pos16;
       Plus_Half   : Boolean)
       return Word32 is
    begin
-      return Word32 (2 * (Freq - 1)) or (if Plus_Half then 1 else 0);
+      return Word32 (2 * (Pos32 (Freq) - 1)) or (if Plus_Half then 1 else 0);
    end CDCLK_CTL_CD_FREQ_DECIMAL;
 
    ----------------------------------------------------------------------------