Skip to content

Conversation

@kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Jan 28, 2026

Add CONFIG_LOG_OUTPUT_FORMAT_LINUX_TIMESTAMP=y to the Linux overlay. This aligns the timestamp format between SOF firmware and host Linux kernel logs.

Add CONFIG_LOG_OUTPUT_FORMAT_LINUX_TIMESTAMP=y to the Linux
overlay. This aligns the timestamp format between SOF firmware
and host Linux kernel logs.

Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Copilot AI review requested due to automatic review settings January 28, 2026 10:36
@kv2019i kv2019i requested a review from dbaluta as a code owner January 28, 2026 10:36
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enables Linux-formatted timestamps in SOF firmware logs to align with host kernel log timestamps, improving log correlation and debugging.

Changes:

  • Enables CONFIG_LOG_OUTPUT_FORMAT_LINUX_TIMESTAMP in the Linux overlay configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

LHTM, Btw will there be a followup PR for GDB on LNL+

# SOF Linux driver does not require FW to retain its
# state, so context save can be disabled
CONFIG_ADSP_IMR_CONTEXT_SAVE=n
CONFIG_LOG_OUTPUT_FORMAT_LINUX_TIMESTAMP=y
Copy link
Member

Choose a reason for hiding this comment

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

I think we can enable this Kconfig option as a project default or an intel_adsp default. It is already enabled for CAVS25, MTL, LNL, PTL, and WCL boards, and is likely just missing for NVL, so there is no need to differentiate it in the Linux overlay.

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