On Tue, Jan 18, 2022 at 12:09:28PM +0100, Linus Walleij wrote: > @@ -655,6 +653,9 @@ static int s3c64xx_spi_prepare_message(struct spi_master *master, > struct spi_device *spi = msg->spi; > struct s3c64xx_spi_csinfo *cs = spi->controller_data; > > + if (!cs) > + return 0; > + > /* Configure feedback delay */ > writel(cs->fb_delay & 0x3, sdd->regs + S3C64XX_SPI_FB_CLK); > If you do revise this that should probably just be a guard on configuring feedback delay rather than aborting out of the entire function since otherwise this isn't extensible if anything else gets added to the function.
Attachment:
signature.asc
Description: PGP signature
