Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: Cortex-A/R speedups #1752

Merged
merged 14 commits into from
Feb 2, 2024
Merged

Feature: Cortex-A/R speedups #1752

merged 14 commits into from
Feb 2, 2024

Conversation

dragonmux
Copy link
Member

@dragonmux dragonmux commented Jan 29, 2024

Detailed description

This PR addresses some of the speed concern with Cortex-A/R targets by speeding up (at least for the firmware) accesses that involve instruction launches using the AP banked register interface to improve throughput by reducing ADIv5 protocol overhead for the DBG_ITR, DBG_DTR{T,R}X and DBG_DCSR core debug interface registers.

Tested against the Renesas RZ/A1LU, this delivers at least a 5x speed up in the firmware and 3.75x speed up in BMDA.

Your checklist for this pull request

Closing issues

@dragonmux dragonmux added the Enhancement General project improvement label Jan 29, 2024
@dragonmux dragonmux added this to the v2.0 release milestone Jan 29, 2024
@dragonmux dragonmux requested a review from esden January 29, 2024 05:01
@dragonmux dragonmux force-pushed the feature/cortex-ar-speedups branch 3 times, most recently from 7fb5417 to 42581ea Compare February 2, 2024 00:04
@dragonmux dragonmux force-pushed the feature/cortex-ar-speedups branch from 42581ea to 016b67d Compare February 2, 2024 01:16
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 016b67d into main Feb 2, 2024
24 checks passed
@dragonmux dragonmux deleted the feature/cortex-ar-speedups branch February 2, 2024 01:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement General project improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants