Re: [PATCH 8/9] [HACK] ARM: imx: work around v7_cpu_resume link error

2013-02-18 Thread Shawn Guo
On Mon, Feb 18, 2013 at 11:11:30PM -0500, Nicolas Pitre wrote: On Tue, 19 Feb 2013, Shawn Guo wrote: On Mon, Feb 18, 2013 at 12:06:32PM -0500, Nicolas Pitre wrote: Try the following instead. It makes the code simpler and easier to debug. It works now. Thanks, Nico. Care

Re: [PATCH 1/1] ARM: dt: add header to define tegra20 clocks

2013-02-18 Thread Shawn Guo
On Thu, Feb 14, 2013 at 10:54:28AM -0700, Stephen Warren wrote: On 02/13/2013 11:38 PM, Hiroshi Doyu wrote: To replace magic number in clocks = tegra_car 28; I like the concept here; I was thinking about doing this today, but you beat me to it:-) Feel free to create the Tegra30 header too,

Re: [PATCH v4 00/32] ldisc patchset

2013-02-20 Thread Shawn Guo
. Michael and Shawn, I'd appreciate if you test with this series although I know it won't WARN because this patchset removes it. On imx51 and imx6q: Tested-by: Shawn Guo shawn@linaro.org -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord

Re: [PATCH v2] pwm i.MX: add devicetree support

2012-09-06 Thread Shawn Guo
pwm i.MX: add functions to enable/disable pwm. pwm i.MX: Use module_platform_driver pwm i.MX: use per clock unconditionally ARM i.MX53: Add pwm support For the series, Reviewed-by: Shawn Guo shawn@linaro.org Documentation/devicetree/bindings/pwm/imx-pwm.txt | 17

Re: [PATCH 1/2] serial: mxs-auart: fix the wrong setting order

2012-09-06 Thread Shawn Guo
-by: Shawn Guo shawn@linaro.org Are you experiencing any user visible problem with this bug? Regards, Shawn --- drivers/tty/serial/mxs-auart.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/serial/mxs-auart.c b/drivers/tty/serial/mxs-auart.c

Re: [PATCH 2/2] serial: mxs-auart: put the device in mxs_auart_probe()

2012-09-06 Thread Shawn Guo
On Thu, Sep 06, 2012 at 10:38:41PM -0400, Huang Shijie wrote: We call the get_device() in the mxs_auart_probe(). For the balance of the reference count, we should put the device in the mxs_auart_remove(). Signed-off-by: Huang Shijie shij...@gmail.com --- drivers/tty/serial/mxs-auart.c |

Re: [PATCH] serial: mxs-auart: put the device when exit or error

2012-09-06 Thread Shawn Guo
-by: Shawn Guo shawn@linaro.org --- drivers/tty/serial/mxs-auart.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/tty/serial/mxs-auart.c b/drivers/tty/serial/mxs-auart.c index 3a667ee..203e512 100644 --- a/drivers/tty/serial/mxs-auart.c +++ b/drivers

Re: Tracking down suspend/resume ext3/mmc issues on imx233

2012-09-07 Thread Shawn Guo
Copy a few more lists to get wider audience ... Regards, Shawn On Thu, Sep 06, 2012 at 10:03:35PM -0700, Mike Thompson wrote: Hi, I'm working on adding power management support for the imx233 on 3.6-rc2. In general I'm working on porting the pm.c file from the Freescale 2.6.35 kernel for

Re: [PATCH 4/4] efikamx: remove Efika MX-specific CHRGVBUS hack from the USB driver

2012-09-11 Thread Shawn Guo
On Mon, Sep 10, 2012 at 10:03:15AM -0500, Matt Sealey wrote: On Sun, Sep 9, 2012 at 10:00 PM, Fabio Estevam feste...@gmail.com wrote: On Wed, Aug 1, 2012 at 2:49 PM, Matt Sealey m...@genesi-usa.com wrote: Since we're removing Efika MX support from the tree (for now), remove the hack present

Re: [PATCH v2 04/15] dmaengine: Pass no_wakeup parameter via device_prep_dma_cyclic() callback

2012-09-17 Thread Shawn Guo
On Mon, Sep 17, 2012 at 10:16:53AM +0300, Peter Ujfalusi wrote: All users of this callback is updated with this patch. The public API (dmaengine_prep_dma_cyclic) is only used by ASoC, which has been updated by the previous patch. Ah, ok. You actually changed dmaengine_prep_dma_cyclic

Re: [PATCH 4/4] drivers/mmc/host/mxs-mmc.c: fix error return code

2012-09-18 Thread Shawn Guo
...@gmail.com Acked-by: Shawn Guo shawn@linaro.org --- drivers/mmc/host/mxs-mmc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/mmc/host/mxs-mmc.c b/drivers/mmc/host/mxs-mmc.c index 80d1e6d..afd8fdd 100644 --- a/drivers/mmc/host/mxs-mmc.c +++ b/drivers/mmc/host/mxs-mmc.c

Re: [PATCH v3 04/15] dmaengine: Pass flags via device_prep_dma_cyclic() callback

2012-09-18 Thread Shawn Guo
...@vista-silicon.com CC: Zhangfei Gao zhangfei@marvell.com CC: Shawn Guo shawn@linaro.org CC: Laxman Dewangan ldewan...@nvidia.com --- ... drivers/dma/imx-dma.c | 2 +- drivers/dma/imx-sdma.c| 2 +- ... drivers/dma/mxs-dma.c | 2 +- Acked-by: Shawn Guo shawn

Re: [PATCH v9 10/13] ARM: SoC: convert imx6q to SoC descriptor

2012-09-12 Thread Shawn Guo
On Wed, Sep 12, 2012 at 04:58:23PM +0200, Arnd Bergmann wrote: From: Marc Zyngier marc.zyng...@arm.com Convert the imx6q platform to use the SoC descriptor to provide its SMP and CPU hotplug operations. Cc: Shawn Guo shawn@linaro.org Signed-off-by: Marc Zyngier marc.zyng...@arm.com

Re: [PATCH 04/16] ARM: imx: move platform_data definitions

2012-09-13 Thread Shawn Guo
On Wed, Sep 12, 2012 at 03:52:25PM +, Arnd Bergmann wrote: On Tuesday 11 September 2012, Guennadi Liakhovetski wrote: ipu.h is used by the dmaengine and IRQ driver under drivers/dma/ipu/, and by its users drivers/media/platform/soc_camera/mx3_camera.c and drivers/video/mx3fb.c. It has

Re: [PATCH v2 04/15] dmaengine: Pass no_wakeup parameter via device_prep_dma_cyclic() callback

2012-09-17 Thread Shawn Guo
Gao zhangfei@marvell.com CC: Shawn Guo shawn@linaro.org CC: Laxman Dewangan ldewan...@nvidia.com --- drivers/dma/at_hdmac.c| 3 ++- drivers/dma/ep93xx_dma.c | 4 +++- drivers/dma/imx-dma.c | 2 +- drivers/dma/imx-sdma.c| 2 +- drivers/dma/mmp_tdma.c

Re: [PATCH] dma: mxs_dma: Add the missing entries for mx6

2012-07-28 Thread Shawn Guo
On Wed, Jul 25, 2012 at 08:54:02AM -0300, Fabio Estevam wrote: From: Fabio Estevam fabio.este...@freescale.com Add the missing entries for mx6. Signed-off-by: Fabio Estevam fabio.este...@freescale.com --- Build tested only, as I currently do not have mx6 board handy.

Re: [GIT PULL] GPIO changes for v3.6

2012-07-28 Thread Shawn Guo
On Sat, Jul 28, 2012 at 12:37:05AM +0200, Linus Walleij wrote: Better page the MXC people so they can scream if something goes wild. Just tested the driver a little bit, and it still works for me. -- Regards, Shawn -- To unsubscribe from this list: send the line unsubscribe linux-kernel in

Re: i.MX pwm patches

2012-08-31 Thread Shawn Guo
On Tue, Aug 28, 2012 at 01:48:15PM +0200, Sascha Hauer wrote: Hi All, The following patches are an overdue work on the i.MX pwm driver. - introduce SoC specific functions to make the driver easier to maintain - use peripheral clock for pwm output unconditionally - separate the two clocks

Re: [PATCH 5/9] pwm i.MX: add devicetree support

2012-08-31 Thread Shawn Guo
On Tue, Aug 28, 2012 at 01:48:20PM +0200, Sascha Hauer wrote: From: Philipp Zabel p.za...@pengutronix.de Signed-off-by: Philipp Zabel p.za...@pengutronix.de Signed-off-by: Sascha Hauer s.ha...@pengutronix.de --- drivers/pwm/pwm-imx.c | 25 +++-- 1 file changed, 19

Re: [PATCH 9/9] pwm i.MX: add devicetree support

2012-08-31 Thread Shawn Guo
Shouldn't this patch be merged into #5? Also, a binding doc should be added. Regards, Shawn On Tue, Aug 28, 2012 at 01:48:24PM +0200, Sascha Hauer wrote: From: Philipp Zabel p.za...@pengutronix.de Signed-off-by: Philipp Zabel p.za...@pengutronix.de Signed-off-by: Sascha Hauer

Re: [PATCH 8/9] ARM i.MX53: Add pwms to dtsi

2012-08-31 Thread Shawn Guo
On Tue, Aug 28, 2012 at 01:48:23PM +0200, Sascha Hauer wrote: Signed-off-by: Sascha Hauer s.ha...@pengutronix.de Conflicts: arch/arm/mach-imx/clk-imx51-imx53.c Yeah, I know you have sorted out conflicts :) --- arch/arm/boot/dts/imx53.dtsi| 14 ++

Re: [PATCH 8/9] ARM i.MX53: Add pwms to dtsi

2012-08-31 Thread Shawn Guo
On Fri, Aug 31, 2012 at 03:07:23PM +0200, Sascha Hauer wrote: clk_register_clkdev(clk[ssi1_ipg_gate], NULL, 63fcc000.ssi); clk_register_clkdev(clk[ssi2_ipg_gate], NULL, 50014000.ssi); clk_register_clkdev(clk[ssi3_ipg_gate], NULL, 63fd.ssi); +

Re: [PATCH v4 1/7] mfd: add syscon driver based on regmap

2012-08-31 Thread Shawn Guo
On Wed, Aug 29, 2012 at 06:56:52PM +0800, Dong Aisheng wrote: +config MFD_SYSCON +bool System Controller Register R/W Based on Regmap If the driver only compiles and works for device tree probe, we should have the following? depends on OF +select REGMAP_MMIO +

Re: [PATCH v4 7/7] mfd: anatop-mfd: remove anatop driver

2012-08-31 Thread Shawn Guo
On Wed, Aug 29, 2012 at 06:56:58PM +0800, Dong Aisheng wrote: -config MFD_ANATOP - bool Support for Freescale i.MX on-chip ANATOP controller - depends on SOC_IMX6Q - help - Select this option to enable Freescale i.MX on-chip ANATOP - MFD controller. This controller

Re: [PATCH v4 2/7] ARM: imx6q: add iomuxc gpr support into syscon

2012-08-31 Thread Shawn Guo
On Wed, Aug 29, 2012 at 06:56:53PM +0800, Dong Aisheng wrote: From: Dong Aisheng dong.aish...@linaro.org Include headfile for easy using. Acked-by: Stephen Warren swar...@wwwdotorg.org Signed-off-by: Dong Aisheng dong.aish...@linaro.org --- arch/arm/boot/dts/imx6q.dtsi |5 +

Re: [PATCH v2 0/9] Add device tree support for on-chip SRAM

2012-09-02 Thread Shawn Guo
I do not understand the point of introducing those imx patches, 1 ~ 4 and 7, 8. They are all unnecessary churns to me. IMO, 4 patches are enough. * genalloc: add a global pool list, allow to find pools by phys address * misc: Generic on-chip SRAM allocation driver * ARM i.MX: remove

Re: [PATCH v2 5/9] genalloc: add a global pool list, allow to find pools by phys address

2012-09-02 Thread Shawn Guo
On Fri, Aug 31, 2012 at 11:27:00AM +0200, Philipp Zabel wrote: This patch keeps all created pools in a global list and adds two functions that allow to retrieve the gen_pool pointer from a known physical address and from a device tree node. Signed-off-by: Philipp Zabel p.za...@pengutronix.de

Re: [PATCH v2 6/9] misc: Generic on-chip SRAM allocation driver

2012-09-02 Thread Shawn Guo
On Fri, Aug 31, 2012 at 11:27:01AM +0200, Philipp Zabel wrote: This driver requests and remaps a memory region as configured in the device tree. It serves memory from this region via the genalloc API. Other drivers can retrieve the genalloc pool from a phandle pointing to this drivers'

Re: [PATCH v4 1/7] mfd: add syscon driver based on regmap

2012-09-02 Thread Shawn Guo
On Mon, Sep 03, 2012 at 10:31:03AM +0800, Dong Aisheng wrote: I think of_node_put should be moved out from here and put into syscon_node_to_regmap and syscon_regmap_lookup_by_compatible. I guess no, if you want to move of_node_put into syscon_regmap_lookup_by_phandle, then

[PATCH] kernel/sys.c: call disable_nonboot_cpus in kernel_restart

2012-09-03 Thread Shawn Guo
Like kernel_power_off calls disable_nonboot_cpus, we may want to have kernel_restart call disable_nonboot_cpus as well. Doing so can help the machines that require boot cpu be the last alive cpu during reboot to survive with kernel restart. Signed-off-by: Shawn Guo shawn@linaro.org

Re: [PATCH v3 0/4] Add device tree support for on-chip SRAM

2012-09-03 Thread Shawn Guo
if devm_request_and_ioremap fails. - Squashed i.MX changes together. For the series: Reviewed-by: Shawn Guo shawn@linaro.org Though it will be very simple one, we may still need a binding doc for that sram driver. Regards, Shawn -- To unsubscribe from this list: send the line unsubscribe

Re: [PATCH v5 3/4] regulator: anatop-regulator: convert to use syscon to access anatop register

2012-09-04 Thread Shawn Guo
On Tue, Sep 04, 2012 at 11:20:10AM +0800, Dong Aisheng wrote: + + anatop_np = of_get_parent(np); + if (!anatop_np) + return -ENODEV; + sreg-anatop = syscon_node_to_regmap(anatop_np); + if (IS_ERR(sreg-anatop)) + return PTR_ERR(sreg-anatop); + With

Re: [PATCH v2 1/4] [RFC] clk: new locking scheme for reentrancy

