[GitHub] ccollins476ad closed issue #3: Unable to compile sample on zephyr 1.11

2018-07-03 Thread GitBox
ccollins476ad closed issue #3: Unable to compile sample on zephyr 1.11 URL: https://github.com/apache/mynewt-mcumgr/issues/3 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] ccollins476ad commented on issue #2: Image upload broken

2018-07-03 Thread GitBox
ccollins476ad commented on issue #2: Image upload broken URL: https://github.com/apache/mynewt-mcumgr-cli/issues/2#issuecomment-402248195 The fix to newtmgr will be automatically included in mcumgr. By the way, the fix was just merged to the newtmgr repo.

[GitHub] carlescufi commented on issue #7: Upstreaming of Zephyr downstream modifications

2018-07-03 Thread GitBox
carlescufi commented on issue #7: Upstreaming of Zephyr downstream modifications URL: https://github.com/apache/mynewt-mcumgr/pull/7#issuecomment-402234451 @utzig thanks! This is an automated message from the Apache Git

[GitHub] utzig closed pull request #7: Upstreaming of Zephyr downstream modifications

2018-07-03 Thread GitBox
utzig closed pull request #7: Upstreaming of Zephyr downstream modifications URL: https://github.com/apache/mynewt-mcumgr/pull/7 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] carlescufi commented on issue #2: Image upload broken

2018-07-03 Thread GitBox
carlescufi commented on issue #2: Image upload broken URL: https://github.com/apache/mynewt-mcumgr-cli/issues/2#issuecomment-402202003 @ccollins476ad thanks for the info. We'll wait until that PR is merged to close this.

[GitHub] JoeHut commented on issue #7: Upstreaming of Zephyr downstream modifications

2018-07-03 Thread GitBox
JoeHut commented on issue #7: Upstreaming of Zephyr downstream modifications URL: https://github.com/apache/mynewt-mcumgr/pull/7#issuecomment-402200028 Looks like that is everything :+1: ``` $ diff -r mynewt-mcumgr zephyr/ext/lib/mgmt/mcumgr Only in mynewt-mcumgr/cborattr:

[GitHub] ccollins476ad commented on issue #2: Image upload broken on Windows

2018-07-03 Thread GitBox
ccollins476ad commented on issue #2: Image upload broken on Windows URL: https://github.com/apache/mynewt-mcumgr-cli/issues/2#issuecomment-402198760 I believe this is an issue that was recently introduced into newtmgr (and therefore mcumgr-cli). The problem is that the CLI tool recently

[GitHub] carlescufi commented on issue #2: Image upload broken on Windows

2018-07-03 Thread GitBox
carlescufi commented on issue #2: Image upload broken on Windows URL: https://github.com/apache/mynewt-mcumgr-cli/issues/2#issuecomment-402196482 Not sure if this is an MCUmgr or MCUboot issue actually. For now I will keep it here, but there is something definitely broken. @sjanc could you

[GitHub] carlescufi opened a new pull request #7: Upstream of Zephyr downstream modifications

2018-07-03 Thread GitBox
carlescufi opened a new pull request #7: Upstream of Zephyr downstream modifications URL: https://github.com/apache/mynewt-mcumgr/pull/7 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] carlescufi commented on issue #7: Upstream of Zephyr downstream modifications

2018-07-03 Thread GitBox
carlescufi commented on issue #7: Upstream of Zephyr downstream modifications URL: https://github.com/apache/mynewt-mcumgr/pull/7#issuecomment-402194450 CC @utzig @ccollins476ad @JoeHut This is an automated message from the

[GitHub] JoeHut commented on issue #5: cmd: img_mgmt: Add an extra byte to the 'hash' buffer

2018-07-03 Thread GitBox
JoeHut commented on issue #5: cmd: img_mgmt: Add an extra byte to the 'hash' buffer URL: https://github.com/apache/mynewt-mcumgr/pull/5#issuecomment-402156773 @carlescufi Sure. This is an automated message from the Apache

Re: 1 Wire HAL

2018-07-03 Thread Kevin Townsend
Hi Marko, That sounds sensible to me, with timer+GPIO beind the sensible place to start in my opinion with UART an interesting second option if you have a free UART HW block available. I'm happy to contribute one or two drivers for common 1-wire sensors or devices, and help out

Re: 1 Wire HAL

2018-07-03 Thread marko kiiskila
Hi Kevin, I was thinking separate packages for different implementations, and then syscfg knobs inside the respective ones. E.g. GPIO/timer and UART selection. These packages would implement the ‘1-wire’ API, and chip drivers would then depend on that API. The PJRC API is simple, which I like.

Re: 1 Wire HAL

2018-07-03 Thread Kevin Townsend
Hi Marko, IMHO, The PJRC lib is a sensible basis for a Mynewt 1-Wire package since it's seen a fair amount of real-world use, and the underlying implementation could be configured via the syscfg.yml file (timer+GPIO or something else like UART if you have one free)? Kevin On 03/07/18

