Re: [PATCH v2 0/6] of: overlays: New target methods

2016-05-27 Thread Frank Rowand
On 5/27/2016 7:46 AM, Pantelis Antoniou wrote: > Hi Frank, > >> On May 27, 2016, at 00:15 , Frank Rowand wrote: >> >> On 5/16/2016 1:18 PM, Pantelis Antoniou wrote: >>> This patchset implements two new target methods. >>> >>> A target index

Re: [PATCH v2 0/6] of: overlays: New target methods

2016-05-26 Thread Frank Rowand
On 5/16/2016 1:18 PM, Pantelis Antoniou wrote: > This patchset implements two new target methods. > > A target index method which allows selecting different > targets according to an argument using an extended API and > a target root method that fences the target only > to a specific given root. >

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 10:48 AM, Mark Brown wrote: > On Wed, May 25, 2016 at 08:32:51AM -0700, Frank Rowand wrote: >> On 5/25/2016 2:20 AM, Mark Rutland wrote: > >>> Linux for legacy reasons, documenting it as a binding is not necessarily >>> in anyone's best interest

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 10:49 AM, Rob Herring wrote: > On Tue, May 24, 2016 at 04:34:50PM -0700, Frank Rowand wrote: >> On 5/24/2016 11:32 AM, Mark Brown wrote: >>> On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: >>>> On 05/24/2016 07:20 PM, Mark Brown wrote:

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 9:06 AM, Mark Rutland wrote: > On Wed, May 25, 2016 at 08:25:44AM -0700, Frank Rowand wrote: >> On 5/24/2016 10:41 AM, Mark Rutland wrote: >>> On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >>>> +Normally SPI buses are assigned dynamic

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 8:59 AM, Mark Rutland wrote: > On Wed, May 25, 2016 at 08:32:51AM -0700, Frank Rowand wrote: >> On 5/25/2016 2:20 AM, Mark Rutland wrote: >>> On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: >>>> On 5/24/2016 10:41 AM, Mark Rutland wrote: &g

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/25/2016 2:20 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at 01:41:26PM -0700, Frank Rowand wrote: >> On 5/24/2016 10:41 AM, Mark Rutland wrote: >>> On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >>>> Document how to use devicetree alias

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-25 Thread Frank Rowand
On 5/24/2016 10:41 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >> Document how to use devicetree aliases to assign a stable >> bus number to a spi bus. >> >> Signed-off-by: Christer Weinigel >> >> --- >> >> Trivial documentation change. >> >> Not ha

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Frank Rowand
On 5/24/2016 4:34 PM, Frank Rowand wrote: > On 5/24/2016 11:32 AM, Mark Brown wrote: >> On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: >>> On 05/24/2016 07:20 PM, Mark Brown wrote: >> >>>> I'm not sure this is something we want to supp

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Frank Rowand
On 5/24/2016 11:32 AM, Mark Brown wrote: > On Tue, May 24, 2016 at 08:03:48PM +0200, Christer Weinigel wrote: >> On 05/24/2016 07:20 PM, Mark Brown wrote: > >>> I'm not sure this is something we want to support at all, I can't >>> immediately see anything that does this deliberately in the SPI >>

Re: [PATCH] devicetree - document using aliases to set spi bus number.

2016-05-24 Thread Frank Rowand
On 5/24/2016 10:41 AM, Mark Rutland wrote: > On Tue, May 24, 2016 at 06:39:20PM +0200, Christer Weinigel wrote: >> Document how to use devicetree aliases to assign a stable >> bus number to a spi bus. >> >> Signed-off-by: Christer Weinigel >> >> --- >> >> Trivial documentation change. >> >> Not ha

Re: [PATCH] scripts/dtc: dt_to_config - report kernel config options for a devicetree

2016-04-29 Thread Frank Rowand
On 4/28/2016 11:44 PM, Geert Uytterhoeven wrote: > On Fri, Apr 29, 2016 at 8:39 AM, Gaurav Minocha > wrote: >> On Thu, Apr 28, 2016 at 3:32 PM, Rob Herring wrote: >>> On Thu, Apr 28, 2016 at 4:46 PM, Frank Rowand >>> wrote: >>>> From: Frank Rowand

Re: [PATCH] scripts/dtc: dt_to_config - report kernel config options for a devicetree

2016-04-28 Thread Frank Rowand
On 4/28/2016 3:32 PM, Rob Herring wrote: > On Thu, Apr 28, 2016 at 4:46 PM, Frank Rowand wrote: >> From: Frank Rowand >> >> Determining which kernel config options need to be enabled for a >> given devicetree can be a painful process. Create a new tool to >>

Re: [PATCH 2/2] tty: serial: msm_serial add info message

2016-04-28 Thread Frank Rowand
On 4/28/2016 1:51 PM, Greg Kroah-Hartman wrote: > On Sat, Apr 23, 2016 at 10:17:05AM -0700, Frank Rowand wrote: >> From: Frank Rowand >> >> Failure to enable DMA by the msm_serial driver is silent. >> Add a message to report the failure. >> >> Signed-off-b

Re: [PATCH] scripts/dtc: dt_to_config - usage message

2016-04-28 Thread Frank Rowand
$ scripts/dtc/dt_to_config --help Usage: dt_to_config [options] device-tree... device_tree is: dts_file | dtb_file | proc_device-tree Valid options: -b ignore driver black list -c FILE Read kernel config options from FILE --config FILEsyno

