浏览代码

[ymodem] 修改YMODEM_USING_FILE_TRANSFER功能的开启条件
目前开启的逻辑存在问题,ymodem的文件传输功能开启的前提是得有完整的文件系统,
但是目前,这个开启逻辑错了。
用户选择YMODEM_USING_FILE_TRANSFER功能后select DFS没有任何意义,因为除了需要依赖DFS之外还要依赖其他的例如FATFS,需要和SDIO适配等等等
正确的做法应该是用户配置好完整的文件系统后,ymodem文件传输才能有机会开启,
因此是depends on DFS 而非select DFS
更改之后,只要用户开启了文件系统,在选定YMODEM时,会自动开启文件传输功能,这也是用户希望看到的,直接一键搞定,不需要再多点一步

Meco Man 4 年之前
父节点
当前提交
4aaf52ae9e
共有 2 个文件被更改,包括 3 次插入3 次删除
  1. 2 2
      components/utilities/Kconfig
  2. 1 1
      components/utilities/ymodem/SConscript

+ 2 - 2
components/utilities/Kconfig

@@ -11,8 +11,8 @@ config RT_USING_RYM
 
 
         config YMODEM_USING_FILE_TRANSFER
         config YMODEM_USING_FILE_TRANSFER
         bool "Enable file transfer feature"
         bool "Enable file transfer feature"
-        select RT_USING_DFS
-        default n
+        depends on RT_USING_DFS
+        default y
     endif
     endif
 
 
 config RT_USING_ULOG
 config RT_USING_ULOG

+ 1 - 1
components/utilities/ymodem/SConscript

@@ -7,7 +7,7 @@ ymodem.c
  
  
 CPPPATH = [cwd]
 CPPPATH = [cwd]
 
 
-if GetDepend('RT_USING_DFS') and GetDepend('YMODEM_USING_FILE_TRANSFER'):
+if GetDepend('YMODEM_USING_FILE_TRANSFER'):
     src += ['ry_sy.c']
     src += ['ry_sy.c']
 
 
 group   = DefineGroup('Utilities', src, depend = ['RT_USING_RYM'], CPPPATH = CPPPATH)
 group   = DefineGroup('Utilities', src, depend = ['RT_USING_RYM'], CPPPATH = CPPPATH)