Re: [PATCH v2 1/2] ARM: DT: msm: Add Qualcomm's PRNG driver binding document

2013-10-14 Thread Stanimir Varbanov
Hi Stephen, Thanks for the review. On 10/11/2013 11:46 PM, Stephen Boyd wrote: On 10/11, Stanimir Varbanov wrote: This adds Qualcomm PRNG driver device tree binding documentation to use as an example in dts trees. Signed-off-by: Stanimir Varbanov svarba...@mm-sol.com ---

Re: [PATCH v2 2/2] hwrng: msm: Add PRNG support for MSM SoC's

2013-10-14 Thread Stanimir Varbanov
Hi Stephen, Thanks for the review. On 10/11/2013 11:37 PM, Stephen Boyd wrote: On 10/11, Stanimir Varbanov wrote: This adds a driver for hardware random number generator present on Qualcomm MSM SoC's. Signed-off-by: Stanimir Varbanov svarba...@mm-sol.com --- Just nitpicks. diff --git

[PATCH v3 04/13] usb: phy: msm: Remove unnecessarily check for valid regulators.

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com Whether regulators are available or not is checked at driver probe. If they are not available driver will refuse to load, so no need to check them again. Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- drivers/usb/phy/phy-msm-usb.c | 10 --

[PATCH v3 13/13] usb: phy: msm: Add device tree support and binding information

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com Allows MSM OTG controller to be specified via device tree. Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- .../devicetree/bindings/usb/msm-hsusb.txt | 58 +++ drivers/usb/phy/phy-msm-usb.c | 59

[PATCH v3 01/13] usb: phy: msm: Move mach depndend code to platform data

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com This patch fix compilation error and is an intermediate step before the addition of DeviceTree support for newer targets. Cc: David Brown dav...@codeaurora.org Cc: Daniel Walker dwal...@fifo99.com Cc: Felipe Balbi ba...@ti.com Cc: Greg Kroah-Hartman

[PATCH v3 12/13] usb: phy: msm: Properly check core interrupt number

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com IRQ with number 0 is valid case, so check for negative numbers instead. Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- drivers/usb/phy/phy-msm-usb.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/phy/phy-msm-usb.c

[PATCH v3 10/13] usb: phy: msm: Switch clock consumer strings

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com Replace the USB specific clock names in driver with the more standard 'core' and 'iface' ... names. Cc: David Brown dav...@codeaurora.org Cc: Stephen Boyd sb...@codeaurora.org Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com ---

[PATCH v3 09/13] usb: phy: msm: Remove unused pclk_src_name

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com There are no references to 'pclk_src_name' in plaform code, so it is unused. Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- drivers/usb/phy/phy-msm-usb.c | 26 +- include/linux/usb/msm_hsusb.h |5 - 2 files changed,

[PATCH v3 05/13] usb: phy: msm: Fix WARNING: quoted string split across lines

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com This fixes checkpatch.pl warnings. Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- drivers/usb/phy/phy-msm-usb.c | 33 +++-- 1 file changed, 11 insertions(+), 22 deletions(-) diff --git a/drivers/usb/phy/phy-msm-usb.c

[PATCH v3 03/13] usb: phy: msm: Migrate to Managed Device Resource allocation

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com Move memory, regulators, clocks and irq allocation to devm_* variants. Properly check for valid clk handles. Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- drivers/usb/phy/phy-msm-usb.c | 192 - 1 file

[PATCH v3 06/13] usb: phy: msm: Fix WARNING: Prefer seq_puts to seq_printf

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com This fixes checkpatch.pl warnings. Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- drivers/usb/phy/phy-msm-usb.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/usb/phy/phy-msm-usb.c

[PATCH v3 02/13] usb: phy: msm: Move global regulators variables to driver state

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- drivers/usb/phy/phy-msm-usb.c | 82 - include/linux/usb/msm_hsusb.h |3 ++ 2 files changed, 42 insertions(+), 43 deletions(-) diff --git

