amd/acp7x: smart_amp: enable Maxim DSM for 2x max98388 on TDM2

[LongWang100]

Summary

Bring up the Maxim DSM smart amplifier path on ACP7x (Steam Deck /
Gainsborough) with two max98388 amplifiers on a 2-slot TDM2 link.

This patch fixes a series of bugs discovered during hardware bring-up
and wires up the full topology pipeline (playback → smart_amp →
ref-capture → DSM feedback).

Changes

smart_amp.c

• IPC3 source buffer fix: bound_module is only populated by
  module_bind() in module_adapter_ipc4.c and is NULL on IPC3
  platforms; dereferencing it caused a DSP fault. Switch to comparing
  producer pipeline IDs, which works under both IPC versions.
• 16-byte DSP buffer alignment: allocate via mod_alloc_align()
  to satisfy Xtensa HiFi AE_L32X2/AE_S64 alignment requirements;
  without this the DSP raises LoadStoreAlignmentCause.
• Null-guard source_buf after re-prepare to prevent stale pointer
  reuse.

maxim_dsm_stub.c

• Fix stub output fields: the upstream stub returned DSM_API_OK
  without writing omemsizerequestedbytes, so get_handle_size() returned
  0, the PRIVATE memblk was never allocated, dsmhandle stayed NULL, and
  every subsequent DSM call failed with -EINVAL.
• The fixed stub also passes audio through (input → output) so the glue
  layer and topology can be validated without libdsm.a.

smart_amp_maxim_dsm.c

• Fix DSM_FF_BUF_DB_SZ: upstream used FF_BUF_SZ * FF_MAX_CH_NUM
  = 192, too small for a 2 ms period (192 samples) + pre-fill frame (96).
  Change to FF_BUF_SZ * 4 = 384 to cover periods up to 3 ms.
• Add ivFormat=3 (DSM_IV_FORMAT_INTERLEAVED_16_BIT_MSB) support:
  each max98388 interleaves V and I within one TDM slot using the sample
  MSB as a flag; the upstream 4*idx stride read only one MSB polarity and
  returned INSUFFICIENT_INPUT_DATA.
• Replace memcpy_s with dsm_buf_shift_down() (memmove-based):
  SOF's memcpy_s rejects overlapping src/dst with -EINVAL, which
  occurs whenever period > 2 ms and caused silent buffer desynchronisation.
• Change if to while in ff_proc/fb_proc: a 2 ms period
  delivers 2 DSM frames; the previous if drained only one per call,
  causing w_ptr to grow until overflow.
• Remove duplicate dsm_api_ff_process() call in the 32-bit else
  branch: libdsm decrements *ipNrSamples in-place; a second call would
  double-advance the adaptive state (excursion/thermal model, RDC, LMS).
• Set ibsamples = fb_fr_sz_samples * nchannels = 96: libdsm divides
  by nChannels internally; passing 48 gave 24 % ffFrameSize != 0 and
  returned INVALID_PARAM on every call.

sof-acp_7_x_i2s.m4

• Switch TDM2 pipeline from passthrough to smart-amplifier +
  amp-ref-capture.
• Define all required SMART_* macros for 2-channel configuration.
• Wire cross-pipeline SectionGraph: demux → smart_amp feedback buffer.
• Set REF_CHMAP = {0x01, 0x02} so the echo reference second channel
  comes from slot 1 (second amp), not the non-existent slot 2.
• Declare SMART_UUID explicitly to make the active build mode visible.

pipe-amp-ref-capture.m4

• Allow platforms to pre-define REF_CHMAP to override the default
  4-slot mapping, which routes ch1 to in ch2 (non-existent on 2-slot
  hardware) and permanently silences ch1 via mux.c:215.

app/maxim_dsm_overlay.conf

• Add Kconfig overlay documenting the three-step DSM bring-up sequence:
  • Step 1: PASSTHRU_AMP — verify audio path without DSM
  • Step 2: MAXIM_DSM_STUB — validate glue layer and topology
  • Step 3: real libdsm.a — full DSM protection

Test plan

• Step 1 (PASSTHRU_AMP): stereo playback on ACP7x, no DSP fault
• Step 2 (MAXIM_DSM_STUB): DSM path exercised, DSM_API_OK returned,
  dsmhandle non-NULL, audio passes through
• Step 3 (real libdsm): Rdc/excursion monitoring active, V/I sense
  correct on both channels, no INSUFFICIENT_INPUT_DATA / INVALID_PARAM
• Re-prepare path (suspend/resume): no stale pointer crash
• Period > 2 ms: buffer pointers stable, no overflow

🤖 Generated with Claude Code

[sofci]

Can one of the admins verify this patch?

reply test this please to run this test once