On 9/9/24 21:29, Marek Vasut wrote: > The cmd53_buf is a 4 byte buffer, embed it into the struct wilc_sdio. Is this change really desirable ? I mean, looking at git log, it looks like this kzalloc is voluntary, for usage with DMA (see [1] as initial fix and [2] as the final patch which has landed in the driver) [1] https://patchwork.kernel.org/project/linux-wireless/patch/20220728152037.386543-1-michael@xxxxxxxx/ [2] https://lore.kernel.org/all/20220809075749.62752-1-ajay.kathat@xxxxxxxxxxxxx/ Alexis -- Alexis Lothoré, Bootlin Embedded Linux and Kernel engineering https://bootlin.com