8000 stm32/nucleo_wb55: Fix mpremote file transfer over uart repl. by andrewleech · Pull Request #8713 · micropython/micropython · GitHub
[go: up one dir, main page]

Skip to content

stm32/nucleo_wb55: Fix mpremote file transfer over uart repl. #8713

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

Closed

Conversation

andrewleech
Copy link
Contributor

Runs some isr and flash erase functions from ram so the uart rx is not blocked by background flash operations.

This is intended to demonstrate an alternate solution to fixing #8386 on a per board/project basis.

Runs some isr and flash erase functions from ram.

// Configure ram isr / functions for board.
extern void ram_isr_fn_init();
#define MICROPY_BOARD_STARTUP ram_isr_fn_init
Copy link
Member

Choose a reason for hiding this comment

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

This should probably only be configured if USE_RAM_ISR_UART_FLASH_FN is enabled.

Also, maybe call it board_startup. And put void in the arg list.

#include <py/mphal.h>

// This is the default MICROPY_BOARD_STARTUP on stm32
extern void powerctrl_check_enter_bootloader();
Copy link
Member

Choose a reason for hiding this comment

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

You should be able to include powerctrl.h to get this declaration.

@dpgeorge
Copy link
Member
dpgeorge commented Aug 8, 2022

TODO: this needs to be generalised to work on any stm32 board that needs it.

@andrewleech
Copy link
Contributor Author

This has been replaced / generalised in #9054

@andrewleech andrewleech closed this Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0