Skip to content

Add multi-interface support for MCT7123 and HiPNUC - #8

Closed
changchuanyong wants to merge 5 commits into
masterfrom
feature/mct7123-multi-interface-support
Closed

Add multi-interface support for MCT7123 and HiPNUC#8
changchuanyong wants to merge 5 commits into
masterfrom
feature/mct7123-multi-interface-support

Conversation

@changchuanyong

Copy link
Copy Markdown
Contributor

No description provided.

@Luo1imasi

Copy link
Copy Markdown
Member

该仓库架构思想采用面向接口的分层驱动架构,通过简单工厂创建不同 IMU 后端。目前的代码实现没有遵守该原则,例如mct7123_serial_driver和mct7123_can_driver分成了多个文件。
此外有太多不必要的修改,比如对函数形参、对外暴露接口、protocol以及hipnuc协议代码的修改,imu数据结构也没必要统一,不同品牌imu有各自的数据结构,放在自己的driver下维护就行了,保证接口一致即可。
为了保证每次提交都清晰可读易于维护,尽量最小修改来实现所需功能,减少vibe coding增加的人工维护难度。

@changchuanyong
changchuanyong force-pushed the feature/mct7123-multi-interface-support branch from 025d756 to cd94935 Compare August 3, 2026 07:43
@changchuanyong
changchuanyong deleted the feature/mct7123-multi-interface-support branch August 4, 2026 02:57
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