[Review Comment]: Retraction of compressed itf transactions not flexible enough

openhwgroup / core-v-xif

RISC-V eXtension interface that provides a generalized framework suitable to implement custom coprocessors and ISA extensions

https://docs.openhwgroup.org/projects/openhw-group-core-v-xif/en

Other

53 stars 23 forks source link

[Review Comment]: Retraction of compressed itf transactions not flexible enough #200

Closed Silabs-ArjanB closed 3 months ago

Silabs-ArjanB commented 3 months ago

Comment

Proposed Resolution

See above

Addition Info

No response

christian-herber-nxp commented 3 months ago

@Silabs-ArjanB The reason why we removed it was, that the assumption was that you would just change instr without de-asserting valid. Practically, that will surely work. That would be inline with the last sentence of your comment.

Let me know the path you prefer.

Silabs-ArjanB commented 3 months ago

I am fine with just changing instr while keeping valid asserted (without introducing an id), but this is not allowed as per above rules as it would require the processor to change hartid.