On Tue, Aug 24, 2021 at 11:40:33AM +0100, Lucas Tanure wrote: > + if (master->max_transfer_size) { > + bus = kmemdup(®map_spi, sizeof(*bus), GFP_KERNEL); We shouldn't be peering into the controller structure, use spi_max_transfer_size() instead. > + bus->max_raw_read = bus->max_raw_write = master->max_transfer_size(spi); Just write two assignment statements, it's more legible all round.