lowRISC / opentitan

OpenTitan: Open source silicon root of trust
https://www.opentitan.org
Apache License 2.0
2.5k stars 745 forks source link

[chip dv] Deprecate jtag_riscv_agent #15486

Open sriyerg opened 1 year ago

sriyerg commented 1 year ago

The jtag_dmi_agent supercedes the jtag_riscv_agent work and provides a more ergonomic way to access the chip over JTAG. As a step towards tech debt reduction, we should replace it with jtag_dmi_agent / sba_access_utils_pkg methods at the chip level soon.

The replacement should be hopefully straightforward, and should take about a day to complete.

@cindychip @tjaychen

msfschaffner commented 1 year ago

@sriyerg do you want to do this deprecation as part of M2, or can this wait until later?

msfschaffner commented 1 year ago

I am moving this into M3 for now - please relabel if that is not accurate.

tjaychen commented 1 year ago

is it fair to say that deprecation of the existing agent (in favor of the new more flexible / abstracted one) is a clean-up activity best left to backlog?

sriyerg commented 1 year ago

I will prefer keeping this as a low priority M3 item. Would it be ok to reevaluate this later when we get close to M3?

tjaychen commented 1 year ago

yes sounds good.