]> git.baikalelectronics.ru Git - kernel.git/commit
spi: Fix tegra20 build with CONFIG_PM=n once again
authorLinus Torvalds <torvalds@linux-foundation.org>
Mon, 25 Oct 2021 17:46:41 +0000 (10:46 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Mon, 25 Oct 2021 17:46:41 +0000 (10:46 -0700)
commit8aff507d47985dcc2cb0069fc5ec911faa7f45ac
tree3e6e722dfd2893d8903092884986fa19cbebcdfe
parentd665250fc3b1b7981c383a41acaa64d0dfc6140a
spi: Fix tegra20 build with CONFIG_PM=n once again

Commit 37c021b21645 ("spi: Fix tegra20 build with CONFIG_PM=n") already
fixed the build without PM support once.  There was an alternative fix
by Guenter in commit 66168ea3f4ba ("spi: tegra20-slink: Declare runtime
suspend and resume functions conditionally"), and Mark then merged the
two correctly in d3e47025c1b7 ("Merge tag 'v5.15-rc2' into spi-5.15").

But for some inexplicable reason, Mark then merged things _again_ in
commit d2ecf03e3aa8 ("Merge tag 'v5.15-rc3' into spi-5.15"), and screwed
things up at that point, and the __maybe_unused attribute on
tegra_slink_runtime_resume() went missing.

Reinstate it, so that alpha (and other architectures without PM support)
builds cleanly again.

Btw, this is another prime example of how random back-merges are not
good.  Just don't do them.  Subsystem developers should not merge my
tree in any normal circumstances.  Both of those merge commits pointed
to above are bad: even the one that got the merge result right doesn't
even mention _why_ it was done, and the one that got it wrong is
obviously broken.

Reported-by: Guenter Roeck <linux@roeck-us.net>
Cc: Mark Brown <broonie@kernel.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/spi/spi-tegra20-slink.c