Nrfx 5848 add gpio edriver #1
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
Update hal_nordic with headers for eGPIO. Signed-off-by: Magdalena Pastula <magdalena.pastula@nordicsemi.no>
jaz1-nordic
reviewed
Jun 24, 2024
magp-nordic
force-pushed
the
NRFX-5848-add-gpio-edriver
branch
from
c4ef8ae
to
c9a63a3
Compare
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 eGPIO to common Nordic peripherals. 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>
magp-nordic
force-pushed
the
NRFX-5848-add-gpio-edriver
branch
from
c9a63a3
to
6f74f10
Compare
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>
magp-nordic
force-pushed
the
NRFX-5848-add-gpio-edriver
branch
from
6f74f10
to
6c04740
Compare
masz-nordic
reviewed
Jul 2, 2024
| compatible = "nordic,nrf-egpio"; | ||
| gpio-controller; | ||
| #gpio-cells = < 0x2 >; | ||
| ngpios = < 0x10 >; |
There was a problem hiding this comment.
A way to detect pin conflicts would be needed in future.
hubertmis
reviewed
Jul 2, 2024
There was a problem hiding this comment.
This file should be included in a eperiph snippet. Building any sample with this snippet should result in using eGPIO. None of the samples should be updated to use eGPIO by default.
|
Moved to: 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.