From: Tom Rini <trini@kernel.crashing.org>
To: acurtis@onz.com
Cc: dan@embeddededge.com, linuxppc-embedded@lists.linuxppc.org
Subject: Re: 8260/8xx Embedded Boot patch
Date: Tue, 27 Aug 2002 17:36:12 -0700 [thread overview]
Message-ID: <20020828003612.GD22875@opus.bloom.county> (raw)
In-Reply-To: <20020827172829.3580.h002.c007.wm@mail.directvinternet.com.criticalpath.net>
On Tue, Aug 27, 2002 at 05:28:29PM -0700, acurtis@onz.com wrote:
>
> > The first thing I saw was the define_bool CONFIG_ALL_PPC n.
> > If this was actually needed someplace, there's a bug there that needs to
> > be fixed.
>
> This is not required. All of the xxxx8260_defconfig files define this so you do
> not get all of the generic Mac stuff like IDE and video. If it is not appropriate
> take it out. It just serves to weed out a lot of options you are just going to
> turn-off anyway.
They shouldn't show up on 8260, since they don't show up on other,
non-CONFIG_ALL_PPC platforms. Again, if they show up it's a bug. The
bug is doing:
dep_{tristate,bool} 'Some pmac-specific driver' CONFIG_PMACSPEC $CONFIG_ALL_PPC
It should be:
if [ "$CONFIG_ALL_PPC" = "y" ]; then
{bool,tristate} 'Some pmac-specific driver' CONFIG_PMACSPEC
fi
--
Tom Rini (TR1265)
http://gate.crashing.org/~trini/
** Sent via the linuxppc-embedded mail list. See http://lists.linuxppc.org/
next prev parent reply other threads:[~2002-08-28 0:36 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-08-28 0:06 8260/8xx Embedded Boot patch Allen Curtis
2002-08-28 0:23 ` Tom Rini
2002-08-28 0:28 ` acurtis
2002-08-28 0:36 ` Tom Rini [this message]
2002-08-28 4:03 ` Dan Malek
2002-08-28 4:16 ` Allen Curtis
2002-08-28 6:06 ` Dan Malek
-- strict thread matches above, loose matches on Subject: below --
2002-08-28 17:08 Curtis, Allen
2002-08-28 18:44 ` Dan Malek
2002-08-28 19:03 Curtis, Allen
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=20020828003612.GD22875@opus.bloom.county \
--to=trini@kernel.crashing.org \
--cc=acurtis@onz.com \
--cc=dan@embeddededge.com \
--cc=linuxppc-embedded@lists.linuxppc.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).