On Tue, May 28, 2024, at 13:57, Arnd Bergmann wrote: > > for (i = 0; i < fragment->fragment_cnt && offset < ish_fw->size; i++) > { > dma_bufs[i] = dma_alloc_coherent(dev->devc, fragment_size, > &dma_addr, GFP_KERNEL); > + dma_bufs[i] = dma_alloc_coherent(dev->devc, fragment_size, > + &dma, GFP_KERNEL); > if (!dma_bufs[i]) > return -ENOMEM; > Sorry about this one, the duplicate linen was an incorrect rebase. I've fixed this one locally but did not resend the series since I'm still waiting for other review comments. Arnd