linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [RFC/PATCH] ARM: dove: Remove UBI support from defconfig
@ 2014-01-27 13:26 Ezequiel Garcia
  2014-01-27 15:00 ` Jason Cooper
  2014-02-05  6:25 ` Jason Cooper
  0 siblings, 2 replies; 5+ messages in thread
From: Ezequiel Garcia @ 2014-01-27 13:26 UTC (permalink / raw)
  To: linux-arm-kernel

As NAND support is not enabled by default, it's hard to see
why we'd want to have UBI support. Let's remove it.

Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
---
 arch/arm/configs/dove_defconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/configs/dove_defconfig b/arch/arm/configs/dove_defconfig
index 1101054..7242b11 100644
--- a/arch/arm/configs/dove_defconfig
+++ b/arch/arm/configs/dove_defconfig
@@ -48,7 +48,6 @@ CONFIG_MTD_CFI_INTELEXT=y
 CONFIG_MTD_CFI_STAA=y
 CONFIG_MTD_PHYSMAP=y
 CONFIG_MTD_M25P80=y
-CONFIG_MTD_UBI=y
 CONFIG_BLK_DEV_LOOP=y
 CONFIG_BLK_DEV_RAM=y
 CONFIG_BLK_DEV_RAM_COUNT=1
-- 
1.8.1.5

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

* [RFC/PATCH] ARM: dove: Remove UBI support from defconfig
  2014-01-27 13:26 [RFC/PATCH] ARM: dove: Remove UBI support from defconfig Ezequiel Garcia
@ 2014-01-27 15:00 ` Jason Cooper
  2014-01-27 15:14   ` Ezequiel Garcia
  2014-02-05  6:25 ` Jason Cooper
  1 sibling, 1 reply; 5+ messages in thread
From: Jason Cooper @ 2014-01-27 15:00 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jan 27, 2014 at 10:26:29AM -0300, Ezequiel Garcia wrote:
> As NAND support is not enabled by default, it's hard to see
> why we'd want to have UBI support. Let's remove it.

I'd rather add support for the nand.

thx,

Jason.

> 
> Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
> ---
>  arch/arm/configs/dove_defconfig | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm/configs/dove_defconfig b/arch/arm/configs/dove_defconfig
> index 1101054..7242b11 100644
> --- a/arch/arm/configs/dove_defconfig
> +++ b/arch/arm/configs/dove_defconfig
> @@ -48,7 +48,6 @@ CONFIG_MTD_CFI_INTELEXT=y
>  CONFIG_MTD_CFI_STAA=y
>  CONFIG_MTD_PHYSMAP=y
>  CONFIG_MTD_M25P80=y
> -CONFIG_MTD_UBI=y
>  CONFIG_BLK_DEV_LOOP=y
>  CONFIG_BLK_DEV_RAM=y
>  CONFIG_BLK_DEV_RAM_COUNT=1
> -- 
> 1.8.1.5
> 

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

* [RFC/PATCH] ARM: dove: Remove UBI support from defconfig
  2014-01-27 15:00 ` Jason Cooper
@ 2014-01-27 15:14   ` Ezequiel Garcia
  2014-01-27 15:19     ` Jason Cooper
  0 siblings, 1 reply; 5+ messages in thread
From: Ezequiel Garcia @ 2014-01-27 15:14 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jan 27, 2014 at 10:00:40AM -0500, Jason Cooper wrote:
> On Mon, Jan 27, 2014 at 10:26:29AM -0300, Ezequiel Garcia wrote:
> > As NAND support is not enabled by default, it's hard to see
> > why we'd want to have UBI support. Let's remove it.
> 
> I'd rather add support for the nand.
> 

Well, there isn't any Dove board (that we currently support) with a NAND
device. Would you still want to add it?

In that case, we should consider adding UBIFS as well. Having CONFIG_UBI
by itself doesn't make any sense.
-- 
Ezequiel Garc?a, Free Electrons
Embedded Linux, Kernel and Android Engineering
http://free-electrons.com

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

* [RFC/PATCH] ARM: dove: Remove UBI support from defconfig
  2014-01-27 15:14   ` Ezequiel Garcia
@ 2014-01-27 15:19     ` Jason Cooper
  0 siblings, 0 replies; 5+ messages in thread
From: Jason Cooper @ 2014-01-27 15:19 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jan 27, 2014 at 12:14:30PM -0300, Ezequiel Garcia wrote:
> On Mon, Jan 27, 2014 at 10:00:40AM -0500, Jason Cooper wrote:
> > On Mon, Jan 27, 2014 at 10:26:29AM -0300, Ezequiel Garcia wrote:
> > > As NAND support is not enabled by default, it's hard to see
> > > why we'd want to have UBI support. Let's remove it.
> > 
> > I'd rather add support for the nand.
> > 
> 
> Well, there isn't any Dove board (that we currently support) with a NAND
> device. Would you still want to add it?

sh*t.  you're right.  My memory is failing me.  I'll queue this up.

thx,

Jason.

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

* [RFC/PATCH] ARM: dove: Remove UBI support from defconfig
  2014-01-27 13:26 [RFC/PATCH] ARM: dove: Remove UBI support from defconfig Ezequiel Garcia
  2014-01-27 15:00 ` Jason Cooper
@ 2014-02-05  6:25 ` Jason Cooper
  1 sibling, 0 replies; 5+ messages in thread
From: Jason Cooper @ 2014-02-05  6:25 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Jan 27, 2014 at 10:26:29AM -0300, Ezequiel Garcia wrote:
> As NAND support is not enabled by default, it's hard to see
> why we'd want to have UBI support. Let's remove it.
> 
> Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com>
> ---
>  arch/arm/configs/dove_defconfig | 1 -
>  1 file changed, 1 deletion(-)

Added to mvebu/defconfig

thx,

Jason.

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

end of thread, other threads:[~2014-02-05  6:25 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-01-27 13:26 [RFC/PATCH] ARM: dove: Remove UBI support from defconfig Ezequiel Garcia
2014-01-27 15:00 ` Jason Cooper
2014-01-27 15:14   ` Ezequiel Garcia
2014-01-27 15:19     ` Jason Cooper
2014-02-05  6:25 ` Jason Cooper

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