[GitHub] kasjer commented on issue #1101: HiFive1 add missing peripheral driver

kasjer commented on issue #1101: HiFive1 add missing peripheral driver URL: https://github.com/apache/mynewt-core/pull/1101#issuecomment-390331215 @mkiiskila Thanks for review, I addressed all you points. I also allowed to have rxbuf set to NULL. Before I checked STM32 code I saw that it

[GitHub] kasjer commented on a change in pull request #1101: HiFive1 add missing peripheral driver

kasjer commented on a change in pull request #1101: HiFive1 add missing peripheral driver URL: https://github.com/apache/mynewt-core/pull/1101#discussion_r189391132 ## File path: hw/mcu/sifive/fe310/src/hal_spi.c ## @@ -0,0 +1,384 @@ +/* + * Licensed to the Apache

[GitHub] kasjer commented on a change in pull request #1101: HiFive1 add missing peripheral driver

kasjer commented on a change in pull request #1101: HiFive1 add missing peripheral driver URL: https://github.com/apache/mynewt-core/pull/1101#discussion_r189391169 ## File path: hw/mcu/sifive/fe310/src/hal_spi.c ## @@ -0,0 +1,384 @@ +/* + * Licensed to the Apache

[GitHub] kasjer commented on a change in pull request #1101: HiFive1 add missing peripheral driver

kasjer commented on a change in pull request #1101: HiFive1 add missing peripheral driver URL: https://github.com/apache/mynewt-core/pull/1101#discussion_r189391159 ## File path: hw/mcu/sifive/fe310/src/hal_spi.c ## @@ -0,0 +1,384 @@ +/* + * Licensed to the Apache

[GitHub] kasjer commented on a change in pull request #1101: HiFive1 add missing peripheral driver

kasjer commented on a change in pull request #1101: HiFive1 add missing peripheral driver URL: https://github.com/apache/mynewt-core/pull/1101#discussion_r189391145 ## File path: hw/mcu/sifive/fe310/src/hal_spi.c ## @@ -0,0 +1,384 @@ +/* + * Licensed to the Apache

[GitHub] wes3 closed pull request #1105: hw/drivers: Add TLC5971 driver

wes3 closed pull request #1105: hw/drivers: Add TLC5971 driver URL: https://github.com/apache/mynewt-core/pull/1105 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

[mynewt-core] 01/01: Merge pull request #1105 from wes3/tlc5971drv

This is an automated email from the ASF dual-hosted git repository. wes3 pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git commit b61097f703e9d295f881cfa2d73175c53e320910 Merge: a7f4d8c c266d0f Author: wes3 AuthorDate: Fri May

[mynewt-core] branch master updated (a7f4d8c -> b61097f)

This is an automated email from the ASF dual-hosted git repository. wes3 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git. from a7f4d8c Merge pull request #1112 from mlaz/pwm_fixes add 5c01f15 hw/drivers: Add TLC5971 driver add

[GitHub] vrahane commented on a change in pull request #1105: hw/drivers: Add TLC5971 driver

vrahane commented on a change in pull request #1105: hw/drivers: Add TLC5971 driver URL: https://github.com/apache/mynewt-core/pull/1105#discussion_r189353905 ## File path: hw/drivers/tlc5971/src/tlc5971.c ## @@ -0,0 +1,270 @@ +/** + * Licensed to the Apache Software

[GitHub] benmccrea commented on issue #1108: Fix/nrf52 i2c hang

benmccrea commented on issue #1108: Fix/nrf52 i2c hang URL: https://github.com/apache/mynewt-core/pull/1108#issuecomment-390288381 @vrahane Fixed. This is an automated message from the Apache Git Service. To respond to the

[GitHub] 720adams opened a new pull request #424: Update port_os.md

720adams opened a new pull request #424: Update port_os.md URL: https://github.com/apache/mynewt-site/pull/424 Simple spelling Correction This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] mlaz closed pull request #1112: Adding pointer validation assertions to PWM API level.

mlaz closed pull request #1112: Adding pointer validation assertions to PWM API level. URL: https://github.com/apache/mynewt-core/pull/1112 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[mynewt-core] 01/01: Merge pull request #1112 from mlaz/pwm_fixes

This is an automated email from the ASF dual-hosted git repository. mlaz pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git commit a7f4d8c4fa9641b08b61377b1bcef8d6b7e0cb4c Merge: 14a5a37 532e630 Author: Miguel Azevedo

[mynewt-core] branch master updated (14a5a37 -> a7f4d8c)

