Re: [PATCH 1/2] ARM: mach-imx: imx53.dtsi: pinctl update

2012-10-29 Thread Shawn Guo
On Mon, Oct 29, 2012 at 10:13:59PM +0100, Sascha Hauer wrote: On Thu, Oct 25, 2012 at 01:26:39PM +0200, Roland Stigge wrote: This patch supplements pinctl support on i.MX53. Signed-off-by: Roland Stigge sti...@antcom.de Acked-by: Sascha Hauer s.ha...@pengutronix.de Shawn, probably

Re: [PATCH V2 1/3] mmc: esdhc: enable polling to detect card by itself

2012-10-31 Thread Shawn Guo
On Tue, Oct 30, 2012 at 05:30:01PM +0800, yongd wrote: In the current code logic, sdhci_add_host() will enable the polling method (set MMC_CAP_NEEDS_POLL) for a removable card (MMC_CAP_ NONREMOVABLE is not set) whose host's internal card detection method is disabled for some reason

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

2012-09-25 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 Queued for 3.8, thanks. -- To unsubscribe from this list: send the line

Re: [PATCH] pwm: dt: Fix description of second PWM cell

2012-10-06 Thread Shawn Guo
Reding thierry.red...@avionic-design.de Cc: Shawn Guo shawn@linaro.org Cc: Sascha Hauer s.ha...@pengutronix.de Cc: Philipp Zabel p.za...@pengutronix.de Cc: Benoît Thébaudeau benoit.thebaud...@advansee.com Cc: Stephen Warren swar...@wwwdotorg.org --- Documentation/devicetree/bindings/pwm/imx

Re: [PATCH 09/17] ARM: imx: select ARM_CPU_SUSPEND if necessary

2012-10-06 Thread Shawn Guo
' Signed-off-by: Arnd Bergmann a...@arndb.de Cc: Sascha Hauer ker...@pengutronix.de Cc: Shawn Guo shawn@linaro.org --- arch/arm/mach-imx/Kconfig |1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/mach-imx/Kconfig b/arch/arm/mach-imx/Kconfig index afd542a..8e96573 100644

Re: [PATCH] pwm: Get rid of HAVE_PWM

2012-10-06 Thread Shawn Guo
functionality. Signed-off-by: Thierry Reding thierry.red...@avionic-design.de Cc: Russell King li...@arm.linux.org.uk Cc: Shawn Guo shawn@linaro.org Cc: Eric Miao eric.y.m...@gmail.com Cc: Haojian Zhuang haojian.zhu...@gmail.com Cc: Ralf Baechle r...@linux-mips.org Cc: Dmitry Torokhov

Re: [PATCH 1/3] mmc: esdhc: enable polling to detect card by itself

2012-10-07 Thread Shawn Guo
On Fri, Sep 28, 2012 at 06:28:31PM +0800, yongd wrote: In the current code logic, sdhci_add_host() will enable the polling method (set MMC_CAP_NEEDS_POLL) for a removable card (MMC_CAP_ NONREMOVABLE is not set) whose host's internal card detection method is disabled for some reason

Re: [PATCH V2 1/3] mmc: esdhc: enable polling to detect card by itself

2012-11-04 Thread Shawn Guo
On Fri, Nov 02, 2012 at 08:37:41PM +0800, yongd wrote: I got it. So how do you think of such below partition/reorganization? It looks fine to me for imx. Patch-1: For sdhci-esdhc-imx.c, only add MMC_CAP_NEEDS_POLL for ESDHC_CD_NONE. With that improper logic of sdhci_add_host(), this is

Re: Wakeup latency measured with SCHED_TRACER depends on HZ

2012-11-04 Thread Shawn Guo
On Fri, Nov 02, 2012 at 03:29:50PM +0100, Stanislav Meduna wrote: On 31.10.2012 22:41, Stanislav Meduna wrote: on an embedded platform using a Freescale i.MX28 ARM processor I am experiencing a strange phenomenon - the latencies reported are dependent of HZ OK, the problem is that the

Re: [PATCH V2 1/3] mmc: esdhc: enable polling to detect card by itself

2012-11-05 Thread Shawn Guo
On Mon, Nov 05, 2012 at 11:34:49AM +0800, yongd wrote: From the code logic, without SDHCI_QUIRK_BROKEN_CARD_DETECTION for ESDHC_CD_GPIO, when your card (using GPIO detection) is removed, we can know the card's absence through the fake CARD_PRESENT flag in esdhc_readl_le(). As a result, we

Re: [PATCH V2 1/3] mmc: esdhc: enable polling to detect card by itself

2012-11-06 Thread Shawn Guo
On Tue, Nov 06, 2012 at 04:49:42PM +0800, yongd wrote: From your info, we can see that on your platform, those pins (including power, clk, DATA) necessary for MMC_SEND_STATUS transaction still keep connected for some time just after the GPIO's level changes due to card removable. And if we

Re: [PATCH 4/7] gpio/mxs: convert to use linear irqdomain

2012-10-19 Thread Shawn Guo
On 19 October 2012 18:22, Linus Walleij linus.wall...@linaro.org wrote: No, which defconfig shall I use for this driver? mxs_defconfig Shawn -- 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

Re: [PATCH] RFC: leds-pwm: don't disable pwm when setting brightness to 0

2012-10-25 Thread Shawn Guo
On Wed, Oct 24, 2012 at 03:52:46PM +0200, Uwe Kleine-König wrote: This fixes disabling the LED on i.MX28. The PWM hardware delays using the newly set pwm-config until the beginning of a new period. It's very likely that pwm_disable is called before the current period ends. In case the LED was

Re: [PATCH 2/2] pinctrl: mxs: Make PINCTRL_MXS select PINMUX PINCONF

2012-11-11 Thread Shawn Guo
have them after 'bool' below, otherwise, Acked-by: Shawn Guo shawn@linaro.org bool config PINCTRL_IMX23 bool - select PINMUX - select PINCONF select PINCTRL_MXS config PINCTRL_IMX28 bool - select PINMUX - select PINCONF select

Re: [PATCH 1/1] ARM: dts: imx6q-sabresd: add volume up/down gpio keys

2012-11-12 Thread Shawn Guo
On Mon, Nov 12, 2012 at 04:26:18PM +0800, Liu Ying wrote: Add volume up/down gpio keys support in imx6q-sabresd.dts. Signed-off-by: Liu Ying ying@freescale.com Applied, thanks. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to

Re: [PATCH 1/3] mmc: esdhc: enable polling to detect card by itself

2012-10-17 Thread Shawn Guo
On Tue, Oct 16, 2012 at 09:01:40PM -0700, Yong Ding wrote: Shawn, Thanks for your comment. And sorry for my so late due to illness:-) SDHCI_QUIRK_BROKEN_CARD_DETECTION is used for notifying we don't use the host internal card detection method so that we don't need enable/disable those

Re: [PATCH 7/7] gpio/tegra: convert to use linear irqdomain

2012-10-18 Thread Shawn Guo
On Tue, Oct 16, 2012 at 09:23:33PM +0200, Linus Walleij wrote: The MXS driver tries to do the work of irq_domain_add_linear() s/MXS/tegra by reserving a bunch of descriptors somewhere and keeping track of the base offset, then calling irq_domain_add_legacy(). Let's stop doing that and simply

Re: [PATCH 4/7] gpio/mxs: convert to use linear irqdomain

2012-10-18 Thread Shawn Guo
the linear IRQ domain. Cc: Rob Herring rob.herr...@calxeda.com Cc: Grant Likely grant.lik...@secretlab.ca Cc: Shawn Guo shawn@linaro.org Signed-off-by: Linus Walleij linus.wall...@linaro.org --- drivers/gpio/gpio-mxs.c | 19 ++- 1 file changed, 6 insertions(+), 13 deletions

Re: [PATCH 1/3] mmc: esdhc: enable polling to detect card by itself

2012-10-18 Thread Shawn Guo
Can you fix your mailer to use bottom posting rather than top posting, and have texts wrap around column 70? Otherwise, you message stands a good chance to be ignored by people. On Wed, Oct 17, 2012 at 11:27:17PM -0700, Yong Ding wrote: Shawn, Thanks. Oh, sorry I really have missed the fact u

Re: [PATCH] [ARM] Unconditional call to smp_cross_call on UP crashes (take #2)

2012-07-13 Thread Shawn Guo
Hi Pantelis, In case you do not know the process of ARM core patches. You need to put the patch into Russell's patch system [1] after review gets done. Then the patch will show up on Russell's git tree. Regards, Shawn [1] http://www.arm.linux.org.uk/developer/patches/ On Fri, May 25, 2012 at

Re: linux-next: manual merge of the arm-soc tree with the i2c-embedded tree

2012-07-17 Thread Shawn Guo
On Tue, Jul 17, 2012 at 04:20:01PM +0100, Mark Brown wrote: Looking at what's specified in the platform data in the current kernel it seems like there's a mixture of things in there that are board and silicon specific. We've got parameters like the the speed mode and timeout which are

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

2012-07-18 Thread Shawn Guo
On 19 July 2012 07:23, John Stultz johns...@us.ibm.com wrote: Actually, this change opens up a bunch of other races, as any caller of alarmtimer_get_rtcdev() could have the rtcdevice removed under it. We'll need to have proper reference counting w/ get/put calls, probably also adding a

Re: [Patch -v3 1/4] Migrate shutdown/reboot to boot cpu.

2013-04-15 Thread Shawn Guo
removed pertaining to the reboot_cpu command line parameter. Signed-off-by: Robin Holt h...@sgi.com To: Shawn Guo shawn@linaro.org It works on my hardware - i.MX6 Quad. Tested-by: Shawn Guo shawn@linaro.org -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body

Re: [PATCH] cpufreq: cpu0: Put cpu parent node after using it

2013-04-16 Thread Shawn Guo
On Mon, Apr 15, 2013 at 12:39:37PM +0530, Viresh Kumar wrote: Parent node must be put after using it to balance its usage count. This was missing in cpufreq-cpu0 driver. Fix it. Signed-off-by: Viresh Kumar viresh.ku...@linaro.org Acked-by: Shawn Guo shawn@linaro.org --- drivers

Re: [PATCH] video: mxsfb: Fix colors display on lower color depth

2013-04-18 Thread Shawn Guo
Copy Sascha. Shawn On Thu, Apr 18, 2013 at 11:23:52AM +0200, Maxime Ripard wrote: The current code always registers as a 32 bits display, and uses the hardware to drop the MSB of each color to abjust to the interface width used by the panel. This results on 18 bits (and probably 16 bits

Re: [PATCH 0/2] cpufreq/regulator: Handle regulators that defer probe with device tree bindings

2013-04-05 Thread Shawn Guo
On Thu, Apr 04, 2013 at 11:21:46PM -0500, Nishanth Menon wrote: Nishanth Menon (2): regulator: core: return err value for regulator_get if there is no DT binding cpufreq: cpufreq-cpu0: defer probe when regulator is not ready For both, Acked-by: Shawn Guo shawn@linaro.org

Re: [PATCH] mmc: mxs-mmc: Add support for non-removable cards

2013-04-05 Thread Shawn Guo
On Fri, Apr 05, 2013 at 01:18:38AM -0300, Alexandre Pereira da Silva wrote: Some boards and card slots doesn't have card detect feature available. In that case allow to mark the cards as non-removable, via devicetree. Signed-off-by: Alexandre Pereira da Silva aletes@gmail.com Marc

Re: [PATCH] ARM: dts: imx23-olinuxino: Set the sdcard as non-removable

2013-04-05 Thread Shawn Guo
On Fri, Apr 05, 2013 at 01:16:53AM -0300, Alexandre Pereira da Silva wrote: The imx23-olinuxino sdcard doesn't support card detect. So it sounds more like a broken-cd rather than non-removable? Shawn Signed-off-by: Alexandre Pereira da Silva aletes@gmail.com ---

Re: [PATCH] ARM: dts: imx23-olinuxino: mark sdcard cd as broken

2013-04-07 Thread Shawn Guo
On Sat, Apr 06, 2013 at 10:42:10AM -0300, Alexandre Pereira da Silva wrote: The imx23-olinuxino sdcard doesn't have card detect. Signed-off-by: Alexandre Pereira da Silva aletes@gmail.com Applied, thanks. --- arch/arm/boot/dts/imx23-olinuxino.dts |1 + 1 file changed, 1

Re: [PATCH] ARM: dts: cfa10036: Add touchscreen support to the CFA-10049

2013-04-07 Thread Shawn Guo
On Fri, Apr 05, 2013 at 02:33:02PM +0200, Alexandre Belloni wrote: Signed-off-by: Alexandre Belloni alexandre.bell...@free-electrons.com Applied, thanks. --- arch/arm/boot/dts/imx28-cfa10049.dts |5 + 1 file changed, 5 insertions(+) diff --git

Re: [GIT PULL] dt: run C pre-processor on *.dts, create some standard headers

2013-04-07 Thread Shawn Guo
On Fri, Apr 05, 2013 at 12:46:50PM -0600, Stephen Warren wrote: Rob, it might be worth keeping this in a separate branch in linux-next so you can pull it out if it causes any issues. I've been using these patches for quite a while now, but there's always opportunity for surprises on

Re: [PATCH RFC] ARM: dts: mxs: leave card detect out of common mmc pins config

2013-04-08 Thread Shawn Guo
On Mon, Apr 08, 2013 at 12:12:20PM +0200, Hector Palacios wrote: MicroSD card sockets don't usually have card detect line. This pin is actually not needed for the MMC to work and it is more of a platform design decission to have it. The card detect pin already has a configuration entry of its

Re: [PATCH RFC] ARM: dts: mxs: leave card detect out of common mmc pins config

2013-04-08 Thread Shawn Guo
On Mon, Apr 08, 2013 at 03:58:05PM +0200, Hector Palacios wrote: On 04/08/2013 02:48 PM, Shawn Guo wrote: On Mon, Apr 08, 2013 at 12:12:20PM +0200, Hector Palacios wrote: MicroSD card sockets don't usually have card detect line. This pin is actually not needed for the MMC to work

Re: [PATCH 1/3] ARM: mx28: add auart2 2 pins pinmux to imx28.dtsi

2013-04-09 Thread Shawn Guo
On Mon, Apr 08, 2013 at 02:57:31PM +0200, Eric Bénard wrote: Add auart2 2 pins configuration on its main pads Signed-off-by: Eric Bénard e...@eukrea.com All 3 patches applied, thanks. --- arch/arm/boot/dts/imx28.dtsi | 11 +++ 1 file changed, 11 insertions(+) diff --git

Re: [PATCH 3/6] i2c: Ignore return value of i2c_del_adapter()

2013-03-31 Thread Shawn Guo
the return value and assume that it will always succeed (which it will). In a subsequent patch the return type of i2c_del_adapter() will be made void. ... Cc: Shawn Guo shawn@linaro.org ... drivers/i2c/busses/i2c-mxs.c | 5 + Acked-by: Shawn Guo shawn@linaro.org

Re: [PATCH] Revert lockdep: check that no locks held at freeze time

2013-03-31 Thread Shawn Guo
: Shawn Guo shawn@linaro.org The same BUG was observed on IMX/MXS systems, so 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 at http://vger.kernel.org

Re: [PATCH] ARM: cfa10036: add one wire bitbanging to the cfa10049

2013-03-31 Thread Shawn Guo
On Fri, Mar 29, 2013 at 07:46:41PM +0100, Alexandre Belloni wrote: Signed-off-by: Alexandre Belloni alexandre.bell...@free-electrons.com Applied, thanks. --- arch/arm/boot/dts/imx28-cfa10049.dts | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git

Re: [PATCH] pinctrl: pinctrl-mxs: document the missing pull-ups

2013-03-31 Thread Shawn Guo
On Sat, Mar 30, 2013 at 12:51:12AM +0100, Alexandre Belloni wrote: Some pins on the i.mx23 and i.mx28 are missing pull-ups, document that oddity because it was difficult to know the expected behaviour. Signed-off-by: Alexandre Belloni alexandre.bell...@free-electrons.com Applied, thanks.

Re: [PATCH] cpufreq: cpufreq-cpu0: No need to check cpu number in init()

2013-04-01 Thread Shawn Guo
-cpu would be different. But the driver doesn't have any dependency on cpu0 as such. We don't mind which cpu of a system is policy-cpu and so this check is just not required. Remove it. Signed-off-by: Viresh Kumar viresh.ku...@linaro.org Acked-by: Shawn Guo shawn@linaro.org

Re: [PATCH] cpufreq: cpufreq-cpu0: Call CPUFREQ_POSTCHANGE notifier for failure cases too

2013-04-01 Thread Shawn Guo
broadcast CPUFREQ_POSTCHANGE notifier for these failure cases too, but with old frequency. Signed-off-by: Viresh Kumar viresh.ku...@linaro.org 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

Re: [PATCH 1/2] pwm: mxs: Remove unused *dev from struct mxs_pwm_chip

2013-04-01 Thread Shawn Guo
On Mon, Apr 01, 2013 at 10:48:13AM +0800, Axel Lin wrote: Signed-off-by: Axel Lin axel@ingics.com Acked-by: Shawn Guo shawn@linaro.org --- drivers/pwm/pwm-mxs.c |2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/pwm/pwm-mxs.c b/drivers/pwm/pwm-mxs.c index a53d309

Re: [PATCH] clocksource: mxs_timer: Add semicolon at end of line

2013-04-02 Thread Shawn Guo
On Tue, Apr 02, 2013 at 01:30:32PM -0300, Fabio Estevam wrote: Fix the following build error: drivers/clocksource/mxs_timer.c:304:1: error: expected ',' or ';' at end of input What's your gcc version? I'm using gcc version 4.6.3 (Ubuntu/Linaro 4.6.3-1ubuntu5), and do not see the compile

Re: [PATCH] clocksource: mxs_timer: Add semicolon at end of line

2013-04-02 Thread Shawn Guo
On 3 April 2013 10:29, Shawn Guo shawn@linaro.org wrote: On Tue, Apr 02, 2013 at 01:30:32PM -0300, Fabio Estevam wrote: Fix the following build error: drivers/clocksource/mxs_timer.c:304:1: error: expected ',' or ';' at end of input What's your gcc version? I'm using gcc version 4.6.3

Re: [PATCH 1/7] kbuild: create an include chroot for DT bindings

2013-03-18 Thread Shawn Guo
file (e.g. macros to create a node), but not define any information that the kernel needs to share. These may be placed directly into arch/*/boot/dts/ along with the DT files themselves. Cc: Shawn Guo shawn@linaro.org Cc: Hiroshi Doyu hd...@nvidia.com Acked-by: Michal Marek mma

Re: [PATCH 1/3] ARM: imx: enable anatop suspend/resume

2013-03-20 Thread Shawn Guo
On Wed, Mar 20, 2013 at 01:39:38PM -0400, Anson Huang wrote: anatop module have sereval configurations for user to reduce the power consumption in suspend, provide suspend/resume interface for further use and enable fet_odrive to reduce CORE LDO leakage during suspend. Signed-off-by: Anson

Re: [PATCH 2/3] ARM: imx: enable periphery well bias for suspend

2013-03-20 Thread Shawn Guo
Forgot mentioning in patch #1, for patches touching arch/arm/, it's good enough to send them to linux-arm-kernel list. Copying list linux-kernel isn't so necessary. On Wed, Mar 20, 2013 at 01:39:39PM -0400, Anson Huang wrote: enable periphery charge pump for well biasing at suspend to reduce

Re: [PATCH 1/3] ARM: imx: enable anatop suspend/resume

2013-03-20 Thread Shawn Guo
On Wed, Mar 20, 2013 at 04:45:54PM -0400, Anson Huang wrote: +void imx_anatop_pre_suspend(void) static? This function will be called outside this file, so we can NOT use static here. Sorry, I mistakenly put the comment there. Shawn -- To unsubscribe from this list: send the line

Re: [PATCH 3/3] ARM: imx: enable RBC to support anatop LPM mode

2013-03-20 Thread Shawn Guo
On Wed, Mar 20, 2013 at 01:39:40PM -0400, Anson Huang wrote: RBC is to control whether some ANATOP sub modules can enter lpm mode when SOC is into STOP mode, if RBC is enabled and PMIC_VSTBY_REQ is set, ANATOP will have below behaviors: 1. Digital LDOs(CORE, SOC and PU) are bypassed; 2.

Re: [PATCH 3/3] ARM: imx: enable RBC to support anatop LPM mode

2013-03-20 Thread Shawn Guo
On Wed, Mar 20, 2013 at 01:39:40PM -0400, Anson Huang wrote: RBC is to control whether some ANATOP sub modules can enter lpm mode when SOC is into STOP mode, if RBC is enabled and PMIC_VSTBY_REQ is set, ANATOP will have below behaviors: 1. Digital LDOs(CORE, SOC and PU) are bypassed; 2.

Re: BUG: soft lockup when recording audio on MX28EVK with ASoC sgtl5000

2013-03-25 Thread Shawn Guo
On Mon, Mar 25, 2013 at 01:10:54PM +0100, Marek Vasut wrote: Dear Hector Palacios, CCing Shawn, this might also explain the touchscreen issue. Hello, I just tried recording audio on Freescale's MX28EVK that uses ASoC sgtl5000 (kernel v3.8) with: arecord -M -f cd sound.wav

Re: [PATCH] cpufreq: cpu0: Fix mistake in Documentation example

2013-03-26 Thread Shawn Guo
On Tue, Mar 26, 2013 at 08:10:46PM +0530, Viresh Kumar wrote: clock-latency is incorrectly written as transition-latency in an example present in Documentation of cpufreq-cpu0. Fix it. Signed-off-by: Viresh Kumar viresh.ku...@linaro.org Acked-by: Shawn Guo shawn@linaro.org

Re: [PATCH v9 RESEND 0/4] Add generic driver for on-chip SRAM

2013-03-27 Thread Shawn Guo
On Wed, Mar 27, 2013 at 09:33:34AM +0100, Philipp Zabel wrote: Am Mittwoch, den 20.03.2013, 11:52 +0100 schrieb Philipp Zabel: Hi, last time I posted was a bit close to the merge window, so I'm reposting now. Greg, Arnd, could you take the first two patches? Ping, can I do anything to

Re: [PATCH v9 RESEND 3/4] media: coda: use genalloc API

2013-03-27 Thread Shawn Guo
On Wed, Mar 20, 2013 at 11:52:46AM +0100, Philipp Zabel wrote: This patch depends on genalloc: add devres support, allow to find a managed pool by device, which provides the of_get_named_gen_pool and dev_get_gen_pool functions. Signed-off-by: Philipp Zabel p.za...@pengutronix.de Acked-By:

Re: [PATCH v9 RESEND 4/4] ARM: dts: add sram for imx53 and imx6q

2013-03-27 Thread Shawn Guo
On Wed, Mar 27, 2013 at 03:29:10PM -0700, Andrew Morton wrote: On Wed, 20 Mar 2013 11:52:47 +0100 Philipp Zabel p.za...@pengutronix.de wrote: Signed-off-by: Philipp Zabel p.za...@pengutronix.de Reviewed-by: Shawn Guo shawn@linaro.org Acked-by: Grant Likely grant.lik...@secretlab.ca

Re: [PATCH] cpufreq: cpufreq-cpu0: use the exact frequency for clk_set_rate()

2013-03-27 Thread Shawn Guo
the clock. Signed-off-by: Guennadi Liakhovetski g.liakhovet...@gmx.de Can I have your ack for this one, please? 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 v9 RESEND 4/4] ARM: dts: add sram for imx53 and imx6q

2013-03-27 Thread Shawn Guo
On Wed, Mar 20, 2013 at 11:52:47AM +0100, Philipp Zabel wrote: Signed-off-by: Philipp Zabel p.za...@pengutronix.de Reviewed-by: Shawn Guo shawn@linaro.org Acked-by: Grant Likely grant.lik...@secretlab.ca --- Changes since v8: - Changed device tree compatible string to mmio-sram

Re: [PATCH v9 RESEND 4/4] ARM: dts: add sram for imx53 and imx6q

2013-03-28 Thread Shawn Guo
On Thu, Mar 28, 2013 at 10:05:18AM +0100, Philipp Zabel wrote: + + ocram: ocram@f800 { + compatible = fsl,imx-ocram, mmio-sram; We should probably just drop fsl,imx-ocram. Shawn I thought that in the future somebody might want to implement some

Re: [PATCH] ARM: imx: fix typo DEBUG_IMX50_IMX53_UART

2013-03-11 Thread Shawn Guo
: Mon Mar 11 14:16:00 2013 +0100 ARM: imx: fix typo DEBUG_IMX50_IMX53_UART Commit f8c95fe (ARM: imx: support DEBUG_LL uart port selection for all i.MX SoCs) had a typo that DEBUG_IMX50_IMX53_UART should be DEBUG_IMX53_UART. Signed-off-by: Paul Bolle pebo...@tiscali.nl Signed-off-by: Shawn Guo shawn

Re: [PATCH] arm: tegra: fuse: export chip id and revision

2013-03-11 Thread Shawn Guo
On Mon, Mar 11, 2013 at 08:37:21PM +0800, Danny Huang wrote: Expose tegra chip id and revision in /proc/cpu/chipinfo for user mode usage. It should probably be done in a more generic way. Check it out Documentation/ABI/testing/sysfs-devices-soc. Shawn -- To unsubscribe from this list: send

Re: [PATCH 1/2] cpufreq: cpufreq-cpu0: support for clock which are not in DT yet.

2013-03-12 Thread Shawn Guo
the following in kernel will just help cpufreq-cpu0 find the clock, if you instantiate cpufreq-cpu0 driver in the same way that highbank-cpufreq does. clk_register_clkdev(cpuclk, NULL, cpufreq-cpu0.0); Shawn Cc: Rafael J. Wysocki r...@sisk.pl Cc: Santosh Shilimkar santosh.shilim...@ti.com Cc: Shawn

Re: [PATCH 2/2] cpufreq: cpufreq-cpu0: provide compatibility string for DT matchup

2013-03-12 Thread Shawn Guo
/cpus, and it does not belong to device tree. Shawn Cc: Rafael J. Wysocki r...@sisk.pl Cc: Santosh Shilimkar santosh.shilim...@ti.com Cc: Shawn Guo shawn@linaro.org Cc: linux-kernel@vger.kernel.org Cc: cpuf...@vger.kernel.org Cc: linux...@vger.kernel.org Cc: linux-o...@vger.kernel.org

Re: [PATCH] dma: mxs-dma: Convert to devm_ioremap_resource()

2013-03-12 Thread Shawn Guo
On Mon, Mar 11, 2013 at 08:50:33PM -0300, Fabio Estevam wrote: Converting to devm_ioremap_resource() can make the code cleaner and smaller. While you are there, you may want to use devm_kzalloc() and devm_clk_get() as well. Shawn Signed-off-by: Fabio Estevam feste...@gmail.com ---

Re: [PATCH] video: mxsfb: Convert to devm_ioremap_resource()

2013-03-12 Thread Shawn Guo
On Mon, Mar 11, 2013 at 08:52:17PM -0300, Fabio Estevam wrote: From: Fabio Estevam fabio.este...@freescale.com Converting to devm_ioremap_resource() can make the code cleaner and smaller. While you are there, you may want to use devm_kzalloc() and devm_clk_get() as well. Shawn

Re: [PATCH] dma: mxs-dma: Convert to devm_ioremap_resource()

2013-03-12 Thread Shawn Guo
On 12 March 2013 20:04, Shawn Guo shawn@linaro.org wrote: On Mon, Mar 11, 2013 at 08:50:33PM -0300, Fabio Estevam wrote: Converting to devm_ioremap_resource() can make the code cleaner and smaller. While you are there, you may want to use devm_kzalloc() and devm_clk_get() as well. I even

Re: [PATCH] video: mxsfb: Convert to devm_ioremap_resource()

2013-03-12 Thread Shawn Guo
On 12 March 2013 20:06, Shawn Guo shawn@linaro.org wrote: On Mon, Mar 11, 2013 at 08:52:17PM -0300, Fabio Estevam wrote: From: Fabio Estevam fabio.este...@freescale.com Converting to devm_ioremap_resource() can make the code cleaner and smaller. While you are there, you may want to use

v3.9-rc1: [BUG: swapper/1 still has locks held!]

2013-03-03 Thread Shawn Guo
I'm running v3.9-rc1 kernel on an ARM platform (i.MX28) with arch/arm/mxs_defconfig and getting the following BUG warning. Shawn [5.681034] = [5.685885] [ BUG: swapper/1 still has locks held! ] [5.690885] 3.9.0-rc1 #774 Not tainted [5.694734]

Re: v3.9-rc1: [BUG: swapper/1 still has locks held!]

2013-03-05 Thread Shawn Guo
On Mon, Mar 04, 2013 at 10:52:10AM +0800, Shawn Guo wrote: I'm running v3.9-rc1 kernel on an ARM platform (i.MX28) with arch/arm/mxs_defconfig and getting the following BUG warning. git bisect points me to the commit 6aa9707 (lockdep: check that no locks held at freeze time). Shawn

Re: [PATCH] ARM: mxs: cfa10049: Fix fb initialisation function

2013-03-06 Thread Shawn Guo
On Tue, Mar 05, 2013 at 04:13:35PM +0100, Maxime Ripard wrote: Commit 1fe42740 (ARM: dts: mxs: Add the LCD to the 10049 board) seem to have been applied with some fuzzyness, and the framebuffer initialisation code for the CFA-10049 ended up in the CFA-10037 initialisation function.

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

2012-12-04 Thread Shawn Guo
On Tue, Dec 04, 2012 at 08:34:02AM -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. Signed-off-by: Mark Langsdorf

Re: [PATCH v2] gpio: devm_gpio_* support should not depend on GPIOLIB

2013-02-13 Thread Shawn Guo
Sorry for the late response due to Chinese New Year Holidays here. On Sun, Feb 10, 2013 at 07:46:30AM +0300, Max Filippov wrote: On Sat, Jan 19, 2013 at 2:33 AM, Linus Walleij linus.wall...@linaro.org wrote: On Fri, Jan 18, 2013 at 8:57 AM, Shawn Guo shawn@linaro.org wrote: Some

Re: [PATCH] ASoC: fsl: fix allyesconfig build for imx-pcm

2013-02-05 Thread Shawn Guo
On Tue, Feb 05, 2013 at 04:42:25PM +, Arnd Bergmann wrote: Patch 25b8d314 ASoC: fsl: fix multiple definition of init_module changed the Makefile for imx-pcm.ko to build two modules that both contain the imx-pcm.c file, which nicely solves the problem of building both versions of the driver

Re: [PATCH 3/3] ARM: dts: mxs: Add the LCD to the 10049 board

2013-01-29 Thread Shawn Guo
On Fri, Jan 25, 2013 at 09:54:07AM +0100, Maxime Ripard wrote: Signed-off-by: Maxime Ripard maxime.rip...@free-electrons.com --- arch/arm/boot/dts/imx28-cfa10049.dts | 99 ++ arch/arm/mach-mxs/mach-mxs.c | 22 2 files changed, 121

Re: [PATCH 2/4] ARM: imx: set CKO1 parent clock source in imx6q sabresd

2013-01-29 Thread Shawn Guo
Gary, For arch/arm/ patches, list linux-arm-ker...@lists.infradead.org rather than linux-kernel@vger.kernel.org should be copied. One comment blow. On Tue, Jan 29, 2013 at 03:49:39PM +0800, Gary Zhang wrote: in imx6q sabresd board, wm8962 uses CKO1 as MCLK. set ahb as CKO1 parent clock source

Re: [PATCH 0/2] PM / OPP : export symbol consolidation

2013-01-29 Thread Shawn Guo
/ is patch #2 in this series. Mark Langsdorf (1): PM / OPP: Export more symbols for module usage Nishanth Menon (1): PM / OPP: switch exported symbols to GPL variant The series is needed by cpufreq-cpu0 and imx6q-cpufreq module build. So, for both patches, Acked-by: Shawn Guo shawn

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

2013-01-30 Thread Shawn Guo
LDO ramp up speed */ + if (anatop_reg-delay_reg) { It might be safer to check if (anatop_reg-delay_bit_width)? Otherwise, Acked-by: Shawn Guo shawn@linaro.org Shawn + /* + * the delay for LDO ramp up time is + * based on the register setting, we

Re: [PATCH] mmc: sdhci-esdhc-imx: Remove unused variables

2013-01-30 Thread Shawn Guo
On Wed, Jan 30, 2013 at 09:11:21PM +0100, Thierry Reding wrote: Commit 3f175a6 (mmc: sdhci-esdhc-imx: remove ESDHC_CD_GPIO handling from IO accessory) removed all the code that was using these variables, so it is safe to drop them. Signed-off-by: Thierry Reding

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

2013-01-30 Thread Shawn Guo
On Wed, Jan 30, 2013 at 09:24:31PM +0100, Thierry Reding wrote: Commit 25b8d31 (ASoC: fsl: fix multiple definition of init_module) fixed a build error due to multiple symbol definitions when building as a module. However, it causes the build to break when the driver is builtin because the

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

2013-01-30 Thread Shawn Guo
On Thu, Jan 31, 2013 at 10:57:06AM -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 V3] regulators: anatop: add set_voltage_time_sel interface

2013-01-30 Thread Shawn Guo
-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 the FAQ at http://www.tux.org/lkml/

Re: [PATCH 1/2] ARM: mxs: dts: Add gpio-keys for the rotary to the CFA-10049

2013-02-03 Thread Shawn Guo
On Thu, Jan 31, 2013 at 11:00:23AM +0100, Maxime Ripard wrote: Signed-off-by: Maxime Ripard maxime.rip...@free-electrons.com Applied both, thanks. --- arch/arm/boot/dts/imx28-cfa10049.dts | 14 ++ 1 file changed, 14 insertions(+) diff --git

Re: [PATCHv2] ARM: mxs: dt: Add Crystalfontz CFA-10037 device tree support

2013-01-25 Thread Shawn Guo
On Fri, Jan 25, 2013 at 10:00:35AM +0100, Maxime Ripard wrote: The CFA-10037 is another expansion board for the CFA-10036 module, with only a USB Host, a Ethernet device and a lot of gpios. Signed-off-by: Maxime Ripard maxime.rip...@free-electrons.com Applied, thanks. -- To unsubscribe from

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

2013-01-26 Thread Shawn Guo
Mark, On Fri, Jan 25, 2013 at 01:46:45PM -0600, Mark Langsdorf wrote: +static int hb_cpufreq_driver_init(void) +{ + struct device *cpu_dev; + struct clk *cpu_clk; + struct device_node *np; + int ret; + + for_each_child_of_node(of_find_node_by_path(/cpus), np) +

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

2013-01-28 Thread Shawn Guo
Rafael, On Sat, Jan 26, 2013 at 11:24:12PM +0100, Rafael J. Wysocki wrote: As we are moving to multiplatform build, this init function will run on all other platforms built together with highbank. We should probably eliminate that effect. That change can be made on top of the Mark's

Re: [PATCH 2/3] ARM: dts: mxs: Add muxing options for the third PWM

2013-01-28 Thread Shawn Guo
On Fri, Jan 25, 2013 at 09:54:06AM +0100, Maxime Ripard wrote: Signed-off-by: Maxime Ripard maxime.rip...@free-electrons.com Applied, thanks. --- arch/arm/boot/dts/imx28.dtsi | 10 ++ 1 file changed, 10 insertions(+) diff --git a/arch/arm/boot/dts/imx28.dtsi

Re: [PATCH 2/2] ARM: dts: cfa10049: Change the SPI3 bus to spi-gpio

2013-01-28 Thread Shawn Guo
On Fri, Jan 25, 2013 at 09:39:35AM +0100, Maxime Ripard wrote: The DAC found on the last chip select requires a word length of 12 bits, which is not supported by the SSP controller of the iMX28. Use bitbanging for that bus to support such a length. Signed-off-by: Maxime Ripard

Re: linux-next: build failure after merge of the gpio-lw tree

2013-01-20 Thread Shawn Guo
On Sat, Jan 19, 2013 at 10:40:45AM +1100, Stephen Rothwell wrote: Hi Shawn, On Fri, 18 Jan 2013 16:02:13 +0800 Shawn Guo shawn@linaro.org wrote: My bad, sorry for that. I just sent a v2 in reply to this message for fixing the error. I spent some time trying to install a ppc64

Re: [PATCH 05/15] ASoC: fsl: fiq and dma cannot both be modules

2013-01-21 Thread Shawn Guo
a...@arndb.de Cc: Liam Girdwood l...@ti.com Cc: Mark Brown broo...@opensource.wolfsonmicro.com Cc: Shawn Guo shawn@linaro.org Cc: alsa-de...@alsa-project.org I sent a fix [1] for that queued by Mark. Mark, Is the patch on the way to 3.8-rc? Shawn [1] http://thread.gmane.org

Re: [PATCH 09/15] media: coda: don't build on multiplatform

2013-01-21 Thread Shawn Guo
...@pengutronix.de Cc: Shawn Guo shawn@linaro.org Acked-by: Shawn Guo shawn@linaro.org Cc: Mauro Carvalho Chehab mche...@redhat.com Cc: linux-me...@vger.kernel.org --- drivers/media/platform/Kconfig |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/platform

Re: [PATCH 13/15] USB: ehci: make orion and mxc bus glues coexist

2013-01-21 Thread Shawn Guo
have my ACK with the changes below rolled into your patch. Acked-by: Shawn Guo shawn@linaro.org ---8 diff --git a/drivers/usb/host/ehci-mxc.c b/drivers/usb/host/ehci-mxc.c index 177b354..a685945 100644 --- a/drivers/usb/host/ehci-mxc.c +++ b/drivers/usb/host/ehci-mxc.c @@ -37,7 +37,7

Re: [PATCH 13/15] USB: ehci: make orion and mxc bus glues coexist

2013-01-21 Thread Shawn Guo
On Tue, Jan 22, 2013 at 02:11:18PM +0800, Shawn Guo wrote: Alan, Thanks for the patch. I just gave it try. The USB Host port still works for me with a couple of fixes on your changes integrated (one for compiling and the other for probing). So you have my ACK with the changes below

Re: [PATCH] ARM: mxs: dt: Add Crystalfontz CFA-10037 device tree support

2013-01-23 Thread Shawn Guo
On Tue, Jan 22, 2013 at 05:35:51PM +0100, Maxime Ripard wrote: The CFA-10037 is another expansion board for the CFA-10036 module, with only a USB Host, a Ethernet device and a lot of gpios. Signed-off-by: Maxime Ripard maxime.rip...@free-electrons.com ---

[PATCH v2] gpio: devm_gpio_* support should not depend on GPIOLIB

2013-01-17 Thread Shawn Guo
of devres.c. It also removes the empty version of devm_gpio_* functions for !GENERIC_GPIO build from linux/gpio.h, and moves the function declarations from asm-generic/gpio.h into linux/gpio.h. Signed-off-by: Shawn Guo shawn@linaro.org --- drivers/gpio/Kconfig |3 +++ drivers/gpio

Re: linux-next: build failure after merge of the gpio-lw tree

2013-01-18 Thread Shawn Guo
On Fri, Jan 18, 2013 at 02:03:46PM +1100, Stephen Rothwell wrote: Hi Linus, After merging the gpio-lw tree, today's linux-next build (powerpc ppc64_defconfig) failed like this: My bad, sorry for that. I just sent a v2 in reply to this message for fixing the error. I spent some time trying

Re: [PATCH 2/2] of: use platform_device_add

2013-02-16 Thread Shawn Guo
On Fri, Jan 18, 2013 at 01:40:00AM +, Grant Likely wrote: This allows platform_device_add a chance to call insert_resource on all of the resources from OF. At a minimum this fills in proc/iomem and presumably makes resource tracking and conflict detection work better. However, it has the

Re: WARNING: at drivers/tty/tty_buffer.c:476 (tty is NULL)

2013-02-16 Thread Shawn Guo
Hi Peter, On Mon, Feb 11, 2013 at 09:42:30AM -0500, Peter Hurley wrote: Can you reproduce after using the following patch series? [PATCH v3 00/23] ldisc fixes I'm constantly seeing this warning on my board (imx6q - ARM Cortex-A9 Quad) with -next kernel, while I believe v3.8-rc kernel works

Re: [PATCH 2/2] of: use platform_device_add

2013-02-16 Thread Shawn Guo
On Sun, Feb 17, 2013 at 11:03:35AM +0800, Shawn Guo wrote: On Fri, Jan 18, 2013 at 01:40:00AM +, Grant Likely wrote: This allows platform_device_add a chance to call insert_resource on all of the resources from OF. At a minimum this fills in proc/iomem and presumably makes resource

Re: [PATCH] clk: mxs: Fix sparse warnings

2013-02-17 Thread Shawn Guo
'mxs_clk_wait' was not declared. Should it be static? Signed-off-by: Fabio Estevam fabio.este...@freescale.com Acked-by: Shawn Guo shawn@linaro.org --- drivers/clk/mxs/clk.c |1 + 1 file changed, 1 insertion(+) diff --git a/drivers/clk/mxs/clk.c b/drivers/clk/mxs/clk.c index

Re: [PATCH v9 4/4] ARM: dts: add sram for imx53 and imx6q

2013-02-17 Thread Shawn Guo
On Wed, Feb 13, 2013 at 04:46:42PM +0100, Philipp Zabel wrote: Signed-off-by: Philipp Zabel p.za...@pengutronix.de Reviewed-by: Shawn Guo shawn@linaro.org Acked-by: Grant Likely grant.lik...@secretlab.ca --- Changes since v8: - Changed device tree compatible string to mmio-sram

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

2013-02-17 Thread Shawn Guo
On Sat, Feb 16, 2013 at 12:14:49AM -0500, Nicolas Pitre wrote: On Fri, 15 Feb 2013, Russell King - ARM Linux wrote: On Thu, Feb 14, 2013 at 11:47:50PM +0100, Arnd Bergmann wrote: Patch c08e20d24 arm: Add v7_invalidate_l1 to cache-v7.S moves the v7_invalidate_l1 symbol out of

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 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 to send a patch for it? I'd like to apply it. Shawn diff --git a/arch/arm/mach-imx/headsmp.S

  1   2   3   4   5   6   7   8   9   10   >