Skip to content

logger: Describe runtime logger trace filtration usage - #248

Merged
deb-intel merged 1 commit into
thesofproject:masterfrom
ktrzcinx:runtime-filter
Sep 22, 2020
Merged

logger: Describe runtime logger trace filtration usage#248
deb-intel merged 1 commit into
thesofproject:masterfrom
ktrzcinx:runtime-filter

Conversation

@ktrzcinx

@ktrzcinxktrzcinx commented Jun 4, 2020

Copy link
Copy Markdown
Member

Each mechanism in logger tool chould be described in sof-dox.

Signed-off-by: Karol Trzcinski karolx.trzcinski@linux.intel.com

Reference changes from thesofproject/sof#2965

@ktrzcinxktrzcinx added the enhancement New feature or request label Jun 4, 2020
Comment threaddeveloper_guides/debugability/logger/index.rst
Comment threaddeveloper_guides/debugability/logger/index.rst Outdated
@ktrzcinx
ktrzcinxforce-pushed the runtime-filter branch 2 times, most recently from 02a61a7 to d8cec4eCompareSeptember 18, 2020 06:50

@lgirdwoodlgirdwood left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@deb-intel fyi - just small technical updates needed.

Comment threaddeveloper_guides/debugability/logger/index.rst Outdated
Comment threaddeveloper_guides/debugability/logger/index.rst
Each mechanism in logger tool chould be described in sof-dox.
Signed-off-by: Karol Trzcinski <karolx.trzcinski@linux.intel.com>
@deb-intel

Copy link
Copy Markdown
Collaborator

@lgirdwood I accidentally sent a request for a new review from you (sorry). Is this PR ready to be merged?

@lgirdwood

Copy link
Copy Markdown
Member

@deb-intel yes, it's ready. thanks !

@deb-inteldeb-intel left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Will make grammatical updates in a separate PR.

@deb-intel
deb-intel merged commit 39f78e5 into thesofproject:masterSep 22, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancementNew feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@ktrzcinx@deb-intel@lgirdwood@dbaluta