This is an automated email from the ASF dual-hosted git repository. mlaz pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git. from 14a5a37 Senesor error format fix (#) add 532e630 Adding pointer validation assertions to PWM API

[GitHub] MariuszSkamra commented on issue #1113: apps/btshell: Add extend cmd_scan command line parameters

MariuszSkamra commented on issue #1113: apps/btshell: Add extend cmd_scan command line parameters URL: https://github.com/apache/mynewt-core/pull/1113#issuecomment-390245767 @sjanc @andrzej-kaczmarek This is an automated

[GitHub] MariuszSkamra opened a new pull request #1113: apps/btshell: Add extend cmd_scan command line parameters

MariuszSkamra opened a new pull request #1113: apps/btshell: Add extend cmd_scan command line parameters URL: https://github.com/apache/mynewt-core/pull/1113 This patch extends cmd_scan with two parameters, that limit the data printed out to the serial console during scanning. It has

[GitHub] vrahane closed pull request #1111: Senesor error format fix

vrahane closed pull request #: Senesor error format fix URL: https://github.com/apache/mynewt-core/pull/ This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull request

[mynewt-core] branch master updated: Senesor error format fix (#1111)

This is an automated email from the ASF dual-hosted git repository. vipulrahane pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git The following commit(s) were added to refs/heads/master by this push: new 14a5a37 Senesor error format fix

[GitHub] mlaz commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlaz commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r189289036 ## File path: hw/drivers/pwm/pwm_stm32/src/pwm_stm32.c ## @@ -0,0 +1,626 @@ +/* + * Licensed to the Apache Software

[GitHub] mlaz commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlaz commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r189289036 ## File path: hw/drivers/pwm/pwm_stm32/src/pwm_stm32.c ## @@ -0,0 +1,626 @@ +/* + * Licensed to the Apache Software

[GitHub] mlaz commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlaz commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r189288648 ## File path: hw/drivers/pwm/pwm_stm32/src/pwm_stm32.c ## @@ -0,0 +1,626 @@ +/* + * Licensed to the Apache Software

[GitHub] mlaz opened a new pull request #1112: Adding pointer validation assertions to PWM API level.

mlaz opened a new pull request #1112: Adding pointer validation assertions to PWM API level. URL: https://github.com/apache/mynewt-core/pull/1112 This is an automated message from the Apache Git Service. To respond to the

[GitHub] mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r189287008 ## File path: hw/drivers/pwm/pwm_stm32/src/pwm_stm32.c ## @@ -0,0 +1,626 @@ +/* + * Licensed to the Apache Software

[GitHub] alvarop commented on issue #1106: [WIP] NRF52810 Support

alvarop commented on issue #1106: [WIP] NRF52810 Support URL: https://github.com/apache/mynewt-core/pull/1106#issuecomment-390223096 @utzig I didn't address Cortex-M0's. Didn't know there was a problem! I should probably wait for your CMSIS 5 PR to go through then rebase mine onto it to

[GitHub] mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r189282177 ## File path: hw/drivers/pwm/pwm_stm32/src/pwm_stm32.c ## @@ -0,0 +1,626 @@ +/* + * Licensed to the Apache Software

[GitHub] mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r189281440 ## File path: hw/drivers/pwm/pwm_stm32/src/pwm_stm32.c ## @@ -0,0 +1,626 @@ +/* + * Licensed to the Apache Software

[GitHub] mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r189280739 ## File path: hw/drivers/pwm/pwm_stm32/src/pwm_stm32.c ## @@ -0,0 +1,626 @@ +/* + * Licensed to the Apache Software

[GitHub] mlaz commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlaz commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r188974104 ## File path: hw/drivers/pwm/pwm_stm32/src/pwm_stm32.c ## @@ -0,0 +1,626 @@ +/* + * Licensed to the Apache Software

[GitHub] mlaz commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlaz commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r189274243 ## File path: hw/drivers/pwm/pwm_stm32/src/pwm_stm32.c ## @@ -0,0 +1,626 @@ +/* + * Licensed to the Apache Software

[GitHub] mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r189266092 ## File path: hw/drivers/pwm/pwm_stm32/src/pwm_stm32.c ## @@ -0,0 +1,626 @@ +/* + * Licensed to the Apache Software

[GitHub] mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled

mlampert commented on a change in pull request #1102: Feature/stm32 pwm enabled URL: https://github.com/apache/mynewt-core/pull/1102#discussion_r189264996 ## File path: hw/bsp/nucleo-f303re/src/hal_bsp.c ## @@ -80,7 +80,50 @@ bsp_uart_config(int port) assert(port <

[GitHub] sjanc opened a new pull request #102: Documentation update

sjanc opened a new pull request #102: Documentation update URL: https://github.com/apache/mynewt-nimble/pull/102 This moves and updated documentation from -core repo to -nimble repo. Also some initial work on doxygen generation is done.

[GitHub] mkiiskila closed pull request #1110: hal; add convenience routine to return reason for reset as string.

mkiiskila closed pull request #1110: hal; add convenience routine to return reason for reset as string. URL: https://github.com/apache/mynewt-core/pull/1110 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[mynewt-core] branch master updated (f6d76bc -> d03f914)

This is an automated email from the ASF dual-hosted git repository. marko pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git. from f6d76bc Remove ADC sysdefs already present on nrf51xx MCU add 0f034d4 hal; add convenience routine to

[GitHub] mkiiskila commented on a change in pull request #1110: hal; add convenience routine to return reason for reset as string.

mkiiskila commented on a change in pull request #1110: hal; add convenience routine to return reason for reset as string. URL: https://github.com/apache/mynewt-core/pull/1110#discussion_r189236761 ## File path: hw/hal/src/hal_common.c ## @@ -57,3 +57,27 @@

[GitHub] kasjer opened a new pull request #1111: Senesor error format fix

kasjer opened a new pull request #: Senesor error format fix URL: https://github.com/apache/mynewt-core/pull/ Several sensor drivers has formating issues that would be detected if log_error() has __attribute((format..) added. First commit removes errors that can result in crash.

[GitHub] utzig commented on a change in pull request #1110: hal; add convenience routine to return reason for reset as string.

utzig commented on a change in pull request #1110: hal; add convenience routine to return reason for reset as string. URL: https://github.com/apache/mynewt-core/pull/1110#discussion_r189235474 ## File path: hw/hal/src/hal_common.c ## @@ -57,3 +57,27 @@

[GitHub] utzig commented on issue #1106: [WIP] NRF52810 Support

utzig commented on issue #1106: [WIP] NRF52810 Support URL: https://github.com/apache/mynewt-core/pull/1106#issuecomment-390173775 Btw, I also added a CMSIS 5 PR here: https://github.com/apache/mynewt-core/pull/1100 I don't see how or that you solved the issue building on any

[GitHub] utzig closed pull request #1098: Remove ADC sysdefs already present on nrf51xx MCU

utzig closed pull request #1098: Remove ADC sysdefs already present on nrf51xx MCU URL: https://github.com/apache/mynewt-core/pull/1098 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is

[mynewt-core] branch master updated: Remove ADC sysdefs already present on nrf51xx MCU

This is an automated email from the ASF dual-hosted git repository. utzig pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git The following commit(s) were added to refs/heads/master by this push: new f6d76bc Remove ADC sysdefs already present

[GitHub] utzig commented on issue #423: Update port_bsp.md

utzig commented on issue #423: Update port_bsp.md URL: https://github.com/apache/mynewt-site/pull/423#issuecomment-390159868 Since this is about porting to a BSP it makes more sense to use `debug`, so both should outputs should be `debug`.

[mynewt-nimble] branch master updated (d17120a -> 111d91a)

This is an automated email from the ASF dual-hosted git repository. naraj pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git. from d17120a Merge pull request #96 from turon/pr/64-bit add 84eed53 mesh: Move atomic.h to sources new

[mynewt-nimble] 01/01: Merge pull request #101 from michal-narajowski/mesh-atomic-cleanup

This is an automated email from the ASF dual-hosted git repository. naraj pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git commit 111d91a40774753fb552fa56399a6f704cd4648d Merge: d17120a 84eed53 Author: Michał Narajowski

[GitHub] michal-narajowski closed pull request #101: mesh: Move atomic.h to sources

michal-narajowski closed pull request #101: mesh: Move atomic.h to sources URL: https://github.com/apache/mynewt-nimble/pull/101 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[GitHub] michal-narajowski opened a new pull request #101: mesh: Move atomic.h to sources

michal-narajowski opened a new pull request #101: mesh: Move atomic.h to sources URL: https://github.com/apache/mynewt-nimble/pull/101 This header is not a part of Mesh API. This is an automated message from the Apache Git

[GitHub] mkiiskila commented on a change in pull request #1101: HiFive1 add missing peripheral driver

mkiiskila commented on a change in pull request #1101: HiFive1 add missing peripheral driver URL: https://github.com/apache/mynewt-core/pull/1101#discussion_r189193154 ## File path: hw/mcu/sifive/fe310/src/hal_spi.c ## @@ -0,0 +1,384 @@ +/* + * Licensed to the Apache

[GitHub] mkiiskila commented on a change in pull request #1101: HiFive1 add missing peripheral driver

mkiiskila commented on a change in pull request #1101: HiFive1 add missing peripheral driver URL: https://github.com/apache/mynewt-core/pull/1101#discussion_r189194857 ## File path: hw/mcu/sifive/fe310/src/hal_spi.c ## @@ -0,0 +1,384 @@ +/* + * Licensed to the Apache

[GitHub] mkiiskila commented on a change in pull request #1101: HiFive1 add missing peripheral driver

mkiiskila commented on a change in pull request #1101: HiFive1 add missing peripheral driver URL: https://github.com/apache/mynewt-core/pull/1101#discussion_r189194351 ## File path: hw/mcu/sifive/fe310/src/hal_spi.c ## @@ -0,0 +1,384 @@ +/* + * Licensed to the Apache

[mynewt-nimble] branch master updated (8a74223 -> d17120a)

This is an automated email from the ASF dual-hosted git repository. andk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git. from 8a74223 Merge pull request #100 from andrzej-kaczmarek/unaligned-fix add c5a0c7a porting: Add support

[mynewt-nimble] 01/01: Merge pull request #96 from turon/pr/64-bit

This is an automated email from the ASF dual-hosted git repository. andk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git commit d17120a2a1c335b41617c6acc2459370eedecf5e Merge: 8a74223 c5a0c7a Author: Andrzej Kaczmarek

[GitHub] andrzej-kaczmarek closed pull request #96: porting: Add support for running on 64-bit architectures

andrzej-kaczmarek closed pull request #96: porting: Add support for running on 64-bit architectures URL: https://github.com/apache/mynewt-nimble/pull/96 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of

[mynewt-nimble] 01/01: Merge pull request #100 from andrzej-kaczmarek/unaligned-fix

This is an automated email from the ASF dual-hosted git repository. andk pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git commit 8a74223386ff8636710f9bbd7935b4e04da44c98 Merge: 1490ed4 17cf119 Author: Andrzej Kaczmarek

[GitHub] andrzej-kaczmarek closed pull request #100: nimble/ll: Use simply memcpy for copying in resolv

andrzej-kaczmarek closed pull request #100: nimble/ll: Use simply memcpy for copying in resolv URL: https://github.com/apache/mynewt-nimble/pull/100 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[mynewt-nimble] branch master updated (1490ed4 -> 8a74223)

This is an automated email from the ASF dual-hosted git repository. andk pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git. from 1490ed4 Merge pull request #99 from sjanc/readme add 17cf119 nimble/ll: Use simply memcpy for copying in

[GitHub] andrzej-kaczmarek commented on issue #100: nimble/ll: Use simply memcpy for copying in resolv

andrzej-kaczmarek commented on issue #100: nimble/ll: Use simply memcpy for copying in resolv URL: https://github.com/apache/mynewt-nimble/pull/100#issuecomment-390130313 @wes3 I also thought this optimization was for a reason, but in the current version we use this only for handling HCI

[GitHub] mkiiskila opened a new pull request #1110: hal; add convenience routine to return reason for reset as string.

mkiiskila opened a new pull request #1110: hal; add convenience routine to return reason for reset as string. URL: https://github.com/apache/mynewt-core/pull/1110 This is an automated message from the Apache Git Service. To

[mynewt-core] branch master updated (9dcf9c8 -> 71e46dd)

This is an automated email from the ASF dual-hosted git repository. naraj pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git. from 9dcf9c8 Merge pull request #1103 from kasjer/lis2dw12-fix add 3e27c7b hw/mcu: Fix typo in nrf52 config

[GitHub] sjanc commented on issue #1096: nimble dependency error

sjanc commented on issue #1096: nimble dependency error URL: https://github.com/apache/mynewt-core/issues/1096#issuecomment-390128906 good catch! This is now fixed on master https://github.com/apache/mynewt-core/pull/1109

[GitHub] michal-narajowski closed pull request #1109: hw/mcu: Fix typo in nrf52 config

michal-narajowski closed pull request #1109: hw/mcu: Fix typo in nrf52 config URL: https://github.com/apache/mynewt-core/pull/1109 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a

[mynewt-core] 01/01: Merge pull request #1109 from michal-narajowski/nrfx52-config

This is an automated email from the ASF dual-hosted git repository. naraj pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git commit 71e46dda999ba264711bf6f5bddc6faef112f599 Merge: 9dcf9c8 3e27c7b Author: Michał Narajowski

[GitHub] michal-narajowski opened a new pull request #1109: hw/mcu: Fix typo in nrf52 config

michal-narajowski opened a new pull request #1109: hw/mcu: Fix typo in nrf52 config URL: https://github.com/apache/mynewt-core/pull/1109 This is an automated message from the Apache Git Service. To respond to the message,