All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arnd Bergmann <arnd@arndb.de>
To: "Gupta, Pekon" <pekon@ti.com>
Cc: "Mohammed, Afzal" <afzal@ti.com>,
	Tony Lindgren <tony@atomide.com>,
	Artem Bityutskiy <artem.bityutskiy@linux.intel.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
	Russell King <rmk+kernel@arm.linux.org.uk>,
	David Woodhouse <dwmw2@infradead.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 21/21] mtd: omap2: allow bulding as a module
Date: Mon, 29 Apr 2013 15:34:10 +0200	[thread overview]
Message-ID: <201304291534.10539.arnd@arndb.de> (raw)
In-Reply-To: <20980858CB6D3A4BAE95CA194937D5E73E9BCFAE@DBDE04.ent.ti.com>

On Wednesday 24 April 2013, Gupta, Pekon wrote:
> [Pekon]: 
> ELM module is required in for Hardware based ECC correction for 
> NAND devices. And this driver has a very small foot-print.
> 
> The only cases this drives would not be used are:
> (a) Using S/W based ECC scheme, which have vey high CPU utilization
> (b) Using single bit ECC scheme, which are becoming obsolete due to
>  increasing NAND densities.
> For most of the cases ELM module will be used with nand-driver. So
> there should be no harm in having this module as built-in, if not used
> in 10% of the use-cases.
> 
> Thus I think it's better to keep this module tied to GPMC module, 
> rather than independent control via KConfig.
> And user should just selects which ECC scheme he would like to
> use via DT, without worrying about KConfig options.

Sorry, I'm lost here and have no idea what you are actually suggesting
we do. Can you phrase that as a patch?

Unless you come up with a better solution, I would still like to queue
up my patch since it fixes an annoying bug and seems to have no downsides.
 
> I'm working in cleaning up omap2-nand driver to remove some 
> redundancies. So would like to know your feedback on same..

Sure, just Cc me on your patches.

	Arnd

WARNING: multiple messages have this Message-ID (diff)
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 21/21] mtd: omap2: allow bulding as a module
Date: Mon, 29 Apr 2013 15:34:10 +0200	[thread overview]
Message-ID: <201304291534.10539.arnd@arndb.de> (raw)
In-Reply-To: <20980858CB6D3A4BAE95CA194937D5E73E9BCFAE@DBDE04.ent.ti.com>

On Wednesday 24 April 2013, Gupta, Pekon wrote:
> [Pekon]: 
> ELM module is required in for Hardware based ECC correction for 
> NAND devices. And this driver has a very small foot-print.
> 
> The only cases this drives would not be used are:
> (a) Using S/W based ECC scheme, which have vey high CPU utilization
> (b) Using single bit ECC scheme, which are becoming obsolete due to
>  increasing NAND densities.
> For most of the cases ELM module will be used with nand-driver. So
> there should be no harm in having this module as built-in, if not used
> in 10% of the use-cases.
> 
> Thus I think it's better to keep this module tied to GPMC module, 
> rather than independent control via KConfig.
> And user should just selects which ECC scheme he would like to
> use via DT, without worrying about KConfig options.

Sorry, I'm lost here and have no idea what you are actually suggesting
we do. Can you phrase that as a patch?

Unless you come up with a better solution, I would still like to queue
up my patch since it fixes an annoying bug and seems to have no downsides.
 
> I'm working in cleaning up omap2-nand driver to remove some 
> redundancies. So would like to know your feedback on same..

Sure, just Cc me on your patches.

	Arnd

WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd@arndb.de>
To: "Gupta, Pekon" <pekon@ti.com>
Cc: Tony Lindgren <tony@atomide.com>,
	"Mohammed, Afzal" <afzal@ti.com>,
	Artem Bityutskiy <artem.bityutskiy@linux.intel.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-mtd@lists.infradead.org" <linux-mtd@lists.infradead.org>,
	Russell King <rmk+kernel@arm.linux.org.uk>,
	David Woodhouse <dwmw2@infradead.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH 21/21] mtd: omap2: allow bulding as a module
Date: Mon, 29 Apr 2013 15:34:10 +0200	[thread overview]
Message-ID: <201304291534.10539.arnd@arndb.de> (raw)
In-Reply-To: <20980858CB6D3A4BAE95CA194937D5E73E9BCFAE@DBDE04.ent.ti.com>

