Re: [PATCH v2] ASoC: tlv320aic3x: Add support for tlv320aic3104

2015-02-03 Thread Benoît Thébaudeau
Dear Jyri Sarha, Sorry to come late. On Mon, Feb 2, 2015 at 3:48 PM, Jyri Sarha wrote: [...] > diff --git a/sound/soc/codecs/tlv320aic3x.c b/sound/soc/codecs/tlv320aic3x.c > index b7ebce0..cb92cdb 100644 > --- a/sound/soc/codecs/tlv320aic3x.c > +++ b/sound/soc/codecs/tlv320aic3x.c [...] > @@ -55

Re: [PATCH] usb: phy: phy-generic: No need to call gpiod_direction_output() twice

2015-02-03 Thread Fabio Estevam
On Tue, Feb 3, 2015 at 6:52 PM, Felipe Balbi wrote: > it doesn't make a difference though, right ? > gpiod_direction_output(NULL, 1) won't do anything. Yes, I will send a v3 without the NULL check. gpiod_set_value returns immediately if desc is NULL: void gpiod_set_value(struct gpio_desc *desc

Re: [PATCH v3 0/2] couple of generic remoteproc enhancements

2015-02-03 Thread Suman Anna
Ohad, > On 01/09/2015 03:21 PM, Suman Anna wrote: >> Hi Ohad, >> >> The following is an updated patchset addressing the previous pending comments >> from v1 & v2, and are rebased onto the latest 3.19-rc3 (are rc independent >> actually). >> >> The patches are mainly developed to support the WkupM

Re: [PATCH] usb: phy: phy-generic: No need to call gpiod_direction_output() twice

2015-02-03 Thread Felipe Balbi
On Tue, Feb 03, 2015 at 06:21:24PM -0200, Fabio Estevam wrote: > Hi Felipe, > > On Sun, Feb 1, 2015 at 3:37 PM, Felipe Balbi wrote: > > > I like this, very much. Two comments though. We requested the gpio with > > _optional(), and NULL is a valid gpio_desc, this if (nop->gpiod_reset) > > is unne

Re: [PATCH v6 2/6] input: touchscreen: ti_am335x_tsc: Remove udelay in interrupt handler

2015-02-03 Thread Dmitry Torokhov
On Fri, Jan 30, 2015 at 10:56:28AM +0530, Vignesh R wrote: > > > On Wednesday 07 January 2015 11:19 AM, Vignesh R wrote: > > From: Brad Griffis > > > > TSC interrupt handler had udelay to avoid reporting of false pen-up > > interrupt to user space. This patch implements workaround suggesting in

OMAP baseline test results for v3.19-rc7

2015-02-03 Thread Paul Walmsley
Here are some basic OMAP test results for Linux v3.19-rc7. Logs and other details at: http://www.pwsan.com/omap/testlogs/test_v3.19-rc7/20150201224202/ Test summary Build: uImage: Pass ( 3/ 3): omap1_defconfig, omap1_defconfig_1510innovator_only, omap1_def

Re: [PATCH 00/11] ARM: OMAP3: legacy clock data move under clk driver

2015-02-03 Thread Mike Turquette
On Tue, Feb 3, 2015 at 11:04 AM, Tony Lindgren wrote: > * Arnd Bergmann [150203 09:03]: >> On Thursday 08 January 2015, Tony Lindgren wrote: >> >> > Great, hopefully this will finally allow Mike to make the >> > generic struct clk private to drivers/clk :) >> > >> > Seems to boot just fine based

Re: [PATCH 00/11] ARM: OMAP3: legacy clock data move under clk driver

2015-02-03 Thread Tony Lindgren
* Arnd Bergmann [150203 09:03]: > On Thursday 08 January 2015, Tony Lindgren wrote: > > > Great, hopefully this will finally allow Mike to make the > > generic struct clk private to drivers/clk :) > > > > Seems to boot just fine based on a quick legacy booting test > > on n900. > > > > Mike, as

Re: [PATCH 00/11] ARM: OMAP3: legacy clock data move under clk driver

2015-02-03 Thread Arnd Bergmann
On Thursday 08 January 2015, Tony Lindgren wrote: > Great, hopefully this will finally allow Mike to make the > generic struct clk private to drivers/clk :) > > Seems to boot just fine based on a quick legacy booting test > on n900. > > Mike, assuming no other issues, can you please apply these

