samples: basic: blinky: eGPIO PoC #3
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
jaz1-nordic
force-pushed
the
NRFX-5849-flpr-comms-handling-and-bitbang
branch
2 times, most recently
from
304fe73
to
87278dc
Compare
nordic-piks
reviewed
Jul 3, 2024
| - ipc | ||
| filter: dt_enabled_alias_with_parent_compat("led4", "gpio-leds") | ||
| depends_on: gpio | ||
| harness: |
There was a problem hiding this comment.
Please change to harness: console
Current configuration fails and twister cannot start any testing.
|
nrf54l15pdk/nrf54l15/cpuapp samples/drivers/mbox/sample.drivers.mbox.nrf54l15_no_multithreading PASSED 2nd and3rd execution: |
|
In general test can be run using this job - please notice parameters: |
jaz1-nordic
force-pushed
the
NRFX-5849-flpr-comms-handling-and-bitbang
branch
from
87278dc
to
1ec99b8
Compare
Update hal_nordic with headers for eGPIO. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
Add binding for emulated GPIO. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
Add Kconfig option for emulated peripherals drivers support. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
Add support for emulated peripherals on nRF54L15 APP. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
Add node for emulated gpio in nRF54L15 APP dts. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
Change default init priority for emulated GPIO, so that it is initialized after IPC. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
Add driver for emulated GPIO. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
Modify blinky sample to use emulated GPIO. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
Adapting icmsg to work without the MULTITHREADING functionality. Dependencies for kernel work_queue, mutexes and other functions related to running multithreaded applications have been 'ifdefed'. Signed-off-by: Jakub Zymelka <jakub.zymelka@nordicsemi.no>
Removed k_sleep dependencies in non-multithreaded runs. Signed-off-by: Jakub Zymelka <jakub.zymelka@nordicsemi.no>
Removed k_sleep dependencies and added 'printk' instead of LOG in non-multithreaded runs. Signed-off-by: Jakub Zymelka <jakub.zymelka@nordicsemi.no>
PoC of eGPIO peripheral. Signed-off-by: Jakub Zymelka <jakub.zymelka@nordicsemi.no>
jaz1-nordic
force-pushed
the
NRFX-5849-flpr-comms-handling-and-bitbang
branch
from
1ec99b8
to
944a7a0
Compare
|
Changes applied here: nrfconnect/sdk-nrf#16393 |
masz-nordic
pushed a commit
that referenced
this pull request
Oct 28, 2024
hci_packet_complete(buf, buf_size) should check whether buf_size is
enough.
For instance, hci_packet_complete can receive buf with buf_size 1,
leading to the buffer overflow in cmd->param_len, which is buf[3].
This can happen when rx_thread() receives two frames in 512 bytes
and the first frame size is 511. Then, rx_thread() will call
hci_packet_complete() with 1.
==5==ERROR: AddressSanitizer: global-buffer-overflow on address
0x000000ad81c2 at pc 0x0000005279b3 bp 0x7fffe74f5b70 sp 0x7fffe74f5b68
READ of size 2 at 0x000000ad81c2 thread T6
#0 0x5279b2 (/root/zephyr.exe+0x5279b2)
#1 0x4d697d (/root/zephyr.exe+0x4d697d)
#2 0x7ffff60e5daa (/lib/x86_64-linux-gnu/libc.so.6+0x89daa)
(BuildId: 2e01923fea4ad9f7fa50fe24e0f3385a45a6cd1c)
0x000000ad81c2 is located 2 bytes to the right of global variable
'rx_thread.frame' defined in 'zephyr/drivers/bluetooth/hci/userchan.c'
(0xad7fc0) of size 512
SUMMARY: AddressSanitizer: global-buffer-overflow
(/root/zephyr.exe+0x5279b2)
Thread T6 created by T2 here:
#0 0x48c17c (/root/zephyr.exe+0x48c17c)
#1 0x530192 (/root/zephyr.exe+0x530192)
#2 0x4dcc22 (/root/zephyr.exe+0x4dcc22)
Thread T2 created by T1 here:
#0 0x48c17c (/root/zephyr.exe+0x48c17c)
#1 0x530192 (/root/zephyr.exe+0x530192)
#2 0x4dcc22 (/root/zephyr.exe+0x4dcc22)
Thread T1 created by T0 here:
#0 0x48c17c (/root/zephyr.exe+0x48c17c)
#1 0x52f36c (/root/zephyr.exe+0x52f36c)
#2 0x5371dc (/root/zephyr.exe+0x5371dc)
#3 0x5312a6 (/root/zephyr.exe+0x5312a6)
#4 0x52ed7b (/root/zephyr.exe+0x52ed7b)
zephyrproject-rtos#5 0x52eddd (/root/zephyr.exe+0x52eddd)
zephyrproject-rtos#6 0x7ffff6083c89 (/lib/x86_64-linux-gnu/libc.so.6+0x27c89)
(BuildId: 2e01923fea4ad9f7fa50fe24e0f3385a45a6cd1c)
==5==ABORTING
Signed-off-by: Sungwoo Kim <iam@sung-woo.kim>
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.
No description provided.