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

[Feature] bsp: cvitek: remove spinor/spinand support #9844

Open
unicornx opened this issue Dec 30, 2024 · 1 comment
Open

[Feature] bsp: cvitek: remove spinor/spinand support #9844

unicornx opened this issue Dec 30, 2024 · 1 comment
Assignees
Labels
Arch: RISC-V BSP related with risc-v BSP: Cvitek BSP related with cvitek BSP

Comments

@unicornx
Copy link
Contributor

Describe problem solved by the proposed feature

和 milkv 确认了一下,市场上默认只对 duo 销售 sd card 版本。spi 的引脚通过邮票孔提供,可供用户在二次开发时,搭配底板的时候,自己焊接相应的元器件。基于以上信息,我建议在 RTT 的 bsp/cvitek 中去掉对 spinor/spinand 的支持。我的理解是一旦代码中提供了,就要对外承诺支持,每次升级或者回归测试时都要考虑相关的影响,可是目前社区里实际上并没有对 rtt 在 spinor/spinand 上的测试和支持有投入。上游主线只支持 sd-card 就好。

Describe your preferred solution

只支持 sd-card 版本

Describe possible alternatives

No response

@unicornx unicornx added BSP: Cvitek BSP related with cvitek Arch: RISC-V BSP related with risc-v BSP labels Dec 30, 2024
@jiuyueshenhua
Copy link

我来清理支持spinor/spinand的代码吧

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arch: RISC-V BSP related with risc-v BSP: Cvitek BSP related with cvitek BSP
Projects
None yet
Development

No branches or pull requests

2 participants