[SQUASH ME] Cleaned Up Figure and Wording

Author: AlexandreSinger

doc/src/tutorials/timing_analysis/index.rst

@@ -15,14 +15,14 @@ multi-clock timing analysis, which tools like OpenSTA have better support for.
 
 .. figure:: timing_analysis_design_cycle.png
 
-    Timing analysis design cycle flow diagram.
+    Post-implementation timing analysis design cycle.
 
-A user design cycle which would use this would perform the following steps:
-    1. Run VPR with the timing commands it can support.
+A user design cycle which would use post-implementation timing analysis could perform the following:
+    1. Run VPR with the timing commands it can support (simplified constraints).
     2. Perform timing analysis on the resulting netlist using OpenSTA with
        more complex timing commands.
-    3. Modify the user design to meet the complex timing constraints.
-    4. Repeat.
+    3. The user can then modify the design to meet the complex timing constraints based on the timing report produced by OpenSTA.
+    4. The design can then be fed back into VPR and the process can repeat until all constraints are met.
 
 Generating the Post-Implementation Netlist for STA
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@@ -47,7 +47,7 @@ directory for convenience:
 
 .. note:: Replace :term:`$VTR_ROOT` with the root directory of the VTR source tree
 
-To perform timing analysis externally to VTR, we will need to create an SDC file
+To perform timing analysis externally to VTR, we will need to provide an SDC file
 which will contain the timing constraints on the clocks and I/Os in the circuit.
 For this tutorial, we will use the following ``clma.sdc`` file:
 
@@ -81,10 +81,10 @@ for timing analysis using VPR.
 
 In this command, we provide the architecture, circuit, the channel width, and
 the SDC file. The other four commands are what generate the necessary netlist
-files for timing simulation:
+files for timing analysis:
  * ``--gen_post_synthesis_netlist on``: This will generate the post-implementation netlist as a Verilog file.
- * ``--gen_post_implementation_sdc on``: This will have VPR generate a new SDC file which contains extra timing information used from proper timing analysis.
- * ``--post_synth_netlist_unconn_inputs gnd``: For timing analysis with OpenSTA we want to be explicit about how we handle unconnected signal ports. Here we just ground them for simplicity.
+ * ``--gen_post_implementation_sdc on``: This will have VPR generate a new SDC file which contains extra timing information based on how VPR implemented the design.
+ * ``--post_synth_netlist_unconn_inputs gnd``: For timing analysis with OpenSTA, we should be explicit about how we handle unconnected signal ports. Here we just ground them for simplicity.
  * ``--post_synth_netlist_module_parameters off``: OpenSTA does not allow parameters to be used in the netlist. This command tells VPR to generate a netlist without using parameters.
 
 Once VPR has completed, we should see the generated Verilog netlist, SDF file, and SDC file:
@@ -98,11 +98,11 @@ Once VPR has completed, we should see the generated Verilog netlist, SDF file, a
 Performing Timing Analysis using OpenSTA
 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-To perform static timing analysis for this tutorial, we will be using OpenSTA.
-Other STA tools can be used, they may use slightly different commands.
+To perform static timing analysis for this tutorial, we will be using OpenSTA (https://github.com/parallaxsw/OpenSTA ).
+Other STA tools can be used, however they may use slightly different commands.
 
 First, install OpenSTA onto your system. Building from source is a good option,
-using the following instructions:
+which can be done using the following instructions:
 https://github.com/parallaxsw/OpenSTA?tab=readme-ov-file#build-from-source
 
 After OpenSTA is installed, we can perfrom static timing analysis on the post-implementation
@@ -119,24 +119,24 @@ It is easiest to write a ``sdf_delays.tcl`` file to setup and configure the timi
     # an SDF file. This contains descriptions of the timing arcs of the primitives
     # in the circuit.
     read_liberty $VTR_ROOT/vtr_flow/primitives.lib
-    
+
     # Read the post-implementation netlist generated by VPR.
     read_verilog top_post_synthesis.v
-    
+
     # Link the top-level design.
     link_design top
-    
+
     # Read the post-synthesis SDF file.
     read_sdf top_post_synthesis.sdf
-    
+
     # Read the SDC commands generated by VPR.
     read_sdc top_post_synthesis.sdc
-    
-    # Report the setup and hold timing checks using OpenSTA and write to a file.
+
+    # Report the setup and hold timing checks using OpenSTA and write them to files.
     report_checks -group_path_count 100 -path_delay max > open_sta_report_timing.setup.rpt
     report_checks -group_path_count 100 -path_delay min > open_sta_report_timing.hold.rpt
 
-    # Report the minimum period of the clock and the fmax.
+    # Report the minimum period of the clockis and their fmax.
     report_clock_min_period
 
     # Exit OpenSTA's TCL terminal.