Re: [PATCH] scripts/dtc: dt_to_config - report kernel config options for a devicetree

2016-04-28 Thread Frank Rowand
Example 3 -- show all nodes. Report the value of the kernel config options in the kernel configuration file. $ scripts/dtc/dt_to_config --config=$KBUILD_OUTPUT/.config arch/arm/boot/dts/qcom-apq8074-dragonboard.dts -dDc---y- : sleep_clk : fixed-clock : arch/powerpc/platforms/512x/clock-com

Re: [PATCH] scripts/dtc: dt_to_config - example 2

2016-04-28 Thread Frank Rowand
Example 2 -- show all nodes. $ scripts/dtc/dt_to_config arch/arm/boot/dts/qcom-apq8074-dragonboard.dts -dDc- : sleep_clk : fixed-clock : arch/powerpc/platforms/512x/clock-commonclk.c : CONFIG_COMMON_CLK : none -dDc- : sleep_clk : fixed-clock : drivers/clk/clk-fixed-rate.c : CONF

Re: [PATCH] scripts/dtc: dt_to_config - example 1

2016-04-28 Thread Frank Rowand
Example 1 -- show only the nodes that have complications. Things like multiple compatible values, multiple matching drivers, etc. $ scripts/dtc/dt_to_config --include-suspect arch/arm/boot/dts/qcom-apq8074-dragonboard.dts -dDc- : sleep_clk : fixed-clock : arch/powerpc/platforms/512x/cl

[PATCH] scripts/dtc: dt_to_config - report kernel config options for a devicetree

2016-04-28 Thread Frank Rowand
From: Frank Rowand Determining which kernel config options need to be enabled for a given devicetree can be a painful process. Create a new tool to find the drivers that may match a devicetree node compatible, find the kernel config options that enable the driver, and optionally report whether

Re: [PATCH 2/2] tty: serial: msm_serial add info message

2016-04-25 Thread Frank Rowand
On 4/25/2016 2:35 PM, Stephen Boyd wrote: > On 04/25, Frank Rowand wrote: >> >> This is in msm_request_tx_dma(). I should have made the message >> "msm_serial: TX DMA not enabled\n" and added a similar message >> to msm_request_rx_dma(). >> >> Th

Re: [PATCH 2/2] tty: serial: msm_serial add info message

2016-04-25 Thread Frank Rowand
On 4/25/2016 1:48 PM, Stephen Boyd wrote: > On 04/23, Frank Rowand wrote: >> From: Frank Rowand >> >> Failure to enable DMA by the msm_serial driver is silent. >> Add a message to report the failure. >> >> Signed-off-by: Frank Rowand >> --- >>

Re: regression: 3a878c430fd6 ("tty: serial: msm: Add TX DMA support") drops data

2016-04-23 Thread Frank Rowand
On 4/21/2016 1:42 PM, Andy Shevchenko wrote: > On Thu, Apr 21, 2016 at 4:00 AM, Frank Rowand wrote: >> On 4/20/2016 9:48 AM, Frank Rowand wrote: < snip > >> I'm not sure if the driver was in DMA or PIO mode, both >> before the change and after. This seems like a

[PATCH 2/2] tty: serial: msm_serial add info message

2016-04-23 Thread Frank Rowand
From: Frank Rowand Failure to enable DMA by the msm_serial driver is silent. Add a message to report the failure. Signed-off-by: Frank Rowand --- drivers/tty/serial/msm_serial.c |1 + 1 file changed, 1 insertion(+) Index: b/drivers/tty/serial/msm_serial.c

[PATCH 1/2] tty: serial: msm_serial regression fix data corruption

2016-04-23 Thread Frank Rowand
From: Frank Rowand Commit 3a878c430fd6 ("tty: serial: msm: Add TX DMA support") regression. The calculation of tx_count was moved from the old msm_handle_tx(), now renamed msm_handle_tx_pio(), to the new msm_handle_tx(). The move left out one size test. The regression seen on the qc

[PATCH 0/2] tty: serial: msm_serial regression and add info message

2016-04-23 Thread Frank Rowand
Commit 3a878c430fd6 ("tty: serial: msm: Add TX DMA support") resulted in dropped characters and invalid characters in pio mode. Fix the problem and add an additional information message that was important in diagnosing the problem (reporting that DMA mode was not enabled).

Re: [PATCH] of: document refcount incrementation of of_get_cpu_node()

