[Libre-soc-bugs] [Bug 713] PartitionedSignal enhancement to add partition-context-aware lengths

[bugzilla-daemon at libre-soc.org]

https://bugs.libre-soc.org/show_bug.cgi?id=713

--- Comment #23 from Jacob Lifshay <programmerjake at gmail.com> ---
(In reply to Luke Kenneth Casson Leighton from comment #22)
> the only thing being as i said, it produces different
> overall widths depending on the input lane_shape which
> is a showstopper.
> 
> if overall width (64 bit) is given as the input it fixes that.
> 
> does mean a safety check is needed, elwidth=0b11 means
> 4 partitions, 4 partitions of 24 bit obviously won't fit
> in 64 bit!

why go through all that when you can just derive the signal width from the lane
shapes -- avoiding wasting extra bits or running out of bits, as well as
avoiding the need to manually specify size? Having the size be computed doesn't
really cause problems imho...

-- 
You are receiving this mail because:
You are on the CC list for the bug.