Re: [PATCH 00/37] ARM: dts: Fix fixed regulators enable GPIO polarity

2015-10-13 Thread Shawn Guo
On Tue, Oct 13, 2015 at 05:17:24PM +0300, Laurent Pinchart wrote: > Hi Shawn, > > On Tuesday 13 October 2015 22:09:46 Shawn Guo wrote: > > On Tue, Oct 13, 2015 at 12:12:29AM +0300, Laurent Pinchart wrote: > > > Laurent Pinchart (37): > > ... > > > >

Re: [PATCH 00/37] ARM: dts: Fix fixed regulators enable GPIO polarity

2015-10-13 Thread Shawn Guo
On Tue, Oct 13, 2015 at 12:12:29AM +0300, Laurent Pinchart wrote: > Laurent Pinchart (37): ... > ARM: imx6sx-sdb: Fix typo in regulator enable GPIO property ... > ARM: dts: imx6qdl-tx6: Fix regulator enable GPIO polarity ... > ARM: dts: imx23-evk: Fix regulator enable GPIO polarity > ARM: d

Re: [PATCH v2 0/3] arm:irchip: IRQCHIP_DECLARE macro is now accessible

2015-10-13 Thread Shawn Guo
On Tue, Oct 13, 2015 at 02:56:12PM +0200, Joel Porquet wrote: > The IRQCHIP_DECLARE macro migrated to 'include/linux/irqchip.h', making it > globally accessible. > > See commit 91e20b5040c67c51aad88cf87db4305c5bd7f79d ("irqchip: Move > IRQCHIP_DECLARE macro to include/linux/irqchip.h"). > > This

Re: [PATCH V3 3/6] PM / OPP: Prefix exported opp routines with dev_pm_opp_

2015-09-06 Thread Shawn Guo
ach-imx/mach-imx6q.c | 2 +- ... > drivers/cpufreq/imx6q-cpufreq.c | 6 +++--- Acked-by: Shawn Guo -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html

Re: [PATCH V3 2/6] PM / OPP: Rename opp init/free table routines

2015-09-06 Thread Shawn Guo
h 'add' and 'free' with 'remove'. > > Reported-by: Pavel Machek > Reviewed-by: Stephen Boyd > Signed-off-by: Viresh Kumar > --- > arch/arm/mach-imx/mach-imx6q.c | 2 +- ... > drivers/cpufreq/imx6q-cpufreq.c | 6 +++--- Acked-by: S

Re: [PATCH] arm:irqchip: IRQCHIP_DECLARE macro is now accessible

2015-09-05 Thread Shawn Guo
On Fri, Sep 04, 2015 at 04:54:21PM -0400, Joel Porquet wrote: > On Monday, July 13, 2015 11:20:35 PM Shawn Guo wrote: > > On Tue, Jul 07, 2015 at 04:02:53PM -0400, Joel Porquet wrote: > > > The IRQCHIP_DECLARE macro migrated to 'include/linux/irqchip.h', making

Re: [PATCH] arm:irqchip: IRQCHIP_DECLARE macro is now accessible

2015-07-13 Thread Shawn Guo
e/linux/irqchip.h"). > > This patch adds inclusions of 'include/linux/irqchip.h' and replaces uses of > macro OF_DECLARE_2 with IRQCHIP_DECLARE. > > Signed-off-by: Joel Porquet > --- ... > arch/arm/mach-imx/gpc.c | 7 ++- Acked-by: Shawn Guo

Re: [PATCH v2 01/13] ARM: kill off set_irq_flags usage

2015-07-12 Thread Shawn Guo
; Cc: Sekhar Nori > Cc: Kevin Hilman > Cc: Jason Cooper > Cc: Andrew Lunn > Cc: Sebastian Hesselbarth > Cc: Gregory Clement > Acked-by: Hans Ulli Kroll > Cc: Shawn Guo > Cc: Sascha Hauer > Cc: Imre Kaloz > Acked-by: Krzysztof Halasa > Cc: Greg Ungerer > Cc

Re: [PATCH v4 14/21] ARM: imx6: convert GPC to stacked domains

2015-01-20 Thread Shawn Guo
d > with old DTs, and old kernels with updated DTs won't even boot. > > Tested-by: Stefan Agner > Acked-by: Stefan Agner > Signed-off-by: Marc Zyngier Acked-by: Shawn Guo -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body o

Re: [PATCH] PM / OPP: Remove ARCH_HAS_OPP

2014-06-08 Thread Shawn Guo
_HAS_OPP symbol > has become redundant and can be removed. Do so. > > Signed-off-by: Mark Brown > --- ... > arch/arm/mach-imx/Kconfig | 1 - Acked-by: Shawn Guo -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a m

Re: [PATCH 3/8] ARM: dts: imx6: update pcie to bring in line with new binding

2014-04-29 Thread Shawn Guo
On Fri, Mar 28, 2014 at 05:52:54PM +0100, Lucas Stach wrote: > The new bindings drops one clock, renames the others and > drops the old interrupt mapping. > > Signed-off-by: Lucas Stach It does not apply on my branch, and I had to apply it manually. Please check my imx/dt branch to see if I did

Re: [PATCH 4/8] PCI: imx6: use new clock names

2014-04-03 Thread Shawn Guo
On Fri, Mar 28, 2014 at 05:52:55PM +0100, Lucas Stach wrote: > As defined in the new binding. > > Signed-off-by: Lucas Stach > --- > drivers/pci/host/pci-imx6.c | 74 > ++--- > 1 file changed, 29 insertions(+), 45 deletions(-) > > diff --git a/drivers/pc

Re: [PATCH 1/8] ARM: imx6q-clk: parent lvds_gate from lvds_sel

2014-04-02 Thread Shawn Guo
On Fri, Mar 28, 2014 at 05:52:52PM +0100, Lucas Stach wrote: > Allows fror proper refcounting of the parent clocks > when enabling the clock output on CLK1/2 pads. > > Signed-off-by: Lucas Stach Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in

Re: [PATCH 2/2] ARM: dts: imx: fix invalid #address-cells value

2014-03-14 Thread Shawn Guo
u > Cc: Sascha Hauer > Cc: Arnd Bergmann > Cc: Stephen Warren > Cc: Bjorn Helgaas > Cc: Simon Horman > Cc: Thierry Reding > Cc: Ben Dooks > Cc: linux-tegra > Cc: Kukjin Kim > Cc: Shawn Guo > Cc: Grant Likely > --- > arch/arm/boot/dts/imx6qdl.dtsi |

Re: [PATCH 1/2] ARM: DT: fix gic interrupt controller documentation

2014-03-14 Thread Shawn Guo
On Thu, Mar 13, 2014 at 03:25:50PM -0700, Tim Harvey wrote: > On Thu, Mar 13, 2014 at 11:25 AM, Jason Gunthorpe > wrote: > > On Thu, Mar 13, 2014 at 11:44:33AM -0600, Stephen Warren wrote: > >> On 03/13/2014 11:40 AM, Tim Harvey wrote: > >> > When using interrupt-maps, the size of a map entry is #

Re: [PATCH v2 5/6] ARM: dts: imx6: add PCIe interrupt mapping properties

2014-03-09 Thread Shawn Guo
On Wed, Mar 05, 2014 at 02:25:50PM +0100, Lucas Stach wrote: > As defined by the common PCI bindings. > > Signed-off-by: Lucas Stach > Acked-by: Arnd Bergmann Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majord...@

Re: [PATCH 5/7] ARM: dts: imx6: add PCIe interrupt mapping properties

2014-03-04 Thread Shawn Guo
On Fri, Feb 28, 2014 at 06:28:41PM +0100, Lucas Stach wrote: > As defined by the common PCI bindings. > > Signed-off-by: Lucas Stach > --- > arch/arm/boot/dts/imx6qdl.dtsi | 11 +++ > 1 file changed, 11 insertions(+) > > diff --git a/arch/arm/boot/dts/imx6qdl.dtsi b/arch/arm/boot/dts/im

Re: [PATCH v2 1/7] cpufreq: cpufreq-cpu0: allow optional safe voltage during frequency transitions

2014-01-28 Thread Shawn Guo
viders, mux, etc.) have to > >> to be operated at a higher voltage level, called the safe voltage level. > >> This patch adds optional support to temporarily switch to a safe voltage > >> level during CPU frequency transitions. > >> > >> Cc: Shawn Guo &g

Re: [PATCH v2 1/7] cpufreq: cpufreq-cpu0: allow optional safe voltage during frequency transitions

2014-01-26 Thread Shawn Guo
PU (and any other > components in the CPU clock domain such as dividers, mux, etc.) have to > to be operated at a higher voltage level, called the safe voltage level. > This patch adds optional support to temporarily switch to a safe voltage > level during CPU frequency transitions. &

Re: [PATCH 1/6] cpufreq: cpufreq-cpu0: allow optional safe voltage during frequency transitions

2014-01-13 Thread Shawn Guo
On Mon, Jan 13, 2014 at 07:51:25PM +0530, Thomas Abraham wrote: > Yes, true. Thanks for pointing out. Instead of using index, will using > a tuple of for safe frequency and safe voltage be fine ? I'm fine with it. Shawn -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-s

Re: [PATCH 5/6] arm: exynos4-dt: statically add platform device for cpufreq-cpu0 platform driver

2014-01-12 Thread Shawn Guo
On Thu, Jan 09, 2014 at 09:29:24PM +0530, Thomas Abraham wrote: > +static struct platform_device cpufreq_device = { > + .name = "cpufreq-cpu0", > + .id = -1, > +}; > + > static void __init exynos4_dt_machine_init(void) > { > exynos_cpuidle_init(); > exyno

Re: [PATCH 1/6] cpufreq: cpufreq-cpu0: allow optional safe voltage during frequency transitions

2014-01-12 Thread Shawn Guo
Hi Thomas, On Thu, Jan 09, 2014 at 09:29:20PM +0530, Thomas Abraham wrote: > @@ -19,6 +19,10 @@ Optional properties: > - cooling-min-level: > - cooling-max-level: > Please refer to Documentation/devicetree/bindings/thermal/thermal.txt. > +- safe-opp-index: Certain platforms require that du

Re: [PATCH] MAINTAINERS: Add DesignWare, i.MX6, Armada, R-Car PCI host maintainers

2013-12-12 Thread Shawn Guo
On Thu, Dec 12, 2013 at 01:29:00PM -0700, Bjorn Helgaas wrote: > On Thu, Dec 12, 2013 at 12:00 AM, Shawn Guo wrote: > > Hi Bjorn, > > > > On Wed, Dec 11, 2013 at 11:32:37AM -0700, Bjorn Helgaas wrote: > >> +PCI DRIVER FOR IMX6 > >> +M: Shawn Guo > >

Re: [PATCH] MAINTAINERS: Add DesignWare, i.MX6, Armada, R-Car PCI host maintainers

2013-12-11 Thread Shawn Guo
Hi Bjorn, On Wed, Dec 11, 2013 at 11:32:37AM -0700, Bjorn Helgaas wrote: > +PCI DRIVER FOR IMX6 > +M: Shawn Guo Thanks for the nomination. But I think a better person for this position would be Richard Zhu (copied). He knows the driver and controller much better than myself, an

Re: [PATCH V5 6/9] ARM: mxs: Provide regulator to pwm-backlight

2013-03-25 Thread Shawn Guo
t;fixed-regulator" is too generic. But it shouldn't an issue right away, and we can fix that later. Acked-by: Shawn Guo > + compatible = "regulator-fixed"; > + regulator-name = "backlight-power"; > + }; >

Re: [PATCH V4 6/9] ARM: mxs: Provide regulator to pwm-backlight

2013-03-19 Thread Shawn Guo
On Tue, Mar 19, 2013 at 11:59:30AM -0700, Andrew Chew wrote: > The pwm-backlight driver now takes a mandatory regulator that is gotten > during driver probe. Initialize a dummy regulator to satisfy this > requirement. > > Signed-off-by: Andrew Chew > --- > arch/arm/boot/dts/imx23-evk.dts |

Re: [PATCH V4 0/9] Add mandatory regulator for all users of pwm-backlight.

2013-03-19 Thread Shawn Guo
On Tue, Mar 19, 2013 at 11:59:24AM -0700, Andrew Chew wrote: > Many backlights are enabled via GPIO. We can generalize the GPIO to a > fixed regulator. > I think we should push the series of "Runtime Interpreted Power Sequences" moving forward, which should be useful this case and many other case

Re: [PATCH] clk: let mxs specific clk-div clock type be a generic clock type

2013-03-18 Thread Shawn Guo
the Samsung platforms as well. So let the mxs specfic clk-div > clock type be a generic clock type that all platforms can utilize. > > Cc: Shawn Guo > Cc: Mike Turquette > Signed-off-by: Thomas Abraham > --- > drivers/clk/Makefile |1 + > dri

Re: [PATCH] mmc: dt: Add card-detection properties to core binding.

2012-08-23 Thread Shawn Guo
On Wed, Aug 22, 2012 at 02:05:35PM -0400, Chris Ball wrote: > Signed-off-by: Chris Ball Acked-by: Shawn Guo Will move sdhci-esdhc-imx driver to this soon. Regards, Shawn > --- > Documentation/devicetree/bindings/mmc/mmc.txt | 9 +++-- > 1 file changed, 7 insertions(+),

Re: [PATCH] mmc: dt: Add 'broken-cd' DT binding

2012-08-22 Thread Shawn Guo
Hi Chris, On Tue, Aug 21, 2012 at 10:48:43AM -0400, Chris Ball wrote: > Aside: the bindings do not match the code. The bindings document says > to use "fsl,cd-internal", and imx51-babbage.dts does so -- but the code > doesn't check for "fsl,cd-internal", it checks for "fsl,cd-controller": > >

Re: [PATCH] mmc: dt: Add 'broken-cd' DT binding

2012-08-21 Thread Shawn Guo
On Tue, Aug 21, 2012 at 10:48:43AM -0400, Chris Ball wrote: > Aside: the bindings do not match the code. The bindings document says > to use "fsl,cd-internal", and imx51-babbage.dts does so -- but the code > doesn't check for "fsl,cd-internal", it checks for "fsl,cd-controller": > > if (o

Re: [PATCH] mmc: dt: Add 'broken-cd' DT binding

2012-08-21 Thread Shawn Guo
On Tue, Aug 21, 2012 at 11:03:59PM +0530, Thomas Abraham wrote: > Ok, I agree with Rob. I was mixing pin muxing here. So if we have > 'cd-gpios' and 'broken-cd' as generic bindings, would the following be > valid? > > [A] cd-gpios not present , broken-cd not present : This means that > there is n

Re: [PATCH v2] video: s3c-fb: Add device tree support

2012-03-31 Thread Shawn Guo
On Fri, Mar 30, 2012 at 09:25:14PM +0530, Rabin Vincent wrote: > On Fri, Mar 30, 2012 at 11:23, Thomas Abraham > wrote: > > +    - samsung,fimd-display: The fimd controller is interfaced with the a > > +      display device such as a lcd panel. This property should specify the > > +      phandle

Re: [PATCH 03/17] ARM: mark const init data with __initconst instead of __initdata

2012-03-30 Thread Shawn Guo
On Fri, Mar 30, 2012 at 11:50:24AM +0200, Uwe Kleine-König wrote: > Hello Shawn, > > On Fri, Mar 30, 2012 at 02:11:36PM +0800, Shawn Guo wrote: > > On Thu, Mar 29, 2012 at 11:12:20PM +0200, Uwe Kleine-König wrote: > > ... > > > diff --git a/arch/arm/mach-imx/imx51-

Re: [PATCH 03/17] ARM: mark const init data with __initconst instead of __initdata

2012-03-29 Thread Shawn Guo
On Thu, Mar 29, 2012 at 11:12:20PM +0200, Uwe Kleine-König wrote: ... > diff --git a/arch/arm/mach-imx/imx51-dt.c b/arch/arm/mach-imx/imx51-dt.c > index 5cca573..7e00748 100644 > --- a/arch/arm/mach-imx/imx51-dt.c > +++ b/arch/arm/mach-imx/imx51-dt.c > @@ -102,7 +102,7 @@ static struct sys_timer im

Re: [PATCH 12/31] ARM: amba: mxs: get rid of NO_IRQ initializers

2012-01-20 Thread Shawn Guo
On Fri, Jan 20, 2012 at 09:26:28AM +, Russell King - ARM Linux wrote: > Signed-off-by: Russell King Acked-by: Shawn Guo > --- > arch/arm/mach-mxs/devices/amba-duart.c |2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/arm/mach-mxs/de

Re: [PATCH 05/31] ARM: amba: mxs: convert to use amba_device_alloc

2012-01-20 Thread Shawn Guo
se, we see the compile error below. CC arch/arm/mach-mxs/devices.o arch/arm/mach-mxs/devices.c: In function ‘mxs_add_amba_device’: arch/arm/mach-mxs/devices.c:80:50: error: ‘const struct amba_device’ has no member named ‘init_name’ Other than that, Acked-by: Shawn Guo Regards, Shaw

Re: [GIT PULL] Samsung devel-dma for v3.3

2012-01-05 Thread Shawn Guo
On Tue, Jan 03, 2012 at 09:11:59PM +, Arnd Bergmann wrote: > There were some conflicts between imx changes that went into the dma-slave > tree and those that went into arm-soc. Sascha, Shawn: can you have a look > to make sure the resolution below is correct? The resolution looks correct to me

Re: [PATCH v5 5/5] imx6q: Remove unconditional dependency on l2x0 L2 cache support

2011-12-16 Thread Shawn Guo
rned off > when desired for debugging purposes etc. > > Since SOC_IMX6Q already depends on ARCH_IMX_V6_V7 and > ARCH_IMX_V6_V7 selects MIGHT_HAVE_CACHE_L2X0, there is no need to > select that option explicitly from SOC_IMX6Q. > > Thanks to Shawn Guo for this suggestion. [1] &g

Re: [PATCH v5 1/5] ARM: l2x0/pl310: Refactor Kconfig to be more maintainable

2011-12-16 Thread Shawn Guo
6_V7 > bool "i.MX3, i.MX6" > select AUTO_ZRELADDR if !ZBOOT_ROM > select ARM_PATCH_PHYS_VIRT > + select MIGHT_HAVE_CACHE_L2X0 > help > This enables support for systems based on the Freescale i.MX3 and > i.MX6 > family. Acked-b

Re: [PATCH v4 REPOST 5/5] imx6q: Remove unconditional dependency on l2x0 L2 cache support

2011-12-14 Thread Shawn Guo
On Thu, Dec 15, 2011 at 09:02:20AM +0800, Richard Zhao wrote: > On Wed, Dec 14, 2011 at 03:01:19PM +, Dave Martin wrote: > > On Wed, Dec 14, 2011 at 10:05:04PM +0800, Richard Zhao wrote: > > > On Wed, Dec 14, 2011 at 09:26:24PM +0800, Shawn Guo wrote: > > > > H

Re: [PATCH v4 REPOST 5/5] imx6q: Remove unconditional dependency on l2x0 L2 cache support

2011-12-14 Thread Shawn Guo
_L2X0 and selects MIGHT_HAVE_CACHE_L2X0 instead. > > This makes the l2x0 support optional, so that it can be turned off > when desired for debugging purposes etc. > > Thanks to Shawn Guo for this suggestion. [1] > > Signed-off-by: Dave Martin > > [1] > http://lists.i

Re: Pet Peaves about Platform code, and arch_reset

2011-11-10 Thread Shawn Guo
On Wed, Nov 09, 2011 at 01:46:52PM -0800, Tony Lindgren wrote: > * Tony Lindgren [07 16:11]: > > * Russell King - ARM Linux [06 05:18]: > > > Here's a list of my peaves about current platform code - which are > > > causing me great issue when trying to clean up the arch_reset() stuff: > >

Re: [PATCH 11/13] ARM: gpio: consolidate trivial gpiolib implementations

2011-08-11 Thread Shawn Guo
/plat/gpio.h | 11 +-- > arch/arm/plat-omap/include/plat/gpio.h | 15 +-- > arch/arm/plat-orion/include/plat/gpio.h |6 ++ > arch/arm/plat-spear/include/plat/gpio.h |6 +----- > 25 files changed, 36 insertions(+), 151 deletions(-) > For mx