* [PATCH 00/21] ARM build fixes for 3.10
@ 2013-04-23 16:30 Arnd Bergmann
2013-04-23 16:30 ` [PATCH 21/21] mtd: omap2: allow bulding as a module Arnd Bergmann
0 siblings, 1 reply; 5+ messages in thread
From: Arnd Bergmann @ 2013-04-23 16:30 UTC (permalink / raw)
To: linux-arm-kernel
Cc: alsa-devel, Viresh Kumar, Russell King, Samuel Ortiz,
Arnd Bergmann, Dave Airlie, Artem Bityutskiy, linux-usb,
linux-kernel, Felipe Balbi, spi-devel-general, Grant Likely,
Tony Lindgren, Mark Brown, Alan Stern, dri-devel, linux-mtd,
Greg Kroah-Hartman, Stephen Warren, Chris Ball, David Woodhouse
Hi everyone,
This is a set of bug fixes for build errors and warnings I am
seeing in the various defconfigs and/or allmodconfig builds
on linux-next. The majority of these showed up because we now
include a lot more platforms in allmodconfig than we used to.
Please apply to appropriate maintainer trees or provide
feedback, I can take care of the rest sending those through
arm-soc that I get no feedback on.
Arnd
Arnd Bergmann (21):
Turn off -Wmaybe-uninitialized when building with -Os
ARM: compressed/head.S: work around new binutils warning
ARM: use optimized do_div only for EABI
ARM: kvm: define KVM_ARM_MAX_VCPUS unconditionally
ARM: OMAP2+: add dependencies on ARCH_MULTI_V6/V7
ARM: tegra: call cpu_do_idle from C code
ARM: tegra: unify tegra_idle_device definitions
ASoC: ux500: forward declare msp_i2s_platform_data
spi/sirf: fix MODULE_DEVICE_TABLE
ssb: fix alignment of struct bcma_device_id
drm/tilcdc: use only a single module device table
mmc: sdhci-tegra: fix MODULE_DEVICE_TABLE
misc: mark spear13xx-pcie-gadget as broken
staging: nvec: allow modular build
staging: comedi/dawboard2000: use mdelay for large delays
staging/zsmalloc: don't use pgtable-mapping from modules
tty/serial/sirf: fix MODULE_DEVICE_TABLE
usb: phy: tegra: don't call into tegra-ehci directly
usb: phy: don't mark usb_bind_phy as __init
MFD: ab8500: export ab8500_gpadc_sw_hw_convert properly
mtd: omap2: allow bulding as a module
Makefile | 2 +-
arch/arm/boot/compressed/Makefile | 2 +-
arch/arm/boot/compressed/head.S | 12 ++++++++++++
arch/arm/include/asm/div64.h | 2 +-
arch/arm/kvm/Kconfig | 6 +++---
arch/arm/mach-omap2/Kconfig | 5 +++++
arch/arm/mach-tegra/cpuidle-tegra114.c | 5 ++---
arch/arm/mach-tegra/cpuidle-tegra20.c | 4 +---
arch/arm/mach-tegra/cpuidle-tegra30.c | 4 +---
arch/arm/mach-tegra/cpuidle.c | 2 ++
arch/arm/mach-tegra/cpuidle.h | 4 ++++
arch/arm/mach-tegra/pm.c | 5 +++++
arch/arm/mach-tegra/sleep-tegra20.S | 2 +-
drivers/gpu/drm/tilcdc/tilcdc_panel.c | 1 -
drivers/gpu/drm/tilcdc/tilcdc_slave.c | 1 -
drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 1 -
drivers/mfd/ab8500-gpadc.c | 2 +-
drivers/misc/Kconfig | 2 +-
drivers/mmc/host/sdhci-tegra.c | 2 +-
drivers/mtd/nand/Kconfig | 2 +-
drivers/spi/spi-sirf.c | 2 +-
drivers/staging/comedi/drivers/daqboard2000.c | 14 +++++++-------
drivers/staging/nvec/Kconfig | 10 +++++-----
drivers/staging/zsmalloc/zsmalloc-main.c | 2 +-
drivers/tty/serial/sirfsoc_uart.c | 2 +-
drivers/usb/host/ehci-tegra.c | 10 +++++-----
drivers/usb/otg/otg.c | 2 +-
drivers/usb/phy/tegra_usb_phy.c | 13 +++++++++----
include/linux/mod_devicetable.h | 5 +++--
include/linux/usb/tegra_usb_phy.h | 10 +++++-----
sound/soc/ux500/ux500_msp_i2s.h | 1 +
31 files changed, 82 insertions(+), 55 deletions(-)
--
1.8.1.2
Cc: Alan Stern <stern@rowland.harvard.edu>
Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Cc: Chris Ball <cjb@laptop.org>
Cc: Dave Airlie <airlied@linux.ie>
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: Felipe Balbi <balbi@ti.com>
Cc: Grant Likely <grant.likely@secretlab.ca>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Mark Brown <broonie@kernel.org>
Cc: Russell King <linux@arm.linux.org.uk>
Cc: Samuel Ortiz <sameo@linux.intel.com>
Cc: Stephen Warren <swarren@nvidia.com>
Cc: Tony Lindgren <tony@atomide.com>
Cc: Viresh Kumar <viresh.kumar@st.com>
Cc: alsa-devel@alsa-project.org
Cc: dri-devel@lists.freedesktop.org
Cc: linux-mtd@lists.infradead.org
Cc: linux-usb@vger.kernel.org
Cc: spi-devel-general@lists.sourceforge.net
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 21/21] mtd: omap2: allow bulding as a module
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 18:03 ` Tony Lindgren
0 siblings, 1 reply; 5+ messages in thread
From: Arnd Bergmann @ 2013-04-23 16:30 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Afzal Mohammed, Arnd Bergmann, Tony Lindgren, Artem Bityutskiy,
linux-kernel, linux-mtd, Russell King, David Woodhouse
The omap2 nand device driver calls into the the elm code, which can
be a loadable module, and in that case it cannot be built-in itself.
I can see no reason why the omap2 driver cannot also be a module,
so let's make the option "tristate" in Kconfig to fix this allmodconfig
build error:
ERROR: "elm_config" [drivers/mtd/nand/omap2.ko] undefined!
ERROR: "elm_decode_bch_error_page" [drivers/mtd/nand/omap2.ko] undefined!
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Cc: David Woodhouse <dwmw2@infradead.org>
Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
Cc: Tony Lindgren <tony@atomide.com>
Cc: Afzal Mohammed <afzal@ti.com>
Cc: Russell King <rmk+kernel@arm.linux.org.uk>
Cc: linux-mtd@lists.infradead.org
---
drivers/mtd/nand/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
index 81bf5e5..106b55e 100644
--- a/drivers/mtd/nand/Kconfig
+++ b/drivers/mtd/nand/Kconfig
@@ -109,7 +109,7 @@ config MTD_NAND_OMAP2
config MTD_NAND_OMAP_BCH
depends on MTD_NAND && MTD_NAND_OMAP2 && ARCH_OMAP3
- bool "Enable support for hardware BCH error correction"
+ tristate "Enable support for hardware BCH error correction"
default n
select BCH
select BCH_CONST_PARAMS
--
1.8.1.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 21/21] mtd: omap2: allow bulding as a module
2013-04-23 16:30 ` [PATCH 21/21] mtd: omap2: allow bulding as a module Arnd Bergmann
@ 2013-04-23 18:03 ` Tony Lindgren
2013-04-24 5:34 ` Gupta, Pekon
0 siblings, 1 reply; 5+ messages in thread
From: Tony Lindgren @ 2013-04-23 18:03 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Afzal Mohammed, Artem Bityutskiy, linux-kernel, linux-mtd,
Russell King, David Woodhouse, linux-arm-kernel
* Arnd Bergmann <arnd@arndb.de> [130423 09:37]:
> The omap2 nand device driver calls into the the elm code, which can
> be a loadable module, and in that case it cannot be built-in itself.
> I can see no reason why the omap2 driver cannot also be a module,
> so let's make the option "tristate" in Kconfig to fix this allmodconfig
> build error:
>
> ERROR: "elm_config" [drivers/mtd/nand/omap2.ko] undefined!
> ERROR: "elm_decode_bch_error_page" [drivers/mtd/nand/omap2.ko] undefined!
>
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> Cc: David Woodhouse <dwmw2@infradead.org>
> Cc: Artem Bityutskiy <artem.bityutskiy@linux.intel.com>
> Cc: Tony Lindgren <tony@atomide.com>
> Cc: Afzal Mohammed <afzal@ti.com>
> Cc: Russell King <rmk+kernel@arm.linux.org.uk>
> Cc: linux-mtd@lists.infradead.org
> ---
> drivers/mtd/nand/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
> index 81bf5e5..106b55e 100644
> --- a/drivers/mtd/nand/Kconfig
> +++ b/drivers/mtd/nand/Kconfig
> @@ -109,7 +109,7 @@ config MTD_NAND_OMAP2
>
> config MTD_NAND_OMAP_BCH
> depends on MTD_NAND && MTD_NAND_OMAP2 && ARCH_OMAP3
> - bool "Enable support for hardware BCH error correction"
> + tristate "Enable support for hardware BCH error correction"
> default n
> select BCH
> select BCH_CONST_PARAMS
Acked-by: Tony Lindgren <tony@atomide.com>
While at it, you could also drop the "default n" as that's
the default anyways.
Regards,
Tony
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 21/21] mtd: omap2: allow bulding as a module
2013-04-23 18:03 ` Tony Lindgren
@ 2013-04-24 5:34 ` Gupta, Pekon
2013-04-29 13:34 ` Arnd Bergmann
0 siblings, 1 reply; 5+ messages in thread
From: Gupta, Pekon @ 2013-04-24 5:34 UTC (permalink / raw)
To: Tony Lindgren, Arnd Bergmann
Cc: Mohammed, Afzal, Artem Bityutskiy, linux-kernel@vger.kernel.org,
linux-mtd@lists.infradead.org, Russell King, David Woodhouse,
linux-arm-kernel@lists.infradead.org
> * Arnd Bergmann <arnd@arndb.de> [130423 09:37]:
> > The omap2 nand device driver calls into the the elm code, which can
> > be a loadable module, and in that case it cannot be built-in itself.
> > I can see no reason why the omap2 driver cannot also be a module,
> > so let's make the option "tristate" in Kconfig to fix this allmodconfig
> > build error:
> >
> > ERROR: "elm_config" [drivers/mtd/nand/omap2.ko] undefined!
> > ERROR: "elm_decode_bch_error_page" [drivers/mtd/nand/omap2.ko]
> undefined!
> >
[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.
I'm working in cleaning up omap2-nand driver to remove some
redundancies. So would like to know your feedback on same..
with regards, pekon
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 21/21] mtd: omap2: allow bulding as a module
2013-04-24 5:34 ` Gupta, Pekon
@ 2013-04-29 13:34 ` Arnd Bergmann
0 siblings, 0 replies; 5+ messages in thread
From: Arnd Bergmann @ 2013-04-29 13:34 UTC (permalink / raw)
To: Gupta, Pekon
Cc: Mohammed, Afzal, Tony Lindgren, Artem Bityutskiy,
linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org,
Russell King, David Woodhouse,
linux-arm-kernel@lists.infradead.org
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
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-04-29 13:34 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-04-23 16:30 [PATCH 00/21] ARM build fixes for 3.10 Arnd Bergmann
2013-04-23 16:30 ` [PATCH 21/21] mtd: omap2: allow bulding as a module Arnd Bergmann
2013-04-23 18:03 ` Tony Lindgren
2013-04-24 5:34 ` Gupta, Pekon
2013-04-29 13:34 ` Arnd Bergmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox