NXP FRDM-MCXN947测评:提交了关于ETH设备的PR

目录

概述

提交PR的过程

操作上的变化


概述

前一篇博文中已经提到,RT-Thread的官方驱动中的有点小缺陷,我们通过修改SConscript文件解决了这个问题。为了彻底解决这个问题方便大家使用,我想把这个解决方案作为PR提交给官方,让他们将修改融入RT-Thread的代码中。最终经过努力,在官方人员的指导下,我完善了解决方案,并最终并加入到官方代码中。

提交PR的过程

RT-Thread有一个运作得非常成功的社区,一直鼓励大家提交PR来改进和完善软件的。有关PR提交的步骤,在其社区有一个详细的说明:RT-Thread-向RT-Thread 提交Pull Request(PR)RT-Thread问答社区 - RT-Thread。他们还专门制作了一个B站视频:从0教你如何给RT-Thread贡献代码_哔哩哔哩_bilibili

我也按照其要求,向其Github仓库提交了PR:Update SConscript by zealsoftstudio · Pull Request #8705 · RT-Thread/rt-thread (github.com)

很快,管理员就对这个PR做出了评论。他指出我的解决方案不太符合RT-Thread的习惯。他认为应该使用BSP_USING_ETH 而不是RT_USING_PHY,并建议我参考一下stm32f407-atk-explorer开发板。根据管理员的建议,我修改了board目录下的Kconfig文件,在其中添加了如下代码:

    config BSP_USING_ETH
        bool "Enable Ethernet"
        default n
        select RT_USING_LWIP
        select RT_USING_NETDEV
        select RT_USING_SAL

然后,再次提交了修改代码。

当天晚上,管理员就批准了提交,将修改融合进了主线代码。

操作上的变化

由于最终的修改和最初的想法有所不同,所以如果用户是从最新的代码开始NXP FRDM-MCXN947的开发,有一点操作上的变化和上一篇博文不同。就是在menuconfig中,不是在在RT-Thread Components -> Device Drivers中选择Using ethernet phy device drivers,而是在Hardware Drivers Config->On-chip Peripheral Drivers中选择Enable Ethernet。

总体来说,这次PR的提交非常顺利。 

  • 3
    点赞
  • 10
    收藏
    觉得还不错? 一键收藏
  • 打赏
    打赏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包

打赏作者

神一样的老师

你的鼓励将是我创作的最大动力

¥1 ¥2 ¥4 ¥6 ¥10 ¥20
扫码支付:¥1
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值