Re: [Cocci] [PATCH v13 3/6] clk: Make clk API return per-user struct clk instances

2015-02-03 Thread Quentin Lambert
Hello, Julia asked me to have a look and see if I can help. On 02/02/2015 23:50, Mike Turquette wrote: Quoting Stephen Boyd (2015-02-02 14:35:59) On 02/02/15 13:31, Julia Lawall wrote: On Mon, 2 Feb 2015, Stephen Boyd wrote: Julia, Is there a way we can write a coccinelle script to check fo

[PATCH] ARM: dra7xx: hwmod: drop .modulemode from pcie1/2 hwmods

2015-02-03 Thread grygorii.strashko
From: Grygorii Strashko Now DRA7xx pcie1/2 hwmods define PRCM configuration as following: .clkctrl_offs = DRA7XX_CM_PCIE_CLKSTCTRL_OFFSET, .rstctrl_offs = DRA7XX_RM_L3INIT_RSTCTRL_OFFSET, .modulemode = MODULEMODE_SWCTRL, which is completely wrong because DRA7XX_CM_PCIE_CLKSTCTRL_OFFSET is

Re: [PATCH v13 3/6] clk: Make clk API return per-user struct clk instances

2015-02-03 Thread Tony Lindgren
* Tero Kristo [150203 00:49]: > On 02/03/2015 09:03 AM, Tomeu Vizoso wrote: > > > >I think you got it right, just wanted to mention that we can and > >probably should make the clk_get_parent_* calls in the consumer API to > >return per-user clk instances but that we need to make sure first that >

Re: am335x: device tree entry for PRM_RTSTIME

2015-02-03 Thread Nishanth Menon
On 02/03/2015 03:07 AM, Yegor Yefremov wrote: > I need to change the reset length. This is not a problem via "devmem > 0x44e00f04 32 0x10ff", but I would like to specify this in DTS > file. > > So far I could find these macros, but they seem not be used anywhere: > > arch/arm/mach-omap2/prm33

Re: [PATCH v2] ASoC: tlv320aic3x: Add support for tlv320aic3104

2015-02-03 Thread Mark Brown
On Mon, Feb 02, 2015 at 04:48:05PM +0200, Jyri Sarha wrote: > Changes since the first version of the patch > - Added "ti,tlv320aic3104" to tlv320aic3x_of_match table > What surprises me is how come the code worked even before this change? Applied, thanks. I2C has the interesting feature that i

[PATCH] ti-soc-thermal: Delete an unnecessary check before the function call "cpufreq_cooling_unregister"

2015-02-03 Thread SF Markus Elfring
From: Markus Elfring Date: Tue, 3 Feb 2015 11:15:14 +0100 The cpufreq_cooling_unregister() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfri

Re: [PATCH 1/3] ARM: OMAP2+: gpmc: Fix writing in gpmc_cs_set_memconf

2015-02-03 Thread Roger Quadros
On 02/02/15 19:08, Tony Lindgren wrote: > * Roger Quadros [150126 01:38]: >> On 24/01/15 22:28, Semen Protsenko wrote: >>> Some GPMC_CONFIG7 register bits marked as "RESERVED", means they >>> shouldn't be overwritten. A typical approach to handle such bits called >>> "Read-Modify-Write". Writing p

am335x: device tree entry for PRM_RTSTIME

2015-02-03 Thread Yegor Yefremov
I need to change the reset length. This is not a problem via "devmem 0x44e00f04 32 0x10ff", but I would like to specify this in DTS file. So far I could find these macros, but they seem not be used anywhere: arch/arm/mach-omap2/prm33xx.h:#define AM33XX_PRM_RSTTIME_OFFSET 0x0004 arch/a

Re: [PATCH v13 3/6] clk: Make clk API return per-user struct clk instances

2015-02-03 Thread Tero Kristo
On 02/03/2015 09:03 AM, Tomeu Vizoso wrote: On 02/02/2015 11:41 PM, Mike Turquette wrote: Quoting Tero Kristo (2015-02-02 11:32:01) On 02/01/2015 11:24 PM, Mike Turquette wrote: Quoting Tomeu Vizoso (2015-01-23 03:03:30) Moves clock state to struct clk_core, but takes care to change as little