[GitHub] carlescufi commented on issue #2: Image upload broken on Windows

2018-07-03 Thread GitBox
carlescufi commented on issue #2: Image upload broken on Windows URL: https://github.com/apache/mynewt-mcumgr-cli/issues/2#issuecomment-402137495 @utzig thanks! I now realize I get this error on Linux as well after updating `mcumgr`, so this is not a Windows issue. Here's the log:

[GitHub] carlescufi commented on issue #2: Image upload broken on Windows

2018-07-03 Thread GitBox
carlescufi commented on issue #2: Image upload broken on Windows URL: https://github.com/apache/mynewt-mcumgr-cli/issues/2#issuecomment-402133202 @ccollins476ad I would report the exact mcumgr version, but it simply shows: ``` C:\Users\Carles\go\bin>mcumgr version mcumgr

[GitHub] carlescufi commented on issue #2: Image upload broken on Windows

2018-07-03 Thread GitBox
carlescufi commented on issue #2: Image upload broken on Windows URL: https://github.com/apache/mynewt-mcumgr-cli/issues/2#issuecomment-402132372 I wonder what `Error 3` means? This is an automated message from the Apache Git

[GitHub] carlescufi commented on issue #2: Image upload broken on Windows

2018-07-03 Thread GitBox
carlescufi commented on issue #2: Image upload broken on Windows URL: https://github.com/apache/mynewt-mcumgr-cli/issues/2#issuecomment-402132221 CC @ccollins476ad @utzig @sjanc @lemrey This is an automated message from the

[GitHub] carlescufi opened a new issue #2: Image upload broken on Windows

2018-07-03 Thread GitBox
carlescufi opened a new issue #2: Image upload broken on Windows URL: https://github.com/apache/mynewt-mcumgr-cli/issues/2 Running `mcumgr` on Windows with the following settings: ``` C:\Users\Carles\go\bin>mcumgr conn show com3 Connection profiles: com3: type=serial,

[GitHub] carlescufi commented on issue #1: Error: errno 0 when running cross-compiled executable on Yocto ARM system

2018-07-03 Thread GitBox
carlescufi commented on issue #1: Error: errno 0 when running cross-compiled executable on Yocto ARM system URL: https://github.com/apache/mynewt-mcumgr-cli/issues/1#issuecomment-402131783 CC @ccollins476ad @utzig This is

[GitHub] carlescufi commented on issue #6: BLE reassembly support

2018-07-03 Thread GitBox
carlescufi commented on issue #6: BLE reassembly support URL: https://github.com/apache/mynewt-mcumgr/issues/6#issuecomment-402130885 Note that this is required for interaction with Android phones version 5 and below, since they don't support big ATT_MTUs (I am told by @philips77)

[GitHub] carlescufi commented on issue #5: cmd: img_mgmt: Add an extra byte to the 'hash' buffer

2018-07-03 Thread GitBox
carlescufi commented on issue #5: cmd: img_mgmt: Add an extra byte to the 'hash' buffer URL: https://github.com/apache/mynewt-mcumgr/pull/5#issuecomment-402118844 @Olivier-ProGlove @JoeHut will you send a PR to Zephyr to update with this version?

[GitHub] utzig closed pull request #4: Readme.md: Link OS-specific documentation

2018-07-03 Thread GitBox
utzig closed pull request #4: Readme.md: Link OS-specific documentation URL: https://github.com/apache/mynewt-mcumgr/pull/4 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

Re: 1 Wire HAL

2018-07-03 Thread marko kiiskila
Sounds like a candidate for driver interface, not HAL. I agree with Miguel. User could pick the implementation which uses UART, or one which uses hal_timer with hal_gpio. However; Sam, you don’t think a generic driver interface makes sense? If you had driver package which implemented an API

[GitHub] carlescufi opened a new issue #6: BLE reassembly support

2018-07-03 Thread GitBox
carlescufi opened a new issue #6: BLE reassembly support URL: https://github.com/apache/mynewt-mcumgr/issues/6 The BLE transport supports segmentation (i.e. splitting SMP reponses into multiple ATT_MTU-sized notifications) but not reassembly (i.e. reassembling SMP requests that have been

[GitHub] carlescufi commented on issue #3: Unable to compile sample on zephyr 1.11

2018-07-03 Thread GitBox
carlescufi commented on issue #3: Unable to compile sample on zephyr 1.11 URL: https://github.com/apache/mynewt-mcumgr/issues/3#issuecomment-402069628 @ccollins476ad this seems to be obsolete, mind closing it? This is an

[GitHub] carlescufi commented on issue #4: Readme.md: Link OS-specific documentation

2018-07-03 Thread GitBox
carlescufi commented on issue #4: Readme.md: Link OS-specific documentation URL: https://github.com/apache/mynewt-mcumgr/pull/4#issuecomment-402069439 CC @ccollins476ad. This looks good, can you merge this? This is an