Re: [PATCH 1/2] gpio: add a driver for the Synopsys DesignWare APB GPIO block

2011-12-19 Thread Jamie Iles
Hi Rob, On Sun, Dec 18, 2011 at 09:03:31PM -0600, Rob Herring wrote: Jamie, On 12/18/2011 04:13 AM, Jamie Iles wrote: The Synopsys DesignWare block is used in some ARM devices (picoxcell) and can be configured to provide multiple banks of GPIO pins. The first bank (A) can also provide

Re: [PATCH V3 4/7] cpufreq: add generic cpufreq driver

2011-12-19 Thread Jamie Iles
Hi Richard, On Mon, Dec 19, 2011 at 11:21:40AM +0800, Richard Zhao wrote: It support single core and multi-core ARM SoCs. But currently it assume all cores share the same frequency and voltage. Signed-off-by: Richard Zhao richard.z...@linaro.org ---

Re: [PATCH v2 03/10] mfd: twl-core: Add initial DT support for twl4030/twl6030

2011-12-19 Thread Samuel Ortiz
Hi Benoit, On Fri, Dec 09, 2011 at 03:02:34PM +0100, Benoit Cousson wrote: Add initial device-tree support for twl familly chips. The current version is missing the regulator entries due to the lack of DT regulator bindings for the moment. Only the simple sub-modules that do not depend on

Re: [PATCH 2/9] irq: check domain hwirq range for DT translate

2011-12-19 Thread Cousson, Benoit
Hi Rob, While trying your series to take advantage of the generic-irq domain support for OMAP3 INTC, I realized that the following patch is breaking drivers that are using irq_domain_add_simple. The point is that irq_domain_add_simple does not populate hwirq_base or nr_irq, so that test

Re: [PATCH 0/3] of: Add support for named irq and reg

2011-12-19 Thread Cousson, Benoit
Hi Grant and Rob, Gentle ping: Do you have some comment on that series? Thanks, Benoit On 12/5/2011 3:23 PM, Benoit Cousson wrote: Hi Grant Rob, Following the previous patch submission [1], here is an updated series that adds the support for both reg and irq names. A small improvement is

Re: [PATCH v2 03/10] mfd: twl-core: Add initial DT support for twl4030/twl6030

2011-12-19 Thread Cousson, Benoit
Salut Samuel, On 12/19/2011 1:03 PM, Samuel Ortiz wrote: Hi Benoit, On Fri, Dec 09, 2011 at 03:02:34PM +0100, Benoit Cousson wrote: Add initial device-tree support for twl familly chips. The current version is missing the regulator entries due to the lack of DT regulator bindings for the

Re: [PATCH v3 2/3] regulator: mc13892: add device tree probe support

2011-12-19 Thread Shawn Guo
On Mon, Dec 12, 2011 at 11:15:57PM +0800, Shawn Guo wrote: It adds device tree probe support for mc13892-regulator driver. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Mark Brown broo...@opensource.wolfsonmicro.com Cc: Liam Girdwood l...@ti.com Ping? -- Regards, Shawn

Re: [PATCH V3 4/7] cpufreq: add generic cpufreq driver

2011-12-19 Thread Jamie Iles
On Mon, Dec 19, 2011 at 10:19:29PM +0800, Richard Zhao wrote: On Mon, Dec 19, 2011 at 10:05:12AM +, Jamie Iles wrote: Hi Richard, On Mon, Dec 19, 2011 at 11:21:40AM +0800, Richard Zhao wrote: It support single core and multi-core ARM SoCs. But currently it assume all cores share

Re: [PATCH V3 4/7] cpufreq: add generic cpufreq driver

2011-12-19 Thread Rob Herring
On 12/19/2011 08:39 AM, Jamie Iles wrote: On Mon, Dec 19, 2011 at 10:19:29PM +0800, Richard Zhao wrote: On Mon, Dec 19, 2011 at 10:05:12AM +, Jamie Iles wrote: Hi Richard, On Mon, Dec 19, 2011 at 11:21:40AM +0800, Richard Zhao wrote: It support single core and multi-core ARM SoCs. But

Re: [PATCH] x86: convert to generic irq_domain

2011-12-19 Thread Sebastian Andrzej Siewior
* Rob Herring | 2011-12-16 23:18:49 [-0600]: This is needed to avoid struct collisions with the generic irq_domain support being added to kernel/irq/generic-chip.c. Boot tested, but not on a DT enabled platform. I had a patch in my tree for this and it did not work. One bug was fixed by eef24a

Re: [PATCH 2/9] irq: check domain hwirq range for DT translate

2011-12-19 Thread Cousson, Benoit
On 12/19/2011 3:23 PM, Rob Herring wrote: Benoit, On 12/19/2011 06:41 AM, Cousson, Benoit wrote: Hi Rob, While trying your series to take advantage of the generic-irq domain support for OMAP3 INTC, I realized that the following patch is breaking drivers that are using irq_domain_add_simple.

Re: [PATCH v2 04/10] rtc: rtc-twl: Add DT support for RTC inside twl4030/twl6030

2011-12-19 Thread Cousson, Benoit
Hi Alessandro, Gentle ping on this patch. Thanks, Benoit On 12/9/2011 3:02 PM, Benoit Cousson wrote: Add the DT support for the TI rtc-twl present in the twl4030 and twl6030 devices. Signed-off-by: Benoit Coussonb-cous...@ti.com Cc: Alessandro Zummoa.zu...@towertech.it ---

Re: [RFC v2 PATCH 1/3] dt: device tree bindings for DDR memories

2011-12-19 Thread Olof Johansson
Hi, Some comments below, but also a more general question: How much of this generic data makes sense to encode in the device tree? Final hardware configuration usually has to take into consideration board layout/signal delays, etc, and that's not part of this binding. On Mon, Dec 19, 2011 at

Re: [RFC v2 PATCH 2/3] dt: device tree bindings for TI's EMIF sdram controller

2011-12-19 Thread Olof Johansson
Oh wait, when I saw 3/3 I realized the following too: On Mon, Dec 19, 2011 at 6:05 AM, Aneesh V ane...@ti.com wrote: +- phy-type     : string indicating the phy type. Should be one of the +  following: + +  phy-type-omap4  : PHY used in OMAP4 family of SoCs + +  phy-type-dm81xx : PHY used

RE: [PATCH V2 1/4] i2c: tegra: Add __devinit/exit to probe/remove

2011-12-19 Thread Stephen Warren
Shubhrajyoti Datta wrote at Sunday, December 18, 2011 11:54 PM: On Sun, Dec 18, 2011 at 11:59 AM, Stephen Warren swar...@nvidia.com wrote: This fixes some section mismatch build warnings. What was the error was a devinit function called from these? As I Understand It, probe/remove are

Re: [PATCH V2 1/4] cpufreq: add arm soc generic cpufreq driver

2011-12-19 Thread Stephen Boyd
On 12/18/11 17:03, Richard Zhao wrote: Do you have to patch to implement per-cpu udelay? In current code, udelay uses global loops_per_jiffy. We've been carrying forward the timer based udelay patches. They're in the patch tracker as 6873/1, 6874/1, and 6875/1. -- Sent by an employee of

Re: [REPOST RFC PATCH 0/3] New gpio-poweroff driver to turn off platform devices with GPIOs

2011-12-19 Thread Moffett, Kyle D
On Dec 17, 2011, at 04:20, Mark Brown wrote: On Wed, Dec 14, 2011 at 02:34:37PM -0600, Moffett, Kyle D wrote: On Dec 14, 2011, at 07:02, Mark Brown wrote: The regulator API in -next has DT bindings. Based on the description of our hardware, is there a good way that I can wire up

[PATCH v2] gpio: pl061: convert to use generic irq chip

2011-12-19 Thread Rob Herring
From: Rob Herring rob.herr...@calxeda.com Convert the pl061 irq_chip code to use the generic irq chip code. This has the side effect of using 32-bit accesses rather than 8-bit accesses to interrupt registers. The h/w TRM and testing seem to indicate this is fine. Signed-off-by: Rob Herring

[PATCH v2] gpio: pl061: enable interrupts with DT style binding

2011-12-19 Thread Rob Herring
From: Rob Herring rob.herr...@calxeda.com Enable DT interrupt binding support for pl061 gpio lines. If the gpio node has an interrupt-controller property, then it will be setup to handle interrupts on gpio lines. Signed-off-by: Rob Herring rob.herr...@calxeda.com Cc: Grant Likely

