zephyrproject-rtos / openocd

OpenOCD with Zephyr patches for building Zephyr SDK
Other
28 stars 45 forks source link

rtos/zephyr: Remove Zephyr RTOS suport for ARC_SEC cores #53

Closed kokas-a closed 1 year ago

kokas-a commented 1 year ago

In issues https://github.com/zephyrproject-rtos/sdk-ng/issues/631 and https://github.com/zephyrproject-rtos/zephyr/issues/55686 root case is in an attempt to access arc registers at early stage of OpenOCD initialization process. At this stage transport is not configured and this cause fault with error "Assertion `jtag_trst == 0' failed".

As temporary solution this patch remove early access to ARC cores. This also disables Zephyr RTOS support for ARC_SEC cores.

kokas-a commented 1 year ago

@abrodkin, could you please take a look?

abrodkin commented 1 year ago

@kokas-a if that's a temporary solution, it would be good to create another issue where we'll track an enhancement for a better fix, otherwise we'll soon forget about that, I guess ;)

Also, it would be good to figure out why that code was added in the first place, maybe it's not really needed and removal of it is the right solution?

evgeniy-paltsev commented 1 year ago

Hi @kokas-a,

the workaround itself LGTM.

Please also create the issue in https://github.com/foss-for-synopsys-dwc-arc-processors/openocd so we'll fix it proper way later.