Skip to content

Conversation

@ranj063
Copy link
Collaborator

@ranj063 ranj063 commented Sep 23, 2025

Add quotes around the valid values for direction to prevent compilation errors with the latest alsa-lib that fails with the error: ALSA lib conf.c:1224:(parse_value) id is not an integer ALSA lib conf.c:2014:(_snd_config_load_with_include) toplevel:224:17:Unexpected char

Add quotes around the valid values for direction to prevent compilation
errors with the latest alsa-lib that fails with the error:
ALSA lib conf.c:1224:(parse_value) id is not an integer
ALSA lib conf.c:2014:(_snd_config_load_with_include) _toplevel_:224:17:Unexpected char

Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
@ranj063 ranj063 requested a review from jsarha as a code owner September 23, 2025 21:03
Copilot AI review requested due to automatic review settings September 23, 2025 21:03
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

Fixes compilation errors with the latest alsa-lib by adding quotes around string values in the PCM configuration. The change addresses parsing errors where unquoted string identifiers were being treated as integers.

  • Adds quotes around the valid values for direction attribute in PCM configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Collaborator

@singalsu singalsu left a comment

Choose a reason for hiding this comment

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

Great, this would close #10246 !

Copy link
Collaborator

@softwarecki softwarecki left a comment

Choose a reason for hiding this comment

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

Looks cleaner :)

@kv2019i kv2019i merged commit 1495dd0 into thesofproject:main Sep 25, 2025
33 of 45 checks passed
@ranj063 ranj063 deleted the main-092325 branch September 25, 2025 16:43
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.

4 participants