23 Aug
2023
23 Aug
'23
5:49 p.m.
On Wed, Aug 23, 2023 at 10:21:13AM +0100, Charles Keepax wrote:
Commit ac950278b087 ("ASoC: add N cpus to M codecs dai link support") added an additional local params in __soc_pmc_hw_params, for the CPU side of the DAI. The snd_pcm_hw_params struct is pretty large (604 bytes) and keeping two local copies of it makes the stack frame for __soc_pcm_hw_params really large. As the two copies are only used sequentially combine these into a single local variable to shrink the stack frame.
Signed-off-by: Charles Keepax ckeepax@opensource.cirrus.com
Hmm... this might need a little more thought its not clear why this should change the frame size and it only seems to change the frame size on the ARM cross compiler I am using, not x86.
Thanks, Charles