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
dts: Add binding and nodes representing Nordic ECB peripheral #30930
Conversation
Rebased and updated accordingly (used |
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.
LGTM.
I left a suggestion for a comment to clarify why there's a dependency with the BT Controller
bool "nRF AES electronic codebook mode encryption" | ||
depends on HAS_HW_NRF_ECB && !BT_CTLR | ||
depends on $(dt_compat_enabled,$(DT_COMPAT_NORDIC_NRF_ECB)) | ||
depends on !BT_CTLR |
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.
@anangl should we have a comment explaining why !BT_CTRL is a dependency?
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.
I know it was un-commented, already, but perhaps we could add now the clarification.
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.
@ioannisg I added a note that explains this dependency. Please check if it is acceptable.
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.
Looks fine, thanks
Since MDK was updated and now it contains the definition of QDEC peripheral for nRF52805, the DT node representing this peripheral can be processed normally. Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
Add binding for the Nordic ECB (AES electronic codebook mode encryption) peripheral and corresponding devicetree nodes for nRF SoCs that feature this peripheral. Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
Convert the driver so that it creates its instance basing on DT. Remove no longer needed Kconfig option CRYPTO_NRF_ECB_NAME. Also update accordingly the crypto sample. Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
Add check that validates that the base addresses specified in DT nodes representing the ECB peripheral match the addresses of that peripheral defined in MDK. Signed-off-by: Andrzej Głąbek <andrzej.glabek@nordicsemi.no>
Rebased on master, to solve the problem with reported undefined Kconfig options being referenced (those from the recently added |
This PR adds binding for the Nordic ECB (AES electronic codebook mode encryption) peripheral and corresponding devicetree nodes for nRF SoCs that feature this peripheral. The
crypto_nrf_ecb
driver is converted so that it creates its instance basing on DT, and theCRYPTO_NRF_ECB_NAME
Kconfig option is removed as no longer needed.The first commit does an additional minor cleanup in one of the files modified by this PR.