LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: "Robert P. J. Day" <rpjday@crashcourse.ca>
To: Julia Lawall <julia@diku.dk>
Cc: khali@linux-fr.org, i2c@lm-sensors.org,
	linux-kernel@vger.kernel.org, kernel-janitors@vger.kernel.org
Subject: Re: [PATCH 1/4] drivers/i2c: Drop redundant includes of moduleparam.h
Date: Wed, 14 Nov 2007 04:21:51 -0500 (EST)	[thread overview]
Message-ID: <alpine.LFD.0.9999.0711140416160.4673@localhost.localdomain> (raw)
In-Reply-To: <Pine.LNX.4.64.0711140707120.25487@ask.diku.dk>

On Wed, 14 Nov 2007, Julia Lawall wrote:

> > > From: Julia Lawall <julia@diku.dk>
> > >
> > > Drop #include <linux/moduleparam.h> in files that also include #include
> > > <linux/module.h>.  module.h includes moduleparam.h already.
> >
> > i'm not convinced that's a good idea.  while module.h does
> > currently (and unfortunately) include moduleparam.h, there may
> > come a day when those header files are refactored to actually make
> > sense, at which point all those missing moduleparam.h inclusions
> > will cause all sorts of bad things to happen.
>
> They were removed from the drivers/media files in August:
> 9c12224a607a4b22ab86784e3394b52810b9507c
> Perhaps they should be put back there then?

i went down this road before a few months back, checking on how many
source files were unnecessarily including moduleparam.h, and the
result was literally *hundreds*.  i took a shot at cleaning that up,
and even factoring the non-parameter-related content out of that file,
but the change would have been hugely disruptive.

as it is now, since module.h includes moduleparam.h, there is (as you
can see) no reason whatever for *anyone* to include moduleparam.h.
at least right now.

but if anyone ever aspires to tidy up this chaos, then all you'd end
up doing is putting all those includes back in.  so unless there's a
long-term goal here, it would seem safest to just leave things the way
they are.  but that's someone else's decision, not mine.  i'm just
reporting on how quickly things got unpleasant when *i* tried to do
something with this once upon a time.

rday
--

========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry
Waterloo, Ontario, CANADA

http://crashcourse.ca
========================================================================

  reply	other threads:[~2007-11-14  9:25 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-11-13 21:32 Julia Lawall
2007-11-13 21:34 ` Robert P. J. Day
2007-11-14  6:09   ` Julia Lawall
2007-11-14  9:21     ` Robert P. J. Day [this message]
2007-11-15 12:19       ` Jean Delvare
2007-11-15 12:42         ` Robert P. J. Day

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=alpine.LFD.0.9999.0711140416160.4673@localhost.localdomain \
    --to=rpjday@crashcourse.ca \
    --cc=i2c@lm-sensors.org \
    --cc=julia@diku.dk \
    --cc=kernel-janitors@vger.kernel.org \
    --cc=khali@linux-fr.org \
    --cc=linux-kernel@vger.kernel.org \
    --subject='Re: [PATCH 1/4] drivers/i2c: Drop redundant includes of moduleparam.h' \
    /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

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).