Skip to content
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

feat: add Rockchip Crypto #555

Merged
merged 1 commit into from
Nov 8, 2024
Merged

Conversation

RadxaYuntian
Copy link
Member

Description of changes

Add description about Rockchip Crypto and librkcrypto.

Signed-off-by: ZHANG Yuntian <[email protected]>
@RadxaYuntian RadxaYuntian requested review from RadxaStephen and a team as code owners November 7, 2024 06:10

## 检查 Rockchip Crypto 是否开启

Rockchip Crypto 需要在设备树中使能对应节点。使能成功后,系统会自动加载对应驱动。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里是不是要简单举例说明下

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个之后我们会开启的。

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@usui95 之后应该要把加密的测试加到bsp里面

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

我的意思是用户看到这里 他会有困惑,具体是使能哪个节点

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个文档的受众是开发人员,但他们不需要关心具体节点是该如何开启,这是我们在BSP调试阶段就应该实现的。所以客户只需要知道如何判断功能是否正常即可,若没有正常开启,应该通知我们的技术支持来修复这个问题。


## 检查 Rockchip Crypto 是否开启

Rockchip Crypto 需要在设备树中使能对应节点。使能成功后,系统会自动加载对应驱动。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

我的意思是用户看到这里 他会有困惑,具体是使能哪个节点

@peterwang2050 peterwang2050 merged commit a077633 into radxa-docs:main Nov 8, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants