diff options
author | Corey Swenson <cswenson@us.ibm.com> | 2019-06-20 13:40:32 -0500 |
---|---|---|
committer | Daniel M. Crowell <dcrowell@us.ibm.com> | 2019-06-21 19:13:49 -0500 |
commit | 4b81399d6441708373f5694e5db21fa35633405a (patch) | |
tree | 44600845cce17598802598f9768ebfec1f9fc126 /src/include | |
parent | 926ec6f90b177df59e7b8aa9534aa3e6ec003183 (diff) | |
download | talos-hostboot-4b81399d6441708373f5694e5db21fa35633405a.tar.gz talos-hostboot-4b81399d6441708373f5694e5db21fa35633405a.zip |
Assembly for DARN instruction inconsistent
Sometimes generating incorrect L value for the darn instruction.
Removed the L variable and hard-coded into the instruction.
Change-Id: I5b478d2c220858942320f6fea3cb09d0ba6aee42
Reviewed-on: http://rchgit01.rchland.ibm.com/gerrit1/79278
Tested-by: Jenkins Server <pfd-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP Build CI <op-jenkins+hostboot@us.ibm.com>
Tested-by: FSP CI Jenkins <fsp-CI-jenkins+hostboot@us.ibm.com>
Tested-by: Jenkins OP HW <op-hw-jenkins+hostboot@us.ibm.com>
Reviewed-by: Roland Veloz <rveloz@us.ibm.com>
Reviewed-by: Daniel M. Crowell <dcrowell@us.ibm.com>
Diffstat (limited to 'src/include')
-rw-r--r-- | src/include/arch/ppc.H | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/src/include/arch/ppc.H b/src/include/arch/ppc.H index 2e6639da9..a6c60be9e 100644 --- a/src/include/arch/ppc.H +++ b/src/include/arch/ppc.H @@ -427,11 +427,9 @@ ALWAYS_INLINE inline uint64_t getDarn() { register uint64_t rt = 0; - register uint64_t L = 1; // L=1 conditioned random number - asm volatile(".long 0x7C0005E6 | " - "((%0 & 0x1F) << 21) | " - "((%1 & 0x3) << 16)" : - "=r" (rt), "=r" (L)); + asm volatile(".long 0x7C0105E6 | " + "((%0 & 0x1F) << 21)" : + "=r" (rt)); return rt; } |