Commit 8ede848b authored Sep 18, 2021 by Linus Torvalds Committed by Greg Kroah-Hartman Sep 30, 2021

spi: Fix tegra20 build with CONFIG_PM=n



[ Upstream commit efafec27c5658ed987e720130772f8933c685e87 ]

Without CONFIG_PM enabled, the SET_RUNTIME_PM_OPS() macro ends up being
empty, and the only use of tegra_slink_runtime_{resume,suspend} goes
away, resulting in

  drivers/spi/spi-tegra20-slink.c:1200:12: error: ‘tegra_slink_runtime_resume’ defined but not used [-Werror=unused-function]
   1200 | static int tegra_slink_runtime_resume(struct device *dev)
        |            ^~~~~~~~~~~~~~~~~~~~~~~~~~
  drivers/spi/spi-tegra20-slink.c:1188:12: error: ‘tegra_slink_runtime_suspend’ defined but not used [-Werror=unused-function]
   1188 | static int tegra_slink_runtime_suspend(struct device *dev)
        |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~

mark the functions __maybe_unused to make the build happy.

This hits the alpha allmodconfig build (and others).

Reported-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent d193f7db

drivers/spi/spi-tegra20-slink.c

+2 −2

Original line number	Diff line number	Diff line
		@@ -1205,7 +1205,7 @@ static int tegra_slink_resume(struct device *dev)
		}
		#endif

		static int tegra_slink_runtime_suspend(struct device *dev)
		static int __maybe_unused tegra_slink_runtime_suspend(struct device *dev)
		{
		struct spi_master *master = dev_get_drvdata(dev);
		struct tegra_slink_data *tspi = spi_master_get_devdata(master);
		@@ -1217,7 +1217,7 @@ static int tegra_slink_runtime_suspend(struct device *dev)
		return 0;
		}

		static int tegra_slink_runtime_resume(struct device *dev)
		static int __maybe_unused tegra_slink_runtime_resume(struct device *dev)
		{
		struct spi_master *master = dev_get_drvdata(dev);
		struct tegra_slink_data *tspi = spi_master_get_devdata(master);