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.

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

Photon/P1: adds system-part2 v0.7.0-rc.4 as a dependency of system-part1 #1426

Merged
merged 1 commit into from
Nov 7, 2017

Conversation

avtolstoy
Copy link
Member

submission notes
**Important:** Please sanitize/remove any confidential info like usernames, passwords, org names, product names/ids, access tokens, client ids/secrets, or anything else you don't wish to share.

Please Read and Sign the Contributor License Agreement ([Info here](https://github.com/spark/firmware/blob/develop/CONTRIBUTING.md)).

You may also delete this submission notes header if you'd like. Thank you for contributing!

Problem

Due to the introduction of #1421, we will need to have an intermediate release that will enable OTA/Ymodem upgrades to the system firmware with WiFi firmware compression (0.8.0-rc.1). The support for this has been included into 0.7.0-rc.4 (#1420). When upgrading from < 0.7.0-rc.4 right now, the device will be stuck in hardfault state, immediately after system-part1 is flashed.

Solution

In order to avoid that, 0.8.0-rc.1 system-part1 needs to have a dependency on 0.7.0-rc.4 system-part2.

Steps to Test

  1. Have < 0.7.0-rc.4 system firmware running on the device
  2. Make sure SYSTEM_PART1_MODULE_VERSION, SYSTEM_PART2_MODULE_VERSION, SYSTEM_PART3_MODULE_VERSION in system_module_version.mk are >= 0.8.0-rc.1 (300) and build the firmware
  3. Flash system-part1.bin from step #2
  4. Verify that the device is running normally and provided system-part1 has not been applied
  5. Upgrade system firmware running on the device to 0.7.0-rc.4
  6. Flash system-part1.bin from step #2
  7. Verify that the device is running normally and provided system-part1 module has been applied

Example App

N/A

References


Completeness

  • User is totes amazing for contributing!
  • Contributor has signed CLA (Info here)
  • Problem and Solution clearly stated
  • Run unit/integration/application tests on device
  • Added documentation
  • Added to CHANGELOG.md after merging (add links to docs and issues)

…rt1 to avoid bricking devices when upgrading to 0.8.0-rc.1 from < 0.7.0-rc.4 version
@avtolstoy avtolstoy added this to the 0.8.0 milestone Nov 7, 2017
Copy link
Contributor

@m-mcgowan m-mcgowan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice :-)

@m-mcgowan m-mcgowan merged commit 3256ac3 into develop Nov 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants