Skip to content

Commit

Permalink
Update frequency selection related comments
Browse files Browse the repository at this point in the history
  • Loading branch information
davidbiancolin authored Feb 24, 2021
1 parent 58076cf commit 7f5d8c8
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions generators/firechip/src/main/scala/TargetConfigs.scala
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,10 @@ class WithFireSimConfigTweaks extends Config(
new WithFireSimSimpleClocks ++
// Required*: When using FireSim-as-top to provide a correct path to the target bootrom source
new WithBootROM ++
// Optional*: Removing this will require adjusting the UART baud rate and
// potential target-software changes to properly capture UART output
// Optional: This sets the default frequency for all buses in the system to 3.2 GHz
// (since unspecified bus frequencies will use the pbus frequency)
// This frequency selection matches FireSim's legacy selection and is required
// to support 200Gb NIC performance. You may select a smaller value.
new chipyard.config.WithPeripheryBusFrequency(3200.0) ++
// Optional: These three configs put the DRAM memory system in it's own clock domian.
// Removing the first config will result in the FASED timing model running
Expand All @@ -93,7 +95,7 @@ class WithFireSimConfigTweaks extends Config(
new testchipip.WithDefaultSerialTL ++
// Optional: Removing this will require using an initramfs under linux
new testchipip.WithBlockDevice ++
// Required*: Scale default baud rate with periphery bus frequency
// Optional: Set a UART baudrate (this selection matches FireSim's historical value)
new chipyard.config.WithUART(BigInt(3686400L)) ++
// Required: Do not support debug module w. JTAG until FIRRTL stops emitting @(posedge ~clock)
new chipyard.config.WithNoDebug
Expand Down

0 comments on commit 7f5d8c8

Please sign in to comment.