Skip to content

Conversation

@dragonmux
Copy link
Member

Detailed description

In this PR we introduce support for TI's ICEPick Type C which is found on the AM335x parts in front of the ADIv5 TAPs on the JTAG chain, allowing BMD to set the scan chain up to actually see any of the subsequent TAPs and enumerate them. This forms the basis too for the ICEPick Type D found on the TMS570, albeit with some differences.

NB: when a scan chain features an ICEPick on it, the debugger must not touch nRST ever, as doing so blows the ICEPick config away 🙃; similarly, if the scan chain features more than one component with one of these and they are anything other than the last item on the scan chan, things will also break and not work right. Scan is required to be restarted after (re-)configuring one of these and so messing with the scan chain geometry etc. This has deleterious effects in the JTAG discovery code if not right at the end.

Please also note that it is expected any future PRs that make use of this will rewrite this section of the Meson build file to properly specify the dependency. An example can be found as an experimental commit (permalinked here):

target_ti_cortexa = declare_dependency(
sources: files(
'am335x.c',
),
compile_args: ['-DCONFIG_TI_SITARA=1'],
dependencies: [target_cortexar, target_ti_icepick],
)
# If the Cortex-A/R target is not enabled, don't enable TI AM335x part support
if enabled_targets.contains('cortexar')
target_ti = [
target_ti_cortexm,
target_ti_cortexa,
]
else
target_ti = target_ti_cortexm
endif

Your checklist for this pull request

Closing issues

@dragonmux dragonmux added this to the v2.1 release milestone Nov 4, 2025
@dragonmux dragonmux requested a review from esden November 4, 2025 16:47
@dragonmux dragonmux added Enhancement General project improvement New Target New debug target labels Nov 4, 2025
Copy link
Member

@esden esden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@esden esden merged commit e313e59 into main Nov 4, 2025
43 checks passed
@dragonmux dragonmux deleted the feature/ti-icepick-support branch November 4, 2025 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement General project improvement New Target New debug target

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants