Skip to content

ASoC: SOF: ipc4-topology: Print for format type in dbg_audio_format#5693

Open
ujfalusi wants to merge 1 commit intothesofproject:topic/sof-devfrom
ujfalusi:peter/sof/pr/ipc4-type-thing-fix-02
Open

ASoC: SOF: ipc4-topology: Print for format type in dbg_audio_format#5693
ujfalusi wants to merge 1 commit intothesofproject:topic/sof-devfrom
ujfalusi:peter/sof/pr/ipc4-type-thing-fix-02

Conversation

@ujfalusi
Copy link
Collaborator

@ujfalusi ujfalusi commented Mar 13, 2026

8 and 32 bit formats can have different types, print them in debug information to have complete view of the supported formats.

Fixes: c04c2e8 ("ASoC: SOF: ipc4-topology: Add support for 8-bit formats")
Cc: stable@vger.kernel.org

@ujfalusi ujfalusi requested a review from singalsu March 13, 2026 07:30
@ujfalusi ujfalusi force-pushed the peter/sof/pr/ipc4-type-thing-fix-02 branch from 7e78e08 to 5ab0997 Compare March 13, 2026 07:39
@ujfalusi ujfalusi changed the title fixup! ASoC: SOF: ipc4-topology: Add support for 8-bit formats (#2) ASoC: SOF: ipc4-topology: Print for format type in dbg_audio_format Mar 13, 2026

static u32 sof_ipc4_fmt_cfg_to_type(u32 fmt_cfg)
{
/* Fetch the sample type from the fmt for 8 and 32 bit formats */
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nit pick: two spaces between "Fetch" and "the"

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

it was in the original patch which added it, I'm moving it up to be usable in other places, I can fix the comment, let me do that.

bardliao
bardliao previously approved these changes Mar 18, 2026
@ujfalusi ujfalusi force-pushed the peter/sof/pr/ipc4-type-thing-fix-02 branch from 5ab0997 to ad4b870 Compare March 18, 2026 08:45
8 and 32 bit formats can have different types, print them in debug
information to have complete view of the supported formats.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
@ujfalusi
Copy link
Collaborator Author

Changes since v1:

  • Correct the moved comment section in it's new place.

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.

2 participants