Rynk host library#873
Draft
HaoboGu wants to merge 1 commit into
Draft
Conversation
f8b4530 to
73ec477
Compare
Draft
5e801a1 to
be7e4a2
Compare
Collaborator
|
我的想法是这样的, 把 rynk-host 做成纯协议库, 不内封 usb/ble/web 什么的支持, 也不内置 runtime
这样可能更干净方便点 |
Owner
Author
纯协议层面在rmk-types,host是一个library,这部分可以是host侧的core,作为独立的一个crate。设备 连接 断开 读写这些仍然需要实现在rmk-host下面作为子crate或者独立的crate。 也就是说,可以构造成 rmk-host crate + transport crates 这种组合形态,类似embedded-hal + hal implementation。但是最上层的应用里,还是需要同时引用 rmk-host crate + specific transport才能够构建起应用。 |
de1ebe4 to
81fa8a9
Compare
Owner
Author
|
@liyang8246 Updated, and web demo is in #876 |
Collaborator
LGTM :) |
29d1242 to
706093c
Compare
Collaborator
|
我擦点错了 |
d298dcf to
207f90b
Compare
Collaborator
|
感觉这个库好像还可以, 让 ai 跑了一下好像效果还行, 但是还没仔细研究( |
0d0e7d0 to
8d0bcfe
Compare
7b1d523 to
9e6faa3
Compare
Add a runtime-free host client crate for the Rynk protocol, plus native serial and BLE transport crates. Extend firmware-side Rynk protocol handling for host clients and cover the wire format, loopback path, hardware validation example, and host CI checks. Signed-off-by: Haobo Gu <haobogu@outlook.com>
9e6faa3 to
2424070
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.