On Wednesday 24 April 2013, Gupta, Pekon wrote:
> [Pekon]: 
> ELM module is required in for Hardware based ECC correction for 
> NAND devices. And this driver has a very small foot-print.
> 
> The only cases this drives would not be used are:
> (a) Using S/W based ECC scheme, which have vey high CPU utilization
> (b) Using single bit ECC scheme, which are becoming obsolete due to
>  increasing NAND densities.
> For most of the cases ELM module will be used with nand-driver. So
> there should be no harm in having this module as built-in, if not used
> in 10% of the use-cases.
> 
> Thus I think it's better to keep this module tied to GPMC module, 
> rather than independent control via KConfig.
> And user should just selects which ECC scheme he would like to
> use via DT, without worrying about KConfig options.

Sorry, I'm lost here and have no idea what you are actually suggesting
we do. Can you phrase that as a patch?

Unless you come up with a better solution, I would still like to queue
up my patch since it fixes an annoying bug and seems to have no downsides.
 
> I'm working in cleaning up omap2-nand driver to remove some 
> redundancies. So would like to know your feedback on same..

Sure, just Cc me on your patches.

	Arnd

  reply	other threads:[~2013-04-29 13:34 UTC|newest]

Thread overview: 123+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-23 16:30 [PATCH 00/21] ARM build fixes for 3.10 Arnd Bergmann
2013-04-23 16:30 ` Arnd Bergmann
2013-04-23 16:30 ` Arnd Bergmann
2013-04-23 16:30 ` [PATCH 01/21] Turn off -Wmaybe-uninitialized when building with -Os Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:30 ` [PATCH 02/21] ARM: compressed/head.S: work around new binutils warning Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:30 ` [PATCH 03/21] ARM: use optimized do_div only for EABI Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:30 ` [PATCH 04/21] ARM: kvm: define KVM_ARM_MAX_VCPUS unconditionally Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 17:29   ` Christoffer Dall
2013-04-23 17:29     ` Christoffer Dall
2013-04-23 17:32     ` Christoffer Dall
2013-04-23 17:32       ` Christoffer Dall
2013-04-23 16:30 ` [PATCH 05/21] ARM: OMAP2+: add dependencies on ARCH_MULTI_V6/V7 Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 18:01   ` Tony Lindgren
2013-04-23 18:01     ` Tony Lindgren
2013-04-23 19:31     ` Arnd Bergmann
2013-04-23 19:31       ` Arnd Bergmann
2013-04-23 16:30 ` [PATCH 06/21] ARM: tegra: call cpu_do_idle from C code Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-24  2:55   ` Joseph Lo
2013-04-24  2:55     ` Joseph Lo
2013-04-23 16:30 ` [PATCH 07/21] ARM: tegra: unify tegra_idle_device definitions Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:56   ` Daniel Lezcano
2013-04-23 16:56     ` Daniel Lezcano
2013-04-23 18:57     ` Arnd Bergmann
2013-04-23 18:57       ` Arnd Bergmann
2013-04-23 22:04       ` Daniel Lezcano
2013-04-23 22:04         ` Daniel Lezcano
2013-04-24  2:56         ` Joseph Lo
2013-04-24  2:56           ` Joseph Lo
2013-04-23 16:30 ` [PATCH 08/21] ASoC: ux500: forward declare msp_i2s_platform_data Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 18:25   ` Mark Brown
2013-04-23 18:25     ` Mark Brown
     [not found] ` <1366734653-488286-1-git-send-email-arnd-r2nGTMty4D4@public.gmane.org>
2013-04-23 16:30   ` [PATCH 09/21] spi/sirf: fix MODULE_DEVICE_TABLE Arnd Bergmann
2013-04-23 16:30     ` Arnd Bergmann
2013-04-23 16:30     ` Arnd Bergmann
2013-04-23 18:27     ` Mark Brown
2013-04-23 18:27       ` Mark Brown
2013-04-23 16:30 ` [PATCH 10/21] ssb: fix alignment of struct bcma_device_id Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:30 ` [PATCH 11/21] drm/tilcdc: use only a single module device table Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-25 17:39   ` Rob Clark
2013-04-25 17:39     ` Rob Clark
2013-04-23 16:30 ` [PATCH 12/21] mmc: sdhci-tegra: fix MODULE_DEVICE_TABLE Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:46   ` Chris Ball
2013-04-23 16:46     ` Chris Ball
2013-04-23 18:55     ` Arnd Bergmann
2013-04-23 18:55       ` Arnd Bergmann
2013-04-23 19:07       ` Chris Ball
2013-04-23 19:07         ` Chris Ball
2013-04-23 16:30 ` [PATCH 13/21] misc: mark spear13xx-pcie-gadget as broken Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-24  3:20   ` Viresh Kumar
2013-04-24  3:20     ` Viresh Kumar
2013-04-23 16:30 ` [PATCH 14/21] staging: nvec: allow modular build Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 19:21   ` Marc Dietrich
2013-04-23 19:21     ` Marc Dietrich
2013-04-23 19:41     ` Arnd Bergmann
2013-04-23 19:41       ` Arnd Bergmann
2013-04-23 19:55       ` Marc Dietrich
2013-04-23 19:55         ` Marc Dietrich
2013-04-23 20:05         ` Greg Kroah-Hartman
2013-04-23 20:05           ` Greg Kroah-Hartman
2013-04-23 20:14         ` Arnd Bergmann
2013-04-23 20:14           ` Arnd Bergmann
2013-04-23 20:21           ` Marc Dietrich
2013-04-23 20:21             ` Marc Dietrich
2013-04-23 16:30 ` [PATCH 15/21] staging: comedi/dawboard2000: use mdelay for large delays Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:46   ` H Hartley Sweeten
2013-04-23 16:46     ` H Hartley Sweeten
2013-04-23 16:30 ` [PATCH 16/21] staging/zsmalloc: don't use pgtable-mapping from modules Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:30 ` [PATCH 17/21] tty/serial/sirf: fix MODULE_DEVICE_TABLE Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:30 ` [PATCH 18/21] usb: phy: tegra: don't call into tegra-ehci directly Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:42   ` Alan Stern
2013-04-23 16:42     ` Alan Stern
2013-04-23 19:02     ` Arnd Bergmann
2013-04-23 19:02       ` Arnd Bergmann
2013-04-23 17:17   ` Greg Kroah-Hartman
2013-04-23 17:17     ` Greg Kroah-Hartman
2013-04-23 19:05     ` [PATCH v2] " Arnd Bergmann
2013-04-23 19:05       ` Arnd Bergmann
2013-04-23 16:30 ` [PATCH 19/21] usb: phy: don't mark usb_bind_phy as __init Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 17:17   ` Greg Kroah-Hartman
2013-04-23 17:17     ` Greg Kroah-Hartman
2013-04-23 19:07     ` [PATCH v2] " Arnd Bergmann
2013-04-23 19:07       ` Arnd Bergmann
2013-04-23 19:30       ` Felipe Balbi
2013-04-23 19:30         ` Felipe Balbi
2013-04-23 19:46         ` Arnd Bergmann
2013-04-23 19:46           ` Arnd Bergmann
2013-04-23 16:30 ` [PATCH 20/21] MFD: ab8500: export ab8500_gpadc_sw_hw_convert properly Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-26  8:13   ` Linus Walleij
2013-04-26  8:13     ` Linus Walleij
2013-04-26  9:23   ` Samuel Ortiz
2013-04-26  9:23     ` Samuel Ortiz
2013-04-23 16:30 ` [PATCH 21/21] mtd: omap2: allow bulding as a module Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 16:30   ` Arnd Bergmann
2013-04-23 18:03   ` Tony Lindgren
2013-04-23 18:03     ` Tony Lindgren
2013-04-23 18:03     ` Tony Lindgren
2013-04-24  5:34     ` Gupta, Pekon
2013-04-24  5:34       ` Gupta, Pekon
2013-04-29 13:34       ` Arnd Bergmann [this message]
2013-04-29 13:34         ` Arnd Bergmann
2013-04-29 13:34         ` Arnd Bergmann

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=201304291534.10539.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=afzal@ti.com \
    --cc=artem.bityutskiy@linux.intel.com \
    --cc=dwmw2@infradead.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=pekon@ti.com \
    --cc=rmk+kernel@arm.linux.org.uk \
    --cc=tony@atomide.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.