Skip to content

Conversation

@YdrMaster
Copy link
Member

NodeSeq 和 GroupDeserializer 看起来是完全一样的东西,无论出于何种理由这都不对,但我不记得为什么会这样……实话说我并不完全理解这个修改,但是相信它可以解决问题。

其实我已经不觉得用 serde 解析设备树是合理的设计。但是这个项目确实是 serde 生态的一个有意义的扩展,因此仍有保留和继续维护的价值。大概有必要做个重构,重新梳理其实现以及添加更完整的 test 和 ci。

Signed-off-by: YdrMaster <ydrml@hotmail.com>
Copy link
Member

@luojia65 luojia65 left a comment

Choose a reason for hiding this comment

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

我觉得可以,serde解析也是生态系统的一种可能性。这个项目可以作为设备树生态的一种有效对比和参考。

@luojia65 luojia65 merged commit 413e397 into rustsbi:main Oct 10, 2024
@woshiluo woshiluo mentioned this pull request Oct 11, 2024
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