Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-26 Thread Grant Likely
On Mon, 26 May 2014 12:57:32 +0200, Geert Uytterhoeven wrote: > Hi Grant, > > On Mon, May 26, 2014 at 12:48 PM, Grant Likely > wrote: > > On Tue, 20 May 2014 09:38:49 +0200, Geert Uytterhoeven > > wrote: > >> On Tue, May 20, 2014 at 7:50 AM, Grant Lik

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-26 Thread Grant Likely
On Tue, 20 May 2014 09:38:49 +0200, Geert Uytterhoeven wrote: > Hi Grant, > > On Tue, May 20, 2014 at 7:50 AM, Grant Likely > wrote: > >> Why has the overlay system been designed for plugging and unpluging whole > >> overlays? > >> That means the kernel

Re: [PATCH v2 2/2] of: Stop naming platform_device using dcr address

2014-05-26 Thread Grant Likely
On Sat, May 24, 2014 at 5:10 AM, Benjamin Herrenschmidt wrote: > On Fri, 2014-05-23 at 08:36 +0900, Grant Likely wrote: >> There is now a way to ensure all platform devices get a unique name when >> populated from the device tree, and the DCR_NATIVE code path is broken >>

Re: [PATCH v2 2/2] of: Stop naming platform_device using dcr address

2014-05-26 Thread Grant Likely
On Sat, May 24, 2014 at 5:10 AM, Benjamin Herrenschmidt b...@kernel.crashing.org wrote: On Fri, 2014-05-23 at 08:36 +0900, Grant Likely wrote: There is now a way to ensure all platform devices get a unique name when populated from the device tree, and the DCR_NATIVE code path is broken anyway

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-26 Thread Grant Likely
On Tue, 20 May 2014 09:38:49 +0200, Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Grant, On Tue, May 20, 2014 at 7:50 AM, Grant Likely grant.lik...@secretlab.ca wrote: Why has the overlay system been designed for plugging and unpluging whole overlays? That means the kernel has

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-26 Thread Grant Likely
On Mon, 26 May 2014 12:57:32 +0200, Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Grant, On Mon, May 26, 2014 at 12:48 PM, Grant Likely grant.lik...@secretlab.ca wrote: On Tue, 20 May 2014 09:38:49 +0200, Geert Uytterhoeven ge...@linux-m68k.org wrote: On Tue, May 20, 2014 at 7:50

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-26 Thread Grant Likely
On Mon, 26 May 2014 14:55:37 +0300, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: Hi Grant, On May 26, 2014, at 2:23 PM, Grant Likely wrote: On Mon, 26 May 2014 12:57:32 +0200, Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Grant, On Mon, May 26, 2014 at 12:48 PM

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-23 Thread Grant Likely
On Thu, 22 May 2014 18:14:38 -0700, Frank Rowand wrote: > On 5/21/2014 6:16 PM, Grant Likely wrote: > > On Tue, 20 May 2014 19:41:22 -0700, Frank Rowand > > wrote: > >> On 5/18/2014 2:27 AM, Grant Likely wrote: > >>> On Fri, 16 May 2014 11:54:44 +0100, Grant

[PATCH v2 1/2] of: Ensure unique names without sacrificing determinism

2014-05-23 Thread Grant Likely
number to the end of it. Signed-off-by: Grant Likely Cc: Ezequiel Garcia Cc: Rob Herring --- drivers/of/platform.c | 40 +++- 1 file changed, 19 insertions(+), 21 deletions(-) diff --git a/drivers/of/platform.c b/drivers/of/platform.c index d0009b3614af

[PATCH v2 2/2] of: Stop naming platform_device using dcr address

2014-05-23 Thread Grant Likely
and associated ugly #ifdef. The user-visible impact of this patch is that any DCR device on Cell will get a new name in the /sys/devices hierarchy. Signed-off-by: Grant Likely Cc: Rob Herring Cc: Benjamin Herrenschmidt --- arch/powerpc/include/asm/dcr-mmio.h | 4 arch/powerpc/sysdev/dcr.c

Re: [PATCH v1] of/irq: do irq resolution in platform_get_irq_byname()

2014-05-23 Thread Grant Likely
isn't complete because platform_get_irq_byname() > need to be modified the same way. > > Hence, fix it by adding interrupt resolution code at the > platform_get_irq_byname() function too. > > Cc: Russell King > Cc: Rob Herring > Cc: Tony Lindgren > Cc: Grant Likely &

[PATCH v2 0/2] of: Clean up naming of platform devices

2014-05-23 Thread Grant Likely
This series makes of_device_make_bus_id() more reliable in choosing a unique name and clears out some historical cruft. I've rebased it onto Rob's for-next tree so that it plays well with the removal of of_can_translate_address() -- To unsubscribe from this list: send the line "unsubscribe

[PATCH v2 0/2] of: Clean up naming of platform devices

2014-05-23 Thread Grant Likely
This series makes of_device_make_bus_id() more reliable in choosing a unique name and clears out some historical cruft. I've rebased it onto Rob's for-next tree so that it plays well with the removal of of_can_translate_address() -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH v1] of/irq: do irq resolution in platform_get_irq_byname()

