[coreboot-gerrit] Patch merged into coreboot/master: riscv: Add a bandaid for the new toolchain

gerrit at coreboot.org gerrit at coreboot.org
Wed Nov 2 19:01:46 CET 2016


the following patch was just integrated into master:
commit 571c2302525206715c2f12bc14fab931e8cdf757
Author: Ronald G. Minnich <rminnich at gmail.com>
Date:   Fri Oct 28 15:08:59 2016 -0700

    riscv: Add a bandaid for the new toolchain
    
    After I did a new toolchain build, I found the
    the mhartid register value is wrong for Spike.
    The docs seem to agree with Spike, not the
    code the toolchain produces?
    
    Until such time as the bitstreams and toolchain can find
    a way to agree, just hardcode it. We've been playing this game
    for two years now so this is hardly a new approach.
    
    This is intentionally ugly because we really need the
    toolchains and emulators and bitstreams to sync up,
    and that's not happening yet. Lowrisc
    allegedly implements the v1.9 spec but it's PTEs are clearly
    1.7. Once it all settles down we can just use constants
    supplied by the toolchain.
    
    I hope the syncup will have happened by the workshop in November.
    
    This gets spike running again.
    
    Change-Id: If259bcb6b6320ef01ed29a20ce3d2dcfd0bc7326
    Signed-off-by: Ronald G. Minnich <rminnich at gmail.com>
    Reviewed-on: https://review.coreboot.org/17183
    Tested-by: build bot (Jenkins)
    Reviewed-by: Paul Menzel <paulepanter at users.sourceforge.net>
    Reviewed-by: Jonathan Neuschäfer <j.neuschaefer at gmx.net>


See https://review.coreboot.org/17183 for details.

-gerrit



More information about the coreboot-gerrit mailing list