[PATCH v3 08/13] usb: phy: msm: Drop default usb_dr_mode from platform data.

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com USB DT bindings states: ...In case this attribute isn't passed via DT, USB DRD controllers should default to OTG..., so remove redundand field. Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- drivers/usb/phy/phy-msm-usb.c |7 ---

[PATCH v3 07/13] usb: phy: msm: Replace custom enum usb_mode_type with enum usb_dr_mode

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com --- arch/arm/mach-msm/board-msm7x30.c |2 +- arch/arm/mach-msm/board-qsd8x50.c |2 +- drivers/usb/phy/phy-msm-usb.c | 40 ++--- include/linux/usb/msm_hsusb.h

[PATCH v3 00/13] usb: phy: msm: Fixes, cleanups and DT support

2013-10-14 Thread Ivan T. Ivanov
From: Ivan T. Ivanov iiva...@mm-sol.com Hi, This is third version of MSM OTG clean up patches. v3 -- * In patch 1 - functions for reseting PHY and LINK controller just call platform code if available. * New stuff is: cleaning up unneeded/unused driver state variables * Simplified regulator

Re: [PATCH v4 05/17] arch_timer: Move to generic sched_clock framework

2013-10-14 Thread Kevin Hilman
, it allows us to move the sched_clock setup into drivers clocksource out of the arch ports. Cc: Christopher Covington c...@codeaurora.org Cc: Catalin Marinas catalin.mari...@arm.com Signed-off-by: Stephen Boyd sb...@codeaurora.org A boot failure on Exynos5/Arndale showed up in next-20131014[1

Re: [PATCH v4 05/17] arch_timer: Move to generic sched_clock framework

2013-10-14 Thread Stephen Boyd
/Arndale showed up in next-20131014[1], and a subsequent bisect has fingered this patch as the culprit. I haven't had a chance to debug this any further, but wanted to share in case someone else can debug. The console log is below, but don't think there is much useful there as it shows

Re: [PATCH v4 05/17] arch_timer: Move to generic sched_clock framework

2013-10-14 Thread John Stultz
Signed-off-by: Stephen Boyd sb...@codeaurora.org A boot failure on Exynos5/Arndale showed up in next-20131014[1], and a subsequent bisect has fingered this patch as the culprit. I haven't had a chance to debug this any further, but wanted to share in case someone else can debug. The console

Re: [PATCH v4 05/17] arch_timer: Move to generic sched_clock framework

2013-10-14 Thread John Stultz
On Mon, Oct 14, 2013 at 1:14 PM, Kevin Hilman khil...@linaro.org wrote: Stephen Boyd sb...@codeaurora.org writes: Can you try this patch? Yup, your patch gets it booting again. John, how do you want to proceed on this? The version of $SUBJECT patch that made it to -next doesn't match the

Re: [PATCH v3 01/13] usb: phy: msm: Move mach depndend code to platform data

2013-10-14 Thread Felipe Balbi
Hi, On Mon, Oct 14, 2013 at 06:24:28PM +0300, Ivan T. Ivanov wrote: From: Ivan T. Ivanov iiva...@mm-sol.com This patch fix compilation error and is an intermediate step before the addition of DeviceTree support for newer targets. Cc: David Brown dav...@codeaurora.org Cc: Daniel Walker

Re: [PATCH v3 02/13] usb: phy: msm: Move global regulators variables to driver state

2013-10-14 Thread Felipe Balbi
On Mon, Oct 14, 2013 at 06:24:29PM +0300, Ivan T. Ivanov wrote: From: Ivan T. Ivanov iiva...@mm-sol.com Signed-off-by: Ivan T. Ivanov iiva...@mm-sol.com this one looks good. -- balbi signature.asc Description: Digital signature

Re: [PATCH v3 12/13] usb: phy: msm: Properly check core interrupt number

2013-10-14 Thread Felipe Balbi
On Mon, Oct 14, 2013 at 06:24:39PM +0300, Ivan T. Ivanov wrote: From: Ivan T. Ivanov iiva...@mm-sol.com IRQ with number 0 is valid case, so check for negative not entirelly correct... IRQ 0 isn't supposed to be used as a linux IRQ number IIRC. numbers instead. Signed-off-by: Ivan T.