2012-09-04 Thread Shawn Guo
On Wed, Aug 15, 2012 at 04:43:31PM -0700, Mike Turquette wrote: +void __clk_unprepare(struct clk *clk, struct clk *top) +{ if (clk-ops-unprepare) clk-ops-unprepare(clk-hw); - __clk_unprepare(clk-parent); + if (clk != top) +

Re: [PATCH] kernel/sys.c: call disable_nonboot_cpus in kernel_restart

2012-09-04 Thread Shawn Guo
On Tue, Sep 04, 2012 at 03:36:24PM -0700, Andrew Morton wrote: On Mon, 3 Sep 2012 22:24:58 +0800 Shawn Guo shawn@linaro.org wrote: Like kernel_power_off calls disable_nonboot_cpus, we may want to have kernel_restart call disable_nonboot_cpus as well. Doing so can help the machines

Re: RCU warning during UBI file system mount

2012-09-04 Thread Shawn Guo
On Tue, Sep 04, 2012 at 08:43:12AM -0700, Subodh Nijsure wrote: MX28 is ARM device. I removed following from my config and these warnings went away. CONFIG_CPU_IDLE=y CONFIG_CPU_IDLE_GOV_LADDER=y CONFIG_CPU_IDLE_GOV_MENU=y There is no cpuidle implemented on imx28. I do not understand how

Re: [PATCH v3 2/4] misc: Generic on-chip SRAM allocation driver

2012-09-04 Thread Shawn Guo
On Mon, Sep 03, 2012 at 06:02:06PM +0200, Philipp Zabel wrote: +static struct platform_driver sram_driver = { + .driver = { + .name = sram, + .of_match_table = of_match_ptr(sram_dt_ids), + }, + .probe = sram_probe, + .remove =

Re: [PATCH v6 0/4] add syscon driver based on regmap for general registers access

2012-09-04 Thread Shawn Guo
Hi Samuel, The series needs to go via mfd or arm-soc tree as a whole. In case you want to take it through mfd tree, here is my ack. Acked-by: Shawn Guo shawn@linaro.org Otherwise, I can take it via arm-soc tree with your ack. Regards, Shawn On Wed, Sep 05, 2012 at 10:57:12AM +0800, Dong

Re: [PATCH] efikamx: reintroduce Genesi Efika MX Smarttop via device tree

2012-08-08 Thread Shawn Guo
On Tue, Aug 07, 2012 at 04:46:18PM -0500, Matt Sealey wrote: This device tree only supports the final retail board (TO3). It is currently feature equivalent to the MX51 Babbage device tree. The following features have been tested and work as well as can be expected: * Serial port * SD

Re: [PATCH] efikamx: reintroduce Genesi Efika MX Smarttop via device tree

2012-08-09 Thread Shawn Guo
On Thu, Aug 09, 2012 at 09:29:39AM -0500, Matt Sealey wrote: The reason the new kernel depends on the new U-Boot is we're trying to do all pinmux configuration in U-Boot (and we do in-house, and it works). No pinctrl stuff in the kernel or device tree is required at this point. The Old Kernel

Re: [PATCH] efikamx: reintroduce Genesi Efika MX Smarttop via device tree

2012-08-10 Thread Shawn Guo
On Fri, Aug 10, 2012 at 08:36:02AM -0500, Matt Sealey wrote: Requiring it breaks the entire concept of the device tree to describe running hardware. It is not a configuration script. pinctrl should be optional - built in always, but not necessary to turn a board on if it's already configured.

Re: [PATCH] efikamx: reintroduce Genesi Efika MX Smarttop via device tree

2012-08-10 Thread Shawn Guo
On Fri, Aug 10, 2012 at 09:26:36AM -0500, Matt Sealey wrote: If you really think it's necessary then fine, we'll do it. Yes, please do. -- Regards, Shawn -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More

Re: [PATCH 3/6] ARM: imx: build pm-imx5 code only when PM is enabled

2012-08-22 Thread Shawn Guo
of 'imx53_pm_init' arch/arm/mach-imx/include/mach-imx/common.h:155:91: note: previous definition of 'imx53_pm_init' was here Signed-off-by: Arnd Bergmann a...@arndb.de Cc: Shawn Guo shawn@linaro.org Cc: Sascha Hauer s.ha...@pengutronix.de Cc: sta...@vger.kernel.org Acked-by: Shawn Guo shawn

Re: [PATCH 4/6] ARM: imx: fix ksz9021rn_phy_fixup

2012-08-22 Thread Shawn Guo
a...@arndb.de Cc: Artem Bityutskiy dedeki...@gmail.com Cc: Shawn Guo shawn@linaro.org Cc: Sascha Hauer s.ha...@pengutronix.de Acked-by: Shawn Guo shawn@linaro.org -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More

Re: [PATCH 5/6] ARM: imx: select CPU_FREQ_TABLE when needed

2012-08-22 Thread Shawn Guo
? Otherwise, Acked-by: Shawn Guo shawn@linaro.org Building IMX without CPU_FREQ_TABLE results in: arch/arm/plat-mxc/built-in.o: In function `mxc_cpufreq_exit': arch/arm/plat-mxc/cpufreq.c:173: undefined reference to `cpufreq_frequency_table_put_attr' arch/arm/plat-mxc/built-in.o: In function

Re: [PATCH 6/6] ARM: imx: select ARM_CPU_SUSPEND when necessary

2012-08-22 Thread Shawn Guo
Gating in hardware, ARM_CPU_SUSPEND support will never be used on imx5. Maybe we should make head-v7.S only compile for imx6? Regards, Shawn Signed-off-by: Arnd Bergmann a...@arndb.de Cc: Eric Miao eric.m...@linaro.org Cc: Shawn Guo shawn@linaro.org Cc: sta...@vger.kernel.org --- arch

Re: [PATCH 6/6] ARM: imx: select ARM_CPU_SUSPEND when necessary

2012-08-23 Thread Shawn Guo
contains a call to the generic cpu_suspend function, which is only available when selected by the i.MX6 code. As pointed out by Shawn Guo, i.MX does not actually use any s/i.MX/i.MX5 Acked-by: Shawn Guo shawn@linaro.org functions defined in head-v7.S. It is also needed only for the i.MX6 power

Re: [PATCH 1/7] mfd: add imx syscon driver based on regmap

2012-08-24 Thread Shawn Guo
On Wed, Aug 22, 2012 at 03:18:42PM +0800, Dong Aisheng wrote: From: Dong Aisheng dong.aish...@linaro.org Add regmap based imx syscon driver. This is usually used for access misc bits in registers which does not belong to a specific module, for example, IOMUXC GPR and ANATOP. With this

Re: [PATCH 0/4] [RFC] ARM: multiplatform: rename all mach headers

2012-08-24 Thread Shawn Guo
On Wed, Aug 22, 2012 at 12:53:06PM +, Arnd Bergmann wrote: I've created this series some time ago, and updated it now to v3.6-rc1. The idea is to get us a big step closer to the single zImage kernel across multiple ARM platforms by untangling the duplicate header file names. There are

Re: [PATCH 0/5] Add device tree support for on-chip SRAM

2012-08-24 Thread Shawn Guo
On Fri, Aug 24, 2012 at 11:17:41AM +0200, Philipp Zabel wrote: These patches add support to configure the on-chip SRAM via device-tree node and to obtain the resulting genalloc pool from a phandle pointing at the node. This allows drivers to allocate SRAM with the genalloc API without

Re: Shutdown problem in SMP system happened on Tegra20

2012-08-25 Thread Shawn Guo
On Fri, Aug 24, 2012 at 07:21:33PM +0100, Russell King - ARM Linux wrote: Why not just ensure that CONFIG_PM_SLEEP_SMP is enabled if your platform requires that the lowest CPU number be the CPU dealing with reboot? I have CONFIG_PM_SLEEP_SMP enabled for imx6q, but still see the imx6q restart

Re: Shutdown problem in SMP system happened on Tegra20

2012-08-25 Thread Shawn Guo
On Sat, Aug 25, 2012 at 08:52:21AM +0100, Russell King - ARM Linux wrote: On Sat, Aug 25, 2012 at 03:36:27PM +0800, Shawn Guo wrote: On Fri, Aug 24, 2012 at 07:21:33PM +0100, Russell King - ARM Linux wrote: Why not just ensure that CONFIG_PM_SLEEP_SMP is enabled if your platform requires

Re: [PATCH] ARM: mxs: remove duplicated include from mach-mxs.c

2012-08-25 Thread Shawn Guo
On 26 August 2012 09:49, Wei Yongjun weiyj...@gmail.com wrote: From: Wei Yongjun yongjun_...@trendmicro.com.cn From: Wei Yongjun yongjun_...@trendmicro.com.cn Remove duplicated include. Signed-off-by: Wei Yongjun yongjun_...@trendmicro.com.cn Thanks for the patch. But the duplicated

Re: [PATCH v2 1/7] mfd: add syscon driver based on regmap

2012-08-27 Thread Shawn Guo
On Mon, Aug 27, 2012 at 03:24:39PM +0800, Dong Aisheng wrote: From: Dong Aisheng dong.aish...@linaro.org Add regmap based syscon driver. This is usually used for access misc bits in registers which does not belong to a specific module, for example, IMX IOMUXC GPR and ANATOP. With this

Re: [PATCH v2 4/7] regulator: anatop-regulator: convert to use syscon to access anatop register

2012-08-27 Thread Shawn Guo
On Mon, Aug 27, 2012 at 03:24:42PM +0800, Dong Aisheng wrote: From: Dong Aisheng dong.aish...@linaro.org Using syscon to access anatop register. Acked-by: Mark Brown broo...@opensource.wolfsonmicro.com Signed-off-by: Dong Aisheng dong.aish...@linaro.org --- ChangeLog v1-v2: * update to

Re: [PATCH v2 1/2] imx6q: pwm: Add device tree support

2012-08-27 Thread Shawn Guo
On Mon, Aug 27, 2012 at 05:34:31PM +0200, HACHIMI Samir wrote: From: Samir Hachimi shach...@adeneo-embedded.com Add clock look-up for pwm. Add the pinmux support for pwm. Several pin can be set to PwmO for the same Pwm. Signed-off-by: Samir Hachimi shach...@adeneo-embedded.com ---

Re: [PATCH v2 2/2] Enable Stop_enable mode during configuration of pwm.

2012-08-27 Thread Shawn Guo
On Mon, Aug 27, 2012 at 05:34:32PM +0200, HACHIMI Samir wrote: From: Samir Hachimi shach...@adeneo-embedded.com Enable Stop_enable mode during configuration of pwm. Check architecture by looking in driver_data instead of cpu_is_xxx. It does 2 things, and should be split into 2 patches.

Re: [PATCH 0/4] Remove Genesi Efika MX support

2012-08-05 Thread Shawn Guo
On Wed, Aug 01, 2012 at 12:49:27PM -0500, Matt Sealey wrote: In an effort to start afresh we are removing the legacy Efika MX support from the tree in this patch series so as to break ties with any non-device-tree code and minimize the possibility that there are two ways to build for our

Re: [PATCH 4/9] ASoC: imx: Don't use {en,dis}able_fiq() calls

2012-08-06 Thread Shawn Guo
On Mon, Aug 06, 2012 at 03:39:50PM -0500, Matt Sealey wrote: * make sure there's no weird FIQ stuff floating around that has so far relied on SND_SOC_IMX_PCM_FIQ doing select FIQ before I make it not Acked on changing SND_IMX_SOC to SND_SOC_IMX_PCM_FIQ in arch/arm/plat-mxc/Makefile. compile

Re: [PATCH 3/3] IIO: arm: Add LRADC to i.MX28 dts

2012-08-20 Thread Shawn Guo
On Mon, Aug 20, 2012 at 08:29:23AM +0100, Jonathan Cameron wrote: Mostly these prefixes are an excuse for grumpy maintainers to moan at people :) Yeah, I have to become grumpy when my comment gets ignored and I have to make it right (for several times) to avoid my upstream moaning at me. Is it

Re: [PATCH] alarmtimer: add .remove_dev hook to put device

2012-07-10 Thread Shawn Guo
Hi John, Is this patch a valid fix or just a noise? Regards, Shawn On Thu, Jul 05, 2012 at 12:13:07AM +0800, Shawn Guo wrote: The following is a test sequence that installs a rtc module, remove it and installs it again. $ insmod rtc-snvs.ko snvs_rtc 20cc034.snvs-rtc-lp: rtc core

Re: [RFC internal PATCH] mfd/mc13xxx: drop modifying driver's id_table in probe

2012-07-11 Thread Shawn Guo
. Signed-off-by: Uwe Kleine-König u.kleine-koe...@pengutronix.de The code was added by me, and it turns out the change is completely unnecessary. So, Acked-by: Shawn Guo shawn@linaro.org Regards, Shawn -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body

Re: [PATCH 6/6 v8] cpufreq, highbank: add support for highbank cpufreq

2012-12-06 Thread Shawn Guo
mark.langsd...@calxeda.com Cc: shawn@linaro.org Reviewed-by: Shawn Guo shawn@linaro.org -- 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://vger.kernel.org/majordomo-info.html Please read

Re: [PATCH] clk: mxs: Remove unneeded NULL pointer check

2012-12-11 Thread Shawn Guo
On Tue, Dec 11, 2012 at 08:33:34AM -0200, Fabio Estevam wrote: Shawn/Mike, On Wed, Nov 21, 2012 at 7:33 PM, Fabio Estevam feste...@gmail.com wrote: From: Fabio Estevam fabio.este...@freescale.com mxs platform has been converted to device tree. There is no need to check if np is NULL

Re: [PATCH v2 1/3] CLK: get rid of HAVE_CLK_PREPARE

2012-11-22 Thread Shawn Guo
of this change we move declarations of clk_prepare() and clk_unprepare() under HAVE_CLK and provide stubs if this option is not enabled. Reviewed-by: Viresh Kumar viresh.ku...@linaro.org Signed-off-by: Dmitry Torokhov dmitry.torok...@gmail.com Acked-by: Shawn Guo shawn@linaro.org

Re: [PATCH 1/3] CLK: uninline clk_prepare() and clk_unprepare()

2012-11-23 Thread Shawn Guo
On Fri, Nov 23, 2012 at 12:08:58AM -0800, Dmitry Torokhov wrote: On Fri, Nov 23, 2012 at 12:57:54PM +0530, Viresh Kumar wrote: On 23 November 2012 12:49, Dmitry Torokhov dmitry.torok...@gmail.com wrote: Ahh, I see. Then I think my first patch was correct albeit it had bad changelog

[PATCH] clocksource: use clockevents_config_and_register() where possible

2012-11-26 Thread Shawn Guo
positive diff stat. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Andres Salomon dilin...@queued.net Cc: Maxime Ripard maxime.rip...@free-electrons.com Cc: Nicolas Ferre nicolas.fe...@atmel.com --- drivers/clocksource/cs5535-clockevt.c | 11 ++- drivers/clocksource/sunxi_timer.c

Re: [PATCH 6/6 v6] cpufreq, highbank: add support for highbank cpufreq

2012-11-27 Thread Shawn Guo
On Tue, Nov 27, 2012 at 02:04:32PM -0600, Mark Langsdorf wrote: Highbank processors depend on the external ECME to perform voltage management based on a requested frequency. Communication between the A9 cores and the ECME happens over the pl320 IPC channel. ... +static int

Re: [PATCH 6/6 v6] cpufreq, highbank: add support for highbank cpufreq

2012-11-28 Thread Shawn Guo
On Wed, Nov 28, 2012 at 07:16:12AM -0600, Mark Langsdorf wrote: Standard practice is to have cpufreq_set_target() handle voltage transitions and leave clk_set_rate() handle the frequency changes. The standard practice is to have cpufreq_set_target() handle both voltage and frequency

Re: [PATCH 6/6 v6] cpufreq, highbank: add support for highbank cpufreq

2012-11-28 Thread Shawn Guo
On Wed, Nov 28, 2012 at 10:58:02PM +0800, Shawn Guo wrote: On Wed, Nov 28, 2012 at 07:16:12AM -0600, Mark Langsdorf wrote: I'd have to move most of the logic of hb_set_target() into clk_highbank.c:clk_pll_set_rate() and then add extra logic for when cpufreq is not enabled/loaded. You

Re: [PATCH 6/6 v6] cpufreq, highbank: add support for highbank cpufreq

2012-11-28 Thread Shawn Guo
The notifiers in the clk framework might be a better place for this than just simply hacking the logic into the .set_rate callback. Ah, right. How did I forget about that nice piece? I haven't looked at the definition of hb_voltage_change but does the call graph make any clk api calls? Are

Re: [PATCH] ehci-mxc: remove Efika MX-specific CHRGVBUS hack

2012-12-25 Thread Shawn Guo
-by: Matt Sealey m...@genesi-usa.com Tested-by: Steev Klimazewski st...@genesi-usa.com CC: Sascha Hauer ker...@pengutronix.de CC: Alan Stern st...@rowland.harvard.edu Acked-by: Shawn Guo shawn@linaro.org -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body

Re: [PATCH 1/2] mxs timer: use ahbx bus clock to drive the timers on timrotv2

2012-12-25 Thread Shawn Guo
On Fri, Dec 21, 2012 at 03:06:15PM +0100, Torben Hohn wrote: timer resolution of ~32us is pretty low. v2 has 32bits resolution, so we have quite some headroom, and can use the 24MHz clock. v1 has only 16bits, so we only increase v2. So we just exchange the timrot clock in imx28. On imx23

Re: [REGRESSION][3.8.-rc1][ INFO: possible circular locking dependency detected ]

2012-12-25 Thread Shawn Guo
It seems that I'm running into the same locking issue. My setup is: - i.MX28 (ARM) - v3.8-rc1 - mxs_defconfig Shawn [ 602.229899] == [ 602.229905] [ INFO: possible circular locking dependency detected ] [ 602.229926]

Re: [PATCH V7 2/7] ARM: dt: change .dtb build rules to build in dts directory

2012-12-26 Thread Shawn Guo
On Tue, Nov 27, 2012 at 04:29:11PM -0700, Stephen Warren wrote: From: Grant Likely grant.lik...@secretlab.ca The current rules have the .dtb files build in a different directory from the .dts files. The only reason for this is that it was what PowerPC has done historically. This patch

Re: [REGRESSION][3.8.-rc1][ INFO: possible circular locking dependency detected ]

2012-12-27 Thread Shawn Guo
On Wed, Dec 26, 2012 at 10:34:39AM +0800, Shawn Guo wrote: It seems that I'm running into the same locking issue. My setup is: - i.MX28 (ARM) - v3.8-rc1 - mxs_defconfig - The warning is seen when LCD is blanking The warning disappears after reverting patch daee779 (console: implement

Re: [REGRESSION][3.8.-rc1][ INFO: possible circular locking dependency detected ]

2012-12-28 Thread Shawn Guo
On Thu, Dec 27, 2012 at 08:03:24AM -0500, Peter Hurley wrote: On Thu, 2012-12-27 at 16:36 +0800, Shawn Guo wrote: On Wed, Dec 26, 2012 at 10:34:39AM +0800, Shawn Guo wrote: It seems that I'm running into the same locking issue. My setup is: - i.MX28 (ARM) - v3.8-rc1

Re: [PATCH] fb: Rework locking to fix lock ordering on takeover

2012-12-28 Thread Shawn Guo
On Thu, Dec 27, 2012 at 05:53:01AM +0100, Borislav Petkov wrote: On Wed, Dec 26, 2012 at 01:09:51PM -0500, Sasha Levin wrote: This patch can fix the following warning we saw? http://lkml.org/lkml/2012/12/22/53 I will give it a try. Yup, that's the same error I've reported couple

Re: [PATCH] clk: mxs: Use a better name for the USB PHY clock

2012-11-15 Thread Shawn Guo
On Sat, Sep 22, 2012 at 01:54:55PM -0300, Fabio Estevam wrote: From: Fabio Estevam fabio.este...@freescale.com Use a better name for the USB PHY clock. Signed-off-by: Fabio Estevam fabio.este...@freescale.com Mike, I was planning send a pull-request to you with all mxs clock patches

[PATCH v2 0/3] Use helper clockevents_config_and_register()

2013-01-12 Thread Shawn Guo
to export clockevents_config_and_register for fixing the error [1] reported by Fengguang. - Add Acked-by and Tested-by tags given by people (Thanks!) [1] http://thread.gmane.org/gmane.linux.ports.arm.kernel/209152 Shawn Guo (3): clockevents: export clockevents_config_and_register for module use

[PATCH v2 3/3] clocksource: use clockevents_config_and_register() where possible

2013-01-12 Thread Shawn Guo
positive diff stat. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Andres Salomon dilin...@queued.net Cc: Nicolas Ferre nicolas.fe...@atmel.com Acked-by: Maxime Ripard maxime.rip...@free-electrons.com --- drivers/clocksource/cs5535-clockevt.c | 11 ++- drivers/clocksource/sunxi_timer.c

[PATCH v2 1/3] clockevents: export clockevents_config_and_register for module use

2013-01-12 Thread Shawn Guo
clockevents_config_and_register is a handy helper for clockevent drivers, some of which might support module build, so export the symbol. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Thomas Gleixner t...@linutronix.de --- kernel/time/clockevents.c |1 + 1 file changed, 1 insertion

[PATCH v2 2/3] ARM: use clockevents_config_and_register() where possible

2013-01-12 Thread Shawn Guo
positive diff stat. Signed-off-by: Shawn Guo shawn@linaro.org Cc: Nicolas Ferre nicolas.fe...@atmel.com Reviewed-by: Anton Vorontsov cbouatmai...@gmail.com Acked-by: Kukjin Kim kgene@samsung.com Cc: Russell King li...@arm.linux.org.uk Cc: Sascha Hauer s.ha...@pengutronix.de Cc: Jason Cooper ja

Re: [PATCH] pinctrl: pinctrl-mxs: Fix variables' definition type

2013-01-08 Thread Shawn Guo
and unsigned integer expressions [-Wsign-compare] Signed-off-by: Fabio Estevam fabio.este...@freescale.com Acked-by: Shawn Guo shawn@linaro.org -- 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

Re: [PATCH] clk: mxs: Index is always positive

2013-01-08 Thread Shawn Guo
: comparison between signed and unsigned integer expressions [-Wsign-compare] drivers/clk/mxs/clk-imx28.c:244:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] Signed-off-by: Fabio Estevam fabio.este...@freescale.com Acked-by: Shawn Guo shawn

