jdavidberger

I'm less worried about that; only `WishboneConfig` should ever actually look at the granularity directly. It exposes a method `wordAddressInc` and the `Wishbone` class uses that for `byteAddress` which is...

I went ahead and added the deprecation and private access modifier since this was the easiest option. I can still just remove unspecified outright if that is what we want...

> Is this primarily targeting aesthetics of the cli string? More or less yes. Right now the tool I'm targeting is something where I change a few parameters, run, look...

Worth noting that setting mode changes the carrier polynomial, and I get the same polynomials as charles when I set the same mode he had set -- which means we...

I cleaned up WishboneBusInterface too with additional tests for it. Which of the interfaces is meant to be used more? I made the slave factory variant support proper pipelining, whereas...

> Looking at it in more detail, I'm not sure we should keep the totally broken version of isAck etc. around. No working module could have come from the current...

> As a sidenote: found the reason again why I proposed to drop the `& CYC` from the condition in the slave if not pipelined: it's what all the examples...

> ERR - somehow faking an OK seems wrong but I can't think of many huge things that can go wrong... If we already have a way to do the...

In any case, you either have to modify / remove those flags or just mark them as deprecated. If the idea is that we preserve the functionality of those fields...