Remove read IDCODE instruction from VeeR JTAG TAP

[mkurc-ant]

This PR removes support for read IDCODE instruction from the TAP. This addresses #289 where Caliptra is to be connected to a system JTAG as a submodule.

[tmichalak]

@bharatpillilli should we target a different branch ?

[bharatpillilli]

@mcockrell-google - SInce we did a tagged release already, is it OK to push this to the main branch and then we do another tagged RTL release?

[mcockrell-google]

@mcockrell-google - SInce we did a tagged release already, is it OK to push this to the main branch and then we do another tagged RTL release?

Yes - that's probably the best way.

[calebofearth]

I currently have a sync in progress to address #284 - if we push this to dev-public, I'll grab it in dev-integrate and get it into main.

[tmichalak]

@calebofearth I see that #299 depends on this PR. To get green CI we need #302 to go in first. Could you take a look?

[calebofearth]

@calebofearth I see that #299 depends on this PR. To get green CI we need #302 to go in first. Could you take a look?

I see that PR's been merged since making this request, but it looks like a simple and good change.

[tmichalak]

I see that PR's been merged since making this request, but it looks like a simple and good change.

@calebofearth Okay, great. Rebased this PR on the latest dev-public.

[calebofearth]

Is this PR ready to go? @tmichalak or @mkurc-ant , are one of you planning to merge it?

[kgugala]

@calebofearth we're waiting for some review and ACK (or comments if there is sth to address). If you're OK with the code changes here we can merge it

[calebofearth]

@calebofearth we're waiting for some review and ACK (or comments if there is sth to address). If you're OK with the code changes here we can merge it

Ok, thanks. Looks good to me.