Re: [myhdl-list] myhdl.AlwaysCombError: signal (startmpy) used as inout in always_comb function argument

[Christopher F. <chr...@gm...>]

On 2/11/2015 10:54 AM, Josy Boelen wrote:
<snip>
>> Do you have some example code?
>>
>> <snip>
> Henry,
>
> the code is a handful (as usual) but I made a simplified example (as
> Chris also likes to see ...):

:)

>
> def contrivedtwostatemachineexample( Clk, Reset,  StartP, ... )
<snip>
>
>      @always_comb
>      def smcomb():


Why have a single comb vs. two comb?

     def smcombfp():
     ...

     def smcombsn():
     ...

I believe the inout comb detection is a mechanism to
protect against unwanted comb feedback.  If you have
a complicated @always_comb it probably make sense to
split it out rather than remove the comb feedback
detection.

Regards,
Chris