LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH 6/8] mtd: replace CONFIG_8xx by CONFIG_PPC_8xx
@ 2015-03-12 15:24 Christophe Leroy
  2015-03-31  0:55 ` Brian Norris
  0 siblings, 1 reply; 2+ messages in thread
From: Christophe Leroy @ 2015-03-12 15:24 UTC (permalink / raw)
  To: David Woodhouse, Brian Norris
  Cc: linux-kernel, linuxppc-dev, user-mode-linux-devel,
	user-mode-linux-user, linux-fbdev, netdev, linux-serial,
	linux-mtd

Two config options exist to define powerpc MPC8xx:
* CONFIG_PPC_8xx
* CONFIG_8xx
In addition, CONFIG_PPC_8xx also defines CONFIG_CPM1 as
communication co-processor

arch/powerpc/platforms/Kconfig.cputype has contained the following
comment about CONFIG_8xx item for some years:
"# this is temp to handle compat with arch=ppc"

It looks like not many places still have that old CONFIG_8xx used,
so it is likely to be a good time to get rid of it completely ?

Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>

---
 drivers/mtd/maps/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/maps/Kconfig b/drivers/mtd/maps/Kconfig
index ba801d2..e715ae9 100644
--- a/drivers/mtd/maps/Kconfig
+++ b/drivers/mtd/maps/Kconfig
@@ -242,7 +242,7 @@ config MTD_L440GX
 
 config MTD_CFI_FLAGADM
 	tristate "CFI Flash device mapping on FlagaDM"
-	depends on 8xx && MTD_CFI
+	depends on PPC_8xx && MTD_CFI
 	help
 	  Mapping for the Flaga digital module. If you don't have one, ignore
 	  this setting.
-- 
2.1.0


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH 6/8] mtd: replace CONFIG_8xx by CONFIG_PPC_8xx
  2015-03-12 15:24 [PATCH 6/8] mtd: replace CONFIG_8xx by CONFIG_PPC_8xx Christophe Leroy
@ 2015-03-31  0:55 ` Brian Norris
  0 siblings, 0 replies; 2+ messages in thread
From: Brian Norris @ 2015-03-31  0:55 UTC (permalink / raw)
  To: Christophe Leroy
  Cc: David Woodhouse, linux-kernel, linuxppc-dev,
	user-mode-linux-devel, user-mode-linux-user, linux-fbdev, netdev,
	linux-serial, linux-mtd

On Thu, Mar 12, 2015 at 04:24:04PM +0100, Christophe Leroy wrote:
> Two config options exist to define powerpc MPC8xx:
> * CONFIG_PPC_8xx
> * CONFIG_8xx
> In addition, CONFIG_PPC_8xx also defines CONFIG_CPM1 as
> communication co-processor
> 
> arch/powerpc/platforms/Kconfig.cputype has contained the following
> comment about CONFIG_8xx item for some years:
> "# this is temp to handle compat with arch=ppc"
> 
> It looks like not many places still have that old CONFIG_8xx used,
> so it is likely to be a good time to get rid of it completely ?
> 
> Signed-off-by: Christophe Leroy <christophe.leroy@c-s.fr>

I'm not sure if the consistency aspect is worked out for all symbols
(Geert's comment on the cover letter), but this one looks fine.

Applied this patch to l2-mtd.git.

Brian

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-03-31  0:55 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-03-12 15:24 [PATCH 6/8] mtd: replace CONFIG_8xx by CONFIG_PPC_8xx Christophe Leroy
2015-03-31  0:55 ` Brian Norris

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).