target/riscv/pmp.c: Fix pmp_decode_napot()
authorAnup Patel <anup@brainfault.org>
Fri, 14 Dec 2018 19:12:32 +0000 (19:12 +0000)
committerPalmer Dabbelt <palmer@sifive.com>
Thu, 20 Dec 2018 20:26:39 +0000 (12:26 -0800)
commit71a150bc914ef154d1321cc3602a4e80a433fc52
tree443d2b097ee1500acbce6869ba0c0a44a06c6f19
parent40061ac0bc5bdfcfa1234dbf8e2a880fd9fc4c2e
target/riscv/pmp.c: Fix pmp_decode_napot()

Currently, start and end address of a PMP region are not decoded
correctly by pmp_decode_napot().

Let's say we have a 128KB PMP region with base address as 0x80000000.
Now, the PMPADDRx CSR value for this region will be 0x20003fff.

The current pmp_decode_napot() implementation will decode PMPADDRx
CSR as t1=14, base=0x100000000, and range=0x1ffff whereas it should
have decoded PMPADDRx CSR as t1=14, base=0x80000000, and range=0x1fff.

This patch fixes the base value decoding in pmp_decode_napot() when
PMPADDRx CSR is not -1 (i.e. 0xffffffffffffffff).

Signed-off-by: Anup Patel <anup@brainfault.org>
Signed-off-by: Anup Patel <anup.patel@wdc.com>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Palmer Dabbelt <palmer@sifive.com>
Signed-off-by: Palmer Dabbelt <palmer@sifive.com>
target/riscv/pmp.c