diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2020-06-03 21:22:46 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2020-06-03 21:22:46 -0700 |
commit | 6929f71e46bdddbf1c4d67c2728648176c67c555 (patch) | |
tree | b5ebf13e62f7b1de87bff7b65b0c7ac9b81b44f9 /arch/x86/kvm/debugfs.c | |
parent | a98f670e41a99f53acb1fb33cee9c6abbb2e6f23 (diff) | |
download | lwn-6929f71e46bdddbf1c4d67c2728648176c67c555.tar.gz lwn-6929f71e46bdddbf1c4d67c2728648176c67c555.zip |
atomisp: avoid warning about unused function
The atomisp_mrfld_power() function isn't actually ever called, because
the two call-sites have commented out the use because it breaks on some
platforms. That results in:
drivers/staging/media/atomisp/pci/atomisp_v4l2.c:764:12: warning: ‘atomisp_mrfld_power’ defined but not used [-Wunused-function]
764 | static int atomisp_mrfld_power(struct atomisp_device *isp, bool enable)
| ^~~~~~~~~~~~~~~~~~~
during the build.
Rather than commenting out the use entirely, just disable it
semantically instead (using a "0 &&" construct), leaving the call in
place from a syntax standpoint, and avoiding the warning.
I really don't want my builds to have any warnings that can then hide
real issues.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'arch/x86/kvm/debugfs.c')
0 files changed, 0 insertions, 0 deletions