Re: [PATCH] cpufreq: Simplify __cpufreq_remove_dev()

2013-01-09 Thread Shawn Guo
On Wed, Jan 09, 2013 at 04:50:44PM +0530, Viresh Kumar wrote: @Shawn: I believe your driver don't require that ugly code anymore (Though i know there is a situation for that to happen, if we have two cpus, you remove second one and then add it back. With this cpufreq_add_dev() would call init()

Re: [PATCH] cpufreq: Simplify __cpufreq_remove_dev()

2013-01-10 Thread Shawn Guo
On 10 January 2013 16:05, Viresh Kumar viresh.ku...@linaro.org wrote: Another thing, can i have a tested-by from you for both my patches ? remove and add dev? For both: Tested-by: Shawn Guo shawn@linaro.org -- To unsubscribe from this list: send the line unsubscribe linux-kernel

Re: [PATCH 02/15] cpufreq: cpufreq-cpu0: initialize .owner field with THIS_MODULE

2013-08-04 Thread Shawn Guo
is initialized with a valid pointer. This field wasn't initialized for this driver, lets initialize it with THIS_MODULE. Cc: Shawn Guo shawn@linaro.org Signed-off-by: Viresh Kumar viresh.ku...@linaro.org For this and imx6q-cpufreq one, Acked-by: Shawn Guo shawn@linaro.org

Re: [PATCH 3/3] dma: Add Freescale eDMA engine driver support

2013-08-04 Thread Shawn Guo
On Fri, Aug 02, 2013 at 03:55:48PM +0800, Jingchang Lu wrote: Add Freescale enhanced direct memory(eDMA) controller support. The eDMA controller deploys DMAMUXs routing DMA request sources(slot) to eDMA channels. This module can be found on Vybrid and LS-1 SoCs. Signed-off-by: Alison Wang

Re: [PATCH v5] regulator: pfuze100: add pfuze100 regulator driver

2013-07-25 Thread Shawn Guo
On Thu, Jul 25, 2013 at 11:33:18AM +0800, Robin Gong wrote: @@ -0,0 +1,113 @@ +PFUZE100 family of regulators + +Required properties: +- compatible: fsl,pfuze100 +- reg: I2C slave address +- regulators: This is the list of child nodes that specify the regulator + initialization data for

Re: linux-next: Tree for Jul 25 (ahci_imx.c)

2013-07-25 Thread Shawn Guo
On Fri, Jul 26, 2013 at 03:46:42AM +, Zhu Richard-R65037 wrote: -Original Message- From: Randy Dunlap [mailto:rdun...@infradead.org] Sent: Friday, July 26, 2013 2:32 AM To: Stephen Rothwell Cc: linux-n...@vger.kernel.org; linux-kernel@vger.kernel.org; linux-...@vger.kernel.org;

