rt-thread icon indicating copy to clipboard operation
rt-thread copied to clipboard

S32K344 Basic porting : FINSH

Open ppppppsssss opened this issue 1 year ago • 13 comments

拉取/合并请求描述:(PR description)

适配S32K344的基础工程,对LPUART采用寄存器控制,实现FINSH的移植,同时每两秒打印hello验证调度和延时的正确

当前拉取/合并请求的状态 Intent for your PR

必须选择一项 Choose one (Mandatory):

  • [ ] 本拉取/合并请求是一个草稿版本 This PR is for a code-review and is intended to get feedback
  • [ ] 本拉取/合并请求是一个成熟版本 This PR is mature, and ready to be integrated into the repo

代码质量 Code Quality:

我在这个拉取/合并请求中已经考虑了 As part of this pull request, I've considered the following:

  • [ ] 已经仔细查看过代码改动的对比 Already check the difference between PR and old code
  • [ ] 代码风格正确,包括缩进空格,命名及其他风格 Style guide is adhered to, including spacing, naming and other styles
  • [ ] 没有垃圾代码,代码尽量精简,不包含#if 0代码,不包含已经被注释了的代码 All redundant code is removed and cleaned up
  • [ ] 所有变更均有原因及合理的,并且不会影响到其他软件组件代码或BSP All modifications are justified and not affect other components or BSP
  • [ ] 对难懂代码均提供对应的注释 I've commented appropriately where code is tricky
  • [ ] 代码是高质量的 Code in this PR is of high quality
  • [ ] 已经使用formatting 等源码格式化工具确保格式符合RT-Thread代码规范 This PR complies with RT-Thread code specification

ppppppsssss avatar Oct 19 '24 10:10 ppppppsssss

bsp/nxp/S32K3/S32K344, please rename as:

bsp/nxp/s32/s32k344_core

BernardXiong avatar Oct 19 '24 11:10 BernardXiong

我建议命名成 bsp/nxp/s32/s32k344-core 文件夹名称可以用中划线

mysterywolf avatar Oct 19 '24 19:10 mysterywolf

ci加一下。在.github/workflow/bsp_building.yml

supperthomas avatar Oct 20 '24 04:10 supperthomas

好的,更改了 同时移除了nxp的库,改为使用者自行手动添加,添加方法在 readme 中给出

ppppppsssss avatar Oct 21 '24 05:10 ppppppsssss

好的,更改了 同时移除了nxp的库,改为使用者自行手动添加,添加方法在 readme 中给出

麻烦把此ci加一下。在.github/workflow/bsp_building.yml中,添加此BSP

Rbb666 avatar Oct 21 '24 09:10 Rbb666

ci添加参考:

https://club.rt-thread.org/ask/article/62273896dee18b09.html
image

supperthomas avatar Oct 25 '24 01:10 supperthomas

@PillarShi CI方便加下吗?

Rbb666 avatar Nov 20 '24 02:11 Rbb666

C build/board/board.o

56 scons: *** [build/applications/main.o] Error 1

57 In file included from board/board.c:11:

58 board/board.h:16:10: fatal error: S32K344.h: No such file or directory

59 16 | #include "S32K344.h"

60 | ^~~~~~~~~~~

61 compilation terminated.

62 scons: *** [build/board/board.o] Error 1

supperthomas avatar Nov 20 '24 04:11 supperthomas

好的,更改了 同时移除了nxp的库,改为使用者自行手动添加,添加方法在 readme 中给出

为啥要移除?

supperthomas avatar Nov 20 '24 04:11 supperthomas

好的,更改了 同时移除了nxp的库,改为使用者自行手动添加,添加方法在 readme 中给出

为啥要移除?

可以只放编译需要的文件

supperthomas avatar Nov 20 '24 04:11 supperthomas

NXP这里的生成的代码是有许可证问题,考虑到这个是不是最好不要放上来?那么ci就只能做检查,不能编译

同时目前这个版本可能要再改动一下,做到更好的适配

ppppppsssss avatar Nov 21 '24 15:11 ppppppsssss

NXP这里的生成的代码是有许可证问题,考虑到这个是不是最好不要放上来?那么ci就只能做检查,不能编译

同时目前这个版本可能要再改动一下,做到更好的适配

可以考虑制作软件包的方式,参考nrf5x/nrf52840

supperthomas avatar Nov 22 '24 01:11 supperthomas

请问作者这个 pr 还有在推进吗

Rbb666 avatar May 21 '25 01:05 Rbb666