[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 #18 from Luke Kenneth Casson Leighton <lkcl at lkcl.net> ---
(In reply to Jacob Lifshay from comment #16)

> it's easy, we let Shape.cast do all the hard work, SimdLayout just passes
> the inputs to nmigen Shape.cast.

it's massively overengineered.  the very fact that you wrote text 5x longer
than the simple functions i outlined should be telling you everything
you need to know, here.

it also goes far ahead of what i can understand and process, and is absorbing
far more time than it should.

you're rushing ahead and solving problems that haven't yet been demonstrated
to be needed.

i have *no problem at all* with *ending up* with a design that is as
complicated
as SimdKind however it needs to be demonstrated and driven by *actual need*
rather than "predicted non-demonstrated requirements".

can you please therefore write those two simple functions
which will fit into the existing code.

this will then allow progress and effort put into fitting PartitionedSignal
into the ALUs.

*at that* point if we *actually* encounter *actual* needs for advanced features
*then and only then* do we come back and revisit.

yes i used to also overengineer.  it didn't go well.

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