-
Notifications
You must be signed in to change notification settings - Fork 645
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 fromlist] samples: basic: add multicore_hello_world #1552
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
Additionally, replace the local print(sh, level, fmt, ...) macros with PR, PR_ERROR, and PR_WARNING macros. Then remove the print(sh, level, fmt, ...) macros. Signed-off-by: Pisit Sawangvonganan <[email protected]> (cherry picked from commit 055ac61)
WIFI scan result shows junk character in SSID because of the length of ssid is maximum(32 character) which leads to buffer overflow. It required one character for null terminator ‘\0’. Signed-off-by: Kapil Bhatt <[email protected]> (cherry picked from commit 2207fed)
Enable all these tests which run in native_posix in native_sim And add native_sim as default test platform Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit 13b7557)
Enable all these tests which run in native_posix in native_sim, Switch from native_posix to native_sim as default test platform And switch native_posix overlays to native_sim. Signed-off-by: Alberto Escolar Piedras <[email protected]> (cherry picked from commit 7b1813e)
Add a generic NMEA GNSS driver. Signed-off-by: Fabio Baltieri <[email protected]> (cherry picked from commit d14066b)
Add a generic test for GNSS devices. Signed-off-by: Fabio Baltieri <[email protected]> (cherry picked from commit 407e9a4)
Printing fractionals currently put the sign on integer values on the fractional part, for example: longitude : -6.-207483333 Run an extra abs to get rid of the sign there for latitude, longitude and altitude, compute the sign separately so it works for numbers between -1 and 0 as well. Signed-off-by: Fabio Baltieri <[email protected]> (cherry picked from commit 96b9bd4)
Move gnss_publish.h in include/ so that out of tree drivers and tests can use it. Signed-off-by: Fabio Baltieri <[email protected]> (cherry picked from commit f9ab050)
Some modems or networks require PAP authentication for successful LCP handshake. Tested on U-blox SARA-R5 with zephyr,gsm-ppp. Signed-off-by: Emil Lindqvist <[email protected]> (cherry picked from commit 79c2faf)
U-blox SARA-R4 already exists but the behavior is different, requiring a separate driver instance. For instance, R5 autostarts, so this commit also adds support for skipping power on pulse. Signed-off-by: Emil Lindqvist <[email protected]> (cherry picked from commit 1cd7c55)
Added missing command and message handling for use existing modem cmux for DCE role. DCE CMUX connection can be now initialized from DTE side. Signed-off-by: Juha Heiskanen <[email protected]> (cherry picked from commit 093efc4)
Added possibility for link 2 modem mock instance for validating P2P communication without any data stubbing. Signed-off-by: Juha Heiskanen <[email protected]> (cherry picked from commit 6943cd4)
Added unite test for for validate modem CMUX DTE & DCE role communication together. Signed-off-by: Juha Heiskanen <[email protected]> (cherry picked from commit 34d4e50)
Adds two APIs which allow for configuring the cellular network configuration of a cellular network device. like a cellular modem. The first allows for configuring which access technology to use, and optionally, which bands to use. The second allows for getting all supported access technologies are supported, and which bands for each tech are supported. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit ad01169)
Implement modem info pulling using the init and periodic chat scripts Signed-off-by: Lucas Denefle <[email protected]> (cherry picked from commit d1ba79a)
Fixed Kconfig to remove dependency between modem_socket and modem_context, the two do not depend on each other and should be possible to use independently Signed-off-by: Alessio Lei <[email protected]> (cherry picked from commit 733b81d)
Add Lucas Denefle to copyright of the cellular API. Signed-off-by: Lucas Denefle <[email protected]> (cherry picked from commit a5e5c1d)
In cases where the data is bigger than 127 bytes, the first bit of the second byte of the data length field used to always be set. This is wrong as according to the 3GPP 27.010 spec only the first bit of the first byte is the EA bit; all the others denote the data length. Signed-off-by: Tomi Fontanilles <[email protected]> (cherry picked from commit 6564e8b)
…e driver model The current cellular API header is not written to conform with the device driver model. This commit fixes that. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 6f55309)
…ular API Move the implementations of the cellular API in modem_cellular.c to an cellular_driver_structure, and implement this API structure withing the device drivers. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit f949103)
…ementation The modem_cellular.c driver now uses a common script to get signal strenght, which is run on demand. This is more efficient than polling it, and simpler since every modem doesn't have to define their own variant of the script. Additionally, the CSQ handler now stores the "raw" rssi value returned from AT+CSQ to be parsed by the cellular_modem_get_signal implementation. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 572c5e6)
…dler The QCCID handler is Quectel specific and as such may not be part of the modem cellular driver which only supports commands defined in 3GPP TS 27.007 Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 1ee0926)
The chat module contains an array of three lists of matches, one of which are static, two of which are contained within the currently running script. The current match, which is an object stored in one of the three lists, is stored in its own pointer in the chat module context. A memory error occurs when the script is stopped, while the chat module is using one of the matches stored withing the script. This commit clears the match pointer when the script is stopped if the match is stored within the script. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 3ebe814)
The CMUX resync mechanism can get stuck between states MODEM_CMUX_RECEIVE_STATE_RESYNC_0 and MODEM_CMUX_RECEIVE_STATE_RESYNC_1 if the resync flags, which are sent only once in state MODEM_CMUX_RECEIVE_STATE_SOF, are not responded to, or the response is lost. This patch ensures resync flags are sent from states MODEM_CMUX_RECEIVE_STATE_SOF, MODEM_CMUX_RECEIVE_STATE_RESYNC_1 and MODEM_CMUX_RECEIVE_STATE_RESYNC_2 if its determined that the resync flags are not being responded to. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 48fa603)
DLCI receive buffer may overrun if data is not processed fast enough. This error was not reported before this patch, resulting in unexplained missing bytes. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit b3971d0)
Some modems don't start sending resync flags as described in 3G TS 27.010 V2.0.0 5.2.5, which results in the CMUX being stuck in resync mode for said modems. This patch simplifies the resync mechanism to simply drop invalid frames, and wait for atleast two consequtive frame flags (stop+start). Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 3debfc8)
Update the unit test to expect the new simplified resync behavior, and validate that new resync is working. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 7d7f7aa)
Add transmit idle event to modem_pipe_event enum. This will allow modules to await transmit idle before trying to transmit more data, instead of blindly calling modem_pipe_transmit in a loop until all data has been accepted. This will reduce the time spent trying to transmit data while the backend is blocked. Similarly to the RECEIVE_READY event, backends will call modem_pipe_notify_transmit_idle() to indicate that transmit is idle, and the TRANSMIT_IDLE event will be reinvoked when the modem pipe is attached to synchronize the state of the pipe with the user of it. Additionally, the TRANSMIT_IDLE event is also invoked when the modem is opened to further help synchronization with the user of the pipe. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 516af3c)
Make async and interrupt driven UART backends notify transmit idle when transmit is idle. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 8c6a9ee)
Implement transmit idle notification for TTY backend. Since TTY has an "infinite" transmit buffer, we invoke transmit idle immediately after writing the data to the TTY file. The test suite for the TTY backend has been updated to match the new behavior. Signed-off-by: Bjarki Arge Andreasen <[email protected]> (cherry picked from commit 4d99c69)
Add IPC shared memory regions in the global RAM, as well as an enlarged SRAM region for Application core. Upstream PR: zephyrproject-rtos/zephyr#69303 Signed-off-by: Grzegorz Swiderski <[email protected]> (cherry picked from commit 7a5f20bc041a73d89484607bdf0fb8001bf146f3)
Add `zephyr,ipc-icmsg*` nodes, with mboxes specified in a common file. Upstream PR: zephyrproject-rtos/zephyr#69303 Signed-off-by: Grzegorz Swiderski <[email protected]> Signed-off-by: Gerard Marull-Partetas <[email protected]> (cherry picked from commit 035adef5768e79f4466e1f74dcd501efc39e2dc2)
Add support for nRF VEVIF. Upstream PR: zephyrproject-rtos/zephyr#69303 Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit d40fd1f886666d0d61a7a555439857c5654bec55)
Add support for testing BELLBOARD between cpuapp (local) and cpuppr (remote). Upstream PR: zephyrproject-rtos/zephyr#69303 Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 54d64e39a00b6ef9383ea46eb79a409e11ccc189)
Add a new "hello world" sample that boots on multiple cores. This sample can be used to see if Zephyr boots successfully with _minimal_ configuration on SoCs with multiple cores. The following configurations are provided for now: - nRF5340 DK: cpuapp+cpunet - nRF54H20 PDK: cpuapp+cpurad, cpuapp+cpuppr, cpuapp+cpuppr(ram) Note that because of a quoting bug somewhere in the build system, SB_CONFIG_REMOTE_BOARD is specified in .conf files (see the sysbuild/ directory). This should be fixed in the near future. Upstream PR: zephyrproject-rtos/zephyr#69652 Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit d1b3b3ad11878c99abf8501537c4704661573f84)
@gmarull Is this still relevant? |
it is not, upstream PR suffered from more changes, will hopefully get merged at some point and come during an upmerge maybe. |
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.
Add a new "hello world" sample that boots on multiple cores. This sample can be used to see if Zephyr boots successfully with minimal configuration on SoCs with multiple cores.
The following configurations are provided for now:
Note that because of a quoting bug somewhere in the build system, SB_CONFIG_REMOTE_BOARD is specified in .conf files (see the sysbuild/ directory). This should be fixed in the near future.
Upstream PR: zephyrproject-rtos/zephyr#69652
Signed-off-by: Gerard Marull-Paretas [email protected]
(cherry picked from commit d1b3b3ad11878c99abf8501537c4704661573f84)