gmane.linux.ports.arm.kernel

Re: [GIT PULL] Veratile Express DT updates for v3.5

Olof Johansson — 2012-05-26T08:46:38

Hi Pawel,

On Mon, May 21, 2012 at 1:38 AM, Pawel Moll <pawel.moll< at >arm.com> wrote:

Since this is the first time you're late with a pull request, and the
branch is small, I'll cut you some slack. However, next time we get a
pull request like this after the merge window is open, I will punt it
until the next release.

So, I'll queue this up in late/soc and send it up sometime next week.
If the merge window closes early this means it _could_ miss it, but
hopefully not.


To avoid this from happening next time, please make sure the pull
requests reach us by -rc6 or -rc7 time frame.


Thanks,

-Olof

[PATCH 01/15] ARM: mx5: Use clk_prepare_enable/clk_disable_unprepare

Fabio Estevam — 2012-05-25T23:14:42

From: Fabio Estevam <fabio.estevam< at >freescale.com>

Prepare the clock before enabling it.

Cc: <linux-arm-kernel< at >lists.infradead.org>
Signed-off-by: Fabio Estevam <fabio.estevam< at >freescale.com>
---
 arch/arm/mach-imx/mm-imx5.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/mach-imx/mm-imx5.c b/arch/arm/mach-imx/mm-imx5.c
index feeee17..fcad95c 100644
--- a/arch/arm/mach-imx/mm-imx5.c
+++ b/arch/arm/mach-imx/mm-imx5.c
< at >< at > -35,11 +35,11 < at >< at > static void imx5_idle(void)
 return;
 clk_prepare(gpc_dvfs_clk);
 }
-clk_enable(gpc_dvfs_clk);
+clk_prepare_enable(gpc_dvfs_clk);
 mx5_cpu_lp_set(WAIT_UNCLOCKED_POWER_OFF);
 if (!tzic_enable_wake())
 cpu_do_idle();
-clk_disable(gpc_dvfs_clk);
+clk_disable_unprepare(gpc_dvfs_clk);
 }
 
 /*

Re: [linux-pm] [PATCH 0/2] bug fixes for coupled cpuidle

Kevin Hilman — 2012-05-25T22:38:56

Len,

Santosh Shilimkar <santosh.shilimkar< at >ti.com> writes:


Tested-by: Kevin Hilman <khilman< at >ti.com>

Can you pick these up for v3.6?

I don't currently see them in your next branch.

Thanks,

Kevin

[PATCH 13/20] arm: Do not call try_to_freeze() in do_signal()

Richard Weinberger — 2012-05-25T22:04:29

get_signal_to_deliver() already calls try_to_freeze(), there is no
need to call it directly.

Cc: Russell King <linux< at >arm.linux.org.uk>
Cc: linux-arm-kernel< at >lists.infradead.org
Signed-off-by: Richard Weinberger <richard< at >nod.at>
---
 arch/arm/kernel/signal.c |    5 -----
 1 files changed, 0 insertions(+), 5 deletions(-)

diff --git a/arch/arm/kernel/signal.c b/arch/arm/kernel/signal.c
index 434a1f3..aeebb46 100644
--- a/arch/arm/kernel/signal.c
+++ b/arch/arm/kernel/signal.c
< at >< at > -10,7 +10,6 < at >< at >
 #include <linux/errno.h>
 #include <linux/signal.h>
 #include <linux/personality.h>
-#include <linux/freezer.h>
 #include <linux/uaccess.h>
 #include <linux/tracehook.h>
 #include <linux/irqflags.h>
< at >< at > -636,9 +635,6 < at >< at > static void do_signal(struct pt_regs *regs, int syscall)
 }
 }
 
-if (try_to_freeze())
-goto no_signal;
-
 /*
  * Get the signal to deliver.  When running under ptrace, at this
  * point the debugger may change all our registers ...
< at >< at > -678,7 +674,6 < at >< at > static void do_signal(struct pt_regs *regs,

Re: [PATCH v2] RFC: ARM: cache-l2x0: update workaround for PL310 errata 727915

Colin Cross — 2012-05-25T20:04:22

In a previous thread, I wrote that the 4430 I was using had r2p0.  I
can't verify the accuracy of that claim right now, but my patch was
based on that assumption.  Where are you getting r3p1 from?

When I said I couldn't reproduce the issue with clean during boot, I
meant that I couldn't trigger the issue during boot by calling clean
repeatedly.  I could still reproduce the issue after boot even if the
cache was cleaned during boot.

The reference to 727915 came from discussions with TI, but given that
it is a deadlock and not a memory corruption, you're probably right
that it is not the same issue.

Re: [PATCH 2/2] mtd mxc_nand: move ecc strengh setup before nand_scan_tail

Artem Bityutskiy — 2012-05-25T16:55:07

Pushed this one to l2-mtd.git as well, thanks!

Re: [PATCH 2/2] mtd mxc_nand: move ecc strengh setup before nand_scan_tail

Mike Dunn — 2012-05-25T16:29:24


Oops, sorry Sascha.  This was compile-tested only, but I should have ensured
strength was set before call to nand_scan_tail().

Thanks,
Mike

[PATCH 1/6] sizes.h: move from asm-generic to

Alessandro Rubini — 2012-05-25T15:48:00

sizes.h is used throughout the AMBA code and drivers, so the header
should be available to everyone in order to driver AMBA/PrimeCell
peripherals behind a PCI bridge where the host can be any platform
(I'm doing it under x86).

At this step <asm-generic/sizes.h> includes <linux/sizes.h>,
to allow a grace period for both in-tree and out-of-tree drivers.

Signed-off-by: Alessandro Rubini <rubini< at >gnudd.com>
Acked-by: Giancarlo Asnaghi <giancarlo.asnaghi< at >st.com>
Cc: Russell King <linux< at >arm.linux.org.uk>
Cc: Arnd Bergmann <arnd< at >arndb.de>
---
 include/asm-generic/sizes.h |   49 +-----------------------------------------
 include/linux/sizes.h       |   47 +++++++++++++++++++++++++++++++++++++++++
 2 files changed, 49 insertions(+), 47 deletions(-)
 create mode 100644 include/linux/sizes.h

diff --git a/include/asm-generic/sizes.h b/include/asm-generic/sizes.h
index ea5d4ef..1dcfad9 100644
--- a/include/asm-generic/sizes.h
+++ b/include/asm-generic/sizes.h
< at >< at > -1,47 +1,2 < at >< at >
-/*
- * linux/include/asm-generic/sizes.h

Re: [PATCH 1/5] ARM: imx6q: add DT node for apbh-dma

Huang Shijie — 2012-05-25T15:09:39

thanks a lot.

Huang Shijie

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel< at >lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Re: [PATCH 1/2] mtd mxc_nand: use 32bit copy functions

Sascha Hauer — 2012-05-25T14:59:01

Will do.


This makes me wonder a bit, I basically copied the prototype from
the _memcpy_*_io template from arch/arm/kernel/io.c. Should they
be wrong?
otoh I also wondered why there were volatiles in arch/arm/kernel/io.c
in the first place ;)

Sascha

Re: [PATCH v2] RFC: ARM: cache-l2x0: update workaround for PL310 errata 727915

Will Deacon — 2012-05-25T14:56:04

Right, ok. This is definitely a different problem though. Firstly, r3p1 is
not affected by the original erratum and secondly the symptoms do not
include deadlock. The clean during boot also sounds highly suspicious -- is
there a chance you boot the kernel with a dirty L2? Is the L2 enabled or
disabled when entering the kernel [I think u8500 has it enabled]?

Will

Re: [PATCH 1/2] mtd mxc_nand: use 32bit copy functions

Artem Bityutskiy — 2012-05-25T14:58:22

Could you please take a look at the checkpatch.pl updates?

WARNING:LONG_LINE: line over 80 characters
#103: FILE: drivers/mtd/nand/mxc_nand.c:276:
+static void memcpy32_fromio(void *trg, const volatile void __iomem  *src, size_t size)

WARNING:VOLATILE: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#103: FILE: drivers/mtd/nand/mxc_nand.c:276:
+static void memcpy32_fromio(void *trg, const volatile void __iomem  *src, size_t size)

WARNING:VOLATILE: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#107: FILE: drivers/mtd/nand/mxc_nand.c:280:
+       const volatile u32 *s = src;

WARNING:VOLATILE: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#113: FILE: drivers/mtd/nand/mxc_nand.c:286:
+static void memcpy32_toio(volatile void __iomem *trg, const void *src, int size)

WARNING:VOLATILE: Use of volatile is usually wrong: see Documentation/volatile-considered-harmful.txt
#116: FILE: drivers/mtd/nand/mxc

Re: Booting mx27 on linux-next

Sascha Hauer — 2012-05-25T14:51:02

Hi Fabio,

On Fri, May 25, 2012 at 10:56:07AM -0300, Fabio Estevam wrote:

Works for me, tested on next-20120524 on a pcm038 board. I just had
to disable nand support, see the patches I just sent to the list.

Sascha

[STABLE PATCH] ARM: 7409/1: Do not call flush_cache_user_range with mmap_sem held

Will Deacon — 2012-05-25T14:38:04

From: Dima Zavin <dima< at >android.com>

commit 435a7ef52db7d86e67a009b36cac1457f8972391 upstream

We can't be holding the mmap_sem while calling flush_cache_user_range
because the flush can fault. If we fault on a user address, the
page fault handler will try to take mmap_sem again. Since both places
acquire the read lock, most of the time it succeeds. However, if another
thread tries to acquire the write lock on the mmap_sem (e.g. mmap) in
between the call to flush_cache_user_range and the fault, the down_read
in do_page_fault will deadlock.

[will: removed drop of vma parameter as already queued by rmk (7365/1)]

Cc: <stable< at >vger.kernel.org> # 2.6.32+: 4542b6a0: ARM: 7365/1
Cc: <stable< at >vger.kernel.org> # 2.6.32+
Acked-by: Catalin Marinas <catalin.marinas< at >arm.com>
Signed-off-by: Dima Zavin <dima< at >android.com>
Signed-off-by: John Stultz <john.stultz< at >linaro.org>
Signed-off-by: Will Deacon <will.deacon< at >arm.com>
Signed-off-by: Russell King <rmk+kernel< at >arm.linux.org.uk>
---

Greg: This patch is a candidate for -stab

[PATCH 2/2] mtd mxc_nand: move ecc strengh setup before nand_scan_tail

Sascha Hauer — 2012-05-25T14:22:42

Since this commit:

  commit 6a918bade9dab40aaef80559bd1169c69e8d69cb
  Author: Mike Dunn <mikedunn< at >newsguy.com>
  Date:   Sun Mar 11 14:21:11 2012 -0700

     mtd: flash drivers set ecc strength

The mxc_nand driver fails with:

Driver must set ecc.strength when using hardware ECC

This is because nand_scan_tail checks for correct ecc strength
settings, so we must set them up before nand_scan_tail.

Signed-off-by: Sascha Hauer <s.hauer< at >pengutronix.de>
Cc: stable< at >vger.kernel.org
---
 drivers/mtd/nand/mxc_nand.c |   12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/drivers/mtd/nand/mxc_nand.c b/drivers/mtd/nand/mxc_nand.c
index 4d27ddc..aaf042b 100644
--- a/drivers/mtd/nand/mxc_nand.c
+++ b/drivers/mtd/nand/mxc_nand.c
< at >< at > -1485,12 +1485,6 < at >< at > static int __init mxcnd_probe(struct platform_device *pdev)
 else if (mtd->writesize == 4096)
 this->ecc.layout = host->devtype_data->ecclayout_4k;
 
-/* second phase scan */
-if (nand_scan_tail(mtd)) {
-err = -ENXIO;
-goto escan;
-}

[PATCH 1/2] mtd mxc_nand: use 32bit copy functions

Sascha Hauer — 2012-05-25T14:22:41

The following commit changes the function used to copy from/to
the hardware buffer to memcpy_[from|to]io. This does not work
since the hardware cannot handle the byte accesses used by these
functions. Instead of reverting this patch introduce 32bit
correspondents of these functions.

commit 5775ba36ea9c760c2d7e697dac04f2f7fc95aa62
Author: Uwe Kleine-König <u.kleine-koenig< at >pengutronix.de>
Date:   Tue Apr 24 10:05:22 2012 +0200

    mtd: mxc_nand: fix several sparse warnings about incorrect address space

    Signed-off-by: Uwe Kleine-König <u.kleine-koenig< at >pengutronix.de>
    Signed-off-by: Artem Bityutskiy <artem.bityutskiy< at >linux.intel.com>

Signed-off-by: Sascha Hauer <s.hauer< at >pengutronix.de>
Cc: Uwe Kleine-König <u.kleine-koenig< at >pengutronix.de>
---
 drivers/mtd/nand/mxc_nand.c |   36 ++++++++++++++++++++++++++++--------
 1 file changed, 28 insertions(+), 8 deletions(-)

diff --git a/drivers/mtd/nand/mxc_nand.c b/drivers/mtd/nand/mxc_nand.c
index fd14966..4d27ddc 100644
--- a/drivers/mtd/nand/mxc_nand.c

Booting mx27 on linux-next

Fabio Estevam — 2012-05-25T13:56:07

Hi Sascha,

Just tried booting a mx27pdk on linux-next and it failed to boot.

Any patches I am missing?

Just wanted to check with you first prior to start debugging it.

Regards,

Fabio Estevam

[PATCH v4 4/6] gpio: introduce lock mechanism for gpiochip_find

Dong Aisheng — 2012-05-25T13:36:18

From: Dong Aisheng <dong.aisheng< at >linaro.org>

The module lock will be automatically claimed for gpiochip_find function
in case the gpio module is removed during the using of gpiochip instance.
Users are responsible to call gpiochip_put to release the lock after
the using.

Signed-off-by: Dong Aisheng <dong.aisheng< at >linaro.org>
---
 drivers/gpio/gpiolib-of.c  |    5 ++++-
 drivers/gpio/gpiolib.c     |   17 +++++++++++++++++
 include/asm-generic/gpio.h |    2 +-
 3 files changed, 22 insertions(+), 2 deletions(-)

diff --git a/drivers/gpio/gpiolib-of.c b/drivers/gpio/gpiolib-of.c
index b8010a9..d521452 100644
--- a/drivers/gpio/gpiolib-of.c
+++ b/drivers/gpio/gpiolib-of.c
< at >< at > -63,6 +63,7 < at >< at > int of_get_named_gpio_flags(struct device_node *np, const char *propname,
                            int index, enum of_gpio_flags *flags)
 {
 struct gg_data gg_data = { .flags = flags, .out_gpio = -ENODEV };
+struct gpio_chip *chip;
 int ret;
 
 /* .of_xlate might decide to not fill in the flags, so clear it. */
< at >< at > -76,7

[PATCH] ARM: at91: aic can use fast eoi handler type

2012-05-25T12:55:11

From: Ludovic Desroches <ludovic.desroches< at >atmel.com>

The Advanced Interrupt Controller allows to use the fast EOI handler type.
It lets remove the Atmel specific workaround into arch/arm/kernel/irq.c used
to indicate to the AIC the end of the interrupt treatment.

Signed-off-by: Ludovic Desroches <ludovic.desroches< at >atmel.com>
---
 arch/arm/kernel/irq.c                  |    3 ---
 arch/arm/mach-at91/gpio.c              |   10 ++++++++--
 arch/arm/mach-at91/include/mach/irqs.h |    7 -------
 arch/arm/mach-at91/irq.c               |   10 ++++++++--
 4 files changed, 16 insertions(+), 14 deletions(-)

diff --git a/arch/arm/kernel/irq.c b/arch/arm/kernel/irq.c
index 71ccdbf..6236d1a 100644
--- a/arch/arm/kernel/irq.c
+++ b/arch/arm/kernel/irq.c
< at >< at > -85,9 +85,6 < at >< at > void handle_IRQ(unsigned int irq, struct pt_regs *regs)
 generic_handle_irq(irq);
 }
 
-/* AT91 specific workaround */
-irq_finish(irq);
-
 irq_exit();
 set_irq_regs(old_regs);
 }
diff --git a/arch/arm/mach-at91/gpio.c b/arch/arm/mach-at91/gpio

Re: [RFC PATCH 00/11] OMAP System Control Module

Konstantin Baydarov — 2012-05-25T10:50:15

  Hi.

On 05/25/2012 12:25 PM, Eduardo Valentin wrote:
In my latest version I got rid from early API set, check out patch for V3 patch set.
I'll attach patch for current version later.

BR,
    Konstantin Baydarov.

[PATCH 2/3] OMAP: avoid build wdt platform device if with dt support

2012-05-25T10:42:31

From: Xiao Jiang <jgq516< at >gmail.com>

If provided dt support, then skip add wdt platform device as usual.

Signed-off-by: Xiao Jiang <jgq516< at >gmail.com>
---
 arch/arm/mach-omap2/devices.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/arch/arm/mach-omap2/devices.c b/arch/arm/mach-omap2/devices.c
index ae62ece..80d7e3f 100644
--- a/arch/arm/mach-omap2/devices.c
+++ b/arch/arm/mach-omap2/devices.c
< at >< at > -759,7 +759,7 < at >< at > static int __init omap_init_wdt(void)
 char *oh_name = "wd_timer2";
 char *dev_name = "omap_wdt";
 
-if (!cpu_class_is_omap2())
+if (!cpu_class_is_omap2() || of_have_populated_dt())
 return 0;
 
 oh = omap_hwmod_lookup(oh_name);