[GitHub] jacobrosenthal commented on issue #1285: move sensor_itf to driver_itf WIP please evaluate

2018-07-19 Thread GitBox
jacobrosenthal commented on issue #1285: move sensor_itf to driver_itf WIP please evaluate URL: https://github.com/apache/mynewt-core/pull/1285#issuecomment-406446275 2 new totally non functional commits just to take a look at what driver and sensor separation feels like. I pulled the low

[GitHub] sterlinghughes commented on issue #1285: move sensor_itf to driver_itf WIP please evaluate

2018-07-19 Thread GitBox
sterlinghughes commented on issue #1285: move sensor_itf to driver_itf WIP please evaluate URL: https://github.com/apache/mynewt-core/pull/1285#issuecomment-406407802 PRs are welcome (at least -- I'm glad you are looking at this!) - i think generally this looks like a good start, but it s

[GitHub] jacobrosenthal commented on issue #1285: move sensor_itf to driver_itf WIP please evaluate

2018-07-19 Thread GitBox
jacobrosenthal commented on issue #1285: move sensor_itf to driver_itf WIP please evaluate URL: https://github.com/apache/mynewt-core/pull/1285#issuecomment-406406351 My main thrust is getting to writing thinner drivers instead of driver/sensor combos and the absolute quickest path to that

[GitHub] vrahane commented on issue #1285: move sensor_itf to driver_itf WIP please evaluate

2018-07-19 Thread GitBox
vrahane commented on issue #1285: move sensor_itf to driver_itf WIP please evaluate URL: https://github.com/apache/mynewt-core/pull/1285#issuecomment-406371963 I agree. The only issue I have with a generic interface is that some hardware requires more context for physical interfaces than t

[mynewt-newt] branch master updated (01800e7 -> 8ec493c)

2018-07-19 Thread ccollins
This is an automated email from the ASF dual-hosted git repository. ccollins pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-newt.git. from 01800e7 Update newt_windows.rst add bdc3872 Detect and report malformed syscfg.yml files new 8ec49

[mynewt-newt] 01/01: Merge pull request #196 from ccollins476ad/syscfg-yaml-check

2018-07-19 Thread ccollins
This is an automated email from the ASF dual-hosted git repository. ccollins pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-newt.git commit 8ec493c4fb718271d92e9a8aa9c0726b4a142997 Merge: 01800e7 bdc3872 Author: ccollins476ad AuthorDate: Thu Jul 19 11:2

[GitHub] ccollins476ad closed pull request #196: Detect and report malformed `syscfg.yml` files

2018-07-19 Thread GitBox
ccollins476ad closed pull request #196: Detect and report malformed `syscfg.yml` files URL: https://github.com/apache/mynewt-newt/pull/196 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 i

[GitHub] sterlinghughes commented on issue #1285: move sensor_itf to driver_itf WIP please evaluate

2018-07-19 Thread GitBox
sterlinghughes commented on issue #1285: move sensor_itf to driver_itf WIP please evaluate URL: https://github.com/apache/mynewt-core/pull/1285#issuecomment-406371204 Why not just add this functionality into the os_dev framework? It seems like a fairly low code & memory footprint, and use

[GitHub] ccollins476ad opened a new pull request #196: Detect and report malformed `syscfg.yml` files

2018-07-19 Thread GitBox
ccollins476ad opened a new pull request #196: Detect and report malformed `syscfg.yml` files URL: https://github.com/apache/mynewt-newt/pull/196 Prior to this commit, malformed `syscfg.defs` or `syscfg.vals` maps would trigger a panic. Now, newt detects the problem and aborts the operatio

[mynewt-core] branch master updated (ad5edcc -> 3336907)

2018-07-19 Thread marko
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 ad5edcc Merge pull request #1283 from jacobrosenthal/lis2ds-modlog add 431df8a tinycbor/oic; byte string e

[mynewt-core] 01/01: Merge pull request #1275 from mkiiskila/cbor_bytestring_iov

2018-07-19 Thread marko
This is an automated email from the ASF dual-hosted git repository. marko pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/mynewt-core.git commit 33369075573b56667119ea0844e60ee1f6b12cf7 Merge: ad5edcc 431df8a Author: mkiiskila AuthorDate: Thu Jul 19 15:00:26 20

[GitHub] mkiiskila closed pull request #1275: tinycbor/oic; byte string encoding with data as iovec.

2018-07-19 Thread GitBox
mkiiskila closed pull request #1275: tinycbor/oic; byte string encoding with data as iovec. URL: https://github.com/apache/mynewt-core/pull/1275 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] mkiiskila commented on issue #1275: tinycbor/oic; byte string encoding with data as iovec.

2018-07-19 Thread GitBox
mkiiskila commented on issue #1275: tinycbor/oic; byte string encoding with data as iovec. URL: https://github.com/apache/mynewt-core/pull/1275#issuecomment-406251566 Re: having doxygen stuff in .h vs in .c. I did it like this here as the rest of the package is doing it this way.