|
Message
From: Richard Herveille<richard@h...>
Date: Fri Apr 13 08:29:05 CEST 2007
Subject: [oc] simple wb_conbus patch
Why don't you upload it to CVS? You might want to consider increasing the effort a bit and support different bus sizes per port. But that really turns the effort into writing a bridge instead of a simple bus matrix.
Richard
-----Original Message----- From: cores-bounces@o... [mailto:cores-bounces@o...] On Behalf Of Matt Ettus Sent: Thursday, April 12, 2007 11:54 PM To: cores@o...; chisuhua@y... Subject: [oc] simple wb_conbus patch
In the CVS version of wb_conbus, all of the bus widths are set in the core itself using `defines. It is easier to customize this without changing the distributed files if these widths are set by parameters instead. I have modified the wb_conbus_top.v file to do this, and it still passes the regression tests. I haven't tested it further, but the change is pretty straightforward.
The changed file is attached. I hope this is the right forum for this.
Thanks, Matt
PS -- Normally I'd just send a diff, but since the change touches nearly every line and they mostly just remove one character per line (the tick marks), the diff is bigger than the actual file.
|
 |