WIP: backend: Fix ax handshaking req_ready does no longer depend on ax_ready

[thommythomaso]

No description provided.

[micprog]

I am not 100% sure, but there may be some further incorrect handshake dependencies in idma_legalizer: https://github.com/pulp-platform/iDMA/blob/fix/backend/ax_handshaking-tbenz/src/idma_legalizer.sv#L379-L386
Another may be in idma_axi_transport layer: https://github.com/pulp-platform/iDMA/blob/fix/backend/ax_handshaking-tbenz/src/idma_axi_transport_layer.sv#L249-L257
I might be wrong, and I don't believe these affect the AXI path, but would be great if you could look at it quickly as well.

[thommythomaso]

I am not 100% sure, but there may be some further incorrect handshake dependencies in idma_legalizer: https://github.com/pulp-platform/iDMA/blob/fix/backend/ax_handshaking-tbenz/src/idma_legalizer.sv#L379-L386 Another may be in idma_axi_transport layer: https://github.com/pulp-platform/iDMA/blob/fix/backend/ax_handshaking-tbenz/src/idma_axi_transport_layer.sv#L249-L257 I might be wrong, and I don't believe these affect the AXI path, but would be great if you could look at it quickly as well.

I agree to lines #L379-L386 in the legalizer. This has to be rewritten at some point (this is anyway planned soon). So far this has never caused any issues so let's create an issue and postpone rewriting until a bug manifests.
The datapath, although looking a bit weird, should be correct.

[micprog]

I agree to lines #L379-L386 in the legalizer. This has to be rewritten at some point (this is anyway planned soon). So far this has never caused any issues so let's create an issue and postpone rewriting until a bug manifests.
The datapath, although looking a bit weird, should be correct.

Ok, thanks for checking!

[micprog]

Approve, see #6

[thommythomaso]

Issue #6 created.