Skip to content

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