Revision ac8a6eba2a117e0fdc04da62ab568d1b7ca4c8f6 authored by Linus Torvalds on 25 October 2021, 17:46:41 UTC, committed by Linus Torvalds on 25 October 2021, 17:46:41 UTC
Commit efafec27c565 ("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 2bab94090b01 ("spi: tegra20-slink: Declare runtime
suspend and resume functions conditionally"), and Mark then merged the
two correctly in ffb1e76f4f32 ("Merge tag 'v5.15-rc2' into spi-5.15").

But for some inexplicable reason, Mark then merged things _again_ in
commit 59c4e190b10c ("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>
1 parent c2b4385
History
File Mode Size
Kconfig -rw-r--r-- 1.2 KB
Makefile -rw-r--r-- 222 bytes
vxfs.h -rw-r--r-- 9.6 KB
vxfs_bmap.c -rw-r--r-- 7.8 KB
vxfs_dir.h -rw-r--r-- 3.2 KB
vxfs_extern.h -rw-r--r-- 2.7 KB
vxfs_fshead.c -rw-r--r-- 5.2 KB
vxfs_fshead.h -rw-r--r-- 2.5 KB
vxfs_immed.c -rw-r--r-- 2.6 KB
vxfs_inode.c -rw-r--r-- 9.2 KB
vxfs_inode.h -rw-r--r-- 5.7 KB
vxfs_lookup.c -rw-r--r-- 7.2 KB
vxfs_olt.c -rw-r--r-- 3.8 KB
vxfs_olt.h -rw-r--r-- 4.3 KB
vxfs_subr.c -rw-r--r-- 4.5 KB
vxfs_super.c -rw-r--r-- 8.7 KB

back to top