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

ADD SSCI TY51822r3 target #50

Closed
wants to merge 1 commit into from
Closed

ADD SSCI TY51822r3 target #50

wants to merge 1 commit into from

Conversation

toyowata
Copy link
Contributor

  • Add Switch Science mbed TY51822r3 target
  • LPC11U35 interface refactor
    • Change SW_RESET_BUTTON port (it should also work with SWDAP since BL and ISP signal is shorted)
    • Remove unused functions and variables
    • gpio_get_sw_reset() now uses Port In register to read signal level which wrongly used W register

* Add Switch Science mbed TY51822r3 target
* LPC11U35 interface refactor
- Change SW_RESET_BUTTON port (it should work with DIPDAP since BL and
ISP signal is shorted)
- Remove unused functions and variables
- gpio_get_sw_reset() now uses Port In register to read signal level
which wrongly used W register.
#define RESET_PORT (0)
#define RESET_PIN (1)
#define RESET_PORT (1)
#define RESET_PIN (19)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did the reset pin change to 19?

@toyowata
Copy link
Contributor Author

Why did the reset pin change to 19?

Because mbed HDK reference schematics used P1_19 for bootloader reset and most of platforms based on mbed-HDK, used P1_19 for reset button signal AFAIK.
https://developer.mbed.org/teams/mbed/code/mbed-HDK/

The SWDAP should also work with P1_19 since P1_19 and P0_1 are shorted.

@sg-
Copy link
Contributor

sg- commented Feb 19, 2016

This target was added with #57 and will be merged soon. Will track the reset button as a separate issue.

@sg- sg- closed this Feb 19, 2016
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

3 participants