From d529844a9dd0c2aa36eed2b64d5fb3bc27def647 Mon Sep 17 00:00:00 2001 From: Jerry Zhao Date: Fri, 21 Jun 2024 11:30:43 -0700 Subject: [PATCH] Update ClockGroup aggregation error to be more descriptive --- .../chipyard/src/main/scala/clocking/ClockGroupCombiner.scala | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/generators/chipyard/src/main/scala/clocking/ClockGroupCombiner.scala b/generators/chipyard/src/main/scala/clocking/ClockGroupCombiner.scala index 9b53cbef..9508e76f 100644 --- a/generators/chipyard/src/main/scala/clocking/ClockGroupCombiner.scala +++ b/generators/chipyard/src/main/scala/clocking/ClockGroupCombiner.scala @@ -48,9 +48,10 @@ class ClockGroupCombiner(implicit p: Parameters, v: ValName) extends LazyModule val name = combiners(i)._1 i = i + 1 require(g.size >= 1) + val names = g.map(_.name.getOrElse("unamed")) val takes = g.map(_.take).flatten require(takes.distinct.size <= 1, - s"Clock group $name has non-homogeneous requested ClockParameters $takes") + s"Clock group $name has non-homogeneous requested ClockParameters ${names.zip(takes)}") require(takes.size > 0, s"Clock group $name has no inheritable frequencies") (grouped ++ Seq(ClockSinkParameters(take = takes.headOption, name = Some(name))), r)