diff options
author | Linus Torvalds | 2021-05-09 14:03:33 -0700 |
---|---|---|
committer | Linus Torvalds | 2021-05-09 14:03:33 -0700 |
commit | 6dae40aed484ef2f1a3934dcdcd17b7055173e56 (patch) | |
tree | 00c6a6ef14a63d2d0a0a6b91ae2ba899fbd544e9 /README | |
parent | efc58a96adcd29cc37487a60582d9d08b34f6640 (diff) |
fbmem: fix horribly incorrect placement of __maybe_unused
Commit b9d79e4ca4ff ("fbmem: Mark proc_fb_seq_ops as __maybe_unused")
places the '__maybe_unused' in an entirely incorrect location between
the "struct" keyword and the structure name.
It's a wonder that gcc accepts that silently, but clang quite reasonably
warns about it:
drivers/video/fbdev/core/fbmem.c:736:21: warning: attribute declaration must precede definition [-Wignored-attributes]
static const struct __maybe_unused seq_operations proc_fb_seq_ops = {
^
Fix it.
Cc: Guenter Roeck <linux@roeck-us.net>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'README')
0 files changed, 0 insertions, 0 deletions