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

optee: phytium: Add Phytium optee driver support #186

Merged
merged 1 commit into from
Jun 12, 2024

Conversation

tianwei08222
Copy link

This patch adds the Phytium optee driver support.

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign liujianqiang-niu for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

drivers/tee/optee/Kconfig Outdated Show resolved Hide resolved
This patch adds the Phytium optee driver support.

Signed-off-by: Chen Baozi <[email protected]>
Signed-off-by: Wang Yinfeng <[email protected]>
Signed-off-by: Tian Wei <[email protected]>
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • 在Kconfig文件中新增的choices选项可能会影响现有配置的稳定性,需要确保新选项的引入不会破坏现有的make规则。
  • 新增的choices选项增加了配置的复杂性,建议评估对性能和维护成本的影响,并考虑是否有必要对配置进行图形化界面可视化。
  • 在smc_abi.c文件中添加了acpi_match_table的定义,需要检查是否有相应的驱动支持,并且该option的添加是否符合整体架构的设计。

是否建议立即修改:

  • 是,特别是对于Kconfig和driver的修改,需要确保它们不会引入新的问题,并且对用户和项目友好的。

@opsiff
Copy link
Member

opsiff commented Jun 12, 2024

@opsiff opsiff merged commit 5086b5c into deepin-community:linux-6.6.y Jun 12, 2024
2 of 3 checks passed
@opsiff
Copy link
Member

opsiff commented Jun 12, 2024

/ok-to-test

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.

3 participants