From: Mauro Carvalho Chehab <mchehab@s-opensource•com>
To: Linus Torvalds <torvalds@linux-foundation•org>
Cc: Tejun Heo <tj@kernel•org>, Jean Delvare <jdelvare@suse•com>,
Guenter Roeck <linux@roeck-us•net>,
Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung•com>,
Sathya Prakash <sathya.prakash@broadcom•com>,
"James E.J. Bottomley" <jejb@linux•vnet.ibm.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation•org>,
"the arch/x86 maintainers" <x86@kernel•org>,
xen-devel <xen-devel@lists•xenproject.org>,
linux-block <linux-block@vger•kernel.org>,
Linux Media Mailing List <linux-media@vger•kernel.org>,
IDE-ML <linux-ide@vger•kernel.org>,
"linux-fbdev@vger•kernel.org" <linux-fbdev@vger•kernel.org>,
Network Development <netdev@vger•kernel.org>,
Alan Cox <alan@llwyncelyn•cymru>
Subject: Re: Lots of new warnings with gcc-7.1.1
Date: Wed, 12 Jul 2017 09:37:56 -0300 [thread overview]
Message-ID: <20170712093552.15aebacd@vento.lan> (raw)
In-Reply-To: <CA+55aFzXz-PxKSJP=hfHD+mfCX4M6+HMacWMkDz7KB8-3y55qw@mail.gmail.com>
Em Tue, 11 Jul 2017 15:35:15 -0700
Linus Torvalds <torvalds@linux-foundation•org> escreveu:
> [ Very random list of maintainers and mailing lists, at least
> partially by number of warnings generated by gcc-7.1.1 that is then
> correlated with the get_maintainers script ]
Under drivers/media, I fixed a bunch of gcc 7.1 warnings before the
merge window. While most were just noise, some actually pointed to
human errors.
Now, gcc-7.1.1 produces only 6 warnings with W=1 on x86_64 (allyesconfig),
either due to unused-but-set-variable or unused-const-variable. I guess
both warning options are disabled by default. Anyway, I have patches
to fix them already. I'll send you later.
The atomisp staging driver is a completely different beast, with would
produce itself a huge amount of warnings. I ended by adding some
logic on drivers/staging/media/atomisp/ Makefiles to disable them:
ccflags-y += $(call cc-disable-warning, missing-declarations)
ccflags-y += $(call cc-disable-warning, missing-prototypes)
ccflags-y += $(call cc-disable-warning, unused-but-set-variable)
ccflags-y += $(call cc-disable-warning, unused-const-variable)
ccflags-y += $(call cc-disable-warning, suggest-attribute=format)
ccflags-y += $(call cc-disable-warning, implicit-fallthrough)
(there's actually one patch pending related to atomisp, that I'll also
be sending you soon - meant to avoid warnings if compiled with an older
gcc version)
Thanks,
Mauro
next prev parent reply other threads:[~2017-07-12 12:37 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-07-11 22:35 Lots of new warnings with gcc-7.1.1 Linus Torvalds
2017-07-11 23:54 ` Marcel Holtmann
2017-07-12 3:10 ` Guenter Roeck
2017-07-12 3:17 ` Linus Torvalds
2017-07-12 3:41 ` Linus Torvalds
2017-07-12 13:31 ` Arnd Bergmann
2017-07-15 11:03 ` Tejun Heo
2017-07-12 4:19 ` Jakub Kicinski
2017-07-12 12:37 ` Mauro Carvalho Chehab [this message]
2017-07-12 13:10 ` Greg Kroah-Hartman
2017-07-12 13:51 ` Arnd Bergmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170712093552.15aebacd@vento.lan \
--to=mchehab@s-opensource$(echo .)com \
--cc=alan@llwyncelyn$(echo .)cymru \
--cc=b.zolnierkie@samsung$(echo .)com \
--cc=gregkh@linuxfoundation$(echo .)org \
--cc=jdelvare@suse$(echo .)com \
--cc=jejb@linux$(echo .)vnet.ibm.com \
--cc=linux-block@vger$(echo .)kernel.org \
--cc=linux-fbdev@vger$(echo .)kernel.org \
--cc=linux-ide@vger$(echo .)kernel.org \
--cc=linux-media@vger$(echo .)kernel.org \
--cc=linux@roeck-us$(echo .)net \
--cc=netdev@vger$(echo .)kernel.org \
--cc=sathya.prakash@broadcom$(echo .)com \
--cc=tj@kernel$(echo .)org \
--cc=torvalds@linux-foundation$(echo .)org \
--cc=x86@kernel$(echo .)org \
--cc=xen-devel@lists$(echo .)xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox