-
Notifications
You must be signed in to change notification settings - Fork 636
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
[nrf fromtree] net: Add copying of missing flags inside clone_pkt_attributes() #1312
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
rlubos
approved these changes
Sep 21, 2023
jukkar
approved these changes
Sep 21, 2023
…pios Allow configuring NFCT pins as GPIOs from devicetree. This setting is part of the UICR. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 1213ed4) (cherry picked from commit 8e6fa57)
If the users configures this option in devicetree, ie, ``` &uicr { nfct-pins-as-gpios; }; ``` CMake will inject the HAL-specific CONFIG_NFCT_PINS_AS_GPIOS definition, so that the necessary operations are performed during system init. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit e43c2f3) (cherry picked from commit 0e3c0a4)
…e property Instead of CONFIG_NFCT_PINS_AS_GPIOS (about to be deprecated). Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit dd568d0) (cherry picked from commit 848f5bf)
In favor of the new UICR nfct-pins-as-gpios devicetree property. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 0762008) (cherry picked from commit 9332b9b)
…_GPIOS changes So that users know how to migrate to the new option. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit c0bf310) (cherry picked from commit 008212f)
So that the reset GPIO can be configured as nRESET from devicetree. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 55ec77c) (cherry picked from commit 2cc0822)
…eset' So that the HAL's system init function configures the reset GPIO as nRESET if requested by the user. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 4d7d5ce) (cherry picked from commit b8209dd)
…set dt property Instead of using the about to be deprecated CONFIG_GPIO_AS_PINRESET option. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 012fa5c) (cherry picked from commit 0a37f20)
…et property Instead of the about to be deprecated GPIO_AS_PINRESET. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 5d6d6f5) (cherry picked from commit 4b28e8d)
In favor of devicetree property in the UICR node. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit efb8408) (cherry picked from commit 03d992d)
…SET=n Because the option is disabled by default now that it is deprecated. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit c572228) (cherry picked from commit 23975f1)
…precation In favor of the new devicetree 'gpio-as-nreset' property. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 9019e46) (cherry picked from commit eabe439)
…tCryptoInit` If asserts are disabled, there is a warning in 'otPlatCryptoInit' regarding unused variable `err`. This commit fixes that. Signed-off-by: Maciej Baczmanski <[email protected]> (cherry picked from commit d2495b1) (cherry picked from commit 9f513ec)
Test that SC Indication is correctly sent when the client reconnect and the server updated the GATT database since last connection. Test that the indication is sent even if the bond is not restored. Signed-off-by: Théo Battrel <[email protected]> (cherry picked from commit 888a8c6) (cherry picked from commit e176212)
This reverts commit cfd368f. Signed-off-by: Théo Battrel <[email protected]> (cherry picked from commit c2b99c0) (cherry picked from commit 72d67fb)
Previous commit reverted the `link_encrypted` filed of CCC config. Update the test to match the new expected behavior. Signed-off-by: Théo Battrel <[email protected]> (cherry picked from commit 2e13802) (cherry picked from commit c991ee3)
This patch updates the SPI flash to the one currently used in the latest board version. Since these are still 0.X.X board versions, old configurations are not explicitly supported. (cherry picked from commit 3a34840) Signed-off-by: Maximilian Deubel <[email protected]> (cherry picked from commit 04666ef)
…for 0.7.0 version Add support for external flash on the 0.7.0 DK version. (cherry picked from commit 8203646) Signed-off-by: Eivind Jølsgard <[email protected]> (cherry picked from commit 8c7463b)
If RPR server is used then Mesh settings work queue requires more size during provisioning procedure. Signed-off-by: Aleksandr Khromykh <[email protected]> (cherry picked from commit 0ba6e80) Signed-off-by: Alperen Şener <[email protected]> (cherry picked from commit f825ca8)
…P_MESH_INIT cmd In some RPR tests we need to pass alternative composition data when testing CDP128. When bt_mesh_init is called from mesh service registration callback, we can't pass any arguments. Therefore we need to move bt_mesh_init call out from mesh service registration to BTP_MESH_INIT command. Signed-off-by: Pavel Vasilyev <[email protected]> (cherry picked from commit 2445e55) Signed-off-by: Alperen Şener <[email protected]> (cherry picked from commit 0d70886)
Remove this command as composition can only be set in bt_mesh_init call which is called before settings are loaded. Signed-off-by: Pavel Vasilyev <[email protected]> (cherry picked from commit bd82ff2) Signed-off-by: Alperen Şener <[email protected]> (cherry picked from commit 6300170)
… bool in comp cmd This will allow to choose more variants of cdp in future if needed. Signed-off-by: Pavel Vasilyev <[email protected]> (cherry picked from commit 8b4994f) Signed-off-by: Alperen Şener <[email protected]> (cherry picked from commit 247c268)
Fixed issue when reprovisioning is done on a device with both RPR client and server on the same device. Signed-off-by: Ingar Kulbrandstad <[email protected]> (cherry picked from commit a5d15ec) Signed-off-by: Alperen Şener <[email protected]> (cherry picked from commit 327e389)
MarGasiorek
requested review from
jukkar
and removed request for
de-nordic,
nordicjm,
glarsennordic,
alwa-nordic,
PavelVPV,
carlocaione,
cvinayak,
kruithofa,
tejlmand,
ioannisg,
anangl,
jfischer-no,
aescolar and
a team
November 30, 2023 07:23
jukkar
approved these changes
Dec 4, 2023
MarGasiorek
requested review from
koffes,
alexsven,
erikrobstad,
rick1082,
gWacey and
lemrey
as code owners
March 8, 2024 19:16
Is seems that changes are merged. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The PR is cleaning code around flags in net_pkt structure, adds copying of missing flags inside clone_pkt_attributes() and included also additional test for flags copy functionality.