Skip to content

Conversation

@ujfalusi
Copy link
Contributor

@ujfalusi ujfalusi commented Jul 9, 2025

The channel count 7 was somehow missed by the initial commit, which I don't think was on purpose.

The channel count 7 was somehow missed by the initial commit,
which I don't think was on purpose.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
in_valid_bit_depth 16
in_channels 7
in_ch_cfg $CHANNEL_CONFIG_5_POINT_1
in_ch_map $CHANNEL_MAP_5_POINT_1
Copy link
Contributor Author

Choose a reason for hiding this comment

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

these ch config should be fixed, it is not correct.

@ujfalusi
Copy link
Contributor Author

ujfalusi commented Jul 9, 2025

There is no valid channel map for 7 channels and we can only say that channels_min/max for supported channel count.
I don't know how to handle this, but the patch is not correct.
I will close this.

@ujfalusi ujfalusi closed this Jul 9, 2025
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.

1 participant