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: Register PWM device only if device file is not present #1886

Merged

Conversation

rzr
Copy link
Contributor

@rzr rzr commented May 28, 2019

If configured from menuconfig, device inode is already there
there is no need to register it again.

It was observed on STM32F7 using a config with those options:

CONFIG_STM32F7_TIM1=y
CONFIG_STM32F7_TIM1_PWM=y

Relate-to: rzr/webthing-iotjs#3
Forwarded: https://github.com/pando-project/iotjs/pull/
Change-Id: I6134f15ead43babe3e5de6a9f0d185f6629f62d2
IoT.js-DCO-1.0-Signed-off-by: Philippe Coval p.coval@samsung.com

If configured from menuconfig, device inode is already there
there is no need to register it again.

It was observed on STM32F7 using a config with those options:

```
CONFIG_STM32F7_TIM1=y
CONFIG_STM32F7_TIM1_PWM=y
```

Relate-to: rzr/webthing-iotjs#3
Forwarded: jerryscript-project#1886
Change-Id: I6134f15ead43babe3e5de6a9f0d185f6629f62d2
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 535c678 to c7d0301 Compare May 28, 2019 12:28
Copy link
Contributor

@hs0225 hs0225 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

@yichoi yichoi 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 603bea5 into jerryscript-project:master May 29, 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.

3 participants