clarifying comments on setting xer_ov/so
authorLuke Kenneth Casson Leighton <lkcl@lkcl.net>
Thu, 9 Jul 2020 19:00:07 +0000 (20:00 +0100)
committerLuke Kenneth Casson Leighton <lkcl@lkcl.net>
Thu, 9 Jul 2020 19:00:07 +0000 (20:00 +0100)
src/soc/fu/alu/output_stage.py

index f6380c5fd7d5cd9955a8dcd7c196396357201bfc..98e653a5321e6aa6258187375394152f8a62a2a7 100644 (file)
@@ -25,7 +25,9 @@ class ALUOutputStage(CommonOutputStage):
         # XXX see https://bugs.libre-soc.org/show_bug.cgi?id=319#c5
         comb += self.so.eq(xer_so_i[0] | xer_ov_i[0]) # SO
 
-        # copy overflow and sticky-overflow
+        # copy overflow and sticky-overflow.  indicate to CompALU if they
+        # are actually required (oe enabled/set) otherwise the CompALU
+        # can (will) ignore them.
         comb += self.o.xer_so.data.eq(self.so)
         comb += self.o.xer_so.ok.eq(op.oe.oe & op.oe.oe_ok)
         comb += self.o.xer_ov.data.eq(xer_ov_i)