From 41d1a627133a43051f82a0218edb3bd5c8b6fb85 Mon Sep 17 00:00:00 2001 From: Megan Wachs Date: Mon, 5 Mar 2018 15:29:14 -0800 Subject: [PATCH] PLIC: Update RegFieldDesc to reflect the fact that source 0 isn't like all the others --- src/main/scala/devices/tilelink/Plic.scala | 19 ++++++++++++++++--- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/src/main/scala/devices/tilelink/Plic.scala b/src/main/scala/devices/tilelink/Plic.scala index 1a2b2b15..72e72eb3 100644 --- a/src/main/scala/devices/tilelink/Plic.scala +++ b/src/main/scala/devices/tilelink/Plic.scala @@ -168,8 +168,17 @@ class TLPLIC(params: PLICParams, beatBytes: Int)(implicit p: Parameters) extends harts(hart) := ShiftRegister(Reg(next = maxPri) > Cat(UInt(1), threshold(hart)), params.intStages) } - def priorityRegDesc(i: Int) = RegFieldDesc(s"priority_$i", s"Acting priority of interrupt source $i", reset=if (nPriorities > 0) None else Some(1)) - def pendingRegDesc(i: Int) = RegFieldDesc(s"pending_$i", s"Set to 1 if interrupt source $i is pending, regardless of its enable or priority setting.") + def priorityRegDesc(i: Int) = if (i > 0) { + RegFieldDesc(s"priority_$i", s"Acting priority of interrupt source $i", reset=if (nPriorities > 0) None else Some(1)) + } else { + RegFieldDesc("reserved", "", reset=Some(0), access=RegFieldAccessType.R) + } + def pendingRegDesc(i: Int) = if (i > 0) { + RegFieldDesc(s"pending_$i", s"Set to 1 if interrupt source $i is pending, regardless of its enable or priority setting.") + } else { + RegFieldDesc("reserved", "", reset=Some(0), access=RegFieldAccessType.R) + } + def priorityRegField(x: UInt, i: Int) = if (nPriorities > 0) RegField(32, x, priorityRegDesc(i)) else RegField.r(32, x, priorityRegDesc(i)) val priorityRegFields = Seq(PLICConsts.priorityBase -> RegFieldGroup("priority", Some("Acting priorities of each interrupt source. 32 bits for each interrupt source."), priority.zipWithIndex.map{case (p, i) => priorityRegField(p, i)})) @@ -179,7 +188,11 @@ class TLPLIC(params: PLICParams, beatBytes: Int)(implicit p: Parameters) extends val enableRegFields = enables.zipWithIndex.map { case (e, i) => PLICConsts.enableBase(i) -> RegFieldGroup(s"enables_${i}", Some(s"Enable bits for each interrupt source for target $i. 1 bit for each interrupt source."), - e.zipWithIndex.map{case (b, j) => RegField(1, b, RegFieldDesc(s"enable_${i}_${j}", s"Enable interrupt for source $j for target $i.", reset=None))}) + e.zipWithIndex.map{case (b, j) => if (j > 0) { + RegField(1, b, RegFieldDesc(s"enable_${i}_${j}", s"Enable interrupt for source $j for target $i.", reset=None)) + } else { + RegField(1, b, RegFieldDesc("reserved", "", reset=Some(0), access=RegFieldAccessType.R)) + }}) } // When a hart reads a claim/complete register, then the