-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
samples: bluetooth: fast_pair: locator_tag: enable lto in debug config #21123
samples: bluetooth: fast_pair: locator_tag: enable lto in debug config #21123
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: a08ea50b7a8c06658960c335049016ef7ae1136a more detailssdk-nrf:
Github labels
List of changed files detected by CI (3)
Outputs:ToolchainVersion: 4ffa2202d5 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
316a145
to
ed84e81
Compare
Enabled the Link Time Optimization (LTO) in the debug (default) configuration of the Fast Pair Locator Tag sample to get it aligned with the release configuration. Signed-off-by: Kamil Piszczek <Kamil.Piszczek@nordicsemi.no>
ed84e81
to
d8e9a31
Compare
You can find the documentation preview for this PR here. Preview links for modified nRF Connect SDK documents: https://ncsdoc.z6.web.core.windows.net/PR-21123/nrf/releases_and_maturity/releases/release-notes-changelog.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved with a minor nit.
* Updated the partition layout for the ``nrf5340dk/nrf5340/cpuapp/ns`` and ``thingy53/nrf5340/cpuapp/ns`` board targets to accommodate the partitions needed due to change in the TF-M profile configuration. | ||
* Updated: | ||
|
||
* The partition layout for the ``nrf5340dk/nrf5340/cpuapp/ns`` and ``thingy53/nrf5340/cpuapp/ns`` board targets to accommodate the partitions needed due to change in the TF-M profile configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
* The partition layout for the ``nrf5340dk/nrf5340/cpuapp/ns`` and ``thingy53/nrf5340/cpuapp/ns`` board targets to accommodate the partitions needed due to change in the TF-M profile configuration. | |
* The partition layout for the ``nrf5340dk/nrf5340/cpuapp/ns`` and ``thingy53/nrf5340/cpuapp/ns`` board targets to accommodate the partitions needed due to a change in the TF-M profile configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
Improved the changelog entry for the Fast Pair Locator Tag sample that is related to the partition layout changes for nRF53-based targets with TF-m. Signed-off-by: Kamil Piszczek <Kamil.Piszczek@nordicsemi.no>
Enabled the Link Time Optimization (LTO) in the debug (default) configuration of the Fast Pair Locator Tag sample to get it aligned with the release configuration.