117 Star 460 Fork 323

OpenHarmony / kernel_liteos_m

 / 详情

shell中包含了uart.h

Done
Task
Opened this issue  
2021-11-29 19:32

【任务描述】
shell的shellmsg.c中包含了uart.h
【解决方案】

【任务来源】
解读整个逻辑发现:
1,需要device设备端创建一个uart.h文件
2,uart.h文件需要实现UartGetC,其他的非必须

整个看起来我们需要依赖很多外部依赖一样,为啥不直接在shellmsg.c中封装一个api,用于和外界通信,允许注册推送数据的接口?和外界彻底解耦?

Comments (3)

金永生 created任务

LosShellInit的调用也不在LOS_KernelInit中也让人很费解,还需要手动调用才行。

g_shellInputEvent这个作为全局变量和device共享感觉不是很合理,把kernel部分的变量直接暴露给外面;触发的事件是0x01,也缺少统一定义,不太符合代码可移植性;应该封装一个接口给device用,比方PostShellInputEvent().

@金永生 欢迎积极贡献,如果有兴趣,可以提交自己的实现代码,与社区进行互动交流。 :facepunch: :facepunch:

贡献代码的流程如下:

https://gitee.com/openharmony/docs/blob/master/zh-cn/contribute/%E5%8F%82%E4%B8%8E%E8%B4%A1%E7%8C%AE.md

如果您感兴趣想自己实现,请让我们知道。谢谢

@SimonLi @zhushengle 龙哥,看看如果需要修改,可以说下方案的意见。

kenneth set assignee to SimonLi
kenneth changed issue state from 待办的 to 进行中
LeonChan changed issue state from 进行中 to 已完成

Sign in to comment

Status
Assignees
Projects
Milestones
Pull Requests
Successfully merging a pull request will close this issue.
Branches
Planed to start   -   Planed to end
-
Top level
Priority
Duration (hours)
参与者(2)
C
1
https://git.oschina.net/openharmony/kernel_liteos_m.git
git@git.oschina.net:openharmony/kernel_liteos_m.git
openharmony
kernel_liteos_m
kernel_liteos_m

Search