mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Brad Hards <bhards@bigpond.net.au>
To: Tom Rini <trini@kernel.crashing.org>
Cc: lkml <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH/RFC 2.4.19-rc1] Fix dependancies on keybdev.o
Date: Thu, 27 Jun 2002 06:58:04 +1000	[thread overview]
Message-ID: <200206270658.04695.bhards@bigpond.net.au> (raw)
In-Reply-To: <20020626144609.GR3489@opus.bloom.county>

On Thu, 27 Jun 2002 00:46, Tom Rini wrote:
> On Wed, Jun 26, 2002 at 12:36:24PM +1000, Brad Hards wrote:
> > 2. Move keyboard handling code to input subsystem
>
> I think that will work out the best.  How's the attached look?  It moves
> drivers/input/Config.in inside of drivers/char/Config.in and then fixes
> arches which had both.  (Lightly tested from xconfig[1] for all arches
> which got changed).
I'm opposed to further junk going into drivers/char. It is already an 
incomprehensible mess of unrelated code.

> > 3. Do wholesale backport of input subsystem from 2.5
>
> Not really an option (and 2.5 has this problem anyhow) since it's so
> invasive, once it's done.
Concur with the invasiveness angle.

Actually, I've got another idea, based on some stuff that I've been working on 
for the ACPI "its not just power management" issue. 

If you need to set something in drivers/input (per your original patch) that 
depends on things that are set in drivers/char (or drivers/usb, anything that 
comes later), then split the Config.in into two sections. One section 
contains the normal Config.in user-selectable options, and a second 
drivers/input/Config-post.in, that is sourced in at the end of 
arch/foo/config.in and contains only automated Config.in dependancies (ie 
define_bool) but no user selectable options.

Does this make sense? If not, I'll try for a patch that shows it later this 
morning.

Brad
-- 
http://conf.linux.org.au. 22-25Jan2003. Perth, Australia. Birds in Black.

  reply	other threads:[~2002-06-26 21:01 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-06-25 16:06 Tom Rini
2002-06-26  2:36 ` Brad Hards
2002-06-26 14:46   ` Tom Rini
2002-06-26 20:58     ` Brad Hards [this message]
2002-06-26 21:23       ` Tom Rini
2002-06-26 23:43         ` Brad Hards
2002-06-26 23:54           ` Tom Rini

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=200206270658.04695.bhards@bigpond.net.au \
    --to=bhards@bigpond.net.au \
    --cc=linux-kernel@vger.kernel.org \
    --cc=trini@kernel.crashing.org \
    /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

all inboxes | Powered by JetHome®