Skip to content

Code applicable to arm_CR4 is applied also for ARM_CM4 (address == 0)… #6

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vipako
Copy link

@vipako vipako commented Jun 1, 2021

… and optimization: eliminate write to reset_instr, when is not needed

By creating this pull request you agree to the terms in CONTRIBUTING.md.
https://github.com/cypresssemiconductorco/.github/blob/master/CONTRIBUTING.md
--- DO NOT DELETE ANYTHING ABOVE THIS LINE ---

CONTRIBUTING.md also tells you what to expect in the PR process.

Description
Provide the information we need to review your PR. What problem does the pull request solve? "Bug fix" is not a good description.

Related Issue
If you opened an issue before creating the PR, point to it here.

Context
What do we need to know about your development environment, tools, target, and so on. Screenshots are always helpful if there is a UI element to this PR.

… and optimization: eliminate write to reset_instr, when is not needed
@cy-chihsien
Copy link
Contributor

Hi @vipako ,

Thanks for the pull request. Some questions below. Could you please check?

  • Which chip/platform was this patch tested on?
  • From the code, it looks like we can avoid writing a reset instruction for ARM_CM3 in our supported chips (that has 0 address input argument). Could you please elaborate more about the CM4 reference -- "also for ARM_CM4 (address == 0)"?

@vipako
Copy link
Author

vipako commented Jun 2, 2021

Hi @cy-chihsien ,
I found this bug when I ported WHD for CYW4343W on STM32H743 platform and I tested it on this platform.

@cy-chihsien
Copy link
Contributor

@vipako ,
Could you please elaborate more about the issue you saw without this patch? Did you see any error print or symptom?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants