[mynewt-core] branch master updated: apps/ble: Use store/config instead of store/ram

2020-12-11 Thread jerzy
This is an automated email from the ASF dual-hosted git repository. jerzy 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 0f29735 apps/ble: Use store/config instead

[GitHub] [mynewt-core] kasjer merged pull request #2428: apps/ble: Use store/config instead of store/ram

2020-12-11 Thread GitBox
kasjer merged pull request #2428: URL: https://github.com/apache/mynewt-core/pull/2428 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[mynewt-newtmgr] branch master updated: taskstat: Add tab character after the task's name

2020-12-11 Thread vipulrahane
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-newtmgr.git The following commit(s) were added to refs/heads/master by this push: new 59848a5 taskstat: Add tab characte

[mynewt-nimble] branch master updated: ble_store_config: Remove unneeded include

2020-12-11 Thread jerzy
This is an automated email from the ASF dual-hosted git repository. jerzy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-nimble.git The following commit(s) were added to refs/heads/master by this push: new 542806a ble_store_config: Remove unneeded

[GitHub] [mynewt-nimble] kasjer merged pull request #889: ble_store_config: Remove unneeded include

2020-12-11 Thread GitBox
kasjer merged pull request #889: URL: https://github.com/apache/mynewt-nimble/pull/889 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

[GitHub] [mynewt-newtmgr] vrahane merged pull request #179: taskstat: Add tab character after the task's name

2020-12-11 Thread GitBox
vrahane merged pull request #179: URL: https://github.com/apache/mynewt-newtmgr/pull/179 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to g

[GitHub] [mynewt-core] kasjer commented on pull request #2428: apps/ble: Use store/config instead of store/ram

2020-12-11 Thread GitBox
kasjer commented on pull request #2428: URL: https://github.com/apache/mynewt-core/pull/2428#issuecomment-743420209 This https://github.com/apache/mynewt-nimble/pull/889 needs to be merged first to make CI work This is an au

[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2435: hw/drivers/uart: Add uart_open function

2020-12-11 Thread GitBox
apache-mynewt-bot commented on pull request #2435: URL: https://github.com/apache/mynewt-core/pull/2435#issuecomment-743278058 ## Style check summary No suggestions at this time! This is an automat

[GitHub] [mynewt-core] apache-mynewt-bot commented on pull request #2434: hw/mcu/da1469x: Allow to release GPIO pin in ISR

2020-12-11 Thread GitBox
apache-mynewt-bot commented on pull request #2434: URL: https://github.com/apache/mynewt-core/pull/2434#issuecomment-743277983 ## Style check summary No suggestions at this time! This is an automat

[GitHub] [mynewt-core] kasjer opened a new pull request #2435: hw/drivers/uart: Add uart_open function

2020-12-11 Thread GitBox
kasjer opened a new pull request #2435: URL: https://github.com/apache/mynewt-core/pull/2435 This adds thin wrapper to open UART device. Generic os_dev_open has opaque pointer that in case of UART has configuration but it's not clear what should be passed as argument without looking

[GitHub] [mynewt-core] kasjer opened a new pull request #2434: hw/mcu/da1469x: Allow to release GPIO pin in ISR

2020-12-11 Thread GitBox
kasjer opened a new pull request #2434: URL: https://github.com/apache/mynewt-core/pull/2434 When GPIO pin is configured as input with interrupt it seems reasonable to be able to release this interrupt during GPIO interrupt handler without need to go through threaded code. To ac

[GitHub] [mynewt-nimble] v01d commented on a change in pull request #878: Add support for NuttX

2020-12-11 Thread GitBox
v01d commented on a change in pull request #878: URL: https://github.com/apache/mynewt-nimble/pull/878#discussion_r541045887 ## File path: porting/nimble/include/log/log.h ## @@ -30,11 +30,13 @@ log_dummy(void *log, ...) (void)log; } +#if 0 Review comment: Any s

[GitHub] [mynewt-mcumgr] homburg edited a comment on issue #103: macos: `mcumgr` process killed

2020-12-11 Thread GitBox
homburg edited a comment on issue #103: URL: https://github.com/apache/mynewt-mcumgr/issues/103#issuecomment-743159826 Might be this go issue: https://github.com/golang/go/issues/19734#issuecomment-289822644 ~Testing~ Tried strategies from golang#19734 without success:

[GitHub] [mynewt-mcumgr] homburg commented on issue #103: macos: `mcumgr` process killed

2020-12-11 Thread GitBox
homburg commented on issue #103: URL: https://github.com/apache/mynewt-mcumgr/issues/103#issuecomment-743159826 Might be this go issue: https://github.com/golang/go/issues/19734#issuecomment-289822644 Testing This is

[GitHub] [mynewt-mcumgr] homburg opened a new issue #103: macos: `mcumgr` process killed

2020-12-11 Thread GitBox
homburg opened a new issue #103: URL: https://github.com/apache/mynewt-mcumgr/issues/103 I am trying to connect with `mcumgr`, but the process seems to be killed, .eg ```zsh ❯ ./mcumgr --loglevel debug --conntype ble --connstring peer_name=something echo conn_test DEBU[2020-12-

[GitHub] [mynewt-core] caspermeijn commented on pull request #2433: remove OS_TICKS_PER_SEC from syscfg when not needed

2020-12-11 Thread GitBox
caspermeijn commented on pull request #2433: URL: https://github.com/apache/mynewt-core/pull/2433#issuecomment-743115910 LGTM I like that is improves usability in the process of fixing a build failure. The `defunct` option produces a clear message when upgrading. --

[GitHub] [mynewt-newtmgr] vrahane opened a new pull request #179: taskstat: Add tab character after the task's name

2020-12-11 Thread GitBox
vrahane opened a new pull request #179: URL: https://github.com/apache/mynewt-newtmgr/pull/179 - There was a space earlier and if the task name was bigger, it would cause formatting issues *BEFORE* ``` Vipuls-MacBook-Pro-16-inch-2019:fw-signal vipul$ mcumgr --conntype ble

[GitHub] [mynewt-nimble] KKopyscinski commented on a change in pull request #867: ble_store_config: building error if BLE_STORE_CONFIG_PERSIST set to 0

2020-12-11 Thread GitBox
KKopyscinski commented on a change in pull request #867: URL: https://github.com/apache/mynewt-nimble/pull/867#discussion_r540771737 ## File path: nimble/host/store/config/src/ble_store_config.c ## @@ -23,7 +23,9 @@ #include "sysinit/sysinit.h" #include "syscfg/syscfg.h" #in

[GitHub] [mynewt-nimble] KKopyscinski closed pull request #867: ble_store_config: building error if BLE_STORE_CONFIG_PERSIST set to 0

2020-12-11 Thread GitBox
KKopyscinski closed pull request #867: URL: https://github.com/apache/mynewt-nimble/pull/867 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above

[GitHub] [mynewt-nimble] kasjer commented on a change in pull request #867: ble_store_config: building error if BLE_STORE_CONFIG_PERSIST set to 0

2020-12-11 Thread GitBox
kasjer commented on a change in pull request #867: URL: https://github.com/apache/mynewt-nimble/pull/867#discussion_r540759244 ## File path: nimble/host/store/config/src/ble_store_config.c ## @@ -23,7 +23,9 @@ #include "sysinit/sysinit.h" #include "syscfg/syscfg.h" #include