linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Tom Rini <trini@kernel.crashing.org>
To: Dan Malek <dan@embeddededge.com>
Cc: Embedded PPC Linux list <linuxppc-embedded@ozlabs.org>
Subject: Re: cleaning up the Kconfig menu structure -- the bigger picture
Date: Fri, 8 Oct 2004 12:48:07 -0700	[thread overview]
Message-ID: <20041008194807.GG14773@smtp.west.cox.net> (raw)
In-Reply-To: <7079964C-1959-11D9-9299-003065F9B7DC@embeddededge.com>

On Fri, Oct 08, 2004 at 02:39:51PM -0400, Dan Malek wrote:
> 
> On Oct 8, 2004, at 2:02 PM, Tom Rini wrote:
> 
> >But that's what I just described.  Letting the special config options
> >(I don't know if I'd call 85xx_PCI2 "I/O", but it is special) live in
> >platforms/8xx/Kconfig (or ppc/Kconfig.cpm1, or whatever).
> 
> You are conveniently ignoring the 4xx serial port configuration
> options that live in the platforms/4xx/Kconfig right now.  Taking
> debating lessons from dubyah, I guess :-)

No, just that I know Matt wants to see the uart0/uart1 part die and I
bet SERIAL_SICC would be rewritten and moved it was maintained. :)

> If you need change for the sake of change or just because you
> want to disagree with me, then there isn't anything I can do to
> convince you otherwise.  There is precedent from previous
> releases, in the current release with other processors, and from
> extensive experience that you should make you consider my
> suggestions.  I can understand moving the sources to better
> assist the development of code that interacts with these drivers,
> but scattering the unique and interdependent configuration
> options around isn't helpful.

I don't get it.  We do agree that the very specific stuff does indeed
belong somewhere that's obvious to the user, right?  We do agree that
for cpm2, cpm2-specific and not driver related stuff belongs in
platforms/85xx/Kconfig, right?  Are we disagreeing that the non-driver
portions of 8xx_io (and 82xx_io) should just live in syslib/ with
similar functionality bits of other machine types?  And that drivers
should live in drivers/ and asked with other drivers?

> The non-ppc folks updating the
> generic Kconfig files are going to be totally confused by that.

They haven't been confused by all of the other architectures stuff.  In
fact, they've helped clean things up too.

-- 
Tom Rini
http://gate.crashing.org/~trini/

  reply	other threads:[~2004-10-08 19:48 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-10-08 11:16 cleaning up the Kconfig menu structure -- the bigger picture Robert P. J. Day
2004-10-08 15:31 ` Dan Malek
2004-10-08 15:38   ` Robert P. J. Day
2004-10-08 16:37     ` Dan Malek
2004-10-08 16:31       ` Robert P. J. Day
2004-10-08 16:40 ` Tom Rini
2004-10-08 16:42   ` Robert P. J. Day
2004-10-08 17:22   ` Dan Malek
2004-10-08 18:02     ` Tom Rini
2004-10-08 18:39       ` Dan Malek
2004-10-08 19:48         ` Tom Rini [this message]
2004-10-08 20:49           ` Dan Malek
2004-10-08 21:38             ` Tom Rini
2004-10-08 22:03             ` Wolfgang Denk

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=20041008194807.GG14773@smtp.west.cox.net \
    --to=trini@kernel.crashing.org \
    --cc=dan@embeddededge.com \
    --cc=linuxppc-embedded@ozlabs.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;
as well as URLs for NNTP newsgroup(s).