Skip to content
This repository was archived by the owner on Apr 24, 2019. It is now read-only.

Update 6LoWPAN Gateway binaries #110

Closed
wants to merge 1 commit into from

Conversation

artokin
Copy link
Collaborator

@artokin artokin commented Nov 10, 2016

Update 6LoWPAN Gateway binaries to match sal-stack-nanostack#v5.0.7.

@artokin
Copy link
Collaborator Author

artokin commented Nov 10, 2016

CC: @SeppoTakalo

@artokin
Copy link
Collaborator Author

artokin commented Nov 11, 2016

@yogpan01 , would you please review? This PR should be merged once ARMmbed/mbed-os#3240 is merged to mbed OS and the new mbed OS release is used by this application.

@yogpan01
Copy link
Contributor

@SeppoTakalo You can merge this PR whenever you are ready with the merged changes.
FYI @artokin

@SeppoTakalo
Copy link
Contributor

@yogpan01 I don't have write access, so I cannot merge. However, this is not time critical as these should be used only after the linked in PR appears at mbed OS public release.

@MarceloSalazar
Copy link
Contributor

Please do not merge this yet. During OOB 5.3, we tested the GW binaries in the mbed-OS-5.3 branch and was working ok, so it's stable (even if it doesn't have the latest features).

We could merge this for a patch release of this example (e.g. 5.3.1). Thanks.

@JanneKiiskila
Copy link
Contributor

@artokin - can you rebase this patch, please?

@JanneKiiskila
Copy link
Contributor

Are these still hard-coded to channel 12?

@JanneKiiskila
Copy link
Contributor

I don't know if there's any way to have the channel as a dynamic parameter, hard-coding a channel to a binary sounds a bit dubious. Code size shouldn't grow too much if we give it in as a parameter via the JSON.

@artokin artokin force-pushed the update_gw_binaries branch from d7afcd7 to abf3a1d Compare January 2, 2017 13:51
@artokin
Copy link
Collaborator Author

artokin commented Jan 2, 2017

@jankii01 , rebased. These binaries are hard-coded to channel 12.

@JanneKiiskila
Copy link
Contributor

rebuild

@SeppoTakalo
Copy link
Contributor

Uhm.. are these quite old? We are now in Nanostack v6.0.1, and that is going to be released on mbed OS 5.4

@JanneKiiskila
Copy link
Contributor

Yep, these are old - we were told not to merge these to mbedOS 5.3... So, bring in new ones!

@artokin
Copy link
Collaborator Author

artokin commented Feb 24, 2017

GW binaries are updated to use nanostackv6.0.1.
Please note! Thread configuration updated to use harmonized settings: channel 22, PANID 0x0700. Please use #177 for testing.

@JanneKiiskila
Copy link
Contributor

Modify the mbed_app.json also accordingly? also, do we have some inconsistent chmod issues? Try chmod a-x for all those files? 100644 → 100755

Update 6LoWPAN Gateway binaries to match
sal-stack-nanostack#v6.0.1.

Update thread configuration file as Thread channel is changed from
channel 12 to channel 22 and PANID 0700.
@artokin
Copy link
Collaborator Author

artokin commented Feb 24, 2017

good, thread jconfiguration file and binary permissions updated.

@JanneKiiskila
Copy link
Contributor

@artokin @SeppoTakalo - I think the mbedOS 5.4 stuff is now overriding this, so we can close this PR?
#190

@SeppoTakalo
Copy link
Contributor

Yes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants