Skip to content

prudebug: Add BBAI support #7

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

Merged
merged 1 commit into from
Jul 29, 2021
Merged

prudebug: Add BBAI support #7

merged 1 commit into from
Jul 29, 2021

Conversation

DhruvaG2000
Copy link
Contributor

@DhruvaG2000 DhruvaG2000 commented Jul 12, 2021

  • Add feature as requested in issue prudebug support for BBAI #6

  • Details on how I tested this code can be found on this blog: https://dhruvag2000.github.io/Blog-GSoC21/prudebug.html

    • This has been built and tested on BeagleBone AI running Linux beaglebone 4.19.94-ti-xenomai-r64 #1buster SMP PREEMPT Sat May 22 01:02:28 UTC 2021 armv7l GNU/Linux
    • It also contains recording of the terminal session where I show:
    • Building the asm blink LED program.
    • loading the .out file onto PRU1 core1.
    • using prudebug to actually step through each instruction live on the BBAI.

Files modified:

  • modified: prudbg.c
    Added the base addresses for PRU ICSS 1 and 2 by the names AM57x1 and 2

  • modified: prudbg.h
    Changed the default from AM335x to AM57x1

  • renamed: README -> README.md
    Also specify BBAI PRU options in the USAGE section

modified:   prudbg.c
Added the base addresses for PRU ICSS 1 and 2 by the names AM57x1 and 2

modified:   prudbg.h
Changed the default from AM335x to AM57x1

renamed:    README -> README.md
Also specify BBAI PRU options in the USAGE section
@@ -10,14 +10,15 @@
#define PRUDBG_H

// default processor to use if none is specified on the command line when prudebug is started
#define DEFAULT_PROCESSOR_INDEX AM335x
#define DEFAULT_PROCESSOR_INDEX AM57x1

Choose a reason for hiding this comment

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

why change the default processor index? Is BBAI more common than BBB and PocketBeagle?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

why change the default processor index? Is BBAI more common than BBB and PocketBeagle?

Okay, I will revert this.

@MarkAYoder MarkAYoder merged commit 00e3d67 into MarkAYoder:master Jul 29, 2021
@MarkAYoder
Copy link
Owner

Thanks for the port to the BBAI. I've merged it now and hope to test next time I have an BBAI running.

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.

3 participants