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 Nucleo F446ZE port to FreeRTOS_apps #11

Merged
merged 14 commits into from
Jul 14, 2020

Conversation

alsaibie
Copy link
Contributor

A port similar to the Olimex FreeRTOS example but for the Nucleo F446 board.
The project structure is the same, except:

  • the FreeRTOS-POSIX plugin is moved to a lib folder.
  • the serial_transport.h/.c files are moved in a separate folder to highlight that their excluded from the firmware build and are only copied over to XCRE
    Recommended change: move the arm toolchain installation outside of the firmware folder, or have it as a prequisite since it is used by most boards.

@ralph-lange
Copy link
Contributor

Very nice contribution. Thank you very much, @alsaibie. This will take a bit longer to review. I'll ask @pablogs9 for that.

@alsaibie
Copy link
Contributor Author

You're most welcome. I have a nucleo F3 and L4, would you care to have them ported as well? Or having a porting guide under README.md would do, since other Nucleo board setups are very much the same?

Will wait for the review first.

@pablogs9
Copy link
Member

pablogs9 commented Jul 14, 2020

@ralph-lange It looks good to me, but we don't have any Nucleo F446ZE here for test... do you have any? If not I propose to merge this PR since it only modifies a F446ZE-related folder.

Once merged, we have to check this other PR: micro-ROS/micro_ros_setup#139

@ralph-lange
Copy link
Contributor

ralph-lange commented Jul 14, 2020

@ralph-lange It looks good to me, but we don't have any Nucleo F446ZE here for test... do you have any? If not I propose to merge this PR since it only modifies a F446ZE-related folder.

I have none either, but I agree to merge.

@pablogs9 pablogs9 merged commit 3442582 into micro-ROS:dashing Jul 14, 2020
pablogs9 added a commit that referenced this pull request Jul 14, 2020
* added nucleo_extension base files WIP

* fixed list of objects in build

* Ping Pong Example Compiles

* Debugging Support, WIP, connecting but not ping ponging

* enabled NVIC on UART3 and added cortex-debug launch file

* Working up to  line 417 in node.c, deallocate(node_secure_root,...)

* added check for null ptr on deallocate

* allocate instead of reallocate if ptr is null, temporary fix?

* remove debug prints

* gitignore update

* cleanup unused wip calls

* moved FreeRTOS-POSIX to lib dir and transport layer to uros_transport dir

* added quick readme to nucleo_f446ze setup

* clean gitignore and vscode debug configuration
Fix STM32 cube MX project
pablogs9 added a commit that referenced this pull request Jul 14, 2020
* added nucleo_extension base files WIP

* fixed list of objects in build

* Ping Pong Example Compiles

* Debugging Support, WIP, connecting but not ping ponging

* enabled NVIC on UART3 and added cortex-debug launch file

* Working up to  line 417 in node.c, deallocate(node_secure_root,...)

* added check for null ptr on deallocate

* allocate instead of reallocate if ptr is null, temporary fix?

* remove debug prints

* gitignore update

* cleanup unused wip calls

* moved FreeRTOS-POSIX to lib dir and transport layer to uros_transport dir

* added quick readme to nucleo_f446ze setup

* clean gitignore and vscode debug configuration
Fix STM32 cube MX project


Fix ping_pong app
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