Skip to content

SoundWire: intel_ace2x: read DOAIS and DODS from _DSD properties - #4915

Merged
plbossart merged 4 commits into
thesofproject:topic/sof-devfrom
plbossart:sdw/phy
Apr 12, 2024
Merged

plbossart merged 4 commits into
thesofproject:topic/sof-devfrom
plbossart:sdw/phy

Conversation

@plbossart

Copy link
Copy Markdown
Member

This is a change that's needed for newer platforms.

@plbossart

Copy link
Copy Markdown
Member Author

tested on LNL_SDW_AOIC, the logs show the firmware value (0x3) is used for DOAIS instead of the hard-coded 0x1.

[    5.667124] snd_intel_sdw_acpi:sdw_intel_scan_controller: acpi PRP00001:00: ACPI reports 4 SDW Link devices
[    5.667137] soundwire_intel:sdw_intel_probe_controller: acpi PRP00001:00: Creating 4 SDW Link devices
[    5.667257] soundwire_intel:sdw_master_read_intel_prop: soundwire sdw-master-0-0: doais 0x3 dods 0x1
[    5.667270] soundwire_bus:sdw_extract_slave_id: soundwire sdw-master-0-0: SDW Slave Addr: 30025d071101
[    5.667271] soundwire_bus:sdw_extract_slave_id: soundwire sdw-master-0-0: SDW Slave class_id 0x01, mfg_id 0x025d, part_id 0x0711, unique_id 0x0, version 0x3
[    5.667606] soundwire_intel:sdw_master_read_intel_prop: soundwire sdw-master-0-1: doais 0x3 dods 0x1
[    5.667618] soundwire_bus:sdw_extract_slave_id: soundwire sdw-master-0-1: SDW Slave Addr: 130025d071401
[    5.667619] soundwire_bus:sdw_extract_slave_id: soundwire sdw-master-0-1: SDW Slave class_id 0x01, mfg_id 0x025d, part_id 0x0714, unique_id 0x0, version 0x3
[    5.669595] soundwire_intel:sdw_master_read_intel_prop: soundwire sdw-master-0-2: doais 0x3 dods 0x1
[    5.669608] soundwire_bus:sdw_extract_slave_id: soundwire sdw-master-0-2: SDW Slave Addr: 230025d131601
[    5.669609] soundwire_bus:sdw_extract_slave_id: soundwire sdw-master-0-2: SDW Slave class_id 0x01, mfg_id 0x025d, part_id 0x1316, unique_id 0x0, version 0x3
[    5.669853] soundwire_intel:sdw_master_read_intel_prop: soundwire sdw-master-0-3: doais 0x3 dods 0x1

@plbossart
plbossart marked this pull request as ready for review April 10, 2024 17:05
@plbossart
plbossart requested review from bardliao and ujfalusi April 10, 2024 17:05
Use two variables to save the settings, in preparation of a follow-up
change to read values from _DSD properties.

Starting with this patch, the bitfields will be reordered and listed
MSB-first, as shown in the hardware documentation.

Also note that the default for DOAIS is changed from 0x1 (copy-pasted
value?) to 0x3 (hardware default).

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Starting with LNL, the recommendation is to use settings read from DSD
properties instead of hard-coding the values.

The DOAIS and DODS values are completely-specific to Intel and are
stored in a vendor-specific property structure.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Extend previous patches with the DODSE field and property.

Signed-off-by Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Extend previous patches with the DOAISE field and property.

Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
@plbossart

Copy link
Copy Markdown
Member Author

@ujfalusi @ranj063 could I bother you to take a look, 2+ reviewers is better than just one.

@plbossart
plbossart requested a review from ranj063 April 11, 2024 20:18
@plbossart
plbossart merged commit 545b4ef into thesofproject:topic/sof-dev Apr 12, 2024
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