Skip to content

stdio: Remove CONFIG_EOL_IS_XXX#8628

Merged
xiaoxiang781216 merged 1 commit intoapache:masterfrom
no1wudi:master
Feb 23, 2023
Merged

stdio: Remove CONFIG_EOL_IS_XXX#8628
xiaoxiang781216 merged 1 commit intoapache:masterfrom
no1wudi:master

Conversation

@no1wudi
Copy link
Contributor

@no1wudi no1wudi commented Feb 23, 2023

Summary

Remove CONFIG_EOL_IS_XXX relative logic since serial driver will do the convertion.

To pass CI, apache/nuttx-apps#1592 need to be merged first.

Impact

Shoule be none.

Testing

CI

Signed-off-by: Huang Qi <huangqi3@xiaomi.com>
@no1wudi no1wudi closed this Feb 23, 2023
@no1wudi
Copy link
Contributor Author

no1wudi commented Feb 23, 2023

Retrigger CI

@no1wudi no1wudi reopened this Feb 23, 2023
@xiaoxiang781216 xiaoxiang781216 merged commit ca61c40 into apache:master Feb 23, 2023
no1wudi added a commit to no1wudi/nuttx that referenced this pull request Feb 27, 2023
NuttX only treat \n as new line after apache#8628,
so need this conversion to interact with terminal emulator.

Signed-off-by: Huang Qi <huangqi3@xiaomi.com>
xiaoxiang781216 pushed a commit that referenced this pull request Feb 27, 2023
NuttX only treat \n as new line after #8628,
so need this conversion to interact with terminal emulator.

Signed-off-by: Huang Qi <huangqi3@xiaomi.com>
zyfeier pushed a commit to zyfeier/incubator-nuttx that referenced this pull request Feb 28, 2023
NuttX only treat \n as new line after apache#8628,
so need this conversion to interact with terminal emulator.

Signed-off-by: Huang Qi <huangqi3@xiaomi.com>
anchao pushed a commit to anchao/nuttx that referenced this pull request Mar 6, 2023
VELAPLATFO-6182

NuttX only treat \n as new line after apache#8628,
so need this conversion to interact with terminal emulator.

Change-Id: Ib7ad97ede41178a111db1356a1c2d0e77991934c
Signed-off-by: Huang Qi <huangqi3@xiaomi.com>
yamt added a commit to yamt/wasm-micro-runtime that referenced this pull request Jul 31, 2024
wenyongh pushed a commit to bytecodealliance/wasm-micro-runtime that referenced this pull request Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants