ASoC: SOF: Intel: hda-dai: Print the format_val as hexadecimal number
authorPeter Ujfalusi <peter.ujfalusi@linux.intel.com>
Thu, 6 Apr 2023 15:52:19 +0000 (10:52 -0500)
committerMark Brown <broonie@kernel.org>
Thu, 6 Apr 2023 17:04:24 +0000 (18:04 +0100)
The format_val is a set of bitfileds, printing it as a decimal just makes
interpreting it complicated.

In other HDA core code the format_val is printed as hexadecimal also.

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Reviewed-by: Jaska Uimonen <jaska.uimonen@linux.intel.com>
Reviewed-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Link: https://lore.kernel.org/r/20230406155219.18997-1-pierre-louis.bossart@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sof/intel/hda-dai.c

index 0435b7f..44a5d94 100644 (file)
@@ -177,8 +177,8 @@ static int hda_link_dma_hw_params(struct snd_pcm_substream *substream,
        format_val = snd_hdac_calc_stream_format(params_rate(params), params_channels(params),
                                                 params_format(params), link_bps, 0);
 
-       dev_dbg(bus->dev, "format_val=%d, rate=%d, ch=%d, format=%d\n",
-               format_val, params_rate(params), params_channels(params), params_format(params));
+       dev_dbg(bus->dev, "format_val=%#x, rate=%d, ch=%d, format=%d\n", format_val,
+               params_rate(params), params_channels(params), params_format(params));
 
        if (ops->setup_hext_stream)
                ops->setup_hext_stream(sdev, hext_stream, format_val);