ASoC: SOF: Intel: hda: Only check SSP MCLK mask in case of IPC3

IPC4 is using the NHLT blob itself and sends the SSP blob from it directly
to the firmware, there is no need for the MCLK quirk based on the SSP blob
since the SSP blob is in use.

At the same time reword the error, info and debug messages for clarity.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Reviewed-by: Balamurugan C <balamurugan.c@intel.com>
Link: https://patch.msgid.link/20251215130819.31218-1-peter.ujfalusi@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/sof/intel/hda.c b/sound/soc/sof/intel/hda.c
index c1518dbee1b7a96c9db165a00022ed599b766dc7..72c98154c027e59a01f0b441da60c21681a9d523 100644 (file)
--- a/sound/soc/sof/intel/hda.c
+++ b/sound/soc/sof/intel/hda.c
@@ -1611,7 +1611,6 @@ struct snd_soc_acpi_mach *hda_machine_select(struct snd_sof_dev *sdev)
                    mach->mach_params.i2s_link_mask) {
                        const struct sof_intel_dsp_desc *chip = get_chip_info(sdev->pdata);
                        int ssp_num;
-                       int mclk_mask;
 
                        if (hweight_long(mach->mach_params.i2s_link_mask) > 1 &&
                            !(mach->tplg_quirk_mask & SND_SOC_ACPI_TPLG_INTEL_SSP_MSB))
@@ -1636,19 +1635,28 @@ struct snd_soc_acpi_mach *hda_machine_select(struct snd_sof_dev *sdev)
 
                        sof_pdata->tplg_filename = tplg_filename;
 
-                       mclk_mask = check_nhlt_ssp_mclk_mask(sdev, ssp_num);
-
-                       if (mclk_mask < 0) {
-                               dev_err(sdev->dev, "Invalid MCLK configuration\n");
-                               return NULL;
-                       }
-
-                       dev_dbg(sdev->dev, "MCLK mask %#x found in NHLT\n", mclk_mask);
-
-                       if (mclk_mask) {
-                               dev_info(sdev->dev, "Overriding topology with MCLK mask %#x from NHLT\n", mclk_mask);
-                               sdev->mclk_id_override = true;
-                               sdev->mclk_id_quirk = (mclk_mask & BIT(0)) ? 0 : 1;
+                       if (sof_pdata->ipc_type == SOF_IPC_TYPE_3) {
+                               int mclk_mask = check_nhlt_ssp_mclk_mask(sdev,
+                                                                        ssp_num);
+
+                               if (mclk_mask < 0) {
+                                       dev_err(sdev->dev,
+                                               "Invalid MCLK configuration for SSP%d\n",
+                                               ssp_num);
+                                       return NULL;
+                               }
+
+                               if (mclk_mask) {
+                                       sdev->mclk_id_override = true;
+                                       sdev->mclk_id_quirk = (mclk_mask & BIT(0)) ? 0 : 1;
+                                       dev_info(sdev->dev,
+                                                "SSP%d to use MCLK id %d (mask: %#x)\n",
+                                                ssp_num, sdev->mclk_id_quirk, mclk_mask);
+                               } else {
+                                       dev_dbg(sdev->dev,
+                                               "MCLK mask is empty for SSP%d in NHLT\n",
+                                               ssp_num);
+                               }
                        }
                }