From: Christian Dietrich <qy03fugy@stud.informatik.uni-erlangen.de>
To: Tejun Heo <tj@kernel.org>, Jiri Kosina <jkosina@suse.cz>,
Daniel Mack <daniel@caiaq.de>,
Joseph Chan <JosephChan@via.com.tw>,
Florian Tobias Schandinat <FlorianSchandinat@gmx.de>,
Andrew Morton <akpm@linux-foundation.org>,
Jonathan Corbet <corbet@lwn.net>,
Erik-Jan Post <ej.lfs@xs4all.nl>,
linux-kernel@vger.kernel.org, linux-fbdev@vger.kernel.org
Cc: vamos-dev@i4.informatik.uni-erlangen.de
Subject: [PATCH 0/2] Removing dead code
Date: Mon, 23 Aug 2010 09:59:33 +0000 [thread overview]
Message-ID: <cover.1282554255.git.qy03fugy@stud.informatik.uni-erlangen.de> (raw)
Hi all!
As part of the VAMOS[0] research project at the University of
Erlangen we are looking at multiple integrity errors in linux'
configuration system.
I've been running a check on the drivers/video sourcetree for
config Items not defined in Kconfig and found 2 such cases. Sourcecode
blocks depending on these Items are not reachable from a vanilla
kernel -- dead code. I've seen such dead blocks made on purpose
e.g. while integrating new features into the kernel but generally
they're just useless.
Each of the patches in this patchset removes on such dead
config Item, I'd be glad if you consider applying them. I've been
doing deeper analysis of such issues before and can do so again but
I'm not so sure they were fastly usefull.
I build the patches against a vanilla kernel in order to
try if the kernel compiles with this patches
Please keep me informed of this patch getting confirmed /
merged so we can keep track of it.
Regards
Christian Dietrich
[0] http://vamos1.informatik.uni-erlangen.de/
Christian Dietrich (2):
drivers/video: Removing dead FB_VIA_CAMERA config option
drivers/video: Removing dead FB_68328_INVERT config option
drivers/video/68328fb.c | 4 ----
drivers/video/via/accel.c | 16 ----------------
include/linux/via-core.h | 5 +----
3 files changed, 1 insertions(+), 24 deletions(-)
next reply other threads:[~2010-08-23 9:59 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-08-23 9:59 Christian Dietrich [this message]
2010-08-23 9:59 ` [PATCH 1/2] drivers/video: Removing dead FB_VIA_CAMERA config Christian Dietrich
2010-08-23 12:42 ` Jonathan Corbet
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=cover.1282554255.git.qy03fugy@stud.informatik.uni-erlangen.de \
--to=qy03fugy@stud.informatik.uni-erlangen.de \
--cc=FlorianSchandinat@gmx.de \
--cc=JosephChan@via.com.tw \
--cc=akpm@linux-foundation.org \
--cc=corbet@lwn.net \
--cc=daniel@caiaq.de \
--cc=ej.lfs@xs4all.nl \
--cc=jkosina@suse.cz \
--cc=linux-fbdev@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=tj@kernel.org \
--cc=vamos-dev@i4.informatik.uni-erlangen.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
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;
as well as URLs for NNTP newsgroup(s).