Pu Wang

> What's the rationale for this? > > Also it's best not to reformat a bunch of code you're not adding anything to. It makes the diff much larger than...

> Is it about preserving as much as possible naming ? Yes. I'm not sure whether this PR covers all the Stream utility function that returned Streams can be renamed?

> > Yes. > > I'm not sure whether this PR covers all the Stream utility function that returned Streams can be renamed? > > Normaly it should be possible....

> So, the case of the > > ```scala > val someQueue = StreamFifo(...) > val queuePop = someQueue.io.pop > ``` > > Is one where a .combStage is good...

> Ping @pwang7 Let me fix this PR later.

I wrote a test piece, and it works. Thanks! ```Scala import spinal.core._ import spinal.core.sim._ import spinal.lib._ class SpinalEnumTest extends AnyFunSuite { object WorkReqOpCode extends SpinalEnum { val RDMA_WRITE, RDMA_WRITE_WITH_IMM, SEND,...

> I got : > > ``` > [Progress] Start SpinalEnumExample test simulation with seed 1933650510 > NOTE 0 time: in report TSO > NOTE 0 time: in report RDMA_READ...

> Hello @pwang7 did the upgrade fix your issue? Yes, this PR can be closed.

newer version should be fine

Thanks for your reply! @alexforencich I'm using ZYNQ 7000, xc7z020clg400-2. BTW, if I'd like to reuse your RGMII Verilog code, which file to start with?