Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature/electron/system module reorder #1065

Merged
merged 2 commits into from
Jul 29, 2016
Merged

Feature/electron/system module reorder #1065

merged 2 commits into from
Jul 29, 2016

Conversation

avtolstoy
Copy link
Member

@avtolstoy avtolstoy commented Jul 25, 2016 •
edited by technobly
Loading

implements #1033

  • Adds Electron system-part1 dependency on system-part3
  • Electron system parts renaming:
    • system-part3 -> system-part1 (module index unchanged: 3)
    • system-part1 -> system-part2 (module index unchanged: 1)
    • system-part2 -> system-part3 (module index unchanged: 2)

Doneness:

  • Contributor has signed CLA
  • Problem and Solution clearly stated
  • Code peer reviewed
  • API tests compiled
  • Run unit/integration/application tests on device
  • Add documentation

Need to update Electron Datasheet to include alternate memory map for post v0.6.0 electrons.

  • Add to CHANGELOG.md after merging (add links to docs and issues)

ENHANCEMENT

  • [Electron] System parts reordered from 3,1,2 to 1,2,3 to preserve logical flashing order for OTA/YModem when upgrading.

@avtolstoy
Electron system-part1 should depend on system-part3
d601203
@avtolstoy
Renaming system parts on Electron:
eed8f1b 
system-part3 -> system-part1 (module index unchanged = 3)
system-part1 -> system-part2 (module index unchanged = 1)
system-part2 -> system-part3 (module index unchanged = 2)
@technobly
Copy link
Member

Tested with Tinker, verified 1,2,3 is really the old 3,1,2 behind the scenes memory map wise.

Tested with make clean all PLATFORM_ID=10 -s program-cloud SPARK_CORE_ID=123412341234123412341234 SPARK_ACCESS_TOKEN=345634563456345634563456 and while it's in the correct order, the script currently doesn't wait long enough for each part to complete, so that needs work (I'll add an issue for that).

@technobly technobly added this to the 0.6.x milestone Jul 29, 2016
@technobly technobly merged commit 7d68e29 into develop Jul 29, 2016
@technobly technobly deleted the feature/electron/system-module-reorder branch October 27, 2016 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.6.x
Development

Successfully merging this pull request may close these issues.
2 participants
@avtolstoy @technobly