On Tue, Feb 22, 2022 at 11:26:10PM +0530, Krishna Yarlagadda wrote: > Add ACPI ID for Tegra QUAD SPI. Switch to common device property calls. > Skip clock calls that are not updated in ACPI boot. > @@ -1377,6 +1400,8 @@ static int __maybe_unused tegra_qspi_runtime_suspend(struct device *dev) > struct spi_master *master = dev_get_drvdata(dev); > struct tegra_qspi *tqspi = spi_master_get_devdata(master); > > + if (has_acpi_companion(tqspi->dev)) > + return 0; > /* flush all write which are in PPSB queue by reading back */ > tegra_qspi_readl(tqspi, QSPI_COMMAND1); As well as clock stuff this is also skipping flushing of pending writes - is that intentional? It's not called out in the changelog and seems like something that could cause issues if someone runs on a system where the firmware does implement runtime suspend.
Attachment:
signature.asc
Description: PGP signature