All of lore.kernel.org
 help / color / mirror / Atom feed
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
To: Timur Tabi <timur@freescale.com>
Cc: linuxppc-dev@ozlabs.org
Subject: Re: [PATCH] powerpc: remove default=y from PMAC and CHRP Kconfigs
Date: Thu, 09 Oct 2008 07:03:06 +1100	[thread overview]
Message-ID: <1223496186.8157.82.camel@pasglop> (raw)
In-Reply-To: <1223495845-14149-1-git-send-email-timur@freescale.com>

On Wed, 2008-10-08 at 14:57 -0500, Timur Tabi wrote:
> Remove the "default y" from the Kconfig options for CHRP, PMAC, and PMAC64
> platforms.  This patch is a follow-up to "remove CHRP and PMAC support from
> defconfigs, fix Kconfigs", which was applied incompletely.

No. Instead, send a patch that fixes the defconfig's to explicitely set
those to "n". As to whether those should be defaults or not, this is a
different discussion (I'm almost tempted to have everything default to
y).

Ben.

> Signed-off-by: Timur Tabi <timur@freescale.com>
> ---
> 
> Ben, in the future, please apply either all of my patch or none of it.
> Applying half of my patch and "thinking about" the other half doesn't do me
> any good.
> 
>  arch/powerpc/platforms/chrp/Kconfig     |    1 -
>  arch/powerpc/platforms/powermac/Kconfig |    4 ----
>  2 files changed, 0 insertions(+), 5 deletions(-)
> 
> diff --git a/arch/powerpc/platforms/chrp/Kconfig b/arch/powerpc/platforms/chrp/Kconfig
> index 22b4b4e..682afbc 100644
> --- a/arch/powerpc/platforms/chrp/Kconfig
> +++ b/arch/powerpc/platforms/chrp/Kconfig
> @@ -9,4 +9,3 @@ config PPC_CHRP
>  	select PPC_UDBG_16550
>  	select PPC_NATIVE
>  	select PCI
> -	default y
> diff --git a/arch/powerpc/platforms/powermac/Kconfig b/arch/powerpc/platforms/powermac/Kconfig
> index 055990c..85619d3 100644
> --- a/arch/powerpc/platforms/powermac/Kconfig
> +++ b/arch/powerpc/platforms/powermac/Kconfig
> @@ -6,7 +6,6 @@ config PPC_PMAC
>  	select PPC_INDIRECT_PCI if PPC32
>  	select PPC_MPC106 if PPC32
>  	select PPC_NATIVE
> -	default y
>  
>  config PPC_PMAC64
>  	bool
> @@ -16,6 +15,3 @@ config PPC_PMAC64
>  	select MPIC_U3_HT_IRQS
>  	select GENERIC_TBSYNC
>  	select PPC_970_NAP
> -	default y
> -
> -

  reply	other threads:[~2008-10-08 20:07 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-08 19:57 [PATCH] powerpc: remove default=y from PMAC and CHRP Kconfigs Timur Tabi
2008-10-08 20:03 ` Benjamin Herrenschmidt [this message]
2008-10-08 20:14   ` Timur Tabi
2008-10-08 20:39     ` Benjamin Herrenschmidt
2008-10-08 20:44       ` Timur Tabi
2008-10-09 15:59         ` Matt Sealey
2008-10-09 16:20           ` Timur Tabi
2008-10-09 16:49             ` Matt Sealey
2008-10-09 17:08               ` Scott Wood
2008-10-09 17:23               ` Timur Tabi
2008-10-09 20:18                 ` Sven Luther
2008-10-09 21:14                   ` Timur Tabi
2008-10-09 21:28                     ` Sven Luther
2008-10-09 21:32                       ` Timur Tabi
2008-10-10  0:43             ` Paul Mackerras
2008-10-10  1:22               ` Kumar Gala
2008-10-10  1:51                 ` Timur Tabi
2008-10-10  3:01                   ` Benjamin Herrenschmidt
2008-10-10  3:01                 ` Benjamin Herrenschmidt
2008-10-09 16:21           ` Scott Wood
2008-10-08 20:46   ` Scott Wood

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=1223496186.8157.82.camel@pasglop \
    --to=benh@kernel.crashing.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=timur@freescale.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.