Skip to content

Conversation

@cfriedt
Copy link
Member

@cfriedt cfriedt commented Sep 24, 2022

This is mainly to reduce clutter in subsys/logging, but also to make backend management slightly easier.

@cfriedt
Copy link
Member Author

cfriedt commented Sep 24, 2022

  • ordered backends/CMakelists.txt alphabetically too

@cfriedt
Copy link
Member Author

cfriedt commented Sep 24, 2022

  • fix compliance issues

nashif
nashif previously approved these changes Sep 24, 2022
Copy link
Member

@nashif nashif left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

makes sense.

teburd
teburd previously requested changes Sep 24, 2022
Copy link
Contributor

@teburd teburd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good but missed the mtrace Kconfig file split out

@cfriedt
Copy link
Member Author

cfriedt commented Sep 25, 2022

  • created Kconfig.adsp_mtrace

This is mainly to reduce clutter in `subsys/logging`, but also to make
backend management slightly easier.

Signed-off-by: Christopher Friedt <[email protected]>
Copy link
Contributor

@nordic-krch nordic-krch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks good.

@fabiobaltieri fabiobaltieri added this to the v3.3.0 milestone Sep 26, 2022
@cfriedt cfriedt merged commit f558755 into zephyrproject-rtos:main Oct 1, 2022
@cfriedt cfriedt deleted the logging-reorg branch January 24, 2023 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants