On Wed, Sep 21, 2022 at 01:37:49PM +0100, Mark Brown wrote: > On Wed, Sep 21, 2022 at 10:02:25AM +0800, Yang Yingliang wrote: > > On 2022/9/21 2:33, Mark Brown wrote: > > > If we're switching please update to the modern naming and use > > > "controller" rather than the old name. > > > Do you mean to use spi_controller instead of spi_master? Something like > > this: > > 'struct spi_controller * ctlr = devm_spi_alloc_master();' > > Or just use devm_spi_alloc_controller() directly. There's no such thing. The driver needs to explicitly allocate a master or slave and that will result in the slave bit being set correctly in struct spi_controller. Yang's v2 series now calls __devm_spi_alloc_controller() but drivers should never call that directly. Thanks, Lukas