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

Improving platform independence #2

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

Conversation

pbuteau
Copy link

@pbuteau pbuteau commented Dec 10, 2021

No description provided.

@jxwleong
Copy link
Owner

Hi Patrice,
Do you mind add in some description on the PR? 😊

Thanks,
Jason

@pbuteau
Copy link
Author

pbuteau commented Jan 3, 2022

4 small changes:

  1. DISVAL: DISVAL can be found in the BSDL file. It varies depending on the device and corresponds to the values used in the BSCPinMode enum.
  2. bSCInIt: bSCInIt is now initializing the whole bSCellPreloadData and bSCellSampleData arrays. This has no impact if the number of boundary scan cells is a factor of 8 (like in this case), but will have an impact otherwise.
  3. Getting rid (or not) of the bypass bit for the CORTEX M3 TAP using a for loop: This makes the code work in the case where you only have 1 TAP, while still supporting the stm32 case where there are 2 TAPS. Important note: This won't work in every other cases, as the code would require a certain prior knowledge of your jtag chain.
  4. Removed the CORTEX_M3_BOUNDARY_SCAN_CELL_LENGTH define duplicate in BoundaryScan.h

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.

None yet

2 participants