[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 #83 from Jacob Lifshay <programmerjake at gmail.com> ---
(In reply to Luke Kenneth Casson Leighton from comment #81)
> cpart_wid = -min(-lane_shapes[i] // c for i, c in part_counts.items())
> 
> that's definitely dividing the lane_shape _total_ by the part count.

that's because that's computing the width of *one* part. a single lane can
occupy multiple parts (a lane occupies part_counts[i] parts)

> you definitely gave:
> 
> 0: 5
> 1: 6
> 2: 12
> 3: 24

yup, because I *wanted* lanes to be 5, 6, 12, and 24 bits wide, not 5, 6, 6,
and 6. That was to demonstrate how the part width is calculated to fit the
minimum required bits, which can be less than a lane's width, because that lane
occupies multiple parts.

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