]> www.infradead.org Git - users/jedix/linux-maple.git/commit
ASoC: SOF: Intel: hda-stream: Always use at least two BDLE for transfers
authorPeter Ujfalusi <peter.ujfalusi@linux.intel.com>
Thu, 7 Nov 2024 12:15:32 +0000 (14:15 +0200)
committerMark Brown <broonie@kernel.org>
Thu, 7 Nov 2024 13:03:36 +0000 (13:03 +0000)
commit1b1f491dac4f6ee88ec7c36c4fd27880b0a89f41
tree3ad3b716fddd33bd48c656d2b8b6477460ba1b44
parent8121de33460d0f1441e75b6e52dae9317efd0656
ASoC: SOF: Intel: hda-stream: Always use at least two BDLE for transfers

The HDA specification states that the SDnLVI (Last Valid Index) must be at
least 1 (two BDLE entry).

Update the debug prints as well to provide better information.

While the LVI=0 worked so far without issues, it is better to align with
the specification to avoid unforeseen issues with FW loading.

Notes:
- The LVI > 0 rules is valid and honored for audio use cases
- LVI == 0 is used with software controlled (SPIB) transfers only for
  firmware and library loading, which is permitted by the hardware
- This is not spelled out in the specification and it is better to avoid
  cases

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Reviewed-by: Liam Girdwood <liam.r.girdwood@intel.com>
Reviewed-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Link: https://patch.msgid.link/20241107121532.3241-1-peter.ujfalusi@linux.intel.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/sof/intel/hda-stream.c