[libre-riscv-dev] big cleanup / reorg

Luke Kenneth Casson Leighton lkcl at lkcl.net
Wed Jul 31 13:55:14 BST 2019


i did a tidyup today including putting heavily commonly used
(identical) code into two key strategic modules: FPModBase and
FPModBaseChain.

FPModBase sets up self.i and self.o from ispec and ospec, and the
setup and process functions are absolutely cookie-cut identical across
literally every single FP module... so those go into a FPModBase,
which is now the default base class for pretty much absolutely every
FP module.

bear in mind that those are all *combinatorial* modules.

FPModBaseChain now contains the code that creates *pipelines* from
those combinatorial modules, and so derives from the (new)
DynamicPipe.  the *only* place now where DynamicPipe is used is:
FPModBaseChain.

all of the manual (near-duplicated) code that was formerly in setup()
has now gone, replaced with get_chain() which must return the chain of
*combinatorial* blocks that are to have the "data handling" - pipeline
registers - wrapped around them.

importantly: rather than be forced to declare the ispec and ospec, the
new FPModBaseChain returns the ispec and ospec of the *start and end
of the chain*.  this saves a lot of messing about.

right across the board these two cut out a huge amount of duplicated
cookie-cut identical code.

i've also started removing a lot of the FSM code that wasn't really
being used / maintained, as there is still the (original) jon dawson
code that does pretty much the same job, is smaller, and easier to
read.

l.



More information about the libre-riscv-dev mailing list