2016-04-21 Thread Frank Rowand
found. > */ > struct device_node *of_get_cpu_node(int cpu, unsigned int *thread) > { > Nice catch. Reviewed-by: Frank Rowand

Re: [RFC/PATCH] of: of_find_node_by_name - stop dropping reference to 'from' node

2016-04-21 Thread Frank Rowand
On 4/19/2016 10:05 AM, Dmitry Torokhov wrote: > Majority of the callers of of_find_node_by_name() do not expect that it > will drop reference to the 'from' node if it was passed in, causing > potential refcount underflows, etc, so let's stop doing this. > > Most of the callers that were handling d

Re: regression: 3a878c430fd6 ("tty: serial: msm: Add TX DMA support") drops data

2016-04-20 Thread Frank Rowand
On 4/20/2016 9:48 AM, Frank Rowand wrote: > On 4/19/2016 11:07 PM, Ivan Ivanov wrote: >> >>> On Apr 20, 2016, at 02:23, Frank Rowand wrote: >>> >>> Hi Ivan, >>> >>> It appears that I have found a regression caused by >>> 3a878c430fd6

Re: regression: 3a878c430fd6 ("tty: serial: msm: Add TX DMA support") drops data

2016-04-20 Thread Frank Rowand
On 4/19/2016 11:07 PM, Ivan Ivanov wrote: > >> On Apr 20, 2016, at 02:23, Frank Rowand wrote: >> >> Hi Ivan, >> >> It appears that I have found a regression caused by >> 3a878c430fd6 ("tty: serial: msm: Add TX DMA support"). >> >>

regression: 3a878c430fd6 ("tty: serial: msm: Add TX DMA support") drops data

2016-04-19 Thread Frank Rowand
Hi Ivan, It appears that I have found a regression caused by 3a878c430fd6 ("tty: serial: msm: Add TX DMA support"). When I cat a file slightly larger than 126000 bytes on the console, viewed via minicom connected to the serial port, I am losing random chunks of data, almost always three bytes in

[PATCH] .mailmap: add Frank Rowand

2016-04-15 Thread Frank Rowand
Set current email address to replace obsolete email addresses. Signed-off-by: Frank Rowand --- .mailmap |3 +++ 1 file changed, 3 insertions(+) Index: b/.mailmap === --- a/.mailmap +++ b/.mailmap @@ -48,6 +48,9 @@ Felix

Re: [Question] refcount of DT node

2016-04-14 Thread Frank Rowand
On 4/14/2016 3:02 AM, Pantelis Antoniou wrote: > Hi Mark, > >> On Apr 14, 2016, at 12:59 , Mark Rutland wrote: >> >> On Thu, Apr 14, 2016 at 09:48:49AM +0100, Russell King - ARM Linux wrote: >>> On Thu, Apr 14, 2016 at 04:47:57PM +0900, Masahiro Yamada wrote: Hi experts. My underst

Re: [PATCH] of: Add generic handling for hardware incomplete fail state

2016-04-12 Thread Frank Rowand
On 4/12/2016 1:13 PM, Frank Rowand wrote: > Hi Tony, < snip > > With that change, the bulk of your patch looks good, with > minor changes: > > __of_device_is_available() would not need to change. > > __of_device_is_incomplete() would change to check the new &g

Re: [PATCH] of: Add generic handling for hardware incomplete fail state

2016-04-12 Thread Frank Rowand
On 4/12/2016 3:20 PM, Rob Herring wrote: > On Tue, Apr 12, 2016 at 4:41 PM, Frank Rowand wrote: >> On 4/12/2016 1:34 PM, Tony Lindgren wrote: >>> Hi, >>> >>> * Frank Rowand [160412 13:15]: >>>> Hi Tony, >>>> >>>> I agree w

Re: [PATCH] of: Add generic handling for hardware incomplete fail state

2016-04-12 Thread Frank Rowand
On 4/12/2016 1:34 PM, Tony Lindgren wrote: > Hi, > > * Frank Rowand [160412 13:15]: >> Hi Tony, >> >> I agree with the need for some way of handling the incomplete >> hardware issue. I like the idea of having a uniform method for >> all nodes. >> &g

Re: [PATCH] of: Add generic handling for hardware incomplete fail state

2016-04-12 Thread Frank Rowand
Hi Tony, I agree with the need for some way of handling the incomplete hardware issue. I like the idea of having a uniform method for all nodes. I am stumbling over what the status property is supposed to convey and what the "fail-hw-incomplete" is meant to convey. The status property is meant

Re: [PATCH v12 1/5] efi: ARM/arm64: ignore DT memory nodes instead of removing them

2016-02-24 Thread Frank Rowand
On 2/23/2016 3:58 AM, Mark Rutland wrote: > Hi, > > On Mon, Feb 22, 2016 at 05:58:19PM -0800, David Daney wrote: >> From: Ard Biesheuvel >> >> There are two problems with the UEFI stub DT memory node removal >> routine: >> - it deletes nodes as it traverses the tree, which happens to work >> bu

Re: [PATCH] scripts/dtc: dtx_diff - add info to error message

2016-02-12 Thread Frank Rowand
On 2/11/2016 5:41 PM, Rob Herring wrote: > On Thu, Feb 11, 2016 at 6:50 PM, Frank Rowand wrote: >> On 2/2/2016 11:02 AM, Frank Rowand wrote: >>> From: Frank Rowand >>> >>> If kernel config options are not properly set, "make scripts" will not >>

Re: [PATCH] scripts/dtc: dtx_diff - add info to error message

2016-02-11 Thread Frank Rowand
On 2/2/2016 11:02 AM, Frank Rowand wrote: > From: Frank Rowand > > If kernel config options are not properly set, "make scripts" will not > compile dtc. Update the unable to find dtc error message to check > the kernel config and give better advice on how to create

Re: [PATCH] of/irq: Fix msi-map calculation for nonzero rid-base

2016-02-11 Thread Frank Rowand
to be picky, I would like the patch to be split in two for easier bisecting, but if Rob is happy with a single patch I'm ok with that. Rob, will you pick this up? Acked-by: Frank Rowand -Frank

[PATCH] scripts/dtc: dtx_diff - add info to error message

2016-02-02 Thread Frank Rowand
From: Frank Rowand If kernel config options are not properly set, "make scripts" will not compile dtc. Update the unable to find dtc error message to check the kernel config and give better advice on how to create dtc. Reword another error message to increase clarity. Signed-off

Re: [PATCH] scripts/dtc: Update to upstream commit b06e55c88b9b

2016-02-02 Thread Frank Rowand
rmat > 5e78dff guess input file format based on file content or file name > 8b927bf tests: convert `echo -n` to `printf` > 64c46b0 Fix crash with poorly defined #size-cells > > Cc: Frank Rowand > Cc: Grant Likely > Signed-off-by: Rob Herring > --- > scripts/dtc/check

Re: [PATCH 00/13] dtb: amd: Miscelleneous Updates for AMD Seattle DTS

2016-01-29 Thread Frank Rowand
On 1/28/2016 6:43 PM, Olof Johansson wrote: > On Thu, Jan 28, 2016 at 2:20 PM, Suravee Suthikulanit > wrote: >> Hi Olof, >> >> On 1/28/2016 3:39 PM, Olof Johansson wrote: >>> >>> Hi Suravee, >>> >>> On Wed, Jan 27, 2016 at 1:11 PM, Suravee Suthikulpanit >>> wrote: From: Suravee Suthikul

Re: [PATCH] of: resolver: Add missing of_node_put

2016-01-29 Thread Frank Rowand
On 1/29/2016 9:33 AM, Pantelis Antoniou wrote: > Hi Rob, > >> On Jan 29, 2016, at 18:45 , Rob Herring wrote: >> >> On Wed, Jan 27, 2016 at 06:14:00PM +0200, Pantelis Antoniou wrote: >>> Hi Mark, >>> On Jan 27, 2016, at 18:05 , Mark Rutland wrote: On Wed, Jan 27, 2016 at 08:50:17PM

[PATCH] dtc: create tool to diff device trees

2015-12-31 Thread Frank Rowand
From: Frank Rowand Create script to diff device trees. The device tree can be in any of the forms recognized by the dtc compiler: - source - binary blob - file system tree (from /proc/devicetree) If the device tree is a source file, then it is pre-processed in the same way as it would be

Re: [PATCH 1/5] arm: devtree: Set system_rev from DT "/revision"

2015-12-28 Thread Frank Rowand
Adding devicetree-spec, and commenting below. On 12/24/2015 11:02 AM, Pali Rohár wrote: > On Monday 06 July 2015 22:26:09 Pali Rohár wrote: >> With this patch "/revision" DT entry is used to set global system_rev >> variable. DT "/revision" is expected to be u32 numeric value. >> >> TODO: add docu

Re: [PATCH v1 2/2] dt-bindings: sound: add devicetree document for rt5616

2015-12-21 Thread Frank Rowand
On 12/21/2015 6:56 AM, Caesar Wang wrote: > Add the description for rt5616 codec. > > Signed-off-by: Caesar Wang > > --- > > Changes in v1: > - As Heiko comments, remove the not exist option properties. > > Documentation/devicetree/bindings/sound/rt5616.txt | 36 > ++ > 1

Re: [PATCH v2 4/5] arm64: dts: qcom: Alias pm8916 on msm8916 devices

2015-12-09 Thread Frank Rowand
On 11/20/2015 3:31 PM, Stephen Boyd wrote: > Add an alias for pm8916 on msm8916 based SoCs so that the newly > updated dtbTool can find the pmic compatible string and add the > pmic-id element to the QCDT header. > > Signed-off-by: Stephen Boyd > --- > arch/arm64/boot/dts/qcom/apq8016-sbc.dtsi |

Re: [PATCH 5/5] arm: boot: store ATAGs structure into DT "/chosen/linux,atags" entry

2015-11-28 Thread Frank Rowand
On 11/28/2015 9:34 AM, Nicolas Pitre wrote: > On Sat, 28 Nov 2015, Russell King - ARM Linux wrote: > >> On Fri, Nov 27, 2015 at 06:28:50PM -0500, Nicolas Pitre wrote: >>> On Fri, 27 Nov 2015, Arnd Bergmann wrote: >>> I don't mind creating the /proc/atags compatibility hack from the kernel >>>

Re: [PATCH 0/5] ATAGs to DT patches

2015-11-27 Thread Frank Rowand
On 11/27/2015 8:06 PM, Frank Rowand wrote: > On 7/6/2015 1:26 PM, Pali Rohár wrote: >> This patch series converts some legacy ATAGs into DT and provide them to >> userspace. It is needed for userspace applications which needs some >> informations from legacy bootloaders which

Re: [PATCH 0/5] ATAGs to DT patches

2015-11-27 Thread Frank Rowand
On 7/6/2015 1:26 PM, Pali Rohár wrote: > This patch series converts some legacy ATAGs into DT and provide them to > userspace. It is needed for userspace applications which needs some > informations from legacy bootloaders which are not DT compatible. > > Patch series is for now without DT documen

Re: [PATCH 5/5] arm: boot: store ATAGs structure into DT "/chosen/linux,atags" entry

2015-11-25 Thread Frank Rowand
On 11/25/2015 1:03 PM, Tony Lindgren wrote: > * Arnd Bergmann [151125 11:50]: >> On Wednesday 25 November 2015 10:16:44 Tony Lindgren wrote: >>> * Pali Rohár [151123 06:46]: >>>> On Sunday 22 November 2015 07:51:46 Pavel Machek wrote: >>>>> O

Re: [PATCH 5/5] arm: boot: store ATAGs structure into DT "/chosen/linux,atags" entry

2015-11-11 Thread Frank Rowand
Adding devicetree list. Thread starts at http://lists.infradead.org/pipermail/linux-arm-kernel/2015-July/354459.html On 11/5/2015 8:17 AM, Tony Lindgren wrote: > * Pali Rohár [151105 03:41]: >> On Tuesday 13 October 2015 16:37:46 Pali Rohár wrote: >>> On Monday 12 October 2015 13:45:09 Tony Lind

regulator: build warning in regulator_lock_supply()

2015-11-09 Thread Frank Rowand
In the current Linus tree, new build warning. When CONFIG_DEBUG_LOCK_ALLOC is not enabled, drivers/regulator/core.c: In function 'regulator_lock_supply': drivers/regulator/core.c:142:6: warning: unused variable 'i' [-Wunused-variable] Introduced by commit 9f01cd4a915e1 ("regulator: core

Re: [PATCH] Documentation: dt: Add bindings for Secure-only devices

2015-11-03 Thread Frank Rowand
On 10/30/2015 1:07 PM, Peter Maydell wrote: > On 30 October 2015 at 18:28, Rob Herring wrote: >> On Thu, Oct 29, 2015 at 9:01 AM, Peter Maydell >> wrote: >>> +Valid Secure world properties: >>> + >>> +- secure-status : specifies whether the device is present and usable >>> + in the secure world

Re: [GIT PULL] On-demand device probing

2015-10-22 Thread Frank Rowand
On 10/22/2015 7:44 AM, Greg Kroah-Hartman wrote: > > > On Thu, Oct 22, 2015 at 11:05:11AM +0200, Tomeu Vizoso wrote: >> But that's moot currently because Greg believes that the time spent >> probing devices at boot time could be reduced enough so that the order >> in which devices are probed beco

Re: Alternative approach to solve the deferred probe

2015-10-21 Thread Frank Rowand
On 10/21/2015 1:35 PM, Russell King - ARM Linux wrote: > On Wed, Oct 21, 2015 at 08:36:23AM -0700, Frank Rowand wrote: >> On 10/21/2015 1:18 AM, Russell King - ARM Linux wrote: >>> On Tue, Oct 20, 2015 at 08:58:19PM -0700, Frank Rowand wrote: >>>> On 10/20/2015 8:4

Re: [GIT PULL] On-demand device probing

2015-10-21 Thread Frank Rowand
On 10/21/2015 2:12 PM, Rob Herring wrote: > On Wed, Oct 21, 2015 at 1:18 PM, Frank Rowand wrote: >> On 10/21/2015 9:27 AM, Mark Brown wrote: >>> On Wed, Oct 21, 2015 at 08:59:51AM -0700, Frank Rowand wrote: >>>> On 10/19/2015 5:34 AM, Tomeu Vizoso wrote: >>&g

Re: [GIT PULL] On-demand device probing

2015-10-21 Thread Frank Rowand
On 10/21/2015 9:27 AM, Mark Brown wrote: > On Wed, Oct 21, 2015 at 08:59:51AM -0700, Frank Rowand wrote: >> On 10/19/2015 5:34 AM, Tomeu Vizoso wrote: > >>> To be clear, I was saying that this series should NOT affect total >>> boot times much. > >> I

Re: Alternative approach to solve the deferred probe

2015-10-21 Thread Frank Rowand
On 10/21/2015 9:55 AM, Grygorii Strashko wrote: > On 10/21/2015 06:36 PM, Frank Rowand wrote: >> On 10/21/2015 1:18 AM, Russell King - ARM Linux wrote: >>> On Tue, Oct 20, 2015 at 08:58:19PM -0700, Frank Rowand wrote: >>>> On 10/20/2015 8:46 AM, Russell King -

Re: [GIT PULL] On-demand device probing

2015-10-21 Thread Frank Rowand
On 10/19/2015 5:34 AM, Tomeu Vizoso wrote: > On 18 October 2015 at 21:53, Mark Brown wrote: >> On Sun, Oct 18, 2015 at 12:37:57PM -0700, Greg Kroah-Hartman wrote: >>> On Sun, Oct 18, 2015 at 08:29:31PM +0100, Mark Brown wrote: On Fri, Oct 16, 2015 at 11:57:50PM -0700, Greg Kroah-Hartman wrote

Re: Alternative approach to solve the deferred probe

2015-10-21 Thread Frank Rowand
On 10/21/2015 1:18 AM, Russell King - ARM Linux wrote: > On Tue, Oct 20, 2015 at 08:58:19PM -0700, Frank Rowand wrote: >> On 10/20/2015 8:46 AM, Russell King - ARM Linux wrote: < snip > >>> + >>> static bool dri

Re: Alternative approach to solve the deferred probe

2015-10-20 Thread Frank Rowand
On 10/20/2015 8:46 AM, Russell King - ARM Linux wrote: > On Mon, Oct 19, 2015 at 06:21:40PM +0200, Geert Uytterhoeven wrote: >> Hi Russell, >> >> On Mon, Oct 19, 2015 at 5:35 PM, Russell King - ARM Linux >> wrote: > What you can do is print those devices which have failed to probe at > lat

Re: [PATCH v7 19/20] driver core: Allow deferring probes until late init

2015-10-14 Thread Frank Rowand
On 9/29/2015 2:10 AM, Tomeu Vizoso wrote: > Add a field to struct device that instructs the device-driver core to > defer the probe of this device until the late_initcall level. > > By letting all built-in drivers to register before starting to probe, we > can avoid any deferred probes by probing

Re: [PATCH v7 20/20] of/platform: Defer probes of registered devices

2015-10-14 Thread Frank Rowand
On 9/29/2015 2:10 AM, Tomeu Vizoso wrote: > Instead of trying to match and probe platform and AMBA devices right > after each is registered, delay their probes until device_initcall_sync.

Re: [PATCH v6 21/22] driver core: Start processing deferred probes earlier

2015-10-05 Thread Frank Rowand
On 9/21/2015 7:03 AM, Tomeu Vizoso wrote: > Some initcalls in the late level assume that some devices will have > already probed without explicitly checking for that. > > After the recent move to defer most device probes when they are > registered, pressure increased in the late initcall level. >

Re: [PATCH v4 1/9] drivers/of: Introduce of_node_alloc

2015-09-28 Thread Frank Rowand
On 9/28/2015 9:42 AM, Marc Zyngier wrote: > We want to be able to generate "fake" device nodes that can be > used as an identifier for irq domains. For that, we reuse the > dynamic DT layer in order to generate DT nodes in a detached state > (so that it doesn't interfere with the rest of the tree).

Re: [PATCH v2] of_pci_irq: Silence bogus "of_irq_parse_pci() failed ..." messages.

2015-09-09 Thread Frank Rowand
> cause for an early exit. > > Signed-off-by: David Daney > --- > Changes in v2: Move the print function in to of_irq_parse_pci() at a > common error exit point (as suggested by Frank Rowand). > > > drivers/of/of_pci_irq.c | 17 ++--- > 1 file changed, 1

Re: [PATCH v2] of_pci_irq: Silence bogus "of_irq_parse_pci() failed ..." messages.

2015-09-09 Thread Frank Rowand
off-by: David Daney > --- > Changes in v2: Move the print function in to of_irq_parse_pci() at a > common error exit point (as suggested by Frank Rowand). > > > drivers/of/of_pci_irq.c | 17 ++--- > 1 file changed, 10 insertions(+), 7 deletions(-) > >

Re: [PATCH] of_pci_irq: Silence bogus "of_irq_parse_pci() failed ..." messages.

2015-09-07 Thread Frank Rowand
On 9/6/2015 8:50 PM, Frank Rowand wrote: > On 9/6/2015 7:16 PM, Frank Rowand wrote: >> On 9/6/2015 1:46 PM, Rob Herring wrote: >>> On Fri, Sep 4, 2015 at 8:14 PM, Frank Rowand wrote: >>>> On 9/4/2015 12:12 PM, David Daney wrote: >>>>> From: David Dane

Re: [PATCH] of_pci_irq: Silence bogus "of_irq_parse_pci() failed ..." messages.

2015-09-06 Thread Frank Rowand
On 9/6/2015 7:16 PM, Frank Rowand wrote: > On 9/6/2015 1:46 PM, Rob Herring wrote: >> On Fri, Sep 4, 2015 at 8:14 PM, Frank Rowand wrote: >>> On 9/4/2015 12:12 PM, David Daney wrote: >>>> From: David Daney >>>> >>>> It is perfectly legitim

Re: [PATCH] of_pci_irq: Silence bogus "of_irq_parse_pci() failed ..." messages.

2015-09-06 Thread Frank Rowand
On 9/6/2015 1:46 PM, Rob Herring wrote: > On Fri, Sep 4, 2015 at 8:14 PM, Frank Rowand wrote: >> On 9/4/2015 12:12 PM, David Daney wrote: >>> From: David Daney >>> >>> It is perfectly legitimate for a PCI device to have an >>> PCI_INTERRUPT_PIN value

Re: [PATCH] of_pci_irq: Silence bogus "of_irq_parse_pci() failed ..." messages.

2015-09-04 Thread Frank Rowand
On 9/4/2015 6:40 PM, David Daney wrote: > On 09/04/2015 06:14 PM, Frank Rowand wrote: >> On 9/4/2015 12:12 PM, David Daney wrote: >>> From: David Daney >>> >>> It is perfectly legitimate for a PCI device to have an >>> PCI_INTERRUPT_PIN value of zero

Re: [PATCH] of_pci_irq: Silence bogus "of_irq_parse_pci() failed ..." messages.

2015-09-04 Thread Frank Rowand
On 9/4/2015 12:12 PM, David Daney wrote: > From: David Daney > > It is perfectly legitimate for a PCI device to have an > PCI_INTERRUPT_PIN value of zero. This happens if the device doesn't > use interrupts, or on PCIe devices, where only MSI/MSI-X are > supported. > > Silence the annoying "of_

Re: [PATCH v5] dtb: Create a common home for cross-architecture dtsi files.

2015-08-03 Thread Frank Rowand
t; {arch/arm/boot/dts => kernel/dts/arm}/vexpress-v2m-rs1.dtsi | 0 > scripts/Makefile.lib| 3 ++- > 8 files changed, 8 insertions(+), 6 deletions(-) > rename {arch/arm/boot/dts => kernel/dts/arm}/vexpress-v2m-rs1.dtsi (100%) Reviewed-by: Frank

Re: Device Tree at Plumbers, early registration ends Friday

2015-06-01 Thread Frank Rowand
On 5/30/2015 2:36 PM, Frank Rowand wrote: > The "Linux Plumbers Device Tree" track was accepted by the Plumbers > conference. The above is a pasto, ycch. The track is, of course: Device Tree Tools, Validation, and Troubleshooting > > The DEADLINE for EARLY BIRD REG

Device Tree at Plumbers, looking for topics and session leaders

2015-05-30 Thread Frank Rowand
/ Plumbers will be co-located with LinuxCon North America in Seattle (Plumbers is Aug 19-21, Linuxcon is Aug 17-19). Hope to see many of you in Seattle. Regards, Frank Rowand -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message

Device Tree at Plumbers, early registration ends Friday

2015-05-30 Thread Frank Rowand
unced very soon now. Preliminary results are that I will be presenting a talk on DT debugging in that track. LinuxCon North America info is at: http://events.linuxfoundation.org/events/linuxcon-north-america Hope to see many of you in Seattle. Regards, Frank Rowand -- To unsubscribe from

Re: [PATCH/RFC] kbuild: Create a rule for building device tree overlay objects

2015-05-15 Thread Frank Rowand
On 5/15/2015 5:47 PM, Frank Rowand wrote: > On 5/12/2015 7:33 AM, Pantelis Antoniou wrote: >> Hi Geert, >> >>> On May 12, 2015, at 14:56 , Geert Uytterhoeven >>> wrote: >>> >>> This allows to handle device tree overlays like plain devi

Re: [PATCH] regulator: Add SPMI regulator driver

2015-05-15 Thread Frank Rowand
On 5/12/2015 2:39 PM, Stephen Boyd wrote: > Add an SPMI regulator driver for Qualcomm's PM8941 and PM8916 > PMICs. This driver is based largely on code from > codeaurora.org[1]. > > [1] > https://www.codeaurora.org/cgit/quic/la/kernel/msm-3.10/tree/drivers/regulator/qpnp-regulator.c?h=msm-3.10 >

Re: [PATCH/RFC] kbuild: Create a rule for building device tree overlay objects

2015-05-15 Thread Frank Rowand
On 5/12/2015 7:33 AM, Pantelis Antoniou wrote: > Hi Geert, > >> On May 12, 2015, at 14:56 , Geert Uytterhoeven >> wrote: >> >> This allows to handle device tree overlays like plain device trees. >> >> Signed-off-by: Geert Uytterhoeven >> --- >> Questions: >> - Do we want dtso files under arch/

Re: [PATCH] of/flattree: print memory scan node results in CPU endian

2015-04-08 Thread Frank Rowand
On 4/7/2015 11:44 PM, Florian Fainelli wrote: > Commit 51975db0b7333 ("of/flattree: merge early_init_dt_scan_memory() > common code") consolidated some code from PowerPC (typically > big-endian), and ended-up adding a pr_debug() printing reg properties in > big-endian (DT native) format, not CPU en

Re: [patch 2/7] dt: dtb version: document chosen/dtb-info node binding

2015-03-19 Thread Frank Rowand
On 3/19/2015 12:12 PM, Mark Rutland wrote: >>> I'm not sure I see the point in adding a property which is not >>> well-defined and not guarnateed to be in any way stable. >> >> This binding is kind of an odd ball to me. It is clearly _not_ describing >> hardware, which is really the central point

Re: [patch 2/7] dt: dtb version: document chosen/dtb-info node binding

2015-03-19 Thread Frank Rowand
On 3/19/2015 12:32 PM, Russell King - ARM Linux wrote: > On Thu, Mar 19, 2015 at 12:01:42PM -0700, Frank Rowand wrote: >> On 3/19/2015 11:41 AM, Russell King - ARM Linux wrote: >> What??? Why would we ever accept code that tested the dtb version >> instead of the compatible s

Re: [patch 2/7] dt: dtb version: document chosen/dtb-info node binding

2015-03-19 Thread Frank Rowand
On 3/19/2015 11:41 AM, Russell King - ARM Linux wrote: > On Thu, Mar 19, 2015 at 08:23:29AM -0500, Rob Herring wrote: >> On Wed, Mar 18, 2015 at 10:33 PM, Frank Rowand >> wrote: >>> +version >>> + The version of the DTB. This is analagous

Re: [patch 2/7] dt: dtb version: document chosen/dtb-info node binding

2015-03-19 Thread Frank Rowand
On 3/18/2015 8:33 PM, Frank Rowand wrote: > From: Frank Rowand > > Add /chosen/dtb-node binding. > > Signed-off-by: Frank Rowand > --- > Documentation/devicetree/bindings/chosen.txt | 37 +++ > > Index: b/Documentation/dev

Re: [patch 0/7] dt: dtb version: add version info to dtb

2015-03-19 Thread Frank Rowand
On 3/19/2015 1:12 AM, Gregory CLEMENT wrote: > Hi, > > On 19/03/2015 04:29, Frank Rowand wrote: >> Rob, >> >> Can this be added to the next trees to get some test exposure before >> submitting >> to Linus? >> >> There is currently no way to tie

Re: [patch 2/7] dt: dtb version: document chosen/dtb-info node binding

2015-03-19 Thread Frank Rowand
On 3/19/2015 6:49 AM, Mark Rutland wrote: > On Thu, Mar 19, 2015 at 03:33:22AM +0000, Frank Rowand wrote: >> From: Frank Rowand >> >> Add /chosen/dtb-node binding. > > Why? It doesn't matter what the cover says, the commit message should > have a rationale. &

Re: [patch 2/7] dt: dtb version: document chosen/dtb-info node binding

2015-03-19 Thread Frank Rowand
On 3/19/2015 6:23 AM, Rob Herring wrote: > On Wed, Mar 18, 2015 at 10:33 PM, Frank Rowand wrote: >> From: Frank Rowand >> >> Add /chosen/dtb-node binding. > > Why? Please write better commit messages. Will update. > >> >> Signed-off-by: Frank R

[patch 7/7] dt: dtb version: report dtb info

2015-03-18 Thread Frank Rowand
From: Frank Rowand Report the /chosen/dtb-info properties on boot. Signed-off-by: Frank Rowand --- The beginning of the context for hunk 2 includes a line from 2fa645cb2703d9b3786d850db815414dfeefa51d, which is in 4.0-rc4. The author of that commit has submitted a request to revert the

[patch 6/7] dt: dtb version: dtsi files

2015-03-18 Thread Frank Rowand
From: Frank Rowand Create a .dtsi file to contain the /chosen/dtb-info node and populate the properties in that node. Signed-off-by: Frank Rowand --- arch/arm/boot/dts/skeleton.dtsi |2 include/dt-bindings/version.dtsi | 19 + Index: b/include/dt-bindings

[patch 5/7] dt: dtb version: kbuild scripts

2015-03-18 Thread Frank Rowand
From: Frank Rowand After applying this patch, need to add execute permission to the new file scripts/version_dtb_increment_once Modify the dtb compile rules to generate dtb version header files. Create script to increment .version_dtb just once per make of one of more dtbs, and to generate the

[patch 4/7] dt: dtb version: kernel Makefile

2015-03-18 Thread Frank Rowand
From: Frank Rowand Capture the initial value of .version_dtb so that when multiple .dtb files are created in a single make the make scripts will be able to increment .version_dtb only once instead of for each .dtb. Signed-off-by: Frank Rowand --- Makefile

[patch 3/7] dt: dtb version: arm dts Makefile

2015-03-18 Thread Frank Rowand
From: Frank Rowand Remove generated files from the dependencies of .dtb files, where the generated files are created as a result of making a .dtb. Signed-off-by: Frank Rowand --- arch/arm/boot/dts/Makefile | 10 +++ Index: b/arch/arm/boot/dts/Makefile

[patch 2/7] dt: dtb version: document chosen/dtb-info node binding

2015-03-18 Thread Frank Rowand
From: Frank Rowand Add /chosen/dtb-node binding. Signed-off-by: Frank Rowand --- Documentation/devicetree/bindings/chosen.txt | 37 +++ Index: b/Documentation/devicetree/bindings/chosen.txt === --- a/Documentation

[patch 1/7] dt: dtb version: consolidate documentation of chosen node bindings

2015-03-18 Thread Frank Rowand
From: Frank Rowand Documentation of bindings in node /chosen are scattered in several bindings files. If not already in Documentation/devicetree/bindings/chosen.txt, add a pointer in that file to where the property and node bindings are described. This is a clean up in anticipation of adding

[patch 0/7] dt: dtb version: add version info to dtb

2015-03-18 Thread Frank Rowand
Rob, Can this be added to the next trees to get some test exposure before submitting to Linus? There is currently no way to tie a device tree blob (DTB) back to the source and environment used to create it. Add this information, including a DTB version number, which is somewhat analogous to the

[Patch 5/5] dt: unittest: breadcrumbs to reduce pain of future maintainers

2015-03-14 Thread Frank Rowand
From: Frank Rowand Fix warnings pointed out by checkpatch. Checkpatch warns: externs should be avoided in .c files Reducing pain for future maintainers - adding a comment so that anyone trying to find where the extern data is created will be able to find it. (grep will not find that location

[Patch 4/5] dt: unittest: reduce checkpatch noise - line after declarations

2015-03-14 Thread Frank Rowand
From: Frank Rowand Fix warnings pointed out by checkpatch. No bug fixes, but reduce the number of checkpatch warnings so that future problems will stand out better. Signed-off-by: Frank Rowand --- drivers/of/unittest.c |5 + 1 file changed, 5 insertions(+) Index: b/drivers/of

[Patch 3/5] dt: unittest: add const where needed

2015-03-14 Thread Frank Rowand
From: Frank Rowand Fix warnings pointed out by checkpatch. No bugs fixed, but the test code should be a good example of how to use the devicetree API. Signed-off-by: Frank Rowand --- drivers/of/unittest.c |8 1 file changed, 4 insertions(+), 4 deletions(-) Index: b/drivers/of

<    3   4   5   6   7   8   9   10   >