Re: [RFC v2 PATCH 0/3] dt: device tree bindings and data for EMIF and DDR

2011-12-19 Thread Tony Lindgren
* Rob Herring robherri...@gmail.com [111219 14:29]: On 12/19/2011 08:05 AM, Aneesh V wrote: This is an RFC to add new device tree bindings for DDR memories and EMIF - TI's DDR SDRAM controller. The first patch adds bindings for DDR memories. Currently, we have added properties for

Re: [PATCH v3 2/3] regulator: mc13892: add device tree probe support

2011-12-19 Thread Mark Brown
On Mon, Dec 19, 2011 at 10:07:31PM +0800, Shawn Guo wrote: On Mon, Dec 12, 2011 at 11:15:57PM +0800, Shawn Guo wrote: It adds device tree probe support for mc13892-regulator driver. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Mark Brown broo...@opensource.wolfsonmicro.com Cc:

Re: [PATCH v3 1/3] mfd: mc13xxx: add device tree probe support

2011-12-19 Thread Mark Brown
On Mon, Dec 12, 2011 at 11:15:56PM +0800, Shawn Guo wrote: +Sub-nodes: +- regulators : Contain the regulator nodes. The name of regulator node + is being used by mc13xxx regulator driver to find the correct relator + device. Reading this I'm not clear what the name of the node is, or what

Re: [PATCH v2 1/3] regulator: twl: adapt twl-regulator driver to dt

2011-12-19 Thread Mark Brown
On Tue, Dec 13, 2011 at 03:49:33PM +0530, Rajendra Nayak wrote: I'm OK with this but would prefer that OMAP or TWL people were OK with it too. If you do need to respin: +For twl4030 regulators/LDO's ' should *not* be used for plurals except when omitting a duplicated s introduced by one

Re: [PATCH v3 2/3] regulator: mc13892: add device tree probe support

2011-12-19 Thread Shawn Guo
On Tue, Dec 20, 2011 at 12:54:15AM +, Mark Brown wrote: On Mon, Dec 19, 2011 at 10:07:31PM +0800, Shawn Guo wrote: On Mon, Dec 12, 2011 at 11:15:57PM +0800, Shawn Guo wrote: It adds device tree probe support for mc13892-regulator driver. Signed-off-by: Shawn Guo

Re: [PATCH v3 2/3] regulator: mc13892: add device tree probe support

2011-12-19 Thread Mark Brown
On Tue, Dec 20, 2011 at 09:37:12AM +0800, Shawn Guo wrote: It seems that Samuel has applied it. http://article.gmane.org/gmane.linux.drivers.devicetree/10441 Oh, that's a bit unfortunate as looking at the first patch I'm still not happy I understand what the regulator part of the binding is

Re: [REPOST RFC PATCH 0/3] New gpio-poweroff driver to turn off platform devices with GPIOs

2011-12-19 Thread Mark Brown
On Mon, Dec 19, 2011 at 10:56:41AM -0600, Moffett, Kyle D wrote: I still don't understand how the regulator API is supposed to help in my particular case (whole-system poweroff). Well, it depends what you need to do. In the ST-Ericsson case what they needed to do was collapse the core

[PATCHv2 1/2] gpio: add a driver for the Synopsys DesignWare APB GPIO block

2011-12-19 Thread Jamie Iles
The Synopsys DesignWare block is used in some ARM devices (picoxcell) and can be configured to provide multiple banks of GPIO pins. The first bank (A) can also provide IRQ capabilities. v2: - use Rob Herring's irqdomain in generic irq chip patches - use reg property to indicate bank

[PATCHv2 2/2] ARM: picoxcell: use new Synopsys Designware GPIO binding

2011-12-19 Thread Jamie Iles
Signed-off-by: Jamie Iles ja...@jamieiles.com --- arch/arm/boot/dts/picoxcell-pc3x2.dtsi | 19 --- arch/arm/boot/dts/picoxcell-pc3x3.dtsi | 28 +++- 2 files changed, 19 insertions(+), 28 deletions(-) diff --git a/arch/arm/boot/dts/picoxcell-pc3x2.dtsi

Re: [PATCH v3 1/3] mfd: mc13xxx: add device tree probe support

2011-12-19 Thread Shawn Guo
On Tue, Dec 20, 2011 at 12:57:09AM +, Mark Brown wrote: On Mon, Dec 12, 2011 at 11:15:56PM +0800, Shawn Guo wrote: +Sub-nodes: +- regulators : Contain the regulator nodes. The name of regulator node + is being used by mc13xxx regulator driver to find the correct relator + device.

Re: [PATCH v3 2/3] regulator: mc13892: add device tree probe support

2011-12-19 Thread Shawn Guo
On Tue, Dec 20, 2011 at 01:34:25AM +, Mark Brown wrote: On Tue, Dec 20, 2011 at 09:37:12AM +0800, Shawn Guo wrote: It seems that Samuel has applied it. http://article.gmane.org/gmane.linux.drivers.devicetree/10441 Oh, that's a bit unfortunate as looking at the first patch I'm still

Re: [REPOST RFC PATCH 0/3] New gpio-poweroff driver to turn off platform devices with GPIOs

2011-12-19 Thread Moffett, Kyle D
On Dec 19, 2011, at 20:38, Mark Brown wrote: On Mon, Dec 19, 2011 at 10:56:41AM -0600, Moffett, Kyle D wrote: I still don't understand how the regulator API is supposed to help in my particular case (whole-system poweroff). Well, it depends what you need to do. In the ST-Ericsson case what

Re: [PATCH V2 1/4] i2c: tegra: Add __devinit/exit to probe/remove

2011-12-19 Thread Olof Johansson
On Sat, Dec 17, 2011 at 11:29:29PM -0700, Stephen Warren wrote: This fixes some section mismatch build warnings. Signed-off-by: Stephen Warren swar...@nvidia.com Ben, since the patch is trivial, I've picked it up in tegra for-3.3/drivers and will merge that way. Let me know if you disagree.

Re: [PATCH V2 2/4] i2c: tegra: Support DVC controller in device tree

2011-12-19 Thread Olof Johansson
On Sat, Dec 17, 2011 at 11:29:30PM -0700, Stephen Warren wrote: One of the Tegra I2C blocks is inside the DVC (Digital Voltage Controller) block. This block is identical to the rest of the I2C blocks, except that it only supports master mode, it has registers moved around, and it needs some

Re: [PATCH V2 3/4] arm/dt: tegra: Use new compatible value for DVC I2C controller

2011-12-19 Thread Olof Johansson
On Sat, Dec 17, 2011 at 11:29:31PM -0700, Stephen Warren wrote: Update the device tree to indicate which I2C controller is the DVC controller. AUXDATA needs to be updated too, since the compatible value changed. Signed-off-by: Stephen Warren swar...@nvidia.com Thanks, applied to for-3.3/dt.

Re: [PATCH V2 4/4] arm/dt: Add ADT7461 to Seaboard

2011-12-19 Thread Olof Johansson
On Sat, Dec 17, 2011 at 11:29:32PM -0700, Stephen Warren wrote: The ADT7461 is a temperature monitoring IC. One is present on the DVC I2C bus on Seaboard. Signed-off-by: Stephen Warren swar...@nvidia.com Thanks, applied to for-3.3/dt. -Olof ___

Re: [PATCH v3 1/3] mfd: mc13xxx: add device tree probe support

2011-12-19 Thread Mark Brown
On Tue, Dec 20, 2011 at 10:01:02AM +0800, Shawn Guo wrote: On Tue, Dec 20, 2011 at 12:57:09AM +, Mark Brown wrote: On Mon, Dec 12, 2011 at 11:15:56PM +0800, Shawn Guo wrote: +Sub-nodes: +- regulators : Contain the regulator nodes. The name of regulator node + is being used by

Re: [PATCH v3 1/3] mfd: mc13xxx: add device tree probe support

2011-12-19 Thread Shawn Guo
On Tue, Dec 20, 2011 at 01:59:32AM +, Mark Brown wrote: On Tue, Dec 20, 2011 at 10:01:02AM +0800, Shawn Guo wrote: On Tue, Dec 20, 2011 at 12:57:09AM +, Mark Brown wrote: On Mon, Dec 12, 2011 at 11:15:56PM +0800, Shawn Guo wrote: +Sub-nodes: +- regulators : Contain the