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

nuttx: Add Minimal support for stm32f7nucleo #1869

Merged

Conversation

rzr
Copy link
Contributor

@rzr rzr commented Apr 26, 2019

So far, Only console, http module has been tested.

Config file is generated using defconfig,
and contains those mandatory options:

CONFIG_NET_LOCAL=y
CONFIG_NET_TCPBACKLOG=y
CONFIG_PTHREAD_MUTEX_TYPES=y
CONFIG_RR_INTERVAL=100
CONFIG_SCHED_LPWORKPRIORITY=176

Build flags are relying on mcpu flags.

More IO modules to be supported next.

Relate-to: rzr/webthing-iotjs#3
Change-Id: Ib44227e33c6b7d07f11687385d788017350e3827
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com

@rzr rzr force-pushed the sandbox/rzr/review/master branch from 9819bf3 to 6e5405d Compare April 26, 2019 11:45
@LaszloLango
Copy link
Contributor

Please add a documentation like docs/build/Build-for-STM32F4-NuttX.md

@rzr rzr force-pushed the sandbox/rzr/review/master branch 2 times, most recently from 46ca200 to 93432eb Compare April 29, 2019 11:49
So far, Only console, http module has been tested.

Config file is generated using defconfig,
and contains those mandatory options:
```
CONFIG_NET_LOCAL=y
CONFIG_NET_TCPBACKLOG=y
CONFIG_PTHREAD_MUTEX_TYPES=y
CONFIG_RR_INTERVAL=100
CONFIG_SCHED_LPWORKPRIORITY=176
```

Build flags are relying on mcpu flags.

More IO modules to be supported next.

As support is "experimental", documentation is as an annex of SMT32F4 page.

Note for later, once support has been validated for all modules
it would make sense to move contents to specific file for STM32F7 file.

Relate-to: rzr/webthing-iotjs#3
Change-Id: Ib44227e33c6b7d07f11687385d788017350e3827
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com
@rzr rzr force-pushed the sandbox/rzr/review/master branch from 93432eb to 0660463 Compare April 29, 2019 12:29
Copy link
Contributor

@LaszloLango LaszloLango left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@haesik haesik left a comment

Choose a reason for hiding this comment

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

LGTM

@yichoi yichoi merged commit 0c8eb7d into jerryscript-project:master May 2, 2019
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.

4 participants