On 3/31/26 14:10, kernel test robot wrote: > Hi Cássio, > > kernel test robot noticed the following build errors: > > [auto build test ERROR on 46a6512f4a74dd7b18d9a455669c226843fc49ce] > > url: > https://github.com/intel-lab-lkp/linux/commits/C-ssio-Gabriel/ALSA-aoa-i2sbus-clear-stale-prepared-state/20260331-113724 > base: 46a6512f4a74dd7b18d9a455669c226843fc49ce > patch link: > https://lore.kernel.org/r/20260330-aoa-i2sbus-clear-stale-active-v1-1-47a6c0a3ac9e%40gmail.com > patch subject: [PATCH] ALSA: aoa: i2sbus: clear stale prepared state > config: powerpc-allmodconfig > (https://download.01.org/0day-ci/archive/20260401/[email protected]/config) > compiler: powerpc64-linux-gcc (GCC) 15.2.0 > reproduce (this is a W=1 build): > (https://download.01.org/0day-ci/archive/20260401/[email protected]/reproduce) > > If you fix the issue in a separate patch/commit (i.e. not just a new version > of > the same patch/commit), kindly add following tags > | Reported-by: kernel test robot <[email protected]> > | Closes: > https://lore.kernel.org/oe-kbuild-all/[email protected]/ > > All errors (new ones prefixed by >>): > >>> sound/aoa/soundbus/i2sbus/pcm.c:760:25: error: initialization of 'int >>> (*)(struct snd_pcm_substream *, struct snd_pcm_hw_params *)' from >>> incompatible pointer type 'int (*)(struct snd_pcm_substream *)' >>> [-Wincompatible-pointer-types] > 760 | .hw_params = i2sbus_playback_hw_params, > | ^~~~~~~~~~~~~~~~~~~~~~~~~ > sound/aoa/soundbus/i2sbus/pcm.c:760:25: note: (near initialization for > 'i2sbus_playback_ops.hw_params') > sound/aoa/soundbus/i2sbus/pcm.c:313:12: note: 'i2sbus_playback_hw_params' > declared here > 313 | static int i2sbus_playback_hw_params(struct snd_pcm_substream > *substream) > | ^~~~~~~~~~~~~~~~~~~~~~~~~ > sound/aoa/soundbus/i2sbus/pcm.c:829:25: error: initialization of 'int > (*)(struct snd_pcm_substream *, struct snd_pcm_hw_params *)' from > incompatible pointer type 'int (*)(struct snd_pcm_substream *)' > [-Wincompatible-pointer-types] > 829 | .hw_params = i2sbus_record_hw_params, > | ^~~~~~~~~~~~~~~~~~~~~~~ > sound/aoa/soundbus/i2sbus/pcm.c:829:25: note: (near initialization for > 'i2sbus_record_ops.hw_params') > sound/aoa/soundbus/i2sbus/pcm.c:323:12: note: 'i2sbus_record_hw_params' > declared here > 323 | static int i2sbus_record_hw_params(struct snd_pcm_substream > *substream) > | ^~~~~~~~~~~~~~~~~~~~~~~ > > > vim +760 sound/aoa/soundbus/i2sbus/pcm.c > > 756 > 757 static const struct snd_pcm_ops i2sbus_playback_ops = { > 758 .open = i2sbus_playback_open, > 759 .close = i2sbus_playback_close, > > 760 .hw_params = i2sbus_playback_hw_params, > 761 .hw_free = i2sbus_playback_hw_free, > 762 .prepare = i2sbus_playback_prepare, > 763 .trigger = i2sbus_playback_trigger, > 764 .pointer = i2sbus_playback_pointer, > 765 }; > 766 > Oops my bad!
I had added i2sbus_playback_hw_params(), i2sbus_record_hw_params(), and the helper they use without the struct snd_pcm_hw_params *params argument expected by struct snd_pcm_ops.hw_params, which caused the compilation error. After fixing the callback signatures to match the ALSA API, the file builds cleanly. I will send a v2 patch with the proposed fixes. -- Thanks, Cássio
