search

pulp-platform / common_cells

Common SystemVerilog components
Other
493 stars 138 forks source link

Edge propagator ack #130

Closed glaserf closed 2 years ago

glaserf commented 2 years ago

Added a version of the edge propagator that has an acknowledge port which flags if the edge has actually propagated to the receiving side. Useful if the initiating circuit can only safely continue if the described condition is met.

niwis commented 2 years ago

Thank you @glaserf! Could you also add the module to README and CHANGELOG? One general question: There seems to be a lot of common code with edge_propagator. Do you think it would make sense to update edge_propagator such that it wraps edge_propagator_ack (leaving ack_o unconnected) to avoid code duplication?

glaserf commented 2 years ago

Cool! I implemented both suggestions.