Content-Length: 240897 | pFad | http://github.com/RT-Thread/rt-thread/issues/9844

B4 [Feature] bsp: cvitek: remove spinor/spinand support · Issue #9844 · RT-Thread/rt-thread · GitHub
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.

Sign up for GitHub

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

Jump to bottom

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

Closed
unicornx opened this issue Dec 30, 2024 · 2 comments · Fixed by #9884
Closed

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

unicornx opened this issue Dec 30, 2024 · 2 comments · Fixed by #9884
Assignees
@unicornx
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的代码吧

@unicornx unicornx assigned unicornx and unassigned jiuyueshenhua Jan 8, 2025
@unicornx
Copy link
Contributor Author

unicornx commented Jan 8, 2025 •
edited
Loading

I will take it over due to it's a really simple fix and need to be resolved first before #9060.

@unicornx unicornx mentioned this issue Jan 8, 2025
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@unicornx unicornx

Labels
Arch: RISC-V BSP related with risc-v BSP: Cvitek BSP related with cvitek BSP
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

bsp: cvitek: remove support for spinor/spinand unicornx/rt-thread
2 participants
@unicornx @jiuyueshenhua








ApplySandwichStrip

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier!      Saves Data!


--- a PPN by Garber Painting Akron. With Image Size Reduction included!

Fetched URL: http://github.com/RT-Thread/rt-thread/issues/9844

Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy