Re: [PATCH v7 01/10] ARM: davinci: move private EDMA API to arm/common

2013-02-04 Thread Mark Brown
On Mon, Feb 04, 2013 at 09:29:46PM +0100, Linus Walleij wrote: On Mon, Feb 4, 2013 at 8:22 PM, Cyril Chemparathy cy...@ti.com wrote: Based on our experience with fitting multiple subsystems on top of this DMA-Engine driver, I must say that the DMA-Engine interface has proven to be a less

Re: [PATCH 05/73] ARM: ux500: Vsmps3 controlled by SysClkReq1

2013-02-05 Thread Mark Brown
On Mon, Feb 04, 2013 at 10:21:36PM +0100, Linus Walleij wrote: What is needed is some way to model that electronic relationship into both the regulator and clock subsystems (Ulf Hansson is working on the ABx500 clocks as we speak.) Maybe the actual place to have it would be somewhere like

Re: [PATCH v7 01/10] ARM: davinci: move private EDMA API to arm/common

2013-02-05 Thread Mark Brown
On Tue, Feb 05, 2013 at 05:21:48PM +0100, Linus Walleij wrote: For IRQ mode, use the completion callback to push each cookie to NAPI, and thus let the IRQ drive the traffic. The whole purpose of NAPI is to avoid taking interrupts for completion of transfers. Anything that generates interrupts

Re: [PATCH 1/2] extcon: arizona: Remove additional increment for jack flips

2013-02-05 Thread Mark Brown
On Tue, Feb 05, 2013 at 10:08:40AM +, Charles Keepax wrote: Signed-off-by: Charles Keepax ckee...@opensource.wolfsonmicro.com These conflict with patches I've already sent (and which I already posted). I'll roll these into a series and resend. Patch 2 doesn't actually seem to reflect the

Re: [PATCH v7 01/10] ARM: davinci: move private EDMA API to arm/common

2013-02-05 Thread Mark Brown
On Tue, Feb 05, 2013 at 05:06:28PM +, Russell King - ARM Linux wrote: On Tue, Feb 05, 2013 at 04:47:05PM +, Mark Brown wrote: On Tue, Feb 05, 2013 at 05:21:48PM +0100, Linus Walleij wrote: For IRQ mode, use the completion callback to push each cookie to NAPI, and thus let the IRQ

[GIT PULL] regulator updates for v3.8

2013-02-06 Thread Mark Brown
: max8997: Fix using wrong dev argument at various places Marek Szyprowski (1): regulator: max8998: fix incorrect min_uV value for ldo10 Mark Brown (8): Merge remote-tracking branch 'regulator/fix/max77686' into tmp Merge remote-tracking branch 'regulator/fix/max8907' into tmp

Re: [PATCH 3/4] spi: s3c64xx: add gpio quirk for controller

2013-02-07 Thread Mark Brown
On Tue, Feb 05, 2013 at 03:09:43PM -0800, Girish K S wrote: This patch adds support for spi controllers with dedicated clk/miso/mosi/cs pins. It skips the gpio parsing and initialization for controllers that have dedicated pins. if (sdd-tgl_spi != spi) { /* if last mssg on diff

Re: [PATCH 37/77] i2c: convert to idr_alloc()

2013-02-07 Thread Mark Brown
On Wed, Feb 06, 2013 at 11:40:09AM -0800, Tejun Heo wrote: Convert to the much saner new idr interface. Only compile tested. This broke I2C for me in -next today, I saw a spinlock bad magic error calling pm_runtime_enable(). -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH 09/49] ARM: ux500: Update displays in vaux1 consumer list

2013-02-07 Thread Mark Brown
On Thu, Feb 07, 2013 at 04:34:18PM +0100, Linus Walleij wrote: On Wed, Feb 6, 2013 at 11:53 AM, Lee Jones lee.jo...@linaro.org wrote: Add 3 more specific consumers pertaining to the displays found on the u8500 and ST User Interface Boards (UIBs). Signed-off-by: Lee Jones

Re: [PATCH 37/77] i2c: convert to idr_alloc()

2013-02-07 Thread Mark Brown
On Thu, Feb 07, 2013 at 08:32:47AM -0800, Tejun Heo wrote: On Thu, Feb 07, 2013 at 03:28:31PM +, Mark Brown wrote: On Wed, Feb 06, 2013 at 11:40:09AM -0800, Tejun Heo wrote: Convert to the much saner new idr interface. Only compile tested. This broke I2C for me in -next today, I

Re: [PATCH V2 2/4] regulator: tps65090: add DT support

2013-01-29 Thread Mark Brown
On Tue, Jan 29, 2013 at 02:35:16PM +0530, Laxman Dewangan wrote: Add DT support for TI PMIC tps65090 regulator driver. The DT of this device have node regulator and all regulator's node of this device is added under this node. There doesn't seem to be any dependency on the rest of the series

[PATCH] mfd: arizona: Support configuring MICBIASes into bypass mode

2013-01-29 Thread Mark Brown
Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/mfd/arizona-core.c| 12 +++- include/linux/mfd/arizona/pdata.h |1 + 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/drivers/mfd/arizona-core.c b/drivers/mfd/arizona-core.c index d12f21b

Re: [GIT PULL][FOR REVIEW] Regulators: Bring the AB8500 into the 20th century

2013-01-29 Thread Mark Brown
On Tue, Jan 29, 2013 at 04:41:31PM +, Lee Jones wrote: If you'd like me to sent the patches to the MLs for a better review platform I can do, but I'd like to get permission before I go and start upsetting people. Yes. drivers/regulator/Makefile |2 +

Re: [PATCH] regulator: clear state each invocation of of_regulator_match

2013-01-29 Thread Mark Brown
On Tue, Jan 29, 2013 at 12:01:13PM -0700, Stephen Warren wrote: From: Stephen Warren swar...@nvidia.com of_regulator_match() saves some dynamcially allocated state into the match table that's passed to it. By implementation and not contract, for each match table entry, if non-NULL state is

Re: [GIT PULL][FOR REVIEW] Regulators: Bring the AB8500 into the 20th century

2013-01-29 Thread Mark Brown
On Tue, Jan 29, 2013 at 04:41:31PM +, Lee Jones wrote: If you'd like me to sent the patches to the MLs for a better review platform I can do, but I'd like to get permission before I go and start upsetting people. This branch contains 111 patches and has already gone though multiple

Re: [PATCH v6 08/10] spi: omap2-mcspi: convert to dma_request_slave_channel_compat()

2013-01-30 Thread Mark Brown
user implementing EDMA so in the !DT case we can default to the OMAP DMA filter. Acked-by: Mark Brown broo...@opensource.wolfsonmicro.com -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http

Re: [PATCH] spi: spi-fsl-spi: Make spi-fsl-spi usable in cpu mode outside of FSL SOC environments and add a grlib variant normally running on sparc

2013-01-30 Thread Mark Brown
On Wed, Jan 30, 2013 at 11:04:33AM +0100, Andreas Larsson wrote: On 2012-12-28 17:16, Andreas Larsson wrote: This makes the cpu mode of the driver available outside of an FSL SOC and even powerpc environment. This is accomplished by putting things regarding fsl specific code and to cpm

Re: [PATCH] regulator: tps6507x: Fix using wrong dev argument for calling of_regulator_match

2013-01-30 Thread Mark Brown
On Sun, Jan 27, 2013 at 08:54:40PM +0800, Axel Lin wrote: The dev parameter is the device requesting the data. In this case it should be pdev-dev rather than pdev-dev.parent. Applied, thanks. signature.asc Description: Digital signature

Re: [RFT][PATCH v2] regulator: Add missing of_node_put()

2013-01-30 Thread Mark Brown
On Sun, Jan 27, 2013 at 09:16:56PM +0800, Axel Lin wrote: of_find_node_by_name() returns a node pointer with refcount incremented, use of_node_put() on it when done. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH] spi: spi-fsl-spi: Make spi-fsl-spi usable in cpu mode outside of FSL SOC environments and add a grlib variant normally running on sparc

2013-01-30 Thread Mark Brown
On Wed, Jan 30, 2013 at 12:04:38PM +0100, Andreas Larsson wrote: If you would like I'll resend the patch adding you to the CC list. Yes, please. signature.asc Description: Digital signature

Re: [PATCH V3] regulators: anatop: add set_voltage_time_sel interface

2013-01-30 Thread Mark Brown
On Thu, Jan 31, 2013 at 11:23:53AM -0500, Anson Huang wrote: some of anatop's regulators(cpu, vddpu and vddsoc) have register settings about LDO's step time, which will impact the LDO ramp up speed, need to use set_voltage_time_sel interface to add necessary delay everytime LDOs' voltage is

Re: [PATCH] regulator: tps65090: Fix using wrong dev argument for calling of_regulator_match

2013-01-30 Thread Mark Brown
On Wed, Jan 30, 2013 at 08:28:20PM +0800, Axel Lin wrote: The dev parameter is the device requesting the data. In this case it should be pdev-dev rather than pdev-dev.parent. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH 1/2] regulator: max8997: Use of_get_child_count()

2013-01-30 Thread Mark Brown
On Wed, Jan 30, 2013 at 08:52:49PM +0800, Axel Lin wrote: Signed-off-by: Axel Lin axel@ingics.com Applied both, thanks. signature.asc Description: Digital signature

Re: [PATCH] ASoC: fsl: Fix multiple symbol definitions

2013-01-30 Thread Mark Brown
On Thu, Jan 31, 2013 at 08:06:58AM +0100, Thierry Reding wrote: That does indeed look much better than fiddling with the Makefile. My automated ARM builds of linux-next were failing for a few days because of this, so I guess the fix (as well as the MMC one) just hadn't made it into linux-next

Re: [PATCH] spi: spi-gpio: fix compilation warning on 64 bits systems

2013-01-31 Thread Mark Brown
On Thu, Jan 31, 2013 at 11:12:56AM +0100, Maxime Ripard wrote: SPI_GPIO_NO_MOSI and SPI_GPIO_NO_MISO flags are type casted to unsigned long, yet, they are to be stored in an unsigned int field in the spi_gpio_platform_data structure. Applied, thanks. signature.asc Description: Digital

Re: [PATCH v3 1/4] ASoC: tlv320aic3x: Convert mic bias to a supply widget

2013-01-31 Thread Mark Brown
On Thu, Jan 31, 2013 at 04:53:10PM +0530, Hebbar Gururaja wrote: :100644 100644 e7b98f4... f47c3f5... M Documentation/devicetree/bindings/sound/tlv320aic3x.txt :100644 100644 ffd9bc7... 9407fd0... Minclude/sound/tlv320aic3x.h :100644 100644 4989143... 65d09d6... M

Re: [PATCH v3 1/4] ASoC: tlv320aic3x: Convert mic bias to a supply widget

2013-01-31 Thread Mark Brown
On Thu, Jan 31, 2013 at 11:27:07AM +, Hebbar, Gururaja wrote: On Thu, Jan 31, 2013 at 16:54:16, Mark Brown wrote: As mentioned when you posted this before this patch needs to update the machine drivers using the device too. The patch-set 2/4 does the same thing. Since I have da850-evm

Re: [PATCH 1/1] ASoC: Ux500: Fix build error pertaining to missing include file

2013-01-31 Thread Mark Brown
On Thu, Jan 31, 2013 at 12:34:00PM +, Lee Jones wrote: Some compilers complain when building sound for MOP500: sound/soc/ux500/mop500.c:27:27: fatal error: mop500_ab8500.h: No such file or directory Should be kernel version dependant I expect... signature.asc Description:

Re: [PATCH 1/1] ASoC: Ux500: Fix build error pertaining to missing include file

2013-01-31 Thread Mark Brown
On Thu, Jan 31, 2013 at 01:03:40PM +, Lee Jones wrote: On Thu, 31 Jan 2013, Mark Brown wrote: On Thu, Jan 31, 2013 at 12:34:00PM +, Lee Jones wrote: Some compilers complain when building sound for MOP500: sound/soc/ux500/mop500.c:27:27: fatal error: mop500_ab8500.h

Re: [PATCH 1/1] ASoC: Ux500: Fix build error pertaining to missing include file

2013-01-31 Thread Mark Brown
On Thu, Jan 31, 2013 at 02:07:35PM +, Lee Jones wrote: On Thu, 31 Jan 2013, Mark Brown wrote: I'm a bit surprised the compiler would change something like that, I'd expect it to be more the build system. You've lost me. When I compile this file it succeeds. When the person who

Re: [PATCH 1/1] ASoC: Ux500: Fix build error pertaining to missing include file

2013-01-31 Thread Mark Brown
On Thu, Jan 31, 2013 at 02:21:17PM +, Lee Jones wrote: Are you going to take the patch? Yes, I applied it when I sent the original reply. signature.asc Description: Digital signature

Re: [PATCH 05/73] ARM: ux500: Vsmps3 controlled by SysClkReq1

2013-02-26 Thread Mark Brown
On Tue, Feb 26, 2013 at 09:44:12AM +, Lee Jones wrote: Okay, I've just re-read this thread and it appears you're both talking about different things. Linus is talking about a discussion with Bengt regarding how these strange sysclkreq thingies work. When I first read the code they

Re: [PATCH] Regulator: db8500-prcmu - remove incorrect __exit markup

2013-02-28 Thread Mark Brown
On Sun, Feb 24, 2013 at 07:26:25PM -0800, Dmitry Torokhov wrote: Even if bus is not hot-pluggable, the devices can be unbound from the driver via sysfs, so we should not be using __exit annotations on remove() methods. The only exception is drivers registered with platform_driver_probe() which

Re: [PATCH] regulator: core: update kernel documentation for regulator_desc

2013-02-28 Thread Mark Brown
On Thu, Feb 28, 2013 at 06:12:47PM -0600, Nishanth Menon wrote: commit df367931 (regulator: core: Provide regmap get/set bypass operations) introduced regulator_[gs]et_bypass_regmap Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH] regulator: core: fix documentation error in regulator_allow_bypass

2013-02-28 Thread Mark Brown
On Thu, Feb 28, 2013 at 06:44:54PM -0600, Nishanth Menon wrote: commit f59c8f9f (regulator: core: Support bypass mode) has a short documentation error around the regulator_allow_bypass parameter 'enable' which is documented as 'allow'. Applied, thanks. signature.asc Description: Digital

Re: [PATCH] regulator: palmas: use correct device node for DT parsing

2013-02-28 Thread Mark Brown
On Wed, Feb 27, 2013 at 02:23:42PM +, Graeme Gregory wrote: On 27/02/13 14:10, Laxman Dewangan wrote: When device is registered through the DT then regulators node exist in the parent device node of regulator driver. Hence passing parent device node for parsing DT in place of

Re: [PATCH 1/5] regmap: irq: call pm_runtime_put in pm_runtime_get_sync failed case

2013-02-28 Thread Mark Brown
On Thu, Feb 28, 2013 at 03:37:11PM +0800, Li Fei wrote: Even in failed case of pm_runtime_get_sync, the usage_count is incremented. In order to keep the usage_count with correct value and runtime power management to behave correctly, call pm_runtime_put(_sync) in such case. Oh, that is a

Re: [PATCH] regulator: tps6586x: (cosmetic) simplify a conditional

2013-02-28 Thread Mark Brown
On Mon, Feb 25, 2013 at 12:34:09PM +0100, Guennadi Liakhovetski wrote: of_node_put() is called on either branch of a conditional, simplify the code by only calling it once. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH v3 1/4] regulator: core: support shared enable GPIO concept

2013-02-28 Thread Mark Brown
On Mon, Feb 18, 2013 at 06:50:39AM +, Kim, Milo wrote: + pin-gpio = config-ena_gpio; + pin-ena_gpio_invert = config-ena_gpio_invert; + list_add(pin-list, regulator_ena_gpio_list); We should really validate that the invert settings are consistent but it's not so important since

Re: [PATCH v3 0/4] regulator: core: support shared enable GPIO

2013-02-28 Thread Mark Brown
On Mon, Feb 18, 2013 at 06:50:30AM +, Kim, Milo wrote: A Regulator can be enabled by external GPIO pin which is configurable in the regulator_config. At this moment, the GPIO can be owned by only one regulator device. In some devices like LP8788 LDOs, multiple regulators are enabled by

Re: [PATCH] regulator: tps6586x: Use dev_err rather than dev_warn for error message

2013-02-28 Thread Mark Brown
On Wed, Feb 20, 2013 at 10:23:46AM +0800, Axel Lin wrote: tps6586x_regulator_set_slew_rate() returns -EINVAL when having slew rate settings for other than SM0/1, thus use dev_err rather than dev_warn. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH 2/2] drivers/regulator/s5m8767.c: adjust duplicate test

2013-02-28 Thread Mark Brown
On Sun, Feb 24, 2013 at 12:55:34PM +0100, Julia Lawall wrote: From: Julia Lawall julia.law...@lip6.fr Delete successive tests to the same location. Applied, thanks. If you're sending a bunch of patches intended to be applied separately it's probably not worth numbering them, it avoids

Re: [PATCH v2 1/2] regulator: palmas fix SMPS no voltages

2013-02-28 Thread Mark Brown
On Sat, Feb 23, 2013 at 04:35:40PM +, Ian Lartey wrote: From: Graeme Gregory g...@slimlogic.co.uk Number of voltages for SMPS regulators was off by one. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH 4/4] regulator: palmas: Change the DT node property names to follow the convention

2013-02-28 Thread Mark Brown
On Mon, Feb 18, 2013 at 10:44:20AM +0530, J Keerthy wrote: DT node properties should not have _. Replacing them by -. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH 2/2] regulators: max8998.c: use dev_err() instead of printk()

2013-02-28 Thread Mark Brown
On Sat, Feb 23, 2013 at 12:51:26AM -0300, Thiago Farina wrote: Fixes the following checkpatch warning: WARNING: Prefer netdev_err(netdev, ... then dev_err(dev, ... then pr_err(... to printk(KERN_ERR ... This doesn't apply against current mainline, and... @@ -666,7 +666,7 @@ static int

Re: [PATCH] regulators: max8998.c: use dev_err() instead of printk()

2013-02-28 Thread Mark Brown
On Sat, Feb 23, 2013 at 12:52:35AM -0300, Thiago Farina wrote: Fixes the following checkpatch warning: WARNING: Prefer netdev_err(netdev, ... then dev_err(dev, ... then pr_err(... to printk(KERN_ERR ... Applied, thanks (and discarded the previous version as it's subsumed in there).

Re: [PATCH] regulator: twl: Convert twl4030ldo_ops to get_voltage_sel

2013-03-01 Thread Mark Brown
On Sat, Feb 16, 2013 at 10:09:54AM +0800, Axel Lin wrote: This fixes an inconsistent behavior between list_voltage() and get_voltage() because current implementation of get_voltage() does not check the case IS_UNSUP() is true. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH] regulator: twl: Remove VDD1_VSEL_table and VDD2_VSEL_table

2013-03-01 Thread Mark Brown
On Fri, Feb 15, 2013 at 10:23:01PM +0800, Axel Lin wrote: Since commit ba305e31 regulator: twl: fix twl4030 support for smps regulators, VDD1_VSEL_table and VDD2_VSEL_table are not used any more. Remove them. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH] regulator: twl: Convert fixed voltage to use regulator_list_voltage_linear

2013-03-01 Thread Mark Brown
On Sat, Feb 16, 2013 at 10:37:33AM +0800, Axel Lin wrote: Signed-off-by: Axel Lin axel@ingics.com I didn't apply this as there was an earlier version sitting in my tree which for some reason had never been pushed out and merged. Sorry about that. signature.asc Description: Digital

Re: [PATCH] regulator: twl: Remove TWL6030_FIXED_RESOURCE

2013-03-01 Thread Mark Brown
On Sat, Feb 16, 2013 at 10:53:49AM +0800, Axel Lin wrote: TWL6030_FIXED_RESOURCE is not used now, remove it. TWL6030_FIXED_RESOURCE is not used since commit e76ab829cc regulator: twl: Remove references to the twl4030 regulator Applied, thanks. signature.asc Description: Digital signature

Re: [RFC/RFT][PATCH 1/3] regulator: core: Add enable_pulldown flag to indicate pulldown on EN input

2013-03-01 Thread Mark Brown
On Fri, Feb 15, 2013 at 05:19:21PM +0800, Axel Lin wrote: Add enable_pulldown flag to indicate pulldown on EN input when using regulator_enable_regmap and friends APIs. - return (val rdev-desc-enable_mask) != 0; + if (rdev-desc-enable_pulldown) + return (val

Re: [PATCH] regulator: fixed regulator_bulk_enable unwinding code

2013-03-01 Thread Mark Brown
On Fri, Mar 01, 2013 at 12:24:05PM +0100, Andrzej Hajda wrote: Unwinding code disables all successfully enabled regulators. Error is logged for every failed regulator. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH] mfd: palmas: provide irq flags through DT/platform data

2013-03-01 Thread Mark Brown
On Fri, Mar 01, 2013 at 06:04:56PM +0530, Laxman Dewangan wrote: Currently driver sets the irq type to IRQF_TRIGGER_LOW which is causing interrupt registration failure in ARM based SoCs as: [0.208479] genirq: Setting trigger mode 8 for irq 118 failed (gic_set_type+0x0/0xf0) [

Re: [PATCH] mfd: palmas: provide irq flags through DT/platform data

2013-03-01 Thread Mark Brown
On Fri, Mar 01, 2013 at 06:25:24PM +0530, Laxman Dewangan wrote: On Friday 01 March 2013 06:13 PM, Mark Brown wrote: This can't be a generic problem on ARM systems, I'm pretty sure the primary users of palmas would've noticed, this is more of a new feature isn't it? I think it is tested

Re: [PATCH v3] davinci-mcasp: Add support for multichannel playback

2013-03-01 Thread Mark Brown
On Thu, Feb 28, 2013 at 04:07:08PM +0100, Michal Bachraty wrote: Davinci McASP has support for I2S multichannel playback. For I2S playback/receive, each serializer is capable to play 2 channels (L/R) audio data.Serializer function (Playback-receive-none) is configured Applied, thanks. I am

Re: [PATCH] mfd: palmas: provide irq flags through DT/platform data

2013-03-01 Thread Mark Brown
On Fri, Mar 01, 2013 at 12:34:40PM -0700, Stephen Warren wrote: Is Palmas a family of chips rather than a single chip then? That implies that the DT would need two compatible values, e.g.: Yes. compatible = ti,12345, ti,palmas; ... where 12345 is the actual chip name. ... rather than

Re: [PATCH -next] spi: fix return value check in ce4100_spi_probe()

2013-03-01 Thread Mark Brown
On Fri, Feb 22, 2013 at 10:52:35AM +0800, Wei Yongjun wrote: From: Wei Yongjun yongjun_...@trendmicro.com.cn In case of error, the function platform_device_register_full() returns ERR_PTR() and never returns NULL. The NULL test in the return value check should be replaced with IS_ERR().

Re: [PATCH 1/4] documentation: add palmas dts definition

2013-03-01 Thread Mark Brown
On Wed, Feb 27, 2013 at 11:16:57AM -0700, Stephen Warren wrote: I believe what Mark wants is something like the following in this file (take from Documentation/devicetree/bindings/regulator/tps6586x.txt): - regulators: A node that houses a sub-node for each regulator within the device.

Re: [PATCH v2] mfd: as3711: add OF support

2013-03-01 Thread Mark Brown
On Mon, Feb 18, 2013 at 10:57:44AM +0100, Guennadi Liakhovetski wrote: Add device-tree bindings to the AS3711 regulator and backlight drivers. Reviwed-by: Mark Brown broo...@opensource.wolfsonmicro.com signature.asc Description: Digital signature

[PATCH] mfd: tps65912: Declare and use tps65912_irq_exit()

2013-03-01 Thread Mark Brown
Clean up interrupts on exit, silencing a sparse warning caused by tps65912_irq_exit() being defined but not prototyped as we go. Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/mfd/tps65912-core.c |1 + include/linux/mfd/tps65912.h |1 + 2 files changed, 2

[PATCH] mfd: twl4030-audio: Fix argument type for twl4030_audio_disable_resource()

2013-03-01 Thread Mark Brown
Looks like the conversion to enum was missed for the definition of this function, the declaration has been updated. Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/mfd/twl4030-audio.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd

[PATCH] leds: wm8350: Complain if we fail to reenable DCDC

2013-03-01 Thread Mark Brown
Provide some trace, though the hardware is most likely non-functional if this happens. Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/leds/leds-wm8350.c |6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drivers/leds/leds-wm8350.c b/drivers

[PATCH] mfd: wm5102: Update rev B patch for latest evaluation

2013-03-02 Thread Mark Brown
From: Charles Keepax ckee...@opensource.wolfsonmicro.com Signed-off-by: Charles Keepax ckee...@opensource.wolfsonmicro.com Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/mfd/wm5102-tables.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

Re: [PATCH] mfd: palmas: provide irq flags through DT/platform data

2013-03-02 Thread Mark Brown
On Sat, Mar 02, 2013 at 08:13:35PM +0800, Graeme Gregory wrote: Should these also flow down into the various drivers for the IP blocks? eg. compatible = ti,twl6035-regulator, ti,palmas-regulator; Ideally, in case there's chip specific issues. signature.asc Description: Digital signature

Re: [PATCH 1/2] regulator: tps65217: Fix using wrong dev argument for calling of_regulator_match

2013-01-24 Thread Mark Brown
On Thu, Jan 24, 2013 at 10:27:30AM +0800, Axel Lin wrote: The dev parameter is the device requestiong the data. In this case it should be pdev-dev rather than pdev-dev.parent. Applied both, thanks. signature.asc Description: Digital signature

[GIT PULL] regmap updates for v3.4

2013-01-24 Thread Mark Brown
fix. Mark Brown (2): regmap: debugfs: Fix seeking from the cache Merge remote-tracking branch 'regmap/fix/debugfs' into tmp Nestor Ovroy (1): regmap: fix small typo in regmap_bulk_write comment drivers/base/regmap

Re: [PATCH 1/2] i2c-core: Add gpio based bus arbitration implementation

2013-01-24 Thread Mark Brown
On Thu, Jan 24, 2013 at 12:13:29PM +0100, Wolfram Sang wrote: On Sat, Dec 15, 2012 at 11:21:36PM +0900, Mark Brown wrote: also get things like read operations which appear as multiple transactions on the I2C bus so require something higher level than what multi-master provides. I don't

Re: [PATCH 5/6] ARM: regulator: add tps6507x device tree data

2013-01-24 Thread Mark Brown
On Thu, Jan 24, 2013 at 04:25:19PM +0530, Vishwanathrao Badarkhe, Manish wrote: --- arch/arm/boot/dts/tps6507x.dtsi | 47 +++ Why are you putting this in arch/arm? signature.asc Description: Digital signature

Re: [PATCH 6/6 v14] gpio: Add block gpio to several gpio drivers

2013-01-24 Thread Mark Brown
On Thu, Jan 24, 2013 at 01:02:38PM +0100, Stijn Devriendt wrote: As a fictive example, consider the i2c-bitbang driver, which you could optimize by using block-gpio with sda/scl in a single block. By offering the block-gpio API even when you cannot set all bits at once, you could cause

[PATCH] mfd: wm5102: Make DSP scratch registers readable

2013-01-25 Thread Mark Brown
Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/mfd/wm5102-tables.c |8 include/linux/mfd/arizona/registers.h | 16 2 files changed, 24 insertions(+) diff --git a/drivers/mfd/wm5102-tables.c b/drivers/mfd/wm5102-tables.c index

[PATCH] extcon: arizona: Use regulated mode for microphone supply when detecting

2013-01-25 Thread Mark Brown
When starting microphone detection some headsets should be exposed to the fully regulated microphone bias in order to ensure that they behave in an optimal fashion. Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/extcon/Kconfig |2 +- drivers/extcon/extcon

[PATCH] mfd: wm5102: Fix definition of WM5102_MAX_REGISTER

2013-01-25 Thread Mark Brown
Updated in latest datasheet. Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/mfd/wm5102-tables.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mfd/wm5102-tables.c b/drivers/mfd/wm5102-tables.c index f183eca..9933034 100644 --- a/drivers

Re: [PATCH 11/19] regmap: avoid undefined return from regmap_read_debugfs

2013-01-25 Thread Mark Brown
On Fri, Jan 25, 2013 at 02:14:28PM +, Arnd Bergmann wrote: Gcc warns about the case where regmap_read_debugfs tries Are you sure about that function name? to walk an empty map-debugfs_off_cache list, which results in uninitialized variable getting returned. Setting this variable to 0

Re: [PATCH 11/19] regmap: avoid undefined return from regmap_read_debugfs

2013-01-25 Thread Mark Brown
On Sat, Jan 26, 2013 at 12:42:26PM +0800, Mark Brown wrote: On Fri, Jan 25, 2013 at 02:14:28PM +, Arnd Bergmann wrote: Gcc warns about the case where regmap_read_debugfs tries Are you sure about that function name? to walk an empty map-debugfs_off_cache list, which results

Re: [PATCH 1/2] i2c-core: Add gpio based bus arbitration implementation

2013-01-25 Thread Mark Brown
On Thu, Jan 24, 2013 at 12:39:48PM +0100, Wolfram Sang wrote: On Thu, Jan 24, 2013 at 07:18:47PM +0800, Mark Brown wrote: A read is typically implemented as a write of the register address followed by a read of the value, usually with the ability to free the bus in between. If two devices

Re: [PATCH RESEND] ARM: dts: max77686: Add DTS file for max77686 PMIC

2013-01-25 Thread Mark Brown
On Fri, Jan 25, 2013 at 03:46:08AM +0900, Dongjin Kim wrote: --- arch/arm/boot/dts/max77686.dtsi | 156 +++ Why is this in arch/arm? This isn't an ARM-specific chip. signature.asc Description: Digital signature

Re: [PATCH 5/6] ARM: regulator: add tps6507x device tree data

2013-01-25 Thread Mark Brown
On Fri, Jan 25, 2013 at 06:29:49AM +, Vishwanathrao Badarkhe, Manish wrote: On Thu, Jan 24, 2013 at 17:30:51, Mark Brown wrote: I too doubt that whether it should be in architecture specific folder, My code is in reference to below patch: arm/dts: regulator: Add tps65910 device tree

Re: [PATCH v3 01/10] spi/pxa2xx: allow building on a 64-bit kernel

2013-01-25 Thread Mark Brown
On Tue, Jan 22, 2013 at 12:26:24PM +0200, Mika Westerberg wrote: We are going to use it on 64-bit kernel on Intel Lynxpoint so make sure we can build it into such kernel. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH v3 02/10] spi/pxa2xx: fix warnings when compiling a 64-bit kernel

2013-01-25 Thread Mark Brown
On Tue, Jan 22, 2013 at 12:26:25PM +0200, Mika Westerberg wrote: Fix following warnings seen when compiling 64-bit: Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH v3 03/10] spi/pxa2xx: convert to the pump message infrastructure

2013-01-25 Thread Mark Brown
On Tue, Jan 22, 2013 at 12:26:26PM +0200, Mika Westerberg wrote: The SPI core provides infrastructure for standard message queueing so use that instead of handling everything in the driver. This simplifies the driver. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH v3 04/10] spi/pxa2xx: convert to the common clk framework

2013-01-25 Thread Mark Brown
On Tue, Jan 22, 2013 at 12:26:27PM +0200, Mika Westerberg wrote: Convert clk_enable() to clk_prepare_enable() and clk_disable() to clk_disable_unprepare() respectively in order to support the common clk framework. Otherwise we get warnings on the console as the clock is not prepared before it

Re: [PATCH] regulators/db8500: Fix compile failure for drivers/regulator/dbx500-prcmu.c

2013-01-25 Thread Mark Brown
On Thu, Jan 24, 2013 at 10:29:26AM -0500, Steven Rostedt wrote: Building for the snowball board, I ran into this compile failure: Applied, thanks. Please use subject lines appropriate for the subsystem (I see I let the original one through). signature.asc Description: Digital signature

Re: [PATCH 1/2] spi: spi-gpio: Add checks for the dt properties

2013-01-25 Thread Mark Brown
On Fri, Jan 25, 2013 at 09:39:34AM +0100, Maxime Ripard wrote: The bindings assumed that the gpios properties were always there, which made the NO_TX and NO_RX mode not usable from device tree. Add extra checks to make sure that the driver can work if either MOSI or MISO is not used. Applied,

Re: [PATCH] regulator: tps80031: Use IS_ERR to check return value of regulator_register()

2013-01-26 Thread Mark Brown
On Fri, Jan 25, 2013 at 09:29:41PM +0800, Axel Lin wrote: regulator_register() does not return NULL, it returns ERR_PTR on error. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH] regulators: max8998: fix incorrect min_uV value for ldo10

2013-01-26 Thread Mark Brown
On Fri, Jan 25, 2013 at 09:25:40AM +0100, Marek Szyprowski wrote: Correct value for minimal voltage for ldo10 output is 95 uV. This patch fixes the typo introduced by patch adf6178ad5552a7f2f742a8c85343c50 (regulator: max8998: Use uV in voltage_map_desc), what solves broken probe of

Re: [PATCH 11/19] regmap: avoid undefined return from regmap_read_debugfs

2013-01-26 Thread Mark Brown
On Sat, Jan 26, 2013 at 09:17:27AM +, Arnd Bergmann wrote: On Saturday 26 January 2013, Mark Brown wrote: On Fri, Jan 25, 2013 at 02:14:28PM +, Arnd Bergmann wrote: Gcc warns about the case where regmap_read_debugfs tries Are you sure about that function name? Yes

[PATCH] regmap: debugfs: Work around GCC flow analysis false positives

2013-01-26 Thread Mark Brown
Reported-by: Vincent Stehlé v-ste...@ti.com Reported-by: Arnd Bergmann a...@arndb.de Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/base/regmap/regmap-debugfs.c |7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/base/regmap/regmap-debugfs.c b/drivers

Re: [PATCH] regmap: debugfs: Fix compilation warning

2013-01-26 Thread Mark Brown
On Thu, Jan 24, 2013 at 11:12:24AM +0100, Vincent Stehlé wrote: If you mean am I sure the loop iterates at least once, then yes, as we have an explicit check just before the concerned list_for_each_entry(): So, I hadn't realised this was triggered by a new compiler version - Arnd just tried

Re: [PATCH 11/19] regmap: avoid undefined return from regmap_read_debugfs

2013-01-26 Thread Mark Brown
On Sat, Jan 26, 2013 at 09:59:16AM +, Russell King - ARM Linux wrote: Why not code the function in a way that avoids the problem altogether? That'd do the trick too; feel free to submit a patch... signature.asc Description: Digital signature

Re: [PATCH 11/19] regmap: regmap: avoid spurious warning in regmap_read_debugfs

2013-01-26 Thread Mark Brown
On Sat, Jan 26, 2013 at 11:45:35AM +, Arnd Bergmann wrote: Gcc warns about the case where regmap_read_debugfs tries to walk an empty map-debugfs_off_cache list, which would results in uninitialized variable getting returned, if we hadn't checked the same condition just before that.

Re: [PATCH 1/3] regulator: max8907: Fix using wrong dev argument for calling of_regulator_match

2013-01-26 Thread Mark Brown
On Fri, Jan 25, 2013 at 10:20:29AM +0800, Axel Lin wrote: The dev parameter is the device requesting the data. In this case it should be pdev-dev rather than pdev-dev.parent. Applied all, thanks. signature.asc Description: Digital signature

Re: [PATCH] regulator: lp8755: Use LP8755_BUCK_MAX instead of magic number

2013-01-26 Thread Mark Brown
On Sat, Jan 26, 2013 at 01:19:47PM +0800, Axel Lin wrote: Signed-off-by: Axel Lin axel@ingics.com Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH 4/6] davinci: regulator: tps6507x: add device tree support.

2013-01-26 Thread Mark Brown
On Thu, Jan 24, 2013 at 04:25:18PM +0530, Vishwanathrao Badarkhe, Manish wrote: Add device tree based initialization support for TI's tps6507x regulators. Applied, thanks. Please always use subject lines appropriate for the subsystem you are submitting against - this is not a DaVinci change.

Re: [PATCH] mc13892-regulator: correct/refine handling of the SWxHI bit

2013-01-26 Thread Mark Brown
On Mon, Jan 21, 2013 at 11:38:40AM -0600, Matt Sealey wrote: MC13892 PMIC supports a HI bit for 3 of it's 4 buck switcher outputs, which enables a higher set of voltage ranges. Applied, thanks. signature.asc Description: Digital signature

Re: [PATCH] mc13892: sanity check num_regulators parsed vs. registered

2013-01-26 Thread Mark Brown
On Mon, Jan 21, 2013 at 12:25:45PM -0600, Matt Sealey wrote: Imagine a situation where a device tree has a few regulators in an appropriate node: Applied, thanks. Always use subject lines appropriate for the subsystem. signature.asc Description: Digital signature

Re: [PATCH 1/3] spi: Add helper functions for setting up transfers

2013-01-26 Thread Mark Brown
() on the message. Reviewed-by: Mark Brown broo...@opensource.wolfsonmicro.com for the helpers, we should try to get them merged separately to the coccinelle rules if there's issue with those. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord

[PATCH 03/10] mfd: arizona: Register MICVDD supply first to ensure no retries

2013-01-26 Thread Mark Brown
Not strictly required as probe deferral will take care of everything but it makes boot a little smoother. Reported-by: Ryo Tsutsui ryo.tsut...@wolfsonmicro.com Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/mfd/arizona-core.c |4 ++-- 1 file changed, 2 insertions

[PATCH 09/10] mfd: wm5102: Update rev B patch for latest evaluation

2013-01-26 Thread Mark Brown
The latest evaluation of the revision B silicon suggests some changes to the tuning applied for optimal performance. Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/mfd/wm5102-tables.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git

[PATCH 10/10] mfd: arizona: Disable interrupts during resume

2013-01-26 Thread Mark Brown
and only reenabling it again during main resume. The goal is to avoid issues in the situation where an interrupt is asserted during resume (eg, due to it being the wake source) and the interrupt handling gets scheduled prior to the device being able to handle runtime PM. Signed-off-by: Mark Brown

[PATCH 02/10] mfd: wm5102: Mark only extant DSP registers volatile

2013-01-26 Thread Mark Brown
Since regmap sometimes uses volatile as a proxy for readable simply having a blanket condition can mark too many registers as readable. Signed-off-by: Mark Brown broo...@opensource.wolfsonmicro.com --- drivers/mfd/wm5102-tables.c | 11 +++ 1 file changed, 7 insertions(+), 4 deletions

<    3   4   5   6   7   8   9   10   11   12   >