common: pcm: hdmi/sof-hda-dsp/sof-soundwire: Syntax 7 dependent macro to generate the hdmi device sections #413
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…evice
The new HdmiDevice macro can be used to create the hdmi PCM device for a
hardware PCM whose ID can be found in the Iec61937Devices comma separated
device ID list.
It is the caller's responsibility to pass correct device id of an HDMI PCM
as Dev.
The new macro needs Syntax 7 support.
Example for a use by sof-hda-dsp where the HDMI devices are 3, 4 and 5
Macro.0.HdmiDevice { Iec61937Devices "${var:Iec61937Pcms1}" Dev 3 Idx 0 }
Macro.1.HdmiDevice { Iec61937Devices "${var:Iec61937Pcms1}" Dev 4 Idx 1 }
Macro.2.HdmiDevice { Iec61937Devices "${var:Iec61937Pcms1}" Dev 5 Idx 2 }
If.HdmiIec61937 {
Condition {
Type RegexMatch
Regex "((^|,)[345](,|$))"
String "${var:Iec61937Pcms1}"
}
True.Macro.save_hdmi_cfg.HdmiPCMSave { Name "42-sof-hdmi" }
}
The Iec61937Pcms1 contains a match string of "5,4,3"
Signed-off-by: Peter Ujfalusi <[email protected]>
…ation The duplicated code for the hdmi device creation can be removed by using the HdmiDevice macro. The macro needs Syntax 7 to work (variable as parameter support), update the Syntax version of the file as well. Signed-off-by: Peter Ujfalusi <[email protected]>
The duplicated code for the hdmi device creation can be removed by using the HdmiDevice macro. The macro needs Syntax 7 to work (variable as parameter support), update the Syntax version of the file as well. Signed-off-by: Peter Ujfalusi <[email protected]>
|
It looks fine. Let's merge this when alsa-lib is released. |
perexg
changed the title
perexg
pushed a commit
that referenced
this pull request
Aug 2, 2024
The duplicated code for the hdmi device creation can be removed by using the HdmiDevice macro. The macro needs Syntax 7 to work (variable as parameter support), update the Syntax version of the file as well. Closes: #413 Signed-off-by: Peter Ujfalusi <[email protected]> Signed-off-by: Jaroslav Kysela <[email protected]>
perexg
pushed a commit
that referenced
this pull request
Aug 2, 2024
The duplicated code for the hdmi device creation can be removed by using the HdmiDevice macro. The macro needs Syntax 7 to work (variable as parameter support), update the Syntax version of the file as well. Closes: #413 Signed-off-by: Peter Ujfalusi <[email protected]> Signed-off-by: Jaroslav Kysela <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi,
The PR will introduce a generic
HdmiDevicemacro, which can replace duplicated conditions in sof UCM files and potentially used by other devices to generate the needed section for the hdmi PCM devices to allow passthrough.The PR is marked as DNM and is in draft since the change depends on Syntax 7 support, which is only available in git version of alsa-lib, it is not a simple ask for a user (or developer) to use it and the change would break UCM for released alsa-lib.
@perexg, is this something that you were suggesting? I think the macro and it's definition is generic enough, I hope that it will be usable for non SOF setups.