Arteris-IP / tlm2-interfaces

contains TLM2 based interfaces for AXI, ACE, CHI and other standard protocols
Apache License 2.0
44 stars 9 forks source link

operation_resp missing implementation #9

Open mslijepc opened 10 months ago

mslijepc commented 10 months ago

Hi, I can just see the function prototype

https://github.com/Arteris-IP/tlm2-interfaces/blob/8b50c1098f5dba38f025065d91e1a5cf0837e458/axi/pe/ace_target_pe.h#L92

but the implementation is missing. Is this function intended to be used, or is obsolete?

mslijepc commented 10 months ago

I am using simple memory for read-write, and have created callbacks for this purpose which I set in set_operation_cb

eyck commented 10 months ago

For the purposes so far there was no need to implement the operation_resp since it just forwards accesses using the axi4 initiator socket (as noted in #10)

mslijepc commented 10 months ago

In that case, memory should implement ace protocol state machine?

hliu71 commented 10 months ago

you do not need to implement the protocol state machine in the memory. you can do the conneciton it in this way: ace_target_pe->axi_target_pe->memory. to some extend ace_target_pe acts here as adapter between ace and axi, which forwards the normal read/write to axi. Besides it exends with snoop function. you can imagin that, the real function of ace target pe includes ace_target_pe+axi_target_pe.