problem with `define in ekv3.va

[gjcoram]

As distributed, the file ekv3model-main/code/ekv3.va has

`define ALL 1

`define RF

lines 81 and 83. Thus, if one tries to use ekv3_all.va, one gets two module declarations.

I think the intent was that these defines would not be in ekv3.va.

[gjcoram]

Unfortunately, if one comments out both defines above, then the value VS, VD, VG are never set (lines 581-605).

The top of the file declares "module ekv3" when ALL is not defined; I think there should be an `else for setting VS, VD, VG.

[dwarning]

For VS, VD and VG it is solvable by moving the declaration from ekv3_variables_oppoint.va to ekv3_variables.va. But other OP variables remains unset.