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/
next prev parent 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).