Re: linux-next: Tree for Jul 25 (ahci_imx.c)

2013-07-25 Thread Shawn Guo
On Fri, Jul 26, 2013 at 04:58:14AM +, Zhu Richard-R65037 wrote: Hi Shawn: yes, it is. BTW, Should I re-send the AHCI_IMX patch-set or send out one stand-alone patch to fix this issue? I think an incremental fixing patch is good. Shawn -- To unsubscribe from this list: send the line

Re: [PATCH] imx2_wdt: expose module alias for loading from device-tree

2013-07-29 Thread Shawn Guo
On Mon, Jul 29, 2013 at 09:38:18AM +0200, Niels de Vos wrote: Enable auto loading by udev when imx2_wdt is compiled as a module. Signed-off-by: Niels de Vos nde...@redhat.com Cc: Shawn Guo shawn@linaro.org Acked-by: Shawn Guo shawn@linaro.org --- drivers/watchdog/imx2_wdt.c | 1

Re: [PATCH 0/3] of: add update device node status via cmdline feature

2013-08-23 Thread Shawn Guo
The device tree mailing list is changed to devicet...@vger.kernel.org. On Fri, Aug 23, 2013 at 03:09:08PM +0800, Dong Aisheng wrote: I tried the uboot way with fdt command to change the node status, it can work. However, it seems using fdt command is much complicated than the way i did with

Re: [PATCH 0/3] of: add update device node status via cmdline feature

2013-08-23 Thread Shawn Guo
On Fri, Aug 23, 2013 at 03:51:07PM +0800, Shawn Guo wrote: The device tree mailing list is changed to devicet...@vger.kernel.org. On Fri, Aug 23, 2013 at 03:09:08PM +0800, Dong Aisheng wrote: I tried the uboot way with fdt command to change the node status, it can work. However

Re: [PATCH 0/3] of: add update device node status via cmdline feature

2013-08-23 Thread Shawn Guo
On Fri, Aug 23, 2013 at 04:49:55PM +0800, Dong Aisheng wrote: On Fri, Aug 23, 2013 at 03:51:07PM +0800, Shawn Guo wrote: The device tree mailing list is changed to devicet...@vger.kernel.org. On Fri, Aug 23, 2013 at 03:09:08PM +0800, Dong Aisheng wrote: I tried the uboot way with fdt

Re: [PATCH 1/3] ARM: imx6q: refactor some ldb related clocks

2013-08-20 Thread Shawn Guo
On Tue, Aug 20, 2013 at 02:18:27PM -0700, Mike Turquette wrote: Quoting Fabio Estevam (2013-08-20 08:40:52) On Tue, Aug 20, 2013 at 5:38 AM, Liu Ying ying@freescale.com wrote: diff --git a/Documentation/devicetree/bindings/clock/imx6q-clock.txt

Re: [PATCH 0/3] refactor some ldb related clocks

2013-08-20 Thread Shawn Guo
Hi Ying, On Tue, Aug 20, 2013 at 06:08:48PM +0800, Liu Ying wrote: While I admit to having introduced the combination of 1/3.5 fixed divider and configurable 1/1,1/2 divder clocks to describe this fractional divider for the reasons you state, I think the correct solution would be to

Re: [PATCH] dma: ipu: remove unnecessary platform_set_drvdata()

2013-08-21 Thread Shawn Guo
On Wed, Aug 21, 2013 at 06:52:54PM +0900, Jingoo Han wrote: The driver core clears the driver data to NULL after device_release or on probe failure. Thus, it is not needed to manually clear the device driver data to NULL. Signed-off-by: Jingoo Han jg1@samsung.com Acked-by: Shawn Guo

Re: [PATCH] w1: mxc_w1: remove unnecessary platform_set_drvdata()

2013-08-22 Thread Shawn Guo
On Thu, Aug 22, 2013 at 11:20:58AM +0900, Jingoo Han wrote: The driver core clears the driver data to NULL after device_release or on probe failure. Thus, it is not needed to manually clear the device driver data to NULL. Signed-off-by: Jingoo Han jg1@samsung.com Acked-by: Shawn Guo

Re: [PATCH linux-next] ARM: imx: fix imx_init_l2cache storage class

2013-07-10 Thread Shawn Guo
[-Wunused-function] Signed-off-by: Vincent Stehlé vincent.ste...@freescale.com Cc: Shawn Guo shawn@linaro.org Cc: Sascha Hauer ker...@pengutronix.de Cc: Russell King li...@arm.linux.org.uk Cc: triv...@kernel.org --- Hi, Linux next-20130710 breaks compilation of ARM multi_v7_defconfig

<    1   2   3   4   5   6   7   8   9   10   >