[mynewt-core] branch master updated: Add SPI to lps33hw driver (#991)

2018-04-04 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-core.git The following commit(s) were added to refs/heads/master by this push: new b91942a Add SPI to lps33hw driver

[GitHub] brianpax opened a new pull request #993: lps33hw: Added sensor set config function to LPS33HW driver

2018-04-04 Thread GitBox
brianpax opened a new pull request #993: lps33hw: Added sensor set config function to LPS33HW driver URL: https://github.com/apache/mynewt-core/pull/993 This is an automated message from the Apache Git Service. To respond

[GitHub] vrahane commented on issue #940: nrf52xxx_i2c: Fix to allow hal_i2c_clear_bus to read from input-buffer

2018-04-04 Thread GitBox
vrahane commented on issue #940: nrf52xxx_i2c: Fix to allow hal_i2c_clear_bus to read from input-buffer URL: https://github.com/apache/mynewt-core/pull/940#issuecomment-378788330 Sounds good to me. This is an automated

[GitHub] brianpax commented on issue #993: lps33hw: Added sensor set config function to LPS33HW driver

2018-04-04 Thread GitBox
brianpax commented on issue #993: lps33hw: Added sensor set config function to LPS33HW driver URL: https://github.com/apache/mynewt-core/pull/993#issuecomment-378788173 set_config function added in a different PR This is an

[GitHub] brianpax closed pull request #993: lps33hw: Added sensor set config function to LPS33HW driver

2018-04-04 Thread GitBox
brianpax closed pull request #993: lps33hw: Added sensor set config function to LPS33HW driver URL: https://github.com/apache/mynewt-core/pull/993 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:

[GitHub] vrahane commented on issue #993: lps33hw: Added sensor set config function to LPS33HW driver

2018-04-04 Thread GitBox
vrahane commented on issue #993: lps33hw: Added sensor set config function to LPS33HW driver URL: https://github.com/apache/mynewt-core/pull/993#issuecomment-378788239 thanks @brianpax This is an automated message from the

[GitHub] joshgrob commented on issue #699: Sensors/BMA2XX - New PR based on PR #697 changes

2018-04-04 Thread GitBox
joshgrob commented on issue #699: Sensors/BMA2XX - New PR based on PR #697 changes URL: https://github.com/apache/mynewt-core/pull/699#issuecomment-378793911 Sorry, I know the delays here are lame. I am traveling right now, and have it top of mind to test everything one last time before

[GitHub] ncasaril commented on issue #940: nrf52xxx_i2c: Fix to allow hal_i2c_clear_bus to read from input-buffer

2018-04-04 Thread GitBox
ncasaril commented on issue #940: nrf52xxx_i2c: Fix to allow hal_i2c_clear_bus to read from input-buffer URL: https://github.com/apache/mynewt-core/pull/940#issuecomment-378773365 Ok, I'll change this PR to address both the clearing of the bus as well as providing support for i2c for

[GitHub] bgiori opened a new pull request #992: Fix Description header

2018-04-04 Thread GitBox
bgiori opened a new pull request #992: Fix Description header URL: https://github.com/apache/mynewt-core/pull/992 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and

[mynewt-core] branch master updated: Fix Description header

2018-04-04 Thread utzig
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 7e1d1e5 Fix Description header 7e1d1e5 is

[GitHub] utzig closed pull request #992: Fix Description header

2018-04-04 Thread GitBox
utzig closed pull request #992: Fix Description header URL: https://github.com/apache/mynewt-core/pull/992 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 (from a

[GitHub] ncasaril commented on issue #940: nrf52xxx_i2c: Fix to allow hal_i2c_clear_bus to read from input-buffer

2018-04-04 Thread GitBox
ncasaril commented on issue #940: nrf52xxx_i2c: Fix to allow hal_i2c_clear_bus to read from input-buffer URL: https://github.com/apache/mynewt-core/pull/940#issuecomment-378808449 I've changed so the ports are taken into account. Let me know if you think I should move the

[GitHub] vrahane closed pull request #991: Add SPI to lps33hw driver

2018-04-04 Thread GitBox
vrahane closed pull request #991: Add SPI to lps33hw driver URL: https://github.com/apache/mynewt-core/pull/991 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

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179068061 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179079036 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179079462 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179075840 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179067661 ## File path: nimble/host/services/cts/include/services/cts/ble_svc_cts.h ## @@ -0,0 +1,49 @@ +/** + *

[GitHub] rymanluk opened a new pull request #27: nimble/ll: Fix for not mixing BT5.0 and legacy HCI commands

2018-04-04 Thread GitBox
rymanluk opened a new pull request #27: nimble/ll: Fix for not mixing BT5.0 and legacy HCI commands URL: https://github.com/apache/mynewt-nimble/pull/27 With this patch Controller makes sure that Host does not mix BT5 and legacy HCI commands

[GitHub] utzig closed pull request #976: Fix console echo in minimal console

2018-04-04 Thread GitBox
utzig closed pull request #976: Fix console echo in minimal console URL: https://github.com/apache/mynewt-core/pull/976 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] branch master updated: Fix console echo in minimal console

2018-04-04 Thread utzig
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 b11615e Fix console echo in minimal

[GitHub] utzig commented on issue #976: Fix console echo in minimal console

2018-04-04 Thread GitBox
utzig commented on issue #976: Fix console echo in minimal console URL: https://github.com/apache/mynewt-core/pull/976#issuecomment-378563776 Thanks, updated as suggested! This is an automated message from the Apache Git

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179075158 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179067891 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179076006 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179072847 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179071195 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the

[GitHub] andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179076298 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the

[GitHub] sdalu commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
sdalu commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179104518 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the Apache

[GitHub] sdalu commented on a change in pull request #14: ble service: Current Time Service

2018-04-04 Thread GitBox
sdalu commented on a change in pull request #14: ble service: Current Time Service URL: https://github.com/apache/mynewt-nimble/pull/14#discussion_r179106988 ## File path: nimble/host/services/cts/src/ble_svc_cts.c ## @@ -0,0 +1,544 @@ +/** + * Licensed to the Apache

[GitHub] sterlinghughes opened a new pull request #994: Add configuration documentation

2018-04-04 Thread GitBox
sterlinghughes opened a new pull request #994: Add configuration documentation URL: https://github.com/apache/mynewt-core/pull/994 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] andrzej-kaczmarek commented on issue #21: nimble/ll: Fix scanning handle for directed advertising

2018-04-04 Thread GitBox
andrzej-kaczmarek commented on issue #21: nimble/ll: Fix scanning handle for directed advertising URL: https://github.com/apache/mynewt-nimble/pull/21#issuecomment-378578957 ok, so we can assume that if we have TargetA which is an RPA at this point, this patch will guarantee that it will

[mynewt-core] branch master updated (b11615e -> e0b7fc6)

2018-04-04 Thread sterling
This is an automated email from the ASF dual-hosted git repository. sterling pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git. from b11615e Fix console echo in minimal console add 50fb2bf util/cbmem: Add interface to append data from

[GitHub] sterlinghughes closed pull request #933: sys/log: Better handling for long/non-text entries

2018-04-04 Thread GitBox
sterlinghughes closed pull request #933: sys/log: Better handling for long/non-text entries URL: https://github.com/apache/mynewt-core/pull/933 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

[GitHub] InternetofAwesome commented on issue #873: [WIP] sys/metrics: Add framework for logging events data

2018-04-04 Thread GitBox
InternetofAwesome commented on issue #873: [WIP] sys/metrics: Add framework for logging events data URL: https://github.com/apache/mynewt-core/pull/873#issuecomment-378667964 Is there anything blocking getting this in? I am waiting on this PR to get some application level logging merged

[mynewt-core] 01/01: Merge pull request #933 from andrzej-kaczmarek/log-upgrade

2018-04-04 Thread sterling
This is an automated email from the ASF dual-hosted git repository. sterling pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git commit e0b7fc6dbc8a5c08598fcd6292d61a25946a057d Merge: b11615e a72e8c5 Author: Sterling Hughes

[GitHub] vrahane commented on issue #940: nrf52xxx_i2c: Fix to allow hal_i2c_clear_bus to read from input-buffer

2018-04-04 Thread GitBox
vrahane commented on issue #940: nrf52xxx_i2c: Fix to allow hal_i2c_clear_bus to read from input-buffer URL: https://github.com/apache/mynewt-core/pull/940#issuecomment-378732631 @ncasaril Yes, you are right there: > I think I understand that it would need to be resolved to a

[GitHub] Jamirsen opened a new pull request #991: Add SPI to lps33hw driver

2018-04-04 Thread GitBox
Jamirsen opened a new pull request #991: Add SPI to lps33hw driver URL: https://github.com/apache/mynewt-core/pull/991 The previous version fo the lps33hw did not include support for SPI interface. In this PR we add this support. Testing/Verification - via sensor CLI read sensor

[GitHub] sjanc opened a new pull request #26: nimble/ll: Disallow to set address when initiating

2018-04-04 Thread GitBox
sjanc opened a new pull request #26: nimble/ll: Disallow to set address when initiating URL: https://github.com/apache/mynewt-nimble/pull/26 When initiating connection LE Set Random Address command should return Command Disallowed error. This was affecting LL/CON/INI/BV-01-C

[GitHub] sjanc commented on a change in pull request #4: [linux] initial port and osal additions.

2018-04-04 Thread GitBox
sjanc commented on a change in pull request #4: [linux] initial port and osal additions. URL: https://github.com/apache/mynewt-nimble/pull/4#discussion_r167164810 ## File path: porting/linux/os/os_mutex.c ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache Software

[GitHub] sjanc commented on a change in pull request #4: [linux] initial port and osal additions.

2018-04-04 Thread GitBox
sjanc commented on a change in pull request #4: [linux] initial port and osal additions. URL: https://github.com/apache/mynewt-nimble/pull/4#discussion_r179080445 ## File path: porting/linux/config/sdk_config.h ## @@ -0,0 +1,698 @@ + Review comment: is this needed

[GitHub] sjanc commented on a change in pull request #4: [linux] initial port and osal additions.

2018-04-04 Thread GitBox
sjanc commented on a change in pull request #4: [linux] initial port and osal additions. URL: https://github.com/apache/mynewt-nimble/pull/4#discussion_r167164892 ## File path: porting/linux/os/os_mutex.c ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache Software

[GitHub] sjanc commented on a change in pull request #4: [linux] initial port and osal additions.

2018-04-04 Thread GitBox
sjanc commented on a change in pull request #4: [linux] initial port and osal additions. URL: https://github.com/apache/mynewt-nimble/pull/4#discussion_r167163624 ## File path: porting/linux/Makefile.common ## @@ -0,0 +1,177 @@ +# Copyright (c) 2016 Nordic Semiconductor.

[GitHub] sjanc commented on a change in pull request #4: [linux] initial port and osal additions.

2018-04-04 Thread GitBox
sjanc commented on a change in pull request #4: [linux] initial port and osal additions. URL: https://github.com/apache/mynewt-nimble/pull/4#discussion_r167164786 ## File path: porting/linux/os/os_mutex.c ## @@ -0,0 +1,69 @@ +/* + * Licensed to the Apache Software

[GitHub] michal-narajowski commented on issue #147: Cmake fixes

2018-04-04 Thread GitBox
michal-narajowski commented on issue #147: Cmake fixes URL: https://github.com/apache/mynewt-newt/pull/147#issuecomment-378521420 Thanks for the review, @ccollins476ad. Fixed those nitpicks. I think we can merge this now.

[GitHub] andrzej-kaczmarek closed pull request #12: ble service: Device Information Service

2018-04-04 Thread GitBox
andrzej-kaczmarek closed pull request #12: ble service: Device Information Service URL: https://github.com/apache/mynewt-nimble/pull/12 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-nimble] 01/01: Merge pull request #12 from sdalu/dis

2018-04-04 Thread andk
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 7e9ccfeafd791ff3c6c94003e2477d01d0cf880c Merge: eb3ccb3 6adc6d6 Author: Andrzej Kaczmarek

[mynewt-nimble] branch master updated (eb3ccb3 -> 7e9ccfe)

2018-04-04 Thread andk
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 eb3ccb3 Merge pull request #23 from rymanluk/ll_set_resolv_fix add 6adc6d6 implementation of the Device

[GitHub] sjanc commented on a change in pull request #4: [linux] initial port and osal additions.

2018-04-04 Thread GitBox
sjanc commented on a change in pull request #4: [linux] initial port and osal additions. URL: https://github.com/apache/mynewt-nimble/pull/4#discussion_r167165091 ## File path: porting/linux/os/os_sem.h ## @@ -0,0 +1,26 @@ +#ifndef _OS_SEM_H_ Review comment: license

[GitHub] sjanc commented on a change in pull request #4: [linux] initial port and osal additions.

2018-04-04 Thread GitBox
sjanc commented on a change in pull request #4: [linux] initial port and osal additions. URL: https://github.com/apache/mynewt-nimble/pull/4#discussion_r167164236 ## File path: porting/linux/os/os_atomic.c ## @@ -0,0 +1,35 @@ +/* + * Licensed to the Apache Software

[GitHub] sjanc commented on a change in pull request #4: [linux] initial port and osal additions.

2018-04-04 Thread GitBox
sjanc commented on a change in pull request #4: [linux] initial port and osal additions. URL: https://github.com/apache/mynewt-nimble/pull/4#discussion_r167163692 ## File path: porting/linux/config/sdk_config.h ## @@ -0,0 +1,698 @@ + Review comment: miising license

[GitHub] sjanc commented on a change in pull request #4: [linux] initial port and osal additions.

2018-04-04 Thread GitBox
sjanc commented on a change in pull request #4: [linux] initial port and osal additions. URL: https://github.com/apache/mynewt-nimble/pull/4#discussion_r167165034 ## File path: porting/linux/os/os_sem.c ## @@ -0,0 +1,95 @@ +/* + * Licensed to the Apache Software

[GitHub] sjanc commented on a change in pull request #4: [linux] initial port and osal additions.

2018-04-04 Thread GitBox
sjanc commented on a change in pull request #4: [linux] initial port and osal additions. URL: https://github.com/apache/mynewt-nimble/pull/4#discussion_r179079914 ## File path: porting/linux/os/os_atomic.c ## @@ -0,0 +1,35 @@ +/* + * Licensed to the Apache Software