Re: [PATCH v7 2/2] spi: loongson: add bus driver for the loongson spi controller

From: Mark Brown
Date: Fri Apr 14 2023 - 14:32:40 EST


On Wed, Apr 12, 2023 at 12:51:52PM +0800, Yinbo Zhu wrote:

> +static int __maybe_unused loongson_spi_suspend(struct device *dev)
> +{
> + struct loongson_spi *loongson_spi;
> + struct spi_master *master;
> +
> + master = dev_get_drvdata(dev);
> + loongson_spi = spi_master_get_devdata(master);
> +
> + loongson_spi->spcr = loongson_spi_read_reg(loongson_spi, LOONGSON_SPI_SPCR_REG);

This is missing a call to spi_controller_suspend(), and similarly resume
is missing a call to spi_controller_resume(). Otherwise this looks
good.

Attachment: signature.asc
Description: PGP signature