feat[nxp][can]:add mcxa346 can device#11082
Conversation
|
👋 感谢您对 RT-Thread 的贡献!Thank you for your contribution to RT-Thread! 为确保代码符合 RT-Thread 的编码规范,请在你的仓库中执行以下步骤运行代码格式化工作流(如果格式化CI运行失败)。 🛠 操作步骤 | Steps
完成后,提交将自动更新至 如有问题欢迎联系我们,再次感谢您的贡献!💐 |
📌 Code Review Assignment🏷️ Tag: bsp_mcxaReviewers: hywing Changed Files (Click to expand)
📊 Current Review Status (Last Updated: 2025-12-26 21:26 CST)
📝 Review Instructions
|
| PORT_SetPinConfig(PORT3, 28U, &port3_28_pin73_config); | ||
|
|
||
|
|
||
| const port_pin_config_t port1_0_pin135_config = {/* Internal pull-up/down resistor is disabled */ |
There was a problem hiding this comment.
这段配置的删除会不会对SPI外设使能有所影响
我看kconfig中也没有spi0的配置我就全删掉了,如果要保留的话得注意引脚会有冲突
|
@CYFS3 有时间的话可以帮忙看下nxp smart ci报错的原因吗,可以在linux下拉一下工具链然后本地测试看看是不是缺少依赖项使能 |
好的 |
| #define RX_MB_COUNT (1) | ||
| static flexcan_frame_t frame[RX_MB_COUNT]; /* one frame buffer per RX MB */ | ||
| static rt_uint32_t filter_mask = 0; | ||
|
|
|
|
||
| return 0; | ||
| } | ||
| static uint8_t FLEXCAN_GetFirstValidMb(CAN_Type *base) |
|
建议添加一份ci的attach |
好的 |
|
@CYFS3 还有一点点格式上的小问题,请修正下谢谢
|
已修改 |
|
format ci看起来是深路径不识别问题,本地运行也有这种情况,等有空我看看吧 |





拉取/合并请求描述:(PR description)
[
为什么提交这份PR (why to submit this PR)
测试用例
你的解决方案是什么 (what is your solution)
请提供验证的bsp和config (provide the config and bsp)
]
当前拉取/合并请求的状态 Intent for your PR
必须选择一项 Choose one (Mandatory):
代码质量 Code Quality:
我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:
#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up