Recent changes addressed PAGE_SIZE ambiguity in 2/3 locations for struct avs_icl_memwnd2. The unaddressed one causes build errors when PAGE_SIZE != SZ_4K.
Reported-by: kernel test robot lkp@intel.com Closes: https://lore.kernel.org/oe-kbuild-all/202404070100.i3t3Jf7d-lkp@intel.com/ Fixes: 275b583d047a ("ASoC: Intel: avs: ICL-based platforms support") Signed-off-by: Cezary Rojewski cezary.rojewski@intel.com ---
Follow up to the recent series for the avs-driver [1]. With static_asserts() added [2], build problems occur when PAGE_SIZE != SZ_4K. Patch fixing memory window description [3] fixed only 2/3 offending spots.
[1]: https://lore.kernel.org/alsa-devel/20240405090929.1184068-1-cezary.rojewski@... [2]: https://lore.kernel.org/alsa-devel/20240405090929.1184068-13-cezary.rojewski... [3]: https://lore.kernel.org/alsa-devel/20240405090929.1184068-3-cezary.rojewski@...
sound/soc/intel/avs/icl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/sound/soc/intel/avs/icl.c b/sound/soc/intel/avs/icl.c index e8b4983e03e9..7a1ecf241856 100644 --- a/sound/soc/intel/avs/icl.c +++ b/sound/soc/intel/avs/icl.c @@ -66,7 +66,7 @@ static_assert(sizeof(struct avs_icl_memwnd2_desc) == 12); struct avs_icl_memwnd2 { union { struct avs_icl_memwnd2_desc slot_desc[AVS_ICL_MEMWND2_SLOTS_COUNT]; - u8 rsvd[PAGE_SIZE]; + u8 rsvd[SZ_4K]; }; u8 slot_array[AVS_ICL_MEMWND2_SLOTS_COUNT][SZ_4K]; } __packed;
base-commit: 1f58d8580e5da64669d923ced99cb3e29832273b