2014-05-23 Thread Grant Likely
: Grant Likely grant.lik...@linaro.org Cc: Thierry Reding thierry.red...@gmail.com Signed-off-by: Grygorii Strashko grygorii.stras...@ti.com Applied, Thanks. g. --- Changes in v1: - use of_property_match_string() to get IRQ index by name - minor comments fixed RFC: https://lkml.org/lkml

[PATCH v2 2/2] of: Stop naming platform_device using dcr address

2014-05-23 Thread Grant Likely
and associated ugly #ifdef. The user-visible impact of this patch is that any DCR device on Cell will get a new name in the /sys/devices hierarchy. Signed-off-by: Grant Likely grant.lik...@linaro.org Cc: Rob Herring r...@kernel.org Cc: Benjamin Herrenschmidt b...@kernel.crashing.org --- arch/powerpc/include

[PATCH v2 1/2] of: Ensure unique names without sacrificing determinism

2014-05-23 Thread Grant Likely
number to the end of it. Signed-off-by: Grant Likely grant.lik...@linaro.org Cc: Ezequiel Garcia ezequ...@vanguardiasur.com.ar Cc: Rob Herring r...@kernel.org --- drivers/of/platform.c | 40 +++- 1 file changed, 19 insertions(+), 21 deletions(-) diff --git

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-23 Thread Grant Likely
On Thu, 22 May 2014 18:14:38 -0700, Frank Rowand frowand.l...@gmail.com wrote: On 5/21/2014 6:16 PM, Grant Likely wrote: On Tue, 20 May 2014 19:41:22 -0700, Frank Rowand frowand.l...@gmail.com wrote: On 5/18/2014 2:27 AM, Grant Likely wrote: On Fri, 16 May 2014 11:54:44 +0100, Grant

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-21 Thread Grant Likely
raw_spin_unlock_irqrestore(_lock, flags); > + return np; > + } > + Special case for the root node? Could use a comment, and of_allnodes will already point to the root node so this could simply be: if (strcmp(path, "/") == 0) return of_n

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-21 Thread Grant Likely
On Tue, 20 May 2014 19:41:22 -0700, Frank Rowand wrote: > On 5/18/2014 2:27 AM, Grant Likely wrote: > > On Fri, 16 May 2014 11:54:44 +0100, Grant Likely > > wrote: > >> On Thu, 15 May 2014 19:51:17 -0700, Frank Rowand > >> wrote: > >>> On 5/13/

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-21 Thread Grant Likely
On Tue, 20 May 2014 19:55:45 -0700, Frank Rowand wrote: > On 5/13/2014 7:58 AM, Grant Likely wrote: > > > Make of_find_node_by_path() handle aliases as prefixes. To make this > > > work the name search is refactored to search by path component instead > > >

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-21 Thread Grant Likely
On Tue, 20 May 2014 19:55:45 -0700, Frank Rowand frowand.l...@gmail.com wrote: On 5/13/2014 7:58 AM, Grant Likely wrote: Make of_find_node_by_path() handle aliases as prefixes. To make this work the name search is refactored to search by path component instead of by full string

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-21 Thread Grant Likely
On Tue, 20 May 2014 19:41:22 -0700, Frank Rowand frowand.l...@gmail.com wrote: On 5/18/2014 2:27 AM, Grant Likely wrote: On Fri, 16 May 2014 11:54:44 +0100, Grant Likely grant.lik...@linaro.org wrote: On Thu, 15 May 2014 19:51:17 -0700, Frank Rowand frowand.l...@gmail.com wrote: On 5

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-21 Thread Grant Likely
node so this could simply be: if (strcmp(path, /) == 0) return of_node_get(np); Here's a complete patch: commit adc96db6c39ef7b895e75d30dbc69781f6443f1d Author: Grant Likely grant.lik...@linaro.org Date: Thu May 22 11:55:31 2014 +0900 fix trailing '/' case diff

Re: [RFC PATCH] of/irq: do irq resolution in platform_get_irq_byname()

2014-05-20 Thread Grant Likely
On Mon, 19 May 2014 14:57:39 +0200, Thierry Reding wrote: > On Mon, May 19, 2014 at 04:30:59PM +0300, Grygorii Strashko wrote: > [...] > > diff --git a/drivers/of/irq.c b/drivers/of/irq.c > [...] > > /** > > + * of_irq_get_byname - Decode a node's IRQ and return it as a Linux irq > > number >

Re: [RFC PATCH] of/irq: do irq resolution in platform_get_irq_byname()

2014-05-20 Thread Grant Likely
On Mon, 19 May 2014 14:57:39 +0200, Thierry Reding thierry.red...@gmail.com wrote: On Mon, May 19, 2014 at 04:30:59PM +0300, Grygorii Strashko wrote: [...] diff --git a/drivers/of/irq.c b/drivers/of/irq.c [...] /** + * of_irq_get_byname - Decode a node's IRQ and return it as a Linux irq

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-19 Thread Grant Likely
On Fri, 16 May 2014 13:52:42 +0200, Geert Uytterhoeven wrote: > Hi Grant, > > On Fri, May 16, 2014 at 12:58 PM, Grant Likely > wrote: > > On Thu, 15 May 2014 09:20:24 +0200, Geert Uytterhoeven > > wrote: > >> On Thu, May 15, 2014 at 9:12 AM, Pantelis Anto

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-19 Thread Grant Likely
On Fri, 16 May 2014 13:52:42 +0200, Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Grant, On Fri, May 16, 2014 at 12:58 PM, Grant Likely grant.lik...@secretlab.ca wrote: On Thu, 15 May 2014 09:20:24 +0200, Geert Uytterhoeven ge...@linux-m68k.org wrote: On Thu, May 15, 2014 at 9:12

Re: [RFC PATCH 2/9] dt: deps: dependency based device creation

2014-05-18 Thread Grant Likely
Hi Tomasz, Thanks for weighing in on this. Thoughts and comments below. On Sat, 17 May 2014 16:24:19 +0200, Tomasz Figa wrote: > Hi, > > On 14.05.2014 16:05, Grant Likely wrote: > > On Mon, 12 May 2014 18:47:53 +0200, Alexander Holler > > wrote: > >> Use the

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-18 Thread Grant Likely
On Fri, 16 May 2014 11:54:44 +0100, Grant Likely wrote: > On Thu, 15 May 2014 19:51:17 -0700, Frank Rowand > wrote: > > On 5/13/2014 7:58 AM, Grant Likely wrote: > > > Make of_find_node_by_path() handle aliases as prefixes. To make this > > > work the name

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-18 Thread Grant Likely
On Fri, 16 May 2014 11:54:44 +0100, Grant Likely grant.lik...@linaro.org wrote: On Thu, 15 May 2014 19:51:17 -0700, Frank Rowand frowand.l...@gmail.com wrote: On 5/13/2014 7:58 AM, Grant Likely wrote: Make of_find_node_by_path() handle aliases as prefixes. To make this work the name

Re: [RFC PATCH 2/9] dt: deps: dependency based device creation

2014-05-18 Thread Grant Likely
Hi Tomasz, Thanks for weighing in on this. Thoughts and comments below. On Sat, 17 May 2014 16:24:19 +0200, Tomasz Figa tomasz.f...@gmail.com wrote: Hi, On 14.05.2014 16:05, Grant Likely wrote: On Mon, 12 May 2014 18:47:53 +0200, Alexander Holler hol...@ahsoftware.de wrote: Use

Re: [PATCH 1/3] lib: add glibc style strchrnul() variant

2014-05-16 Thread Grant Likely
On Thu, 15 May 2014 15:19:00 -0700, Frank Rowand wrote: > On 5/13/2014 7:58 AM, Grant Likely wrote: > > The strchrnul() variant helpfully returns a the end of the string > > instead of a NULL if the requested character is not found. This can > > simplify string parsing code

[GIT PULL] Devicetree fixes for v3.15

2014-05-16 Thread Grant Likely
Hi Linus, Please pull the following bug fix branch. Thanks, g. The following changes since commit 14186fea0cb06bc43181ce239efe0df6f1af260a: Merge tag 'locks-v3.15-4' of git://git.samba.org/jlayton/linux (2014-05-13 11:33:09 +0900) are available in the git repository at:

Re: [RFC PATCH 2/9] dt: deps: dependency based device creation

2014-05-16 Thread Grant Likely
On Wed, 14 May 2014 23:10:39 +0200, Alexander Holler wrote: > Am 14.05.2014 22:06, schrieb Grant Likely: > > On Wed, 14 May 2014 16:49:05 +0200, Alexander Holler > > wrote: > >> Am 14.05.2014 16:05, schrieb Grant Likely: > >>> On Mon, 12 May 2014 18:47:53 +

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-16 Thread Grant Likely
On Thu, 15 May 2014 09:20:24 +0200, Geert Uytterhoeven wrote: > Hi Pantelis, > > On Thu, May 15, 2014 at 9:12 AM, Pantelis Antoniou > wrote: > >> We also need to think about kexec. Kexec works by sucking the live tree > >> out of the kernel and creating a .dtb from it to pass to the new

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-16 Thread Grant Likely
On Thu, 15 May 2014 19:51:17 -0700, Frank Rowand wrote: > On 5/13/2014 7:58 AM, Grant Likely wrote: > > Make of_find_node_by_path() handle aliases as prefixes. To make this > > work the name search is refactored to search by path component instead > > of by full string.

Re: [PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-16 Thread Grant Likely
On Thu, 15 May 2014 19:51:17 -0700, Frank Rowand frowand.l...@gmail.com wrote: On 5/13/2014 7:58 AM, Grant Likely wrote: Make of_find_node_by_path() handle aliases as prefixes. To make this work the name search is refactored to search by path component instead of by full string. This should

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-16 Thread Grant Likely
On Thu, 15 May 2014 09:20:24 +0200, Geert Uytterhoeven ge...@linux-m68k.org wrote: Hi Pantelis, On Thu, May 15, 2014 at 9:12 AM, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: We also need to think about kexec. Kexec works by sucking the live tree out of the kernel and creating

Re: [RFC PATCH 2/9] dt: deps: dependency based device creation

2014-05-16 Thread Grant Likely
On Wed, 14 May 2014 23:10:39 +0200, Alexander Holler hol...@ahsoftware.de wrote: Am 14.05.2014 22:06, schrieb Grant Likely: On Wed, 14 May 2014 16:49:05 +0200, Alexander Holler hol...@ahsoftware.de wrote: Am 14.05.2014 16:05, schrieb Grant Likely: On Mon, 12 May 2014 18:47:53 +0200

[GIT PULL] Devicetree fixes for v3.15

2014-05-16 Thread Grant Likely
Hi Linus, Please pull the following bug fix branch. Thanks, g. The following changes since commit 14186fea0cb06bc43181ce239efe0df6f1af260a: Merge tag 'locks-v3.15-4' of git://git.samba.org/jlayton/linux (2014-05-13 11:33:09 +0900) are available in the git repository at:

Re: [PATCH 1/3] lib: add glibc style strchrnul() variant

2014-05-16 Thread Grant Likely
On Thu, 15 May 2014 15:19:00 -0700, Frank Rowand frowand.l...@gmail.com wrote: On 5/13/2014 7:58 AM, Grant Likely wrote: The strchrnul() variant helpfully returns a the end of the string instead of a NULL if the requested character is not found. This can simplify string parsing code since

Re: [PATCH 3/3] of: Handle memory@0 node on PPC32 only

2014-05-15 Thread Grant Likely
On Thu, 24 Apr 2014 10:26:42 +0100, Leif Lindholm wrote: > On Wed, Apr 23, 2014 at 02:10:58PM +0100, Grant Likely wrote: > > > Does anyone have a LongTrail DT to hand, and if so does the root have a > > > compatible string? From grepping through the kernel I could only find

Re: [PATCH 2/3] mips: dts: add device_type="memory" where missing

2014-05-15 Thread Grant Likely
On Tue, Apr 22, 2014 at 2:13 PM, Grant Likely wrote: > On Thu, 17 Apr 2014 18:42:00 +0100, Leif Lindholm > wrote: >> A few platforms lack a 'device_type = "memory"' for their memory >> nodes, relying on an old ppc quirk in order to discover its memory. >&

Re: [PATCH 1/3] arm: dts: add device_type="memory" for ste-ccu8540

2014-05-15 Thread Grant Likely
On Tue, Apr 22, 2014 at 2:26 PM, Linus Walleij wrote: > On Thu, Apr 17, 2014 at 7:41 PM, Leif Lindholm > wrote: > >> The current .dts for ste-ccu8540 lacks a 'device_type = "memory"' for >> its memory node, relying on an old ppc quirk in order to discover its >> memory. Add this, to permit that

Re: [PATCH] of: fix CONFIG_OF=n prototype of of_node_full_name()

2014-05-15 Thread Grant Likely
On Thu, 15 May 2014 14:44:30 +1000, Stephen Rothwell wrote: > Make the CONFIG_OF=n prototpe of of_node_full_name() mateh the CONFIG_OF=y > version. > > Fixes compile warnings like this: > > sound/soc/soc-core.c: In function 'soc_check_aux_dev': > sound/soc/soc-core.c:1667:3: warning: passing

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-15 Thread Grant Likely
Hi Pantelis, Thanks for writing this up. A few responses below... On Thu, 15 May 2014 00:12:17 -0700, Pantelis Antoniou wrote: > On May 14, 2014, at 3:08 AM, Grant Likely wrote: > > On Fri, 4 Apr 2014 15:43:55 +0300, Pantelis Antoniou > > wrote: > > The notification i

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-15 Thread Grant Likely
Hi Pantelis, Thanks for writing this up. A few responses below... On Thu, 15 May 2014 00:12:17 -0700, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: On May 14, 2014, at 3:08 AM, Grant Likely wrote: On Fri, 4 Apr 2014 15:43:55 +0300, Pantelis Antoniou pantelis.anton

Re: [PATCH] of: fix CONFIG_OF=n prototype of of_node_full_name()

2014-05-15 Thread Grant Likely
On Thu, 15 May 2014 14:44:30 +1000, Stephen Rothwell s...@canb.auug.org.au wrote: Make the CONFIG_OF=n prototpe of of_node_full_name() mateh the CONFIG_OF=y version. Fixes compile warnings like this: sound/soc/soc-core.c: In function 'soc_check_aux_dev': sound/soc/soc-core.c:1667:3:

Re: [PATCH 1/3] arm: dts: add device_type=memory for ste-ccu8540

2014-05-15 Thread Grant Likely
On Tue, Apr 22, 2014 at 2:26 PM, Linus Walleij linus.wall...@linaro.org wrote: On Thu, Apr 17, 2014 at 7:41 PM, Leif Lindholm leif.lindh...@linaro.org wrote: The current .dts for ste-ccu8540 lacks a 'device_type = memory' for its memory node, relying on an old ppc quirk in order to discover

Re: [PATCH 2/3] mips: dts: add device_type=memory where missing

2014-05-15 Thread Grant Likely
On Tue, Apr 22, 2014 at 2:13 PM, Grant Likely grant.lik...@secretlab.ca wrote: On Thu, 17 Apr 2014 18:42:00 +0100, Leif Lindholm leif.lindh...@linaro.org wrote: A few platforms lack a 'device_type = memory' for their memory nodes, relying on an old ppc quirk in order to discover its memory

Re: [PATCH 3/3] of: Handle memory@0 node on PPC32 only

2014-05-15 Thread Grant Likely
On Thu, 24 Apr 2014 10:26:42 +0100, Leif Lindholm leif.lindh...@linaro.org wrote: On Wed, Apr 23, 2014 at 02:10:58PM +0100, Grant Likely wrote: Does anyone have a LongTrail DT to hand, and if so does the root have a compatible string? From grepping through the kernel I could only find

Re: [RFC PATCH 2/9] dt: deps: dependency based device creation

2014-05-14 Thread Grant Likely
On Wed, 14 May 2014 16:49:05 +0200, Alexander Holler wrote: > Am 14.05.2014 16:05, schrieb Grant Likely: > > On Mon, 12 May 2014 18:47:53 +0200, Alexander Holler > > wrote: > >> Use the properties named 'dependencies' in binary device tree blobs to > >&

Re: [RFC PATCH 5/9] dt: deps: register drivers based on the initialization order based on DT

2014-05-14 Thread Grant Likely
On Wed, May 14, 2014 at 3:58 PM, Alexander Holler wrote: > Am 14.05.2014 16:13, schrieb Grant Likely: > >> On Mon, 12 May 2014 18:47:56 +0200, Alexander Holler >> wrote: >>> >>> The init system currently calls unknown functions with almost unknown >>

Re: [RFC PATCH 0/9] dt: dependencies (for deterministic driver initialization order based on the DT)

2014-05-14 Thread Grant Likely
On Wed, May 14, 2014 at 4:02 PM, Alexander Holler wrote: > Am 14.05.2014 16:19, schrieb Grant Likely: > > >> Rather than a dtb schema change, for the most common properties (irqs, >> clocks, gpios), we could extract dependencies at boot time. I don't like >> the idea o

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-14 Thread Grant Likely
On Wed, 14 May 2014 14:11:52 +0200, Michael Stickel wrote: > Hi Grant, > > Am 14.05.2014 12:08, schrieb Grant Likely: > > More generally I am concerned about whether or not overlays > > will introduce corner cases that can never be handled correctly, > > particular

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-14 Thread Grant Likely
On Wed, 14 May 2014 15:03:35 +0200, Geert Uytterhoeven wrote: > On Wed, May 14, 2014 at 12:08 PM, Grant Likely > wrote: > >> +config OF_OVERLAY > >> + bool "OF overlay support" > >> + depends on OF > >> + select OF_DYNAMIC >

Re: [PATCH 0/6] DT early console initialization

2014-05-14 Thread Grant Likely
e path of the serial port. > > This series is dependent on generic earlycon[1], libfdt support[2], and > vmlinux.lds.h clean-ups[3]. The first 2 are in linux-next already. A git > branch is available here: > > git://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git earlyco

Re: [PATCH 2/6] of: consolidate linker section OF match table declarations

2014-05-14 Thread Grant Likely
ection OF match > table entries which each table declaration can use. > > Cc: Grant Likely > Signed-off-by: Rob Herring Nice! Acked-by: Grant Likely g. > --- > drivers/clocksource/clksrc-of.c | 2 +- > drivers/irqchip/irqchip.h | 7 +++ > include/linux/

Re: [RFC PATCH 0/9] dt: dependencies (for deterministic driver initialization order based on the DT)

2014-05-14 Thread Grant Likely
On Mon, 12 May 2014 18:47:51 +0200, Alexander Holler wrote: > > Hello, > > if I would have to describe the Linux kernels init system (before userspace > starts), it would be like: > Unknown functions with almost unknown functionality are called in an almost > random order. > > That reminded

Re: [RFC PATCH 5/9] dt: deps: register drivers based on the initialization order based on DT

2014-05-14 Thread Grant Likely
On Mon, 12 May 2014 18:47:56 +0200, Alexander Holler wrote: > The init system currently calls unknown functions with almost unknown > functionality in an almost random order. Correct, we've got a module system. Some would say that is a strength! :-) That said, I don't object to optimizing to

Re: [RFC PATCH 2/9] dt: deps: dependency based device creation

2014-05-14 Thread Grant Likely
On Mon, 12 May 2014 18:47:53 +0200, Alexander Holler wrote: > Use the properties named 'dependencies' in binary device tree blobs to build > a dependency based initialization order for platform devices and drivers. > > This is done by building a directed acyclic graph using an adjacency list >

Re: [PATCH v6 05/11] drivers: of: add automated assignment of reserved regions to client devices

2014-05-14 Thread Grant Likely
On Wed, 14 May 2014 10:15:38 +0100, "Jon Medhurst (Tixy)" wrote: > On Sun, 2014-03-02 at 13:40 +0800, Grant Likely wrote: > > On Fri, 28 Feb 2014 14:42:50 +0100, Marek Szyprowski > > wrote: > > > This patch adds code for automated assignment of reserved mem

Re: [PATCH] of/selftest: clean-up of_selftest_platform_populate pass/fail handling

2014-05-14 Thread Grant Likely
On Tue, 13 May 2014 11:18:28 -0500, Rob Herring wrote: > From: Rob Herring > > Move the pass/fail checks into selftest() calls instead of a separate if > condition. Unconditionally calling pass was wrong. > > Signed-off-by: Rob Herring > Cc: Grant Likely A

Re: [PATCH] of: Add of_device_destroy_children() function

2014-05-14 Thread Grant Likely
On Thu, 08 May 2014 18:37:49 +0200, Sylwester Nawrocki wrote: > This patch adds a helper function to unregister devices which > were created by an of_platform_populate() call. The pattern > used here can already be found in multiple drivers. This helper > can now be used instead of repeating

Re: [PATCH] of: Add of_device_destroy_children() function

2014-05-14 Thread Grant Likely
On Thu, 8 May 2014 14:33:39 -0600, Jason Gunthorpe wrote: > On Thu, May 08, 2014 at 06:37:49PM +0200, Sylwester Nawrocki wrote: > > This patch adds a helper function to unregister devices which > > were created by an of_platform_populate() call. The pattern > > used here can already be found in

Re: [PATCH v3 0/7] of: setup dma parameters using dma-ranges and dma-coherent

2014-05-14 Thread Grant Likely
followup series for Keystone SOC which will use this > > > infrastructure. > > > Linus W also wants to use this for ARM integrator platform dma offset > > > issue. > > > > > > Cc: Greg Kroah-Hartman > > > Cc: Russell King > > > Cc

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-14 Thread Grant Likely
On Fri, 4 Apr 2014 15:43:55 +0300, Pantelis Antoniou wrote: > Introduce DT overlay support. > Using this functionality it is possible to dynamically overlay a part of > the kernel's tree with another tree that's been dynamically loaded. > It is also possible to remove node and properties. > >

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-14 Thread Grant Likely
On Fri, 4 Apr 2014 15:43:55 +0300, Pantelis Antoniou pantelis.anton...@konsulko.com wrote: Introduce DT overlay support. Using this functionality it is possible to dynamically overlay a part of the kernel's tree with another tree that's been dynamically loaded. It is also possible to remove

Re: [PATCH v3 0/7] of: setup dma parameters using dma-ranges and dma-coherent

2014-05-14 Thread Grant Likely
...@linuxfoundation.org Cc: Russell King li...@arm.linux.org.uk Cc: Arnd Bergmann a...@arndb.de Cc: Olof Johansson o...@lixom.net Cc: Grant Likely grant.lik...@linaro.org Cc: Rob Herring robh...@kernel.org Cc: Catalin Marinas catalin.mari...@arm.com Cc: Linus Walleij linus.wall...@linaro.org

Re: [PATCH] of: Add of_device_destroy_children() function

2014-05-14 Thread Grant Likely
On Thu, 8 May 2014 14:33:39 -0600, Jason Gunthorpe jguntho...@obsidianresearch.com wrote: On Thu, May 08, 2014 at 06:37:49PM +0200, Sylwester Nawrocki wrote: This patch adds a helper function to unregister devices which were created by an of_platform_populate() call. The pattern used here

Re: [PATCH] of: Add of_device_destroy_children() function

2014-05-14 Thread Grant Likely
On Thu, 08 May 2014 18:37:49 +0200, Sylwester Nawrocki s.nawro...@samsung.com wrote: This patch adds a helper function to unregister devices which were created by an of_platform_populate() call. The pattern used here can already be found in multiple drivers. This helper can now be used

Re: [PATCH] of/selftest: clean-up of_selftest_platform_populate pass/fail handling

2014-05-14 Thread Grant Likely
Likely grant.lik...@linaro.org Acked-by: Grant Likely grant.lik...@linaro.org --- drivers/of/selftest.c | 14 +- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/drivers/of/selftest.c b/drivers/of/selftest.c index fe70b86..c1d7d38 100644 --- a/drivers/of/selftest.c

Re: [PATCH v6 05/11] drivers: of: add automated assignment of reserved regions to client devices

2014-05-14 Thread Grant Likely
On Wed, 14 May 2014 10:15:38 +0100, Jon Medhurst (Tixy) t...@linaro.org wrote: On Sun, 2014-03-02 at 13:40 +0800, Grant Likely wrote: On Fri, 28 Feb 2014 14:42:50 +0100, Marek Szyprowski m.szyprow...@samsung.com wrote: This patch adds code for automated assignment of reserved memory

Re: [RFC PATCH 2/9] dt: deps: dependency based device creation

2014-05-14 Thread Grant Likely
On Mon, 12 May 2014 18:47:53 +0200, Alexander Holler hol...@ahsoftware.de wrote: Use the properties named 'dependencies' in binary device tree blobs to build a dependency based initialization order for platform devices and drivers. This is done by building a directed acyclic graph using an

Re: [RFC PATCH 5/9] dt: deps: register drivers based on the initialization order based on DT

2014-05-14 Thread Grant Likely
On Mon, 12 May 2014 18:47:56 +0200, Alexander Holler hol...@ahsoftware.de wrote: The init system currently calls unknown functions with almost unknown functionality in an almost random order. Correct, we've got a module system. Some would say that is a strength! :-) That said, I don't object

Re: [RFC PATCH 0/9] dt: dependencies (for deterministic driver initialization order based on the DT)

2014-05-14 Thread Grant Likely
On Mon, 12 May 2014 18:47:51 +0200, Alexander Holler hol...@ahsoftware.de wrote: Hello, if I would have to describe the Linux kernels init system (before userspace starts), it would be like: Unknown functions with almost unknown functionality are called in an almost random order. That

Re: [PATCH 2/6] of: consolidate linker section OF match table declarations

2014-05-14 Thread Grant Likely
for creating linker section OF match table entries which each table declaration can use. Cc: Grant Likely grant.lik...@linaro.org Signed-off-by: Rob Herring r...@kernel.org Nice! Acked-by: Grant Likely grant.lik...@linaro.org g. --- drivers/clocksource/clksrc-of.c | 2 +- drivers/irqchip

Re: [PATCH 0/6] DT early console initialization

2014-05-14 Thread Grant Likely
-by: Grant Likely grant.lik...@linaro.org I haven't tested it though. Rob [1] https://lkml.org/lkml/2014/4/18/573 [2] https://lkml.org/lkml/2014/4/22/1202 [3] https://lkml.org/lkml/2014/3/27/285 Rob Herring (6): of: align RESERVEDMEM_OF_DECLARE function callbacks to other callbacks

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-14 Thread Grant Likely
On Wed, 14 May 2014 15:03:35 +0200, Geert Uytterhoeven ge...@linux-m68k.org wrote: On Wed, May 14, 2014 at 12:08 PM, Grant Likely grant.lik...@secretlab.ca wrote: +config OF_OVERLAY + bool OF overlay support + depends on OF + select OF_DYNAMIC + select OF_DEVICE

Re: [PATCH v4 2/8] OF: Introduce DT overlay support.

2014-05-14 Thread Grant Likely
On Wed, 14 May 2014 14:11:52 +0200, Michael Stickel m...@mycable.de wrote: Hi Grant, Am 14.05.2014 12:08, schrieb Grant Likely: More generally I am concerned about whether or not overlays will introduce corner cases that can never be handled correctly, particularly in how multiple

Re: [RFC PATCH 0/9] dt: dependencies (for deterministic driver initialization order based on the DT)

2014-05-14 Thread Grant Likely
On Wed, May 14, 2014 at 4:02 PM, Alexander Holler hol...@ahsoftware.de wrote: Am 14.05.2014 16:19, schrieb Grant Likely: Rather than a dtb schema change, for the most common properties (irqs, clocks, gpios), we could extract dependencies at boot time. I don't like the idea of adding

Re: [RFC PATCH 5/9] dt: deps: register drivers based on the initialization order based on DT

2014-05-14 Thread Grant Likely
On Wed, May 14, 2014 at 3:58 PM, Alexander Holler hol...@ahsoftware.de wrote: Am 14.05.2014 16:13, schrieb Grant Likely: On Mon, 12 May 2014 18:47:56 +0200, Alexander Holler hol...@ahsoftware.de wrote: The init system currently calls unknown functions with almost unknown functionality

Re: [RFC PATCH 2/9] dt: deps: dependency based device creation

2014-05-14 Thread Grant Likely
On Wed, 14 May 2014 16:49:05 +0200, Alexander Holler hol...@ahsoftware.de wrote: Am 14.05.2014 16:05, schrieb Grant Likely: On Mon, 12 May 2014 18:47:53 +0200, Alexander Holler hol...@ahsoftware.de wrote: Use the properties named 'dependencies' in binary device tree blobs to build

[PATCH 0/3] Rework of_find_node_by_path() code

2014-05-13 Thread Grant Likely
This series reworks the of_find_node_by_path() function to search by path component instead of comparing the full path on every node. This makes for a more efficient search and makes it possible to parse things like aliases. This is the second time I'm posting this series. I'm going to apply it

[PATCH 3/3] of: Add a testcase for of_find_node_by_path()

2014-05-13 Thread Grant Likely
Add a testcase for the find_node_by_path() function to make sure it handles all the valid scenarios. Signed-off-by: Grant Likely --- drivers/of/selftest.c | 39 + drivers/of/testcase-data/tests-phandle.dtsi | 6 - 2 files changed, 44

[PATCH 1/3] lib: add glibc style strchrnul() variant

2014-05-13 Thread Grant Likely
will always come back out. Signed-off-by: Grant Likely --- include/linux/string.h | 3 +++ lib/string.c | 15 +++ 2 files changed, 18 insertions(+) diff --git a/include/linux/string.h b/include/linux/string.h index ac889c5ea11b..d36977e029af 100644 --- a/include/linux

[PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-13 Thread Grant Likely
Signed-off-by: Pantelis Antoniou [grant.likely: Rework to not require allocating at runtime] Acked-by: Rob Herring Signed-off-by: Grant Likely --- drivers/of/base.c | 60 +++ 1 file changed, 56 insertions(+), 4 deletions(-) diff --git a/drivers

[PATCH 2/3] of: Make of_find_node_by_path() handle /aliases

2014-05-13 Thread Grant Likely
david.da...@cavium.com Signed-off-by: Pantelis Antoniou pantelis.anton...@konsulko.com [grant.likely: Rework to not require allocating at runtime] Acked-by: Rob Herring r...@kernel.org Signed-off-by: Grant Likely grant.lik...@linaro.org --- drivers/of/base.c | 60

[PATCH 3/3] of: Add a testcase for of_find_node_by_path()

2014-05-13 Thread Grant Likely
Add a testcase for the find_node_by_path() function to make sure it handles all the valid scenarios. Signed-off-by: Grant Likely grant.lik...@linaro.org --- drivers/of/selftest.c | 39 + drivers/of/testcase-data/tests-phandle.dtsi | 6 - 2

[PATCH 1/3] lib: add glibc style strchrnul() variant

2014-05-13 Thread Grant Likely
will always come back out. Signed-off-by: Grant Likely grant.lik...@linaro.org --- include/linux/string.h | 3 +++ lib/string.c | 15 +++ 2 files changed, 18 insertions(+) diff --git a/include/linux/string.h b/include/linux/string.h index ac889c5ea11b..d36977e029af 100644

[PATCH 0/3] Rework of_find_node_by_path() code

2014-05-13 Thread Grant Likely
This series reworks the of_find_node_by_path() function to search by path component instead of comparing the full path on every node. This makes for a more efficient search and makes it possible to parse things like aliases. This is the second time I'm posting this series. I'm going to apply it

Re: [patch 00/32] genirq: Another round of tree wide cleanups

2014-05-08 Thread Grant Likely
13xx to >sparse irq there is only ia64 left. I simplified the code and made >this available only under a legacy config option to avoid that more >people think they need this. > > Full diffstat below. For the whole series: Reviewed-by: Grant Likely It all looks sane

Re: [patch 29/32] genirq: Remove irq_reserve_irq[s]

2014-05-08 Thread Grant Likely
On Wed, 07 May 2014 15:44:21 -, Thomas Gleixner wrote: > No more users. And it's not going to come back. If you need > hotplugable irq chips, use irq domains. > > Signed-off-by: Thomas Gleixner Acked-by: Grant Likely The ironic thing is that irq_domain may end up bringing t

Re: [patch 03/32] genirq: Provide generic hwirq allocation facility

2014-05-08 Thread Grant Likely
On Wed, 07 May 2014 15:44:05 -, Thomas Gleixner wrote: > Not really the solution to the problem, but at least it confines the > mess in the core code and allows to get rid of the create/destroy_irq > variants from hell, i.e. 3 implementations with different semantics > plus the x86 specific

Re: [PATCH 1/4] of/selftest: add testcase for nodes with same name and address

2014-05-08 Thread Grant Likely
On Wed, May 7, 2014 at 10:48 PM, Rob Herring wrote: > From: Rob Herring > > Add a test case for nodes which have the same name and same > non-translatable unit address. > > Signed-off-by: Rob Herring Looks good to me. Reviewed-by: Grant Likely > --- >

Re: [PATCH 1/4] of/selftest: add testcase for nodes with same name and address

2014-05-08 Thread Grant Likely
On Thu, May 8, 2014 at 3:51 AM, Frank Rowand wrote: > On 5/7/2014 2:48 PM, Rob Herring wrote: >> From: Rob Herring >> >> Add a test case for nodes which have the same name and same >> non-translatable unit address. > > If I apply patch 1 and 2 without applying 3 and 4 then console > warnings are

Re: [PATCH 1/4] of/selftest: add testcase for nodes with same name and address

2014-05-08 Thread Grant Likely
On Thu, May 8, 2014 at 3:51 AM, Frank Rowand frowand.l...@gmail.com wrote: On 5/7/2014 2:48 PM, Rob Herring wrote: From: Rob Herring r...@kernel.org Add a test case for nodes which have the same name and same non-translatable unit address. If I apply patch 1 and 2 without applying 3 and 4

Re: [PATCH 1/4] of/selftest: add testcase for nodes with same name and address

2014-05-08 Thread Grant Likely
On Wed, May 7, 2014 at 10:48 PM, Rob Herring robherri...@gmail.com wrote: From: Rob Herring r...@kernel.org Add a test case for nodes which have the same name and same non-translatable unit address. Signed-off-by: Rob Herring r...@kernel.org Looks good to me. Reviewed-by: Grant Likely

Re: [patch 03/32] genirq: Provide generic hwirq allocation facility

2014-05-08 Thread Grant Likely
On Wed, 07 May 2014 15:44:05 -, Thomas Gleixner t...@linutronix.de wrote: Not really the solution to the problem, but at least it confines the mess in the core code and allows to get rid of the create/destroy_irq variants from hell, i.e. 3 implementations with different semantics plus the

Re: [patch 29/32] genirq: Remove irq_reserve_irq[s]

2014-05-08 Thread Grant Likely
On Wed, 07 May 2014 15:44:21 -, Thomas Gleixner t...@linutronix.de wrote: No more users. And it's not going to come back. If you need hotplugable irq chips, use irq domains. Signed-off-by: Thomas Gleixner t...@linutronix.de Acked-by: Grant Likely grant.lik...@secretlab.ca The ironic

<    5   6   7   8   9   10   11   12   13   14   >