[PATCH v2] ARM: pxa: Move iotable mapping inside vmalloc region

2013-11-28 Thread Ezequiel Garcia
: Thu Aug 25 00:35:59 2011 -0400 ARM: move iotable mappings within the vmalloc region While at it, condition the mapping to PXA25x and PXA27x, which are the only platforms where it's used. Cc: Nicolas Pitre Cc: Russell King - ARM Linux Cc: David Heidelberger Signed-off-by: Ezequiel G

Re: [PATCH v2] serial: 8250_dw: Improve unwritable LCR workaround

2013-11-28 Thread Ezequiel Garcia
On Thu, Nov 28, 2013 at 04:47:20PM -0300, Ezequiel Garcia wrote: > Hi Thomas, Tim: > > On Thu, Nov 28, 2013 at 09:30:34AM +0100, Thomas Petazzoni wrote: > > Dear Ezequiel Garcia, > > > > On Wed, 27 Nov 2013 15:54:49 -0300, Ezequiel Garcia wrote: > > > > &g

Re: [PATCH v2] serial: 8250_dw: Improve unwritable LCR workaround

2013-11-28 Thread Ezequiel Garcia
Hi Thomas, Tim: On Thu, Nov 28, 2013 at 09:30:34AM +0100, Thomas Petazzoni wrote: > Dear Ezequiel Garcia, > > On Wed, 27 Nov 2013 15:54:49 -0300, Ezequiel Garcia wrote: > > > > An external device may be keeping the UART busy and preventing LCR > > > from being wri

Re: [PATCH v2] serial: 8250_dw: Improve unwritable LCR workaround

2013-11-27 Thread Ezequiel Garcia
On Tue, Nov 26, 2013 at 03:03:03PM -0800, Tim Kryger wrote: > On Tue, Nov 26, 2013 at 10:36 AM, Ezequiel Garcia > wrote: > > > Since v3.13-rc1, this commit seems to have introduced some oddities on > > some of our boards. See this log snippet: > > > > Serial

Re: [PATCH v2] serial: 8250_dw: Improve unwritable LCR workaround

2013-11-26 Thread Ezequiel Garcia
Hello, On Tue, Oct 01, 2013 at 10:18:08AM -0700, Tim Kryger wrote: > When configured with UART_16550_COMPATIBLE=NO or in versions prior to > the introduction of this option, the Designware UART will ignore writes > to the LCR if the UART is busy. The current workaround saves a copy of > the last

Re: [PATCH] Make the mtdblock read/write skip the bad nand sector

2013-11-25 Thread Ezequiel Garcia
David, On Mon, Nov 25, 2013 at 12:16:10PM +, David Woodhouse wrote: > On Mon, 2013-11-25 at 08:52 -0300, Ezequiel Garcia wrote: > > > > Your understanding is correct: NAND *must* be erased explictly in > > userspace > > before writing. However, keep in mi

Re: [PATCH] Make the mtdblock read/write skip the bad nand sector

2013-11-25 Thread Ezequiel Garcia
On Mon, Nov 25, 2013 at 07:30:33PM +0800, Hans Zhang wrote: > On 2013/11/25 18:23, Richard Genoud wrote: > > > > Well, yes, write through the char device would be a solution. > >> But, *why* are you writing through mtdblock instead? > >> > >>> I think that maybe it's an optional approach through mt

Re: [PATCH] ARM: pxa: Move iotable mapping inside vmalloc region

2013-11-25 Thread Ezequiel Garcia
On Sun, Nov 24, 2013 at 06:26:45PM -0500, Nicolas Pitre wrote: > On Sun, 24 Nov 2013, Ezequiel Garcia wrote: > > > In order to remove the following ugly message: > > > > BUG: mapping for 0x at 0xff00 out of vmalloc space > > > > the iotable mapp

Re: [PATCH] Make the mtdblock read/write skip the bad nand sector

2013-11-25 Thread Ezequiel Garcia
On Mon, Nov 25, 2013 at 09:29:11AM +0800, Hans Zhang wrote: > On 2013/11/22 19:45, Ezequiel Garcia wrote: > >> Thanks for your reply. > >> This is for my embed system upgrade, I reserved one nand partition for > >> filesystem > >> environment which will loa

Re: [PATCH] ARM: pxa: Move iotable mapping inside vmalloc region

2013-11-24 Thread Ezequiel Garcia
On Sun, Nov 24, 2013 at 12:41:25PM +, Russell King - ARM Linux wrote: > On Sun, Nov 24, 2013 at 09:36:11AM -0300, Ezequiel Garcia wrote: > > In order to remove the following ugly message: > > > > BUG: mapping for 0x at 0xff00 out of vmalloc space > >

[PATCH] ARM: pxa: Move iotable mapping inside vmalloc region

2013-11-24 Thread Ezequiel Garcia
: Thu Aug 25 00:35:59 2011 -0400 ARM: move iotable mappings within the vmalloc region While at it, let's add some nicer defines to make the code more readable. Cc: Nicolas Pitre Signed-off-by: Ezequiel Garcia --- arch/arm/mach-pxa/generic.c | 12 +--- 1 file changed, 9 inser

Re: [PATCH] Make the mtdblock read/write skip the bad nand sector

2013-11-22 Thread Ezequiel Garcia
On Fri, Nov 22, 2013 at 10:10:19AM +0800, Hans Zhang wrote: > On 2013/11/22 9:52, Ezequiel Garcia wrote: > > On Fri, Nov 22, 2013 at 09:22:43AM +0800, Hans Zhang wrote: > >> On 2013/11/21 18:59, Richard Genoud wrote: > >> > >> Here's my scenario, I want to

Re: [PATCH 2/2] mtd: cmdlinepart: support master name is not set when parser partition

2013-11-10 Thread Ezequiel Garcia
On Sun, Nov 10, 2013 at 10:06:59PM -0500, Huang Shijie wrote: > On Sat, Nov 09, 2013 at 11:43:06AM +, Caizhiyong wrote: > > From: Cai Zhiyong > > Date: Sat, 9 Nov 2013 17:54:12 +0800 > > > > @@ -143,5 +149,5 @@ MODULE_PARM_DESC(mtdparts, "Partitioning > > specification"); > > module_param(

Re: [PATCH 2/2] mtd: cmdlinepart: use cmdline partition parser lib

2013-11-08 Thread Ezequiel Garcia
On Fri, Nov 08, 2013 at 06:53:29AM +, Caizhiyong wrote: > >> For further information, see "https://lkml.org/lkml/2013/8/6/550"; > > > > Thanks for doing this. Could we please get some acked-by's or, > > preferably, tested-by's from the MTD p

Re: [PATCH -next] mtd: nand: omap: fix error return code in omap_nand_probe()

2013-11-05 Thread Ezequiel Garcia
. (IOW, is the > rest of the function returning a negative error code on the error > paths, or is it returning 0? Of course the answer is the former, but > it's possible to misread it.) If it helps, I can try to tweak the > wording a bit when applying this patch. > > Pekon, ca

Re: [PATCH] ARM: kirkwood: remove lagacy clk workarounds

2013-10-31 Thread Ezequiel Garcia
On Fri, Oct 18, 2013 at 01:45:45PM -0400, Jason Cooper wrote: > On Fri, Oct 18, 2013 at 07:43:54PM +0200, Sebastian Hesselbarth wrote: > > On 10/18/2013 07:27 PM, Sebastian Hesselbarth wrote: > > >On 10/18/2013 06:17 PM, Ezequiel Garcia wrote: > > >>On Fri, Oct 18, 2

Re: [PATCH -next] mtd: remove duplicated include from mtdcore.c

2013-10-30 Thread Ezequiel Garcia
/mtd/mtdcore.c > index 7189089..92311a5 100644 > --- a/drivers/mtd/mtdcore.c > +++ b/drivers/mtd/mtdcore.c > @@ -37,7 +37,6 @@ > #include > #include > #include > -#include > > #include > #include > > Acked-by: Ezequiel Garcia Ooo

Re: [PATCH 2/2] dma: mv_xor: Use high_base mmio where appropriate

2013-10-29 Thread Ezequiel Garcia
On Tue, Oct 29, 2013 at 12:15:18PM -0700, Dan Williams wrote: > On Tue, Oct 29, 2013 at 2:32 AM, Thomas Petazzoni > wrote: > > Dan, Ezequiel, > > > > On Tue, 29 Oct 2013 05:34:08 -0300, Ezequiel Garcia wrote: > > > >> > On Mon, Oct 28, 2013

Re: [PATCH 2/2] dma: mv_xor: Use high_base mmio where appropriate

2013-10-29 Thread Ezequiel Garcia
On Mon, Oct 28, 2013 at 04:47:20PM -0700, Dan Williams wrote: > [ adding dmaengine ] > > On Mon, Oct 28, 2013 at 3:54 PM, Ezequiel Garcia > wrote: > > Despite requesting two memory resources, called 'base' and 'high_base', the > > driver uses explicit

[PATCH 2/2] dma: mv_xor: Use high_base mmio where appropriate

2013-10-28 Thread Ezequiel Garcia
ne the registers with the offset from high_base, and use high_base explicitly where appropriate. Signed-off-by: Ezequiel Garcia --- drivers/dma/mv_xor.c | 3 ++- drivers/dma/mv_xor.h | 25 + 2 files changed, 15 insertions(+), 13 deletions(-) diff --git a/drivers/dma/mv_xor

[PATCH 1/2] dma: mv_xor: Remove unneeded NULL address check

2013-10-28 Thread Ezequiel Garcia
This mmio address is checked at probe-time, which makes this test redundant. Let's just remove it. Signed-off-by: Ezequiel Garcia --- drivers/dma/mv_xor.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/dma/mv_xor.c b/drivers/dma/mv_xor.c index 536dcb8..d00d58e 100644

Re: [PATCH] mtd: cmdlinepart: use cmdline partition parser lib

2013-10-25 Thread Ezequiel Garcia
Hi, On Fri, Oct 25, 2013 at 11:52:14AM +, Caizhiyong wrote: > > > > I'd like to review the patch in detail and test it, but being a bit old, > > the patch doesn't apply as it is on v3.12-rc5. Care to resend an update? > > > I have sent you a 2 patch base on v3.12-rc6. Are there any problems

Re: [PATCH] mtd: cmdlinepart: use cmdline partition parser lib

2013-10-20 Thread Ezequiel Garcia
Hi Cai, Sorry for the late review. See below some minor comments. On Thu, Aug 15, 2013 at 11:19:36AM +, Caizhiyong wrote: > From: Cai Zhiyong > > MTD cmdline partition use cmdline partition parser lib > > reference: https://lkml.org/lkml/2013/8/6/550 > How about some more verbose commit

Re: [PATCH] ARM: kirkwood: remove lagacy clk workarounds

2013-10-18 Thread Ezequiel Garcia
On Fri, Oct 18, 2013 at 10:47:41AM -0400, Jason Cooper wrote: > On Fri, Oct 18, 2013 at 01:54:13PM +0200, Sebastian Hesselbarth wrote: > > With legacy devices converted to DT and a proper ethernet MAC > > workaround, we can now remove the clk workarounds for legacy > > devices. While at it, also cl

Re: [PATCH 0/3] net: mv643xx_eth: various small fixes for v3.12

2013-10-04 Thread Ezequiel Garcia
On Wed, Oct 02, 2013 at 12:57:19PM +0200, Sebastian Hesselbarth wrote: > This patch set comprises some one-liners to fix issues with repeated > loading and unloading of a modular mv643xx_eth driver. > > First two patches take care of the periodic port statistic timer, that > updates statistics by

Re: [PATCH v2 2/2] input: rotary-encoder: Add 'on-each-step' to binding documentation

2013-10-04 Thread Ezequiel Garcia
On Fri, Oct 04, 2013 at 02:19:56PM +0100, Mark Rutland wrote: > On Fri, Oct 04, 2013 at 01:53:23PM +0100, Ezequiel Garcia wrote: > > The driver now supports a new mode to handle the interruptions generated > > by the device: on this new mode an input event is generated on each st

[PATCH v2 2/2] input: rotary-encoder: Add 'on-each-step' to binding documentation

2013-10-04 Thread Ezequiel Garcia
v Cc: Rob Herring Cc: devicet...@vger.kernel.org Signed-off-by: Ezequiel Garcia --- I'm not at all happy with this DT binding as it's way to customized for the current driver. For instance, if we want to support mapping key events (or better arbitrary linux-input event types) it seems th

[PATCH v2 0/2] rotary-encoder: Add new interruption handler

2013-10-04 Thread Ezequiel Garcia
event on each turn of the device. Thanks! Ezequiel Garcia (2): Input: rotary-encoder: Add 'stepped' irq handler input: rotary-encoder: Add 'on-each-step' to binding documentation .../devicetree/bindings/input/rotary-encoder.txt | 1 + drivers/input/m

[PATCH v2 1/2] Input: rotary-encoder: Add 'stepped' irq handler

2013-10-04 Thread Ezequiel Garcia
detection. Cc: Daniel Mack Cc: Dmitry Torokhov Signed-off-by: Ezequiel Garcia --- v1->v2: * Add DT property handling * Replace binary representation by hexa-decimal in the sum encoding drivers/input/misc/rotary_encoder.c | 41 + include/linux/rotary_encode

Re: [PATCH v2 1/2] ARM: kirkwood: retain MAC address for DT ethernet

2013-10-03 Thread Ezequiel Garcia
r we enable the clock, copy a possible invalid MAC address > and disable the clock again. (Added Mike, who might have an opinion about > clk_is_enabled) > > Cc: Jason Cooper > Cc: Andrew Lunn > Cc: Russell King > Cc: Grant Likely > Cc: Benjamin Herrenschmidt

Re: [PATCH 2/2] ARM: kirkwood: remove ethernet clock gate workaround

2013-10-03 Thread Ezequiel Garcia
t; --- > Cc: Jason Cooper > Cc: Andrew Lunn > Cc: Russell King > Cc: Jason Gunthorpe > Cc: Ezequiel Garcia > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > --- > arch/arm/mach-kirkwood/board-dt.c | 14 -- > 1 file changed,

Re: [PATCH] arm: mvebu: use the main timer as clock source from DT

2013-09-25 Thread Ezequiel Garcia
Without this change the kernel is stuck at: 'Calibrating delay loop...'. > > > > Signed-off-by: Jean Pihet > > Acked-by: Ezequiel Garcia > > --- > > drivers/clocksource/Kconfig | 1 + > > 1 file changed, 1 insertion(+) > > Acked-by: Jason Coo

Re: [GIT PULL] clockevents/clocksource : armada-370-xp fixes, sh_cmt, em_sti timer-sp and nomadik changes

2013-09-11 Thread Ezequiel Garcia
Hi Thomas, Daniel, On Mon, Sep 02, 2013 at 10:01:56PM +0200, Daniel Lezcano wrote: [..] > > The following changes since commit cfb6d656d569510ac9239583ce09e4c92ad54719: > > Merge branch 'timers/clockevents-next' of > git://git.linaro.org/people/dlezcano/clockevents into timers/core > (2013-08-

Re: [PATCH v4 1/4] lib: Introduce atomic MMIO modify

2013-08-28 Thread Ezequiel Garcia
Linus, Andrew suggested you might have opinions on this, so I'm cc'ing you. Since you'll probably want some better context, here it is: http://lwn.net/Articles/564709/ On Sat, Aug 24, 2013 at 12:35:29PM -0300, Ezequiel Garcia wrote: > Some platforms have MMIO regions that

Re: [PATCH v4 1/4] lib: Introduce atomic MMIO modify

2013-08-28 Thread Ezequiel Garcia
On Tue, Aug 27, 2013 at 01:37:09PM -0700, Andrew Morton wrote: > On Sat, 24 Aug 2013 12:35:29 -0300 Ezequiel Garcia > wrote: > > > Some platforms have MMIO regions that are shared across orthogonal > > subsystems. This commit implements a possible solution for the > >

Re: [PATCH v4 2/4] ARM: Add atomic_io_modify optimized routines

2013-08-28 Thread Ezequiel Garcia
On Wed, Aug 28, 2013 at 09:53:40AM +0100, Catalin Marinas wrote: > On Sat, Aug 24, 2013 at 04:35:30PM +0100, Ezequiel Garcia wrote: > > Implement arch-specific atomic_io_modify and atomic_io_modify_relaxed, > > which are based on writel/readl_relaxed and writel_relaxe

Re: [PATCH v4 1/4] lib: Introduce atomic MMIO modify

2013-08-27 Thread Ezequiel Garcia
(Adding Andrew Morton in Cc) On Sat, Aug 24, 2013 at 12:35:29PM -0300, Ezequiel Garcia wrote: > Some platforms have MMIO regions that are shared across orthogonal > subsystems. This commit implements a possible solution for the > thread-safe access of such regions through a spinlock-prot

Re: [PATCH v4 1/4] lib: Introduce atomic MMIO modify

2013-08-24 Thread Ezequiel Garcia
On Sat, Aug 24, 2013 at 10:35:34PM +0200, Richard Weinberger wrote: > Am 24.08.2013 21:58, schrieb Ezequiel Garcia: > > On Sat, Aug 24, 2013 at 08:27:10PM +0200, richard -rw- weinberger wrote: > >> On Sat, Aug 24, 2013 at 5:35 PM, Ezequiel Garcia > >> wrote: > >&g

Re: [PATCH v4 1/4] lib: Introduce atomic MMIO modify

2013-08-24 Thread Ezequiel Garcia
On Sat, Aug 24, 2013 at 08:27:10PM +0200, richard -rw- weinberger wrote: > On Sat, Aug 24, 2013 at 5:35 PM, Ezequiel Garcia > wrote: > > Some platforms have MMIO regions that are shared across orthogonal > > subsystems. This commit implements a possible solution for the > &g

[PATCH v4 4/4] watchdog: orion: Use atomic access for shared registers

2013-08-24 Thread Ezequiel Garcia
Since the timer control register is shared with the clocksource driver, use the recently introduced atomic_io_clear_set() to access such register. Signed-off-by: Ezequiel Garcia --- drivers/watchdog/orion_wdt.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers

[PATCH v4 3/4] clocksource: orion: Use atomic access for shared registers

2013-08-24 Thread Ezequiel Garcia
Replace the driver-specific thread-safe shared register API by the recently introduced atomic_io_modify(). Signed-off-by: Ezequiel Garcia --- drivers/clocksource/time-orion.c | 29 +++-- 1 file changed, 11 insertions(+), 18 deletions(-) diff --git a/drivers/clocksource

[PATCH v4 1/4] lib: Introduce atomic MMIO modify

2013-08-24 Thread Ezequiel Garcia
this protects shared-registers, it also serializes access to unrelated/unshared registers. Signed-off-by: Ezequiel Garcia --- include/linux/io.h | 5 + lib/Makefile | 2 +- lib/atomicio.c | 27 +++ 3 files changed, 33 insertions(+), 1 deletion(-) create mode

[PATCH v4 2/4] ARM: Add atomic_io_modify optimized routines

2013-08-24 Thread Ezequiel Garcia
Implement arch-specific atomic_io_modify and atomic_io_modify_relaxed, which are based on writel/readl_relaxed and writel_relaxed/readl_relaxed, respectively. In both cases, by relaxing the readl, perfomance can be improved. Signed-off-by: Ezequiel Garcia --- arch/arm/include/asm/io.h | 4

[PATCH v4 0/4] Introduce atomic MMIO modify

2013-08-24 Thread Ezequiel Garcia
suggested by Will Deacon, to ensure the writel gets completed before the spin_unlock(). Ezequiel Garcia (4): lib: Introduce atomic MMIO modify ARM: Add atomic_io_modify optimized routines clocksource: orion: Use atomic access for shared registers watchdog: orion: Use atomic access

Re: [PATCH v2 4/4] pinctrl: mvebu: Convert to use devm_ioremap_resource

2013-08-24 Thread Ezequiel Garcia
this patch. In particular, the current code is buggy (for there's a leak) and this patch fixes it. Care to write something meaningful and re-send? (except the patch Mike already took, I guess). With that changes you can add: Reviewed-by: Ezequiel Garcia T for the whole series. -- Ezequiel Ga

Re: [PATCH 4/4] pinctrl: mvebu: add missing iounmap

2013-08-23 Thread Ezequiel Garcia
On Fri, Aug 23, 2013 at 08:06:02PM +0200, Linus Walleij wrote: > On Thu, Aug 22, 2013 at 3:22 PM, Ezequiel Garcia > wrote: > > On Thu, Aug 22, 2013 at 08:46:51PM +0800, Jisheng Zhang wrote: > >> Add missing iounmap to prove error path and remove path > >> >

Re: [PATCH 4/4] pinctrl: mvebu: add missing iounmap

2013-08-22 Thread Ezequiel Garcia
Hi Jisheng, On Thu, Aug 22, 2013 at 08:46:51PM +0800, Jisheng Zhang wrote: > Add missing iounmap to prove error path and remove path > > Signed-off-by: Jisheng Zhang > --- > drivers/pinctrl/mvebu/pinctrl-mvebu.c | 24 +--- > 1 file changed, 17 insertions(+), 7 deletions(-)

Re: [PATCH 1/3] ARM: Introduce atomic MMIO clear/set

2013-08-20 Thread Ezequiel Garcia
On Tue, Aug 20, 2013 at 09:32:13AM -0500, Matt Sealey wrote: > On Mon, Aug 19, 2013 at 11:59 AM, Ezequiel Garcia > wrote: > > On Mon, Aug 12, 2013 at 07:29:42PM +0100, Will Deacon wrote: > > > >> This means that you don't have ordering guarantees between the two

Re: [RFC 08/17] ARM: mvebu: remove custom .init_time hook

2013-08-20 Thread Ezequiel Garcia
On Tue, Aug 20, 2013 at 03:12:12PM +0200, Sebastian Hesselbarth wrote: > > Maybe that's why I'm getting only the patches where I'm Cced? > > I try to keep Cc list as short as possible, so I only put Gregory as the > maintainer of mach-mvebu on Cc. > Pff... you're right, I got this because my mv

Re: [RFC 08/17] ARM: mvebu: remove custom .init_time hook

2013-08-20 Thread Ezequiel Garcia
Hi Sebastian, I've only received a few patches of this series. I *think* using "RFC" alone is not correct in LAKML and for some reason you're required to use RFC/PATCH. Maybe that's why I'm getting only the patches where I'm Cced? Could you please re-send this? The series looks promising and I'd

Re: [PATCH 1/3] ARM: Introduce atomic MMIO clear/set

2013-08-19 Thread Ezequiel Garcia
On Mon, Aug 12, 2013 at 07:29:42PM +0100, Will Deacon wrote: > On Sat, Aug 10, 2013 at 01:43:00PM +0100, Ezequiel Garcia wrote: > > Some SoC have MMIO regions that are shared across orthogonal > > subsystems. This commit implements a possible solution for the > > thread-safe a

Re: [PATCH 1/1] Resubmit mtd: mtdoops: fix for a potential memory leak in mtdoops_notify_remove

2013-08-13 Thread Ezequiel Garcia
Hi Nilanjan, On Mon, Aug 12, 2013 at 09:48:35PM -0700, Nilanjan Roychowdhury wrote: > we are allocating cxt->oops_page_used using vmalloc in mtdoops_notify_add for > every mtd_info addition but not freeing it in mtdoops_notify_remove. Also > care is > taken so that we do not free the same pointer

Re: [PATCH 1/1] mtd: mtdoops: fix for a potential memory leak in mtdoops_notify_remove

2013-08-12 Thread Ezequiel Garcia
On Sun, Aug 11, 2013 at 01:11:48PM -0700, Nilanjan Roychowdhury wrote: > we are allocating cxt->oops_page_used using vmalloc in mtdoops_notify_add for > every mtd_info addition but not freeing it in mtdoops_notify_remove > > Signed-off-by: Nilanjan Roychowdhury > --- > drivers/mtd/mtdoops.c |

Re: [PATCH 1/3] ARM: Introduce atomic MMIO clear/set

2013-08-12 Thread Ezequiel Garcia
Sebastian, On Mon, Aug 12, 2013 at 06:44:10PM +0200, Sebastian Hesselbarth wrote: > On 08/12/13 17:46, Ezequiel Garcia wrote: > >> Indeed, syscon looks like a nice match for this use case. > >> (although it still looks like an overkill to me). > >> > >>

Re: [PATCH 1/3] ARM: Introduce atomic MMIO clear/set

2013-08-12 Thread Ezequiel Garcia
On Sat, Aug 10, 2013 at 12:55:53PM -0300, Ezequiel Garcia wrote: > On Sat, Aug 10, 2013 at 07:43:08PM +0400, Alexander Shiyan wrote: > > > On Sat, Aug 10, 2013 at 11:02:38AM -0300, Ezequiel Garcia wrote: > > > > On Sat, Aug 10, 2013 at 04:49:28PM +0400, Alexander Shiyan wro

Re: [PATCH 1/3] ARM: Introduce atomic MMIO clear/set

2013-08-10 Thread Ezequiel Garcia
On Sat, Aug 10, 2013 at 07:43:08PM +0400, Alexander Shiyan wrote: > > On Sat, Aug 10, 2013 at 11:02:38AM -0300, Ezequiel Garcia wrote: > > > On Sat, Aug 10, 2013 at 04:49:28PM +0400, Alexander Shiyan wrote: > > > > > Some SoC have MMIO regions that are shared across o

Re: [PATCH 1/3] ARM: Introduce atomic MMIO clear/set

2013-08-10 Thread Ezequiel Garcia
On Sat, Aug 10, 2013 at 11:02:38AM -0300, Ezequiel Garcia wrote: > On Sat, Aug 10, 2013 at 04:49:28PM +0400, Alexander Shiyan wrote: > > > Some SoC have MMIO regions that are shared across orthogonal > > > subsystems. This commit implements a possible solution for the > &

Re: [PATCH 1/3] ARM: Introduce atomic MMIO clear/set

2013-08-10 Thread Ezequiel Garcia
with clear-set semantics. > > > > Concurrent access is protected with a single spinlock for the > > entire MMIO address space. While this protects shared-registers, > > it also serializes access to unrelated/unshared registers. > > > > Signed-off-by:

[PATCH 2/3] clocksource: orion: Use atomic access for shared registers

2013-08-10 Thread Ezequiel Garcia
Replace the driver-specific thread-safe shared register API by the recently introduced atomic_io_clear_set(). Signed-off-by: Ezequiel Garcia --- drivers/clocksource/time-orion.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/clocksource/time-orion.c b

[PATCH 3/3] watchdog: orion: Use atomic access for shared registers

2013-08-10 Thread Ezequiel Garcia
Since the timer control register is shared with the clocksource driver, use the recently introduced atomic_io_clear_set() to access such register. Signed-off-by: Ezequiel Garcia --- drivers/watchdog/orion_wdt.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/drivers

[PATCH 1/3] ARM: Introduce atomic MMIO clear/set

2013-08-10 Thread Ezequiel Garcia
address space. While this protects shared-registers, it also serializes access to unrelated/unshared registers. Signed-off-by: Ezequiel Garcia --- arch/arm/include/asm/io.h | 5 + arch/arm/kernel/io.c | 24 2 files changed, 29 insertions(+) diff --git a/arch/arm

[PATCH 0/3] Introduce atomic MMIO register clear-set

2013-08-10 Thread Ezequiel Garcia
proper solution for shared-register access. Based in v3.11-rc4. Ezequiel Garcia (3): ARM: Introduce atomic MMIO clear/set clocksource: orion: Use atomic access for shared registers watchdog: orion: Use atomic access for shared registers arch/arm/include/asm/io.h| 5 +

Re: [PATCH v2 0/4] Add phy support for AM335X platform using Generic PHy framework

2013-08-10 Thread Ezequiel Garcia
On Fri, Jul 19, 2013 at 06:04:33PM +0530, George Cherian wrote: > This patch series adds phy support for AM335X platform. > This patch series is based on Generic PHY framework [1]. > > > This series has > - adds dual musb instances support for am335x platform > - adds phy-am-

Re: [PATCH] ATA: sata_mv: Remove uneeded CONFIG_HAVE_CLK ifdefs

2013-07-29 Thread Ezequiel Garcia
Hi Andrew, On Mon, Jul 29, 2013 at 09:04:31PM +0200, Andrew Lunn wrote: > On Mon, Jul 29, 2013 at 12:21:22PM -0300, Ezequiel Garcia wrote: > > If CONFIG_HAVE_CLK is not selected, then all the clk API turn out > > into stubs, so there's no need to have the ifdefs. > > Th

Re: [PATCH] ATA: sata_mv: Remove uneeded CONFIG_HAVE_CLK ifdefs

2013-07-29 Thread Ezequiel Garcia
On Mon, Jul 29, 2013 at 12:00:16PM -0400, Tejun Heo wrote: > On Mon, Jul 29, 2013 at 12:21:22PM -0300, Ezequiel Garcia wrote: > > If CONFIG_HAVE_CLK is not selected, then all the clk API turn out > > into stubs, so there's no need to have the ifdefs. > > The only side-e

[PATCH] ATA: sata_mv: Remove uneeded CONFIG_HAVE_CLK ifdefs

2013-07-29 Thread Ezequiel Garcia
If CONFIG_HAVE_CLK is not selected, then all the clk API turn out into stubs, so there's no need to have the ifdefs. The only side-effect of this patch is the extra tiny kmalloc, but that's not enough reason to have such ugly ifdefs all around the code. Signed-off-by: Ezequ

[PATCH] ATA: sata_mv: Remove unneeded forward declaration

2013-07-29 Thread Ezequiel Garcia
These forward declarations are no longer needed, and are probably historical left-over. Signed-off-by: Ezequiel Garcia --- drivers/ata/sata_mv.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/ata/sata_mv.c b/drivers/ata/sata_mv.c index 35c6b6d..10ea99a 100644 --- a/drivers/ata

Re: [PATCH 3/5] ARM: dove: add MBus DT node

2013-07-29 Thread Ezequiel Garcia
On Mon, Jul 29, 2013 at 04:23:20PM +0200, Sebastian Hesselbarth wrote: > On 07/29/2013 03:52 PM, Ezequiel Garcia wrote: > > Hi Sebastian, > > > > (Ccing devicetree ML) > > > > On Mon, Jul 29, 2013 at 02:36:46PM +0200, Sebastian Hesselbarth wrote: > >> On

Re: [PATCH 4/5] ARM: dove: relocate internal registers device nodes

2013-07-29 Thread Ezequiel Garcia
c: Jason Cooper > Cc: Andrew Lunn > Cc: Ezequiel Garcia > Cc: linux-arm-ker...@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > --- > arch/arm/boot/dts/dove.dtsi | 955 > +-- > 1 file changed, 474 insertions(+), 481 de

Re: [PATCH 0/5] ARM: dove: switch to DT MBus

2013-07-29 Thread Ezequiel Garcia
Hello Sebastian, On Mon, Jul 29, 2013 at 02:31:49PM +0200, Sebastian Hesselbarth wrote: > This patch set converts DT based Dove boards to DT probed MBus to be > introduced with v3.12. > > The first two patches prepare Dove DT nodes for MBus ID remapping by using > preprocessor includes and adding

Re: [PATCH 3/5] ARM: dove: add MBus DT node

2013-07-29 Thread Ezequiel Garcia
Hi Sebastian, (Ccing devicetree ML) On Mon, Jul 29, 2013 at 02:36:46PM +0200, Sebastian Hesselbarth wrote: > On 07/29/2013 02:31 PM, Sebastian Hesselbarth wrote: > > This adds a MBus node including ranges and pcie apertures required later. > > > > Signed-off-by: Sebastian Hesselbarth > > --- > >

Re: linux-next: Tree for Jul 24 (media/usb/stk1160)

2013-07-24 Thread Ezequiel Garcia
Hello Randy, On Wed, Jul 24, 2013 at 10:32:57AM -0700, Randy Dunlap wrote: > On 07/23/13 23:32, Stephen Rothwell wrote: > > Hi all, > > > > Changes since 20130723: > > > > on x86_64: > > > drivers/built-in.o: In function `stk1160_ac97_register': > (.text+0x414e77): undefined reference to `snd

Re: [PATCH V2 6/6] pci: mvebu: Convert to use devm_ioremap_resource

2013-07-23 Thread Ezequiel Garcia
source() and deprecated the use of > > devm_request_and_ioremap(). > > > > While at it, modify mvebu_pcie_map_registers() to propagate error code. > > > > Signed-off-by: Tushar Behera > > CC: linux-...@vger.kernel.org > > CC: Bjorn Helgaas > > Acked-by: Bjorn Helg

Re: [PATCH 0/5] Add phy support for AM335X platform using Generic PHy framework

2013-07-08 Thread Ezequiel Garcia
Hi, On Mon, Jul 08, 2013 at 09:44:33PM +0200, Sebastian Andrzej Siewior wrote: > We need two nodes each one with a glue layer and a musb child node. The > instances crap in kernel has to vanish. Also that means your phy nodes > are wrong. This is not musb with two ports but two musb instances eac

Re: [PATCH 0/6] Marvell Orion SoC irqchip and clocksource

2013-06-11 Thread Ezequiel Garcia
rce > ARM: dove: convert to DT irqchip and clocksource > ARM: kirkwood: convert to DT irqchip and clocksource > I've done some tests on my Kirkwood Openblocks A6 using the latest branch orion-irqchip-for-v3.11-v4. Everything works fine so: Tested-by: Ezequiel Garcia For patches

Re: [PATCH] drivers: memory: Introduce Marvell EBU Device Bus driver

2013-05-16 Thread Ezequiel Garcia
Hi Greg, On Mon, May 13, 2013 at 04:12:34PM -0400, Jason Cooper wrote: > On Tue, Apr 23, 2013 at 04:55:58PM -0300, Ezequiel Garcia wrote: > > On Tue, Apr 23, 2013 at 12:48:22PM -0700, Greg KH wrote: > > > On Tue, Apr 23, 2013 at 04:21:26PM -0300, Ezequiel Garcia wrote: [...]

Re: Splitting stk1160-ac97 as a module (Re: linux-next: Tree for May 1 (media/usb/stk1160))

2013-05-06 Thread Ezequiel Garcia
On Sat, May 04, 2013 at 09:59:50PM +0200, Yann E. MORIN wrote: > Ezequiel, All, > > On Sat, May 04, 2013 at 02:21:44PM -0300, Ezequiel Garcia wrote: > > I'm trying to split the ac97 support into a separate module. > > So far I've managed to do this with two differ

Splitting stk1160-ac97 as a module (Re: linux-next: Tree for May 1 (media/usb/stk1160))

2013-05-04 Thread Ezequiel Garcia
Hi Mauro, On Thu, May 02, 2013 at 11:52:33AM -0300, Mauro Carvalho Chehab wrote: > > > > is unreliable (doesn't do what some people expect) when SND=m and > > SND_AC97_CODEC=m, > > since VIDEO_STK1160_AC97 is a bool. > > Using select is always tricky. > > I can see a few possible fixes for it:

Re: [PATCH] ARM: kirkwood: add button for OpenBlock A6

2013-05-02 Thread Ezequiel Garcia
Hi Gregory, On Thu, May 02, 2013 at 09:43:22PM +0200, Gregory CLEMENT wrote: > The OpenBlocks A6 board has one software-controlled button on the > front side, labeled "INIT", so we add minimal support for this button > in the kernel. > > Signed-off-by: Gregory CLEMENT > --- > arch/arm/boot/dts/

Re: [PATCH] drivers: memory: Introduce Marvell EBU Device Bus driver

2013-04-23 Thread Ezequiel Garcia
On Tue, Apr 23, 2013 at 12:48:22PM -0700, Greg KH wrote: > On Tue, Apr 23, 2013 at 04:21:26PM -0300, Ezequiel Garcia wrote: > > From: Ezequiel Garcia > > > > Marvell EBU SoCs such as Armada 370/XP, Orion5x (88f5xxx) and > > Discovery (mv78xx0) supports a Device Bus c

[PATCH] drivers: memory: Introduce Marvell EBU Device Bus driver

2013-04-23 Thread Ezequiel Garcia
From: Ezequiel Garcia Marvell EBU SoCs such as Armada 370/XP, Orion5x (88f5xxx) and Discovery (mv78xx0) supports a Device Bus controller to access several kinds of memories and I/O devices (NOR, NAND, SRAM, FPGA). This commit adds a driver to handle this controller. So far only Armada 370

Re: [BUG][mvebu] mvneta: cannot request irq 25 on openblocks-ax3

2013-03-19 Thread Ezequiel Garcia
Hi Masami, On Tue, Mar 19, 2013 at 10:12:37PM +0900, Masami Hiramatsu wrote: > > Here I've hit a bug on the recent kernel. As far as I know, this bug > exists on 3.9-rc1 too. > > When I tried the latest mvebu for-next tree > (git://git.infradead.org/users/jcooper/linux.git mvebu/for-next), > I g

Re: [RFC V1 0/8] Add a driver for somagic smi2021

2013-03-17 Thread Ezequiel Garcia
Hi Jon, On Sun, Mar 17, 2013 at 09:01:58PM +0100, Jon Arne Jørgensen wrote: > On Fri, Mar 15, 2013 at 09:08:58AM -0300, Ezequiel Garcia wrote: > > On Thu, Mar 14, 2013 at 03:06:56PM +0100, Jon Arne Jørgensen wrote: > > > This patch-set will add a driver for the So

Re: [RFC V1 5/8] smi2021: Add smi2021_video.c

2013-03-15 Thread Ezequiel Garcia
thout even the implied warranty of > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this pro

Re: [RFC V1 4/8] smi2021: Add smi2021_v4l2.c

2013-03-15 Thread Ezequiel Garcia
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, see <http://www.gnu.org/licenses/

Re: [RFC V1 3/8] smi2021: Add smi2021_i2c.c

2013-03-15 Thread Ezequiel Garcia
ESS FOR A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, see <http://www.gnu.org/licenses/>. > + * > + * This driver is hea

Re: [RFC V1 2/8] smi2021: Add smi2021_main.c

2013-03-15 Thread Ezequiel Garcia
A PARTICULAR PURPOSE. See the > + * GNU General Public License for more details. > + * > + * You should have received a copy of the GNU General Public License > + * along with this program; if not, see <http://www.gnu.org/licenses/>. > + * > + * This driver is heavily influense

Re: [RFC V1 1/8] smi2021: Add the header file

2013-03-15 Thread Ezequiel Garcia
You should have received a copy of the GNU General Public License > + * along with this program; if not, see <http://www.gnu.org/licenses/>. > + * > + * This driver is heavily influensed by the STK1160 driver. > + * Copyright (C) 2012 Ezequiel Garcia > + * Cool ;-) >

Re: [RFC V1 0/8] Add a driver for somagic smi2021

2013-03-15 Thread Ezequiel Garcia
On Thu, Mar 14, 2013 at 03:06:56PM +0100, Jon Arne Jørgensen wrote: > This patch-set will add a driver for the Somagic SMI2021 chip. > > This chip is found inside different usb video-capture devices. > Most of them are branded as EasyCap, but there also seems to be > some other brands selling devi

Re: [PATCH 1/2] ARM: dts: omap3: Add gpmc node

2013-02-16 Thread Ezequiel Garcia
Hi Anil, On Sat, Feb 16, 2013 at 05:08:27PM +0530, Anil Kumar wrote: > Add gpmc DT node. > > Signed-off-by: Anil Kumar > --- > :100644 100644 1acc261... 9f36531... March/arm/boot/dts/omap3.dtsi > arch/arm/boot/dts/omap3.dtsi | 11 +++ > 1 files changed, 11 insertions(+), 0 del

Re: Question about printking

2013-02-04 Thread Ezequiel Garcia
Hi Joe, On Sat, Feb 2, 2013 at 4:43 PM, Joe Perches wrote: > On Sat, 2013-02-02 at 16:30 -0300, Ezequiel Garcia wrote: >> ptr = kmalloc(sizeof(foo)); >> if (!ptr) { >> pr_err("Cannot allocate memory for foo\n"); >> return -ENOMEM; >>

Re: [PATCH 37/62] mtd: convert to idr_alloc()

2013-02-04 Thread Ezequiel Garcia
On Sat, Feb 02, 2013 at 05:20:38PM -0800, Tejun Heo wrote: > Convert to the much saner new idr interface. > > Only compile tested. > Tested-by: Ezequiel Garcia -- Ezequiel García, Free Electrons Embedded Linux, Kernel and Android Engineering http://free-electrons.com -- To unsu

Re: [PATCHv3] ARM: sunxi: gpio: Add Allwinner SoCs GPIO drivers

2013-01-26 Thread Ezequiel Garcia
Hi Maxime, On Sat, Jan 26, 2013 at 6:31 AM, Maxime Ripard wrote: > The IP responsible for the muxing on the Allwinner SoCs are also > handling the GPIOs on the system. This patch adds the needed driver that > relies on the pinctrl driver for most of its operations. > > The number of pins availabl

Re: [RFC/PATCH] scripts/tracing: Add trace_analyze.py tool

2013-01-24 Thread Ezequiel Garcia
On Thu, Jan 24, 2013 at 2:50 AM, Minchan Kim wrote: > On Wed, Jan 23, 2013 at 06:37:56PM -0300, Ezequiel Garcia wrote: > >> >> > 2. Does it support alloc_pages family? >> >kmem event trace already supports it. If it supports, maybe we can >> >

Re: [RFC/PATCH] scripts/tracing: Add trace_analyze.py tool

2013-01-23 Thread Ezequiel Garcia
Hi Minchan, On Wed, Jan 23, 2013 at 1:27 AM, Minchan Kim wrote: > Hi Ezequiel, > > On Tue, Jan 22, 2013 at 06:46:58AM -0300, Ezequiel Garcia wrote: >> From: Ezequiel Garcia >> >> The purpose of trace_analyze.py tool is to perform static >> and dynamic memory an

Re: [RFC/PATCH] scripts/tracing: Add trace_analyze.py tool

2013-01-22 Thread Ezequiel Garcia
Hi Pekka, On Tue, Jan 22, 2013 at 10:41 AM, Pekka Enberg wrote: > (Adding acme to CC.) > > On Tue, Jan 22, 2013 at 11:46 AM, Ezequiel Garcia > wrote: >> From: Ezequiel Garcia >> >> The purpose of trace_analyze.py tool is to perform static >> and dynamic

[RFC/PATCH] scripts/tracing: Add trace_analyze.py tool

2013-01-22 Thread Ezequiel Garcia
From: Ezequiel Garcia The purpose of trace_analyze.py tool is to perform static and dynamic memory analysis using a kmem ftrace log file and a built kernel tree. This script and related work has been done on the CEWG/2012 project: "Kernel dynamic memory allocation tracking and reduction&q

Re: [PATCH 2/4] staging: media: go7007: firmware protection Protection for unfirmware load

2013-01-16 Thread Ezequiel Garcia
Hi Volokh, On Wed, Jan 16, 2013 at 10:42 AM, Volokh Konstantin wrote: > On Wed, Jan 16, 2013 at 04:35:45PM +0300, Dan Carpenter wrote: >> The problem is that the firmware was being unloaded on disconnect? >> > If no firmware was loaded (no exists,wrong or some error) then rmmod fails > with OOPS

Re: [PATCH 01/23] uvc: Replace memcpy with struct assignment

2013-01-12 Thread Ezequiel Garcia
On Wed, Jan 9, 2013 at 9:19 PM, Laurent Pinchart wrote: > Hi Ezequiel, > > On Thursday 27 December 2012 21:49:37 Mauro Carvalho Chehab wrote: >> Em Thu, 27 Dec 2012 18:12:46 -0300 Ezequiel Garcia escreveu: >> > On Tue, Oct 23, 2012 at 4:57 PM, Ezequiel Garcia wrote: &g

Re: [PATCH 01/23] uvc: Replace memcpy with struct assignment

2012-12-27 Thread Ezequiel Garcia
Mauro, On Tue, Oct 23, 2012 at 4:57 PM, Ezequiel Garcia wrote: > This kind of memcpy() is error-prone. Its replacement with a struct > assignment is prefered because it's type-safe and much easier to read. > > Found by coccinelle. Hand patched and reviewed. > Tested by com

<    6   7   8   9   10   11   12   13   >