From: Tom Rini <trini@kernel•crashing.org>
To: Michael Sokolov <msokolov@ivan•Harhan.ORG>
Cc: linux-galileo@source•mvista.com, linuxppc-dev@lists•linuxppc.org
Subject: Re: [PATCH] My GT-64260 enhancements
Date: Mon, 18 Mar 2002 08:00:49 -0700 [thread overview]
Message-ID: <20020318150049.GC3762@opus.bloom.county> (raw)
In-Reply-To: <0203170703.AA00587@ivan.Harhan.ORG>
On Sat, Mar 16, 2002 at 11:03:13PM -0800, Michael Sokolov wrote:
> Tom Rini <trini@kernel•crashing.org> wrote:
>
> > Well, (and I do need to check out the _galileo tree) iirc the stuff to
> > support that option is in one of the generic files. Or generic to the
> > work currently in there. If that's the case, and you're explicitly not
> > going to support it in your files (and ports) then add a -a
> > "$CONFIG_xxx" = "n" .
>
> You still don't get it. I find that logic offensive. It makes me not want to
> use the GT-64260 in my designs because your logic implies that if someone uses
> the GT-64260 s/he wants to do things your way.
Yes, it assumes that if you're going to make a GT-64260 based port and
get it into the main tree you're going to use the common files for said
chipset so that all of the boards can be said to have certain things
working and reduce the ammount of coding needed to be done by you.
Like Troy said in an earlier message, it's great that you've got all of
this work done, but if you want to get it into the main tree and not
just keep up your own fork of it, some of your code will have to be
modified. The current stuff has the feature that it can make use of
CONFIG_SERIAL_TEXT_DEBUG on all ports.
> Just because there is a generic file doesn't mean I'm obligated to use
> it in my ports.
Actually, unless you've got good and preferably technical objections to
it, you are. Once it gets into the main trees, it's not just your board
port, it's everyones board port. And duplication of code isn't
generally a good thing.
If you've done your GT-64260 board without duplicating most of the work
in gt64260_common.c, then we can add in tests for your board(s) or even
just define a CONFIG_STARMON and test for that.
--
Tom Rini (TR1265)
http://gate.crashing.org/~trini/
** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/
next prev parent reply other threads:[~2002-03-18 15:00 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-03-15 6:11 [PATCH] My GT-64260 enhancements Michael Sokolov
2002-03-15 17:04 ` Tom Rini
2002-03-16 8:21 ` Michael Sokolov
2002-03-16 15:15 ` Tom Rini
2002-03-17 7:03 ` Michael Sokolov
2002-03-17 17:51 ` Dan Malek
2002-03-17 20:24 ` David Monro
2002-03-18 15:00 ` Tom Rini [this message]
2002-03-18 15:53 ` [Linux-galileo] " Mark A. Greer
2002-03-18 18:48 ` Tom Rini
2002-03-15 20:05 ` [Linux-galileo] " Nye Liu
-- strict thread matches above, loose matches on Subject: below --
2002-03-17 18:16 Michael Sokolov
2002-03-17 18:42 ` Dan Malek
2002-03-17 20:10 ` Michael Sokolov
2002-03-18 14:54 ` Tom Rini
2002-03-20 0:46 Michael Sokolov
2002-03-19 22:55 ` Mark A. Greer
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=20020318150049.GC3762@opus.bloom.county \
--to=trini@kernel$(echo .)crashing.org \
--cc=linux-galileo@source$(echo .)mvista.com \
--cc=linuxppc-dev@lists$(echo .)linuxppc.org \
--cc=msokolov@ivan$(echo .)Harhan.ORG \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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