Skip to content

Commit

Permalink
p9_tod_init: Update spread spectrum synchronization for Axone
Browse files Browse the repository at this point in the history
Axone has two spread PLLs that can individually synchronized, so the
TOD timer facility grew two enable bits. Set both of them to
synchronize both PLLs at the same time. The enable bits are don't
care on Nimbus/Cumulus so the procedure will still work the same
way there.

Check only the status of one PLL, knowing that if both are enabled,
either both or none will get started by the TOD. This maintains
Nimbus/Cumulus compatibility on the checking part.

Change-Id: I1a84f4531f4eba62903c118711fc6bc84af28fbb
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67952
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: HWSV CI <hwsv-ci+hostboot@us.ibm.com>
Tested-by: Hostboot CI <hostboot-ci+hostboot@us.ibm.com>
Reviewed-by: Soma Bhanutej <soma.bhanu@in.ibm.com>
Reviewed-by: Joseph J. McGill <jmcgill@us.ibm.com>
Reviewed-by: Jennifer A. Stofer <stofer@us.ibm.com>
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/67957
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Reviewed-by: Christian R. Geddes <crgeddes@us.ibm.com>
  • Loading branch information
fenkes-ibm authored and crgeddes committed Oct 30, 2018
1 parent 7b0ac24 commit 93478ad
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/import/chips/p9/procedures/hwp/nest/p9_tod_init.C
Original file line number Diff line number Diff line change
Expand Up @@ -171,8 +171,8 @@ fapi2::ReturnCode sync_spread(
l_tod_value_data),
"Error reading TOD_VALUE_REG");

l_tod_timer_data = (l_tod_value_data + (C_SSCG_START_DELAY << 4)) &
0xFFFFFFFFFFFFFFF0ULL;
l_tod_timer_data = ((l_tod_value_data + (C_SSCG_START_DELAY << 4)) &
0xFFFFFFFFFFFFFFF0ULL) | 0xC;

// Write value > tod_value to TOD Timer Register
for (auto l_chip : l_targets)
Expand Down

0 comments on commit 93478ad

Please sign in to comment.