Re: [PATCH 1/2] ARM: dts: fix gpio-keys wakeup-source property

2015-10-15 Thread Sudeep Holla
Hi Arnd, On 15/10/15 15:57, Linus Walleij wrote: On Tue, Oct 13, 2015 at 3:32 PM, Sudeep Holla <sudeep.ho...@arm.com> wrote: The keyboard driver for GPIO buttons(gpio-keys) checks for one of the two boolean properties to enable gpio buttons as wakeup source: 1. "wakeup-s

Re: [PATCH] cpufreq: arm_big_little: fix frequency check when bL switcher is active

2015-10-08 Thread Sudeep Holla
On 08/10/15 13:55, Jon Medhurst (Tixy) wrote: On Thu, 2015-10-08 at 16:54 +0530, Viresh Kumar wrote: On 08-10-15, 10:23, Jon Medhurst (Tixy) wrote: On Wed, 2015-10-07 at 23:09 +0530, Viresh Kumar wrote: [...] You are right, I had misread the code. I guess my problem is that I'm actually

[PATCH v2] ARM: dts: fix gpio-keys wakeup-source property

2015-10-16 Thread Sudeep Holla
up" property to indictate the wakeup source. This patch fixes it by making use of "wakeup-source" property. Cc: Magnus Damm <magnus.d...@gmail.com> Acked-by: Simon Horman <ho...@verge.net.au> Reviewed-by: Linus Walleij <linus.wall...@linaro.org> Signed-off-by: Sudeep

Re: [PATCH 14/17] media: st-rc: remove misuse of IRQF_NO_SUSPEND flag

2015-10-19 Thread Sudeep Holla
Hi, On 21/09/15 16:47, Sudeep Holla wrote: The device is set as wakeup capable using proper wakeup API but the driver misuses IRQF_NO_SUSPEND to set the interrupt as wakeup source which is incorrect. This patch removes the use of IRQF_NO_SUSPEND flags replacing it with enable_irq_wake instead

Re: [PATCH 04/17] powerpc: mpic: use IRQCHIP_SKIP_SET_WAKE instead of redundant mpic_irq_set_wake

2015-10-19 Thread Sudeep Holla
Hi Ben, On 23/09/15 05:06, Scott Wood wrote: On Mon, 2015-09-21 at 16:47 +0100, Sudeep Holla wrote: mpic_irq_set_wake return -ENXIO for non FSL MPIC and sets IRQF_NO_SUSPEND flag for FSL ones. enable_irq_wake already returns -ENXIO if irq_set_wak is not implemented. Also there's no need to set

Re: [PATCH 11/17] pm: bf609: remove misuse of IRQF_NO_SUSPEND flag

2015-10-19 Thread Sudeep Holla
On 21/09/15 16:47, Sudeep Holla wrote: The IRQF_NO_SUSPEND flag is used to identify the interrupts that should be left enabled so as to allow them to work as expected during the suspend-resume cycle, but doesn't guarantee that it will wake the system from a suspended state, enable_irq_wake

Re: [PATCH 03/17] input: tegra-kbc: drop use of IRQF_NO_SUSPEND flag

2015-10-19 Thread Sudeep Holla
Hi Dmitry, On 05/10/15 11:24, Laxman Dewangan wrote: On Monday 21 September 2015 09:16 PM, Sudeep Holla wrote: diff --git a/drivers/input/keyboard/tegra-kbc.c b/drivers/input/keyboard/tegra-kbc.c index f97c73bd14f8..5f5dbbd51991 100644 --- a/drivers/input/keyboard/tegra-kbc.c +++ b/drivers

Re: [PATCH v2 4/4] arm64: dts: Add BRCM IPROC NAND DT node for NS2

2015-10-20 Thread Sudeep Holla
On 16/10/15 10:08, Anup Patel wrote: The NAND controller on NS2 SoC is compatible with existing BRCM IPROC NAND driver so let's enable it in NS2 DT and NS2 SVK DT. Signed-off-by: Anup Patel Reviewed-by: Ray Jui Reviewed-by: Scott Branden

[PATCH 18/19] ARM: dts: ste: replace legacy *,wakeup property with wakeup-source

2015-10-21 Thread Sudeep Holla
ified "wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: Linus Walleij <linus.wall...@linaro.org> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/arm/boot/dts/ste-href-tvk1281618.dtsi | 2 +- arch/arm/boot/dts/ste-nomadik-s8815.dts|

[PATCH 19/19] ARM: dts: zynq: replace gpio-key,wakeup with wakeup-source property

2015-10-21 Thread Sudeep Holla
quot; with the unified "wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: Michal Simek <michal.si...@xilinx.com> Cc: "Sören Brinkmann" <soren.brinkm...@xilinx.com> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/arm/

[PATCH 13/19] ARM: dts: tegra: replace legacy *,wakeup property with wakeup-source

2015-10-21 Thread Sudeep Holla
ith the unified "wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: Stephen Warren <swar...@wwwdotorg.org> Cc: Thierry Reding <thierry.red...@gmail.com> Cc: Alexandre Courbot <gnu...@gmail.com> Cc: linux-te...@vger.kernel.org Signed-off-by:

[PATCH 03/19] rtc: opal: enable support for the stardard "wakeup-source" property

2015-10-21 Thread Sudeep Holla
he existing "has-tpo" property. Cc: Alessandro Zummo <a.zu...@towertech.it> Cc: Alexandre Belloni <alexandre.bell...@free-electrons.com> Cc: rtc-li...@googlegroups.com Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- drivers/rtc/rtc-opal.c | 5 +++-- 1 file chan

[PATCH 08/19] ARM: dts: spear: replace gpio-key,wakeup with wakeup-source property

2015-10-21 Thread Sudeep Holla
enabled, value of "0" or "1" have no significance. This patch replaces the legacy "gpio-key,wakeup" with the unified "wakeup-source" property which inturn fixes the above mentioned issue. Cc: Viresh Kumar <viresh.ku...@linaro.org> Reviewed-by: Viresh Kuma

[PATCH 07/19] ARM: dts: rockchip: replace gpio-key,wakeup with wakeup-source property

2015-10-21 Thread Sudeep Holla
c: linux-rockc...@lists.infradead.org Cc: Heiko Stuebner <he...@sntech.de> Reviewed-by: Heiko Stuebner <he...@sntech.de> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/arm/boot/dts/rk3066a-bqcurie2.dts | 3 +-- arch/arm/boot/dts/rk3066a-rayeager.dts | 2

[PATCH 05/19] mmc: core/host: enable support for the standard "wakeup-source" property

2015-10-21 Thread Sudeep Holla
dition to the existing "enable-sdio-wakeup" property. Cc: Ulf Hansson <ulf.hans...@linaro.org> Cc: Adrian Hunter <adrian.hun...@intel.com> Cc: linux-...@vger.kernel.org Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- drivers/mmc/core/host.c| 3 ++-

[PATCH 15/19] ARM: dts: omap: replace legacy *,wakeup property with wakeup-source

2015-10-21 Thread Sudeep Holla
ified "wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: "Benoît Cousson" <bcous...@baylibre.com> Cc: Tony Lindgren <t...@atomide.com> Cc: linux-o...@vger.kernel.org Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/arm/boo

[PATCH 16/19] ARM: dts: s5pv210: replace legacy *,wakeup property with wakeup-source

2015-10-21 Thread Sudeep Holla
ith the unified "wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: Marek Szyprowski <m.szyprow...@samsung.com> Cc: Kukjin Kim <kg...@kernel.org> Cc: linux-samsung-...@vger.kernel.org Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/arm

[PATCH 12/19] ARM: dts: exynos: replace legacy *,wakeup property with wakeup-source

2015-10-21 Thread Sudeep Holla
ith the unified "wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: Kukjin Kim <kg...@kernel.org> Cc: Krzysztof Kozlowski <k.kozlow...@samsung.com> Cc: linux-samsung-...@vger.kernel.org Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/ar

[PATCH 17/19] ARM: dts: armada: replace isil,irq2-can-wakeup-machine with wakeup-source property

2015-10-21 Thread Sudeep Holla
ot;wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: Jason Cooper <ja...@lakedaemon.net> Cc: Andrew Lunn <and...@lunn.ch> Cc: Gregory Clement <gregory.clem...@free-electrons.com> Cc: Sebastian Hesselbarth <sebastian.hesselba...@gmail.com> Signe

[PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property

2015-10-21 Thread Sudeep Holla
quot; with the unified "wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: Simon Horman <ho...@verge.net.au> Cc: Magnus Damm <magnus.d...@gmail.com> Cc: linux...@vger.kernel.org Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/

Re: [PATCH 10/19] ARM: dts: at91: replace gpio-key,wakeup with wakeup-source property

2015-10-21 Thread Sudeep Holla
On 21/10/15 11:21, Nicolas Ferre wrote: Le 21/10/2015 12:10, Sudeep Holla a écrit : Though the keyboard driver for GPIO buttons(gpio-keys) will continue to check for/support the legacy "gpio-key,wakeup" boolean property to enable gpio buttons as wakeup source, "wakeup-sou

Re: [PATCH v3 2/2] ARM: dts: bcm5301x: Add BCM SVK DT files

2015-10-21 Thread Sudeep Holla
On 15/10/15 23:24, Jon Mason wrote: Add device tree files for Broadcom Northstar based SVKs. Since the bcm5301x.dtsi already exists, all that is necessary is the dts files to enable the UARTs. With these files, the SVKs are able to boot to shell. Signed-off-by: Jon Mason

[PATCH 09/19] ARM: dts: am335x: replace gpio-key,wakeup with wakeup-source property

2015-10-21 Thread Sudeep Holla
quot; with the unified "wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: "Benoît Cousson" <bcous...@baylibre.com> Cc: Tony Lindgren <t...@atomide.com> Cc: linux-o...@vger.kernel.org Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com&

[PATCH 11/19] ARM: dts: imx: replace legacy *,wakeup property with wakeup-source

2015-10-21 Thread Sudeep Holla
ified "wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: Shawn Guo <shawn...@kernel.org> Cc: Sascha Hauer <ker...@pengutronix.de> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/arm/boot/dts/imx25-eukrea-mbimxsd25-

[PATCH 06/19] input: tegra-kbc: enable support for the standard "wakeup-source" property

2015-10-21 Thread Sudeep Holla
dition to the existing "nvidia,wakeup-source" property. Cc: Laxman Dewangan <ldewan...@nvidia.com> Cc: Dmitry Torokhov <dmitry.torok...@gmail.com> Cc: Thierry Reding <thierry.red...@gmail.com> Cc: linux-in...@vger.kernel.org Cc: linux-te...@vger.kernel.org Signed-off-

[PATCH 10/19] ARM: dts: at91: replace gpio-key,wakeup with wakeup-source property

2015-10-21 Thread Sudeep Holla
quot; with the unified "wakeup-source" property in order to avoid any futher copy-paste duplication. Cc: Nicolas Ferre <nicolas.fe...@atmel.com> Cc: Alexandre Belloni <alexandre.bell...@free-electrons.com> Cc: Jean-Christophe Plagniol-Villard <plagn...@jcrosoft.com> Signed

[PATCH 01/19] Documentation: devicetree: standardize/consolidate on "wakeup-source" property

2015-10-21 Thread Sudeep Holla
. Cc: Rob Herring <robh...@kernel.org> Cc: Pawel Moll <pawel.m...@arm.com> Cc: Mark Rutland <mark.rutl...@arm.com> Cc: Ian Campbell <ijc+devicet...@hellion.org.uk> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- .../devicetree/bindings/power/wakeup-source.tx

[PATCH 00/19] devicetree: standardize/consolidate on "wakeup-source" property

2015-10-21 Thread Sudeep Holla
Hi, This series attempts to consolidate the various existing bindings and standardize on the widely accepted "wakeup-source" property. Regards, Sudeep Sudeep Holla (19): Documentation: devicetree: standardize/consolidate on "wakeup-source" property Documentat

[PATCH 04/19] rtc: isl12057: enable support for the standard "wakeup-source" property

2015-10-21 Thread Sudeep Holla
rce" property in addition to the existing "isil,irq2-can-wakeup-machine" property. Cc: Alessandro Zummo <a.zu...@towertech.it> Cc: Alexandre Belloni <alexandre.bell...@free-electrons.com> Cc: rtc-li...@googlegroups.com Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> ---

[PATCH 02/19] Documentation: devicetree: fix reference to legacy wakeup properties

2015-10-21 Thread Sudeep Holla
; Cc: Pawel Moll <pawel.m...@arm.com> Cc: Mark Rutland <mark.rutl...@arm.com> Cc: Ian Campbell <ijc+devicet...@hellion.org.uk> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- Documentation/devicetree/bindings/crypto/fsl-sec4.txt | 4 ++-- Documentati

Re: [PATCH 14/19] ARM: dts: shmobile/r8a7xxx: replace gpio-key,wakeup with wakeup-source property

2015-10-21 Thread Sudeep Holla
On 21/10/15 11:18, Geert Uytterhoeven wrote: On Wed, Oct 21, 2015 at 12:10 PM, Sudeep Holla <sudeep.ho...@arm.com> wrote: Though the keyboard driver for GPIO buttons(gpio-keys) will continue to check for/support the legacy "gpio-key,wakeup" boolean property to enable gpio b

Re: [PATCH 10/19] ARM: dts: at91: replace gpio-key,wakeup with wakeup-source property

2015-10-21 Thread Sudeep Holla
On 21/10/15 12:01, Ulf Hansson wrote: On 21 October 2015 at 12:25, Sudeep Holla <sudeep.ho...@arm.com> wrote: On 21/10/15 11:21, Nicolas Ferre wrote: Le 21/10/2015 12:10, Sudeep Holla a écrit : Though the keyboard driver for GPIO buttons(gpio-keys) will continue to check for/s

Re: [PATCH 10/19] ARM: dts: at91: replace gpio-key,wakeup with wakeup-source property

2015-10-21 Thread Sudeep Holla
On 21/10/15 12:13, Sudeep Holla wrote: On 21/10/15 12:01, Ulf Hansson wrote: On 21 October 2015 at 12:25, Sudeep Holla <sudeep.ho...@arm.com> wrote: [..] Yes that was my intention for splitting the patches per SoC group. Many SoC maintainers prefer that. Hold on! All p

Re: [PATCH 4/4] arm64: dts: enable idle states for Hi6220

2015-10-09 Thread Sudeep Holla
states { + entry-method = "arm,psci"; Please refer the bindings: ^ should be just "psci" Otherwise looks good. Acked-by: Sudeep Holla <sudeep.ho...@arm.com> Regards, Sudeep -- To unsubscribe from this list: send the line "unsubscribe linux-kerne

Re: [Linaro-acpi] [PATCH v5 0/5] Provide better MADT subtable sanity checks

2015-10-12 Thread Sudeep Holla
On 12/10/15 08:04, Hanjun Guo wrote: On 10/12/2015 11:58 AM, Pat Erley wrote: On 10/11/2015 08:49 PM, Hanjun Guo wrote: On 10/12/2015 11:08 AM, Pat Erley wrote: On 10/05/2015 10:12 AM, Al Stone wrote: On 10/05/2015 07:39 AM, Rafael J. Wysocki wrote: On Wednesday, September 30, 2015

Re: [PATCH] cpufreq: arm_big_little: fix frequency check when bL switcher is active

2015-10-12 Thread Sudeep Holla
On 08/10/15 10:23, Jon Medhurst (Tixy) wrote: [...] diff --git a/drivers/cpufreq/arm_big_little.c b/drivers/cpufreq/arm_big_little.c index f1e42f8..59115a4 100644 --- a/drivers/cpufreq/arm_big_little.c +++ b/drivers/cpufreq/arm_big_little.c @@ -149,6 +149,18 @@ bL_cpufreq_set_rate(u32 cpu,

Re: [Linaro-acpi] [PATCH v5 0/5] Provide better MADT subtable sanity checks

2015-10-13 Thread Sudeep Holla
On 12/10/15 20:25, Rafael J. Wysocki wrote: On Monday, October 12, 2015 10:44:52 AM Sudeep Holla wrote: [...] Instead of just removing the check completely on x86, IMO restrict it to some newer/later version of ACPI so you can still force vendors to fix their ACPI tables at-least

Re: [PATCH 12/17] ARM: OMAP2+: remove misuse of IRQF_NO_SUSPEND flag

2015-10-13 Thread Sudeep Holla
On 12/10/15 21:28, Tony Lindgren wrote: * Tony Lindgren <t...@atomide.com> [151012 13:27]: * Sudeep Holla <sudeep.ho...@arm.com> [150921 08:52]: The IRQF_NO_SUSPEND flag is used to identify the interrupts that should be left enabled so as to allow them to work as expected during

Re: [PATCH] cpufreq: arm_big_little: fix frequency check when bL switcher is active

2015-10-13 Thread Sudeep Holla
On 13/10/15 08:19, Jon Medhurst (Tixy) wrote: On Mon, 2015-10-12 at 14:20 +0100, Sudeep Holla wrote: On 08/10/15 10:23, Jon Medhurst (Tixy) wrote: [...] diff --git a/drivers/cpufreq/arm_big_little.c b/drivers/cpufreq/arm_big_little.c index f1e42f8..59115a4 100644 --- a/drivers/cpufreq

[PATCH 1/2] ARM: dts: fix gpio-keys wakeup-source property

2015-10-13 Thread Sudeep Holla
up" property to indictate the wakeup source. This patch fixes it by making use of "wakeup-source" property. Cc: Simon Horman <ho...@verge.net.au> Cc: Magnus Damm <magnus.d...@gmail.com> Cc: Linus Walleij <linus.wall...@linaro.org> Signed-off-by: Sudeep Holla <

[PATCH 2/2] ARM: dts: change gpio-key,wakeup property to boolean

2015-10-13 Thread Sudeep Holla
..@sntech.de> Cc: linux-rockc...@lists.infradead.org Cc: Viresh Kumar <viresh.ku...@linaro.org> Cc: spear-de...@list.st.com Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/arm/boot/dts/rk3066a-bqcurie2.dts | 3 +-- arch/arm/boot/dts/rk3066a-rayeager.dts | 2 +-

Re: [PATCH 2/2] ARM: dts: change gpio-key,wakeup property to boolean

2015-10-13 Thread Sudeep Holla
On 13/10/15 15:07, Rob Herring wrote: On Tue, Oct 13, 2015 at 8:32 AM, Sudeep Holla <sudeep.ho...@arm.com> wrote: Keyboard driver for GPIO buttons(gpio-keys) checks for the legacy "gpio-key,wakeup" boolean property to enable gpio buttons as wakeup source. Few dts files

Re: [PATCH 12/17] ARM: OMAP2+: remove misuse of IRQF_NO_SUSPEND flag

2015-10-13 Thread Sudeep Holla
On 13/10/15 15:53, Tony Lindgren wrote: * Sudeep Holla <sudeep.ho...@arm.com> [151013 03:46]: On 12/10/15 21:28, Tony Lindgren wrote: * Tony Lindgren <t...@atomide.com> [151012 13:27]: * Sudeep Holla <sudeep.ho...@arm.com> [150921 08:52]: The IRQF_NO_SUSPEND flag i

[PATCH 2/2][UPDATE] ARM: dts: replace gpio-key,wakeup with wakeup-source property

2015-10-13 Thread Sudeep Holla
issue. Cc: Heiko Stuebner <he...@sntech.de> Cc: linux-rockc...@lists.infradead.org Cc: Viresh Kumar <viresh.ku...@linaro.org> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/arm/boot/dts/rk3066a-bqcurie2.dts | 3 +-- arch/arm/boot/dts/rk3066a-rayeager.dts | 2 +- arch/ar

Re: [PATCH v4 1/8] Documentation: add DT binding for ARM System Control and Power Interface(SCPI) protocol

2015-07-08 Thread Sudeep Holla
Hi Mark, On 08/06/15 11:39, Sudeep Holla wrote: This patch adds devicetree binding for System Control and Power Interface (SCPI) Message Protocol used between the Application Cores(AP) and the System Control Processor(SCP). The MHU peripheral provides a mechanism for inter-processor

Re: [patch 1/2] tick/broadcast: Prevent deep idle states if no broadcast device available

2015-07-07 Thread Sudeep Holla
Hi Thomas, On 07/07/15 08:31, Thomas Gleixner wrote: On Mon, 6 Jul 2015, Thomas Gleixner wrote: On Mon, 6 Jul 2015, Sudeep Holla wrote: This triggered the below crash on boot, looks like it's accessing hrtimer-function which is null in periodic mode IIUC Regards, Sudeep Gah. I have

Re: [PATCH 1/1] irqchip: imx-gpcv2: Simplify the implemenation

2015-08-26 Thread Sudeep Holla
typo in $subject On 26/08/15 16:49, Shenwei Wang wrote: Based on Sudeep Holla's review comments, the implementation can be simplified by using the two flags: IRQCHIP_SKIP_SET_WAKE and IRQCHIP_MASK_ON_SUSPEND. This patch enables the flags in the struct irq_chip and removes the unnecessory

Re: [PATCH 1/1] irqchip: imx-gpcv2: Simplify the implemenation

2015-08-26 Thread Sudeep Holla
On 26/08/15 17:44, Shenwei Wang wrote: -Original Message- From: Sudeep Holla [mailto:sudeep.ho...@arm.com] Sent: 2015年8月26日 11:13 To: Wang Shenwei-B38339; ja...@lakedaemon.net Cc: shawn@linaro.org; t...@linutronix.de; Sudeep Holla; linux-arm-ker...@lists.infradead.org; linux

Re: [PATCH v9 1/1] irqchip: imx-gpcv2: IMX GPCv2 driver for wakeup sources

2015-08-26 Thread Sudeep Holla
Hi Thomas, On 25/08/15 20:26, Thomas Gleixner wrote: On Tue, 25 Aug 2015, Sudeep Holla wrote: On 25/08/15 15:54, Shenwei Wang wrote: You don't really understand what happens after a driver calls enable_irq_wake. In suspend state, even the interrupt controller itself is powered off. How can

Re: [PATCH v3 1/1][Resend] misc: sram: add dev_pm_ops to support module power gate

2015-08-26 Thread Sudeep Holla
On 25/08/15 23:03, Shenwei Wang wrote: When system goes into low power states like SUSPEND_MEM and HIBERNATION, the hardware IP block may be powered off to reduce the power consumption. This power down will lost all the data inside the ram. This patch added the dev_pm_ops and implemented two

Re: [PATCH v9 1/1] irqchip: imx-gpcv2: IMX GPCv2 driver for wakeup sources

2015-08-25 Thread Sudeep Holla
On 24/08/15 20:04, Shenwei Wang wrote: IMX7D contains a new version of GPC IP block (GPCv2). It has two major functions: power management and wakeup source management. This patch adds a new irqchip driver to manage the interrupt wakeup sources on IMX7D. Interesting, you mention that this IP

Re: [PATCH v2 1/5] ACPI: add in a bad_madt_entry() function to eventually replace the macro

2015-09-07 Thread Sudeep Holla
nconditionally. Currently I can see there are 2 other users of acpi_parse_entries i.e. PCC and NUMA. So may be it can be made conditional or return success for non-MADT tables from bad_madt_entry ? Other than that, you can add for ARM64 specific parts: Reviewed-and-tested-by: Sudeep Holla <sud

Re: [PATCH v4] pinctrl: mediatek: Implement wake handler and suspend resume

2015-09-08 Thread Sudeep Holla
On 06/09/15 11:39, maoguang meng wrote: On Wed, 2015-09-02 at 14:02 +0800, Daniel Kurtz wrote: Hi maoguang, On Tue, Aug 25, 2015 at 12:27 AM, Sudeep Holla <sudeep.ho...@arm.com> wrote: On 14/08/15 09:38, maoguang.m...@mediatek.com wrote: From: Maoguang Meng <maoguang.m...@med

Re: [PATCH v1 3/3] arm64: dts: add Hi6220 mailbox node

2015-08-25 Thread Sudeep Holla
On 19/08/15 10:37, Leo Yan wrote: On Hi6220, below memory regions in DDR have specific purpose: 0x05e0, - 0x05ef,: For MCU firmware using at runtime; 0x0740,f000 - 0x0740,: For MCU firmware's section; 0x06df,f000 - 0x06df,: For mailbox message data. Unless I am

Re: [PATCH v1 1/3] dt-bindings: mailbox: Document Hi6220 mailbox driver

2015-08-25 Thread Sudeep Holla
On 19/08/15 10:37, Leo Yan wrote: Document the new compatible for Hisilicon Hi6220 mailbox driver. Signed-off-by: Leo Yan leo@linaro.org --- .../bindings/mailbox/hisilicon,hi6220-mailbox.txt | 57 ++ 1 file changed, 57 insertions(+) create mode 100644

Re: [PATCH v4] pinctrl: mediatek: Implement wake handler and suspend resume

2015-09-08 Thread Sudeep Holla
On 08/09/15 10:28, Sudeep Holla wrote: On 06/09/15 11:39, maoguang meng wrote: On Wed, 2015-09-02 at 14:02 +0800, Daniel Kurtz wrote: Hi maoguang, On Tue, Aug 25, 2015 at 12:27 AM, Sudeep Holla <sudeep.ho...@arm.com> wrote: On 14/08/15 09:38, maoguang.m...@mediatek.com wrote:

Re: [PATCH 0/2] irqchip: renesas: Propagate wake-up settings to parent

2015-09-08 Thread Sudeep Holla
ers didn't bother to enable wakeup source correctly. These fixes are meant for v4.3, as commit aec89ef72ba6c944 is already upstream. Looks fine to me, so for the series: Acked-by: Sudeep Holla <sudeep.ho...@arm.com> Note that masking GIC interrupts during suspend has another sid

Re: [PATCH v2 1/5] ACPI: add in a bad_madt_entry() function to eventually replace the macro

2015-09-10 Thread Sudeep Holla
On 09/09/15 20:57, Al Stone wrote: On 09/07/2015 09:32 AM, Sudeep Holla wrote: Hi Al, On 19/08/15 23:07, Al Stone wrote: I finally got a chance to try this series on Juno. Well it exposed a firmware bug in MADT table :) [..] acpi_tbl_entry_handler handler, @@ -245,6

Re: [PATCH v4] pinctrl: mediatek: Implement wake handler and suspend resume

2015-09-14 Thread Sudeep Holla
On 12/09/15 10:50, maoguang meng wrote: > hi Sudeep: > > I test flowlling your blow suggestions,but the system can not be woken. > > beacuse,mtk_eint_suspend will mask it.As we know if eint wakeup system > it must be unmasked before enter suspend flow. > > e.x > > static int __maybe_unused

Re: [PATCH] drivers/power/wakeirq: Call device_init_wakeup from dev_pm_set_wake_irq

2015-09-14 Thread Sudeep Holla
(+Tony in case he has any inputs) Hi Daniel, On 01/09/15 15:53, Daniel Lezcano wrote: The function dev_pm_set_wake_irq is typically called after device_init_wakeup. Instead of summing a couple of call, let's call device_init_wakeup directly from dev_pm_set_wake_irq / dev_pm_clear_wake_irq.

[PATCH v2 3/5] ACPI / processor_idle: replace PREFIX with pr_fmt

2015-09-16 Thread Sudeep Holla
Like few of the other ACPI modules, replace PREFIX with pr_fmt and change all the printk call sites to use pr_* companion functions in processor_idle. Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- drivers/acpi

[PATCH v2 1/5] ACPI / processor : add support for ACPI0010 processor container

2015-09-16 Thread Sudeep Holla
to represent CPU topology and should not be used to hotplug purposes. This patch enables the support for these ACPI processor containers and ensures the generic container/module devices are not created for them. Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> Signed-off-by: Sudeep

[PATCH v2 1/2] ACPI / tables: simplify acpi_parse_entries

2015-09-16 Thread Sudeep Holla
will help to eliminate the need to pass the table end to the handlers. It also moves the check for zero length entry early so that execution of the handler can be avoided. Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- driv

[PATCH v2 2/2] ACPI / tables : remove unused table_end parameter to acpi_tbl_entry_handler

2015-09-16 Thread Sudeep Holla
of that argument namely MADT is also removed. So we can now eliminate the need to pass the table end to the handlers. This patch removes the unused second parameter in acpi_tbl_entry_handler and updates all the handlers accordingly. Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> Signed-off-

[PATCH v2 0/5] ACPI / processor_idle: Add ACPIv6.0 LPI support

2015-09-16 Thread Sudeep Holla
state requests between the processors is required. ACPI supports two different coordination schemes: Platform coordinated and OS initiated. This series aims at providing basic and initial support for platform coordinated LPI states. Sudeep Holla (5): ACPI / processor : add support for ACPI0010

[PATCH v2 2/5] ACPI / sleep: move acpi_processor_sleep to sleep.c

2015-09-16 Thread Sudeep Holla
it depend on CONFIG_ACPI_SYSTEM_POWER_STATES_SUPPORT so that it's not compiled on architecture like ARM64 where S-states are not yet defined in ACPI. Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- drivers/acpi/processor_driver.c

[PATCH v2 5/5] ACPI / processor_idle: Add support for Low Power Idle(LPI) states

2015-09-16 Thread Sudeep Holla
coordination schemes: Platform coordinated and OS initiated. This patch adds initial support for Platform coordination scheme of LPI. Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- drivers/acpi/Kconfig| 3

[PATCH v2 4/5] ACPI / processor_idle : introduce ARCH_SUPPORTS_ACPI_PROCESSOR_CSTATE

2015-09-16 Thread Sudeep Holla
the processor_idle module to support LPI. Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- arch/ia64/Kconfig | 1 + arch/x86/Kconfig | 1 + drivers/acpi/Kconfig | 3 +++ drivers/acpi

Re: [PATCH v2 1/5] ACPI: add in a bad_madt_entry() function to eventually replace the macro

2015-09-11 Thread Sudeep Holla
On 10/09/15 21:43, Al Stone wrote: On 09/10/2015 10:20 AM, Sudeep Holla wrote: [...] From the code inspection, I can see we have 3 users of acpi_parse_entries not just MADT but also PCC and NUMA/SRAT Something like this solves this issue: - if (bad_madt_entry(table_header

Re: [PATCH v4] pinctrl: mediatek: Implement wake handler and suspend resume

2015-09-15 Thread Sudeep Holla
On 15/09/15 03:52, Dmitry Torokhov wrote: On Mon, Sep 14, 2015 at 4:16 AM, Sudeep Holla <sudeep.ho...@arm.com> wrote: [...] This is wrong assumption in the driver. enable_irq_wake doesn't implicitly enable the IRQ. So the disable_irq should be moved to else. And the resume patc

[PATCH REPOST v6 1/8] Documentation: add DT binding for ARM System Control and Power Interface(SCPI) protocol

2015-09-16 Thread Sudeep Holla
control and management of the core/cluster power states, various power domain DVFS including the core/cluster, certain system clocks configuration, thermal sensors and many others. Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> Acked-by: Mark Rutland <mark.rutl...@arm.com> Cc: Rob H

[PATCH REPOST v6 0/8] ARM64: juno: add SCPI mailbox protocol, clock and CPUFreq support

2015-09-16 Thread Sudeep Holla
Cc: linux...@vger.kernel.org Sudeep Holla (8): Documentation: add DT binding for ARM System Control and Power Interface(SCPI) protocol firmware: add support for ARM System Control and Power Interface(SCPI) protocol clk: add support for clocks provided by SCP(System Control Processor

[PATCH REPOST v6 8/8] arm64: dts: add clock support for all the cpus

2015-09-16 Thread Sudeep Holla
This patch adds the CPU clocks so that the CPU DVFS can be enabled. Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> Acked-by: Liviu Dudau <liviu.du...@arm.com> Cc: Jon Medhurst (Tixy) <t...@linaro.org> --- arch/arm64/boot/dts/arm/juno-r1.dts | 6 ++ arch/arm64/boot/dts/a

[PATCH REPOST v6 7/8] arm64: dts: add CPU topology on Juno

2015-09-16 Thread Sudeep Holla
This patch adds CPU topology on Juno. It will be useful for ther other IP blocks depending on this topology. Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> Acked-by: Liviu Dudau <liviu.du...@arm.com> Cc: Jon Medhurst (Tixy) <t...@linaro.org> --- arch/arm64/boot/dts/ar

[PATCH REPOST v6 5/8] cpufreq: arm_big_little: add SCPI interface driver

2015-09-16 Thread Sudeep Holla
the arm_big_little cpufreq driver for such systems. Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> Acked-by: Viresh Kumar <viresh.ku...@linaro.org> Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> Cc: linux...@vger.kernel.org --- MAINTAINERS| 1 +

[PATCH REPOST v6 6/8] arm64: dts: add SRAM, MHU mailbox and SCPI support on Juno

2015-09-16 Thread Sudeep Holla
-by: Sudeep Holla <sudeep.ho...@arm.com> Acked-by: Liviu Dudau <liviu.du...@arm.com> Cc: Jon Medhurst (Tixy) <t...@linaro.org> --- arch/arm64/boot/dts/arm/juno-base.dtsi | 54 ++ 1 file changed, 54 insertions(+) diff --git a/arch/arm64/boot/dts/arm/juno

[PATCH REPOST v6 3/8] clk: add support for clocks provided by SCP(System Control Processor)

2015-09-16 Thread Sudeep Holla
support for the clocks provided by SCP using SCPI protocol. Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> Reviewed-by: Stephen Boyd <sb...@codeaurora.org> Cc: Mike Turquette <mturque...@baylibre.com> Cc: Liviu Dudau <liviu.du...@arm.com> Cc: Lorenzo Pieralisi <lo

[PATCH REPOST v6 4/8] clk: scpi: add support for cpufreq virtual device

2015-09-16 Thread Sudeep Holla
-by: Sudeep Holla <sudeep.ho...@arm.com> Acked-by: Stephen Boyd <sb...@codeaurora.org> Cc: Mike Turquette <mturque...@baylibre.com> Cc: Liviu Dudau <liviu.du...@arm.com> Cc: Lorenzo Pieralisi <lorenzo.pieral...@arm.com> Cc: Jon Medhurst (Tixy) <t...@linaro.org> Cc: li

[PATCH REPOST v6 2/8] firmware: add support for ARM System Control and Power Interface(SCPI) protocol

2015-09-16 Thread Sudeep Holla
. Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> Reviewed-by: Jon Medhurst (Tixy) <t...@linaro.org> Cc: Jassi Brar <jassisinghb...@gmail.com> Cc: Liviu Dudau <liviu.du...@arm.com> Cc: Lorenzo Pieralisi <lorenzo.pieral...@arm.com> --- MAINTAINERS |

Re: [PATCH] modify pl011 driver to let it work as wakeup source

2015-09-17 Thread Sudeep Holla
On 17/09/15 09:31, Zhaoyang Huang wrote: the commit use the latest dev_pm_set_wake_irq API instead of the enable_irq_wake and IRQF_NO_SUSPEND to configure the ttyAMA device to work as the wakeup source As I mentioned earlier[1], if this a based on Juno platform, then it gets NACK. Please

Re: [PATCH] modify pl011 driver to let it work as wakeup source

2015-09-17 Thread Sudeep Holla
On 17/09/15 12:01, Zhaoyang Huang wrote: On 17 September 2015 at 18:39, Sudeep Holla <sudeep.ho...@arm.com <mailto:sudeep.ho...@arm.com>> wrote: On 17/09/15 09:31, Zhaoyang Huang wrote: the commit use the latest dev_pm_set_wake_irq API instead of the ena

Re: [PATCH 2/2] arm64: dts: mt8173: add timer node

2015-09-17 Thread Sudeep Holla
On 17/09/15 15:56, Yingjoe Chen wrote: On Thu, 2015-09-17 at 14:51 +0100, Sudeep Holla wrote: On 16/09/15 03:04, Yingjoe Chen wrote: From: Daniel Kurtz <djku...@chromium.org> Add device node to enable GPT timer. This timer will be used as sched clock source. Interesting any known

Re: [PATCH 2/2] arm64: dts: mt8173: add timer node

2015-09-17 Thread Sudeep Holla
On 16/09/15 03:04, Yingjoe Chen wrote: From: Daniel Kurtz Add device node to enable GPT timer. This timer will be used as sched clock source. Interesting any known issues with or advantage over the arch timers to prefer it as sched clock source. I see even arch

Re: [PATCH v4] pinctrl: mediatek: Implement wake handler and suspend resume

2015-09-11 Thread Sudeep Holla
On 11/09/15 12:22, Chung-Yih Wang (王崇懿) wrote: Hi Sudeep and Maoguang, Please correct me if I am wrong. I think the wake_mask Maoguang implemented is the wake-up configuration and it is how he disabled other unwanted interrupt sources(e.g. audio jacket insertion) during suspend. OK,

Re: [PATCH] ARM: Remove __ref on hotplug cpu die path

2015-09-15 Thread Sudeep Holla
u...@gmail.com> Cc: Linus Walleij <linus.wall...@linaro.org> Cc: Sudeep Holla <sudeep.ho...@arm.com> Cc: Lorenzo Pieralisi <lorenzo.pieral...@arm.com> Cc: Will Deacon <will.dea...@arm.com> Cc: Mark Rutland <mark.rutl...@arm.com> Cc: <linux-o...@vger.kernel.org

[PATCH v3 1/2] ACPI / tables: simplify acpi_parse_entries

2015-10-01 Thread Sudeep Holla
will help to eliminate the need to pass the table end to the handlers. It also moves the check for zero length entry early so that execution of the handler can be avoided. Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- driv

[PATCH v3 2/2] ACPI / tables : remove unused table_end parameter to acpi_tbl_entry_handler

2015-10-01 Thread Sudeep Holla
of that argument namely MADT is also removed. So we can now eliminate the need to pass the table end to the handlers. This patch removes the unused second parameter in acpi_tbl_entry_handler and updates all the handlers accordingly. Cc: "Rafael J. Wysocki" <r...@rjwysocki.net> Signed-off-

Re: [PATCH 2/2] arm64: dts: mt8173: add timer node

2015-10-01 Thread Sudeep Holla
On 01/10/15 15:33, Yingjoe Chen wrote: On Thu, 2015-09-17 at 17:13 +0100, Sudeep Holla wrote: [...] I think your are confusing the system counter with arch timers. System counter is always-on, but the arch timers(logic implementing timers comparators) might not be off when the processor

Re: [PATCH 01/17] rtc: pl031: remove misuse of IRQF_NO_SUSPEND flag

2015-10-02 Thread Sudeep Holla
On 02/10/15 11:40, Linus Walleij wrote: On Mon, Sep 21, 2015 at 8:46 AM, Sudeep Holla <sudeep.ho...@arm.com> wrote: The IRQF_NO_SUSPEND flag is used to identify the interrupts that should be left enabled so as to allow them to work as expected during the suspend-resume cycle, but d

Re: [PATCH v4 2/5] firmware: arm_scpi: Extend to support sensors

2015-10-02 Thread Sudeep Holla
On 15/09/15 17:50, Punit Agrawal wrote: ARM System Control Processor (SCP) provides an API to query and use the sensors available in the system. Extend the SCPI driver to support sensor messages. Signed-off-by: Punit Agrawal <punit.agra...@arm.com> Cc: Sudeep Holla <sudeep.ho..

Re: [PATCH v2 1/2] ACPI / tables: simplify acpi_parse_entries

2015-09-28 Thread Sudeep Holla
On 26/09/15 01:27, Rafael J. Wysocki wrote: On Wednesday, September 16, 2015 01:58:06 PM Sudeep Holla wrote: acpi_parse_entries passes the table end pointer to the sub-table entry handler. acpi_parse_entries itself could validate the end of an entry against the table end using the length

Re: [PATCH v2 1/2] ACPI / tables: simplify acpi_parse_entries

2015-09-28 Thread Sudeep Holla
On 28/09/15 14:50, Rafael J. Wysocki wrote: On Monday, September 28, 2015 11:11:11 AM Sudeep Holla wrote: On 26/09/15 01:27, Rafael J. Wysocki wrote: On Wednesday, September 16, 2015 01:58:06 PM Sudeep Holla wrote: acpi_parse_entries passes the table end pointer to the sub-table entry

Re: [PATCH 15/17] ir-hix5hd2: drop the use of IRQF_NO_SUSPEND

2015-09-23 Thread Sudeep Holla
On 23/09/15 15:26, zhangfei wrote: On 09/21/2015 08:47 AM, Sudeep Holla wrote: This driver doesn't claim the IR transmitter to be wakeup source. It even disables the clock and the IR during suspend-resume cycle. This patch removes yet another misuse of IRQF_NO_SUSPEND. Cc: Mauro Carvalho

Re: [PATCH v2 4/5] ACPI / processor_idle : introduce ARCH_SUPPORTS_ACPI_PROCESSOR_CSTATE

2015-09-24 Thread Sudeep Holla
On 24/09/15 11:31, Ashwin Chaugule wrote: Hi Sudeep, On 16 September 2015 at 09:59, Sudeep Holla <sudeep.ho...@arm.com> wrote: ACPI 6.0 adds a new method to specify the CPU idle states(C-states) called Low Power Idle(LPI) states. Since new architectures like ARM64 use only LPIs, int

Re: [PATCH 07/17] input: tegra-kbc: drop use of IRQF_NO_SUSPEND flag

2015-09-23 Thread Sudeep Holla
USPEND while registering the interrupt. This patch removes the use of IRQF_NO_SUSPEND flag. Cc: Samuel Ortiz <sa...@linux.intel.com> Cc: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- drivers/mfd/qcom_rpm.c | 2 +- 1 file changed, 1

Re: [PATCH V3 1/2] ACPI / EC: Fix broken big-endian 64bit platforms using 'global_lock'

2015-09-23 Thread Sudeep Holla
On 15/09/15 09:34, Viresh Kumar wrote: global_lock is defined as an unsigned long and accessing only its lower 32 bits from sysfs is incorrect, as we need to consider other 32 bits for big endian 64 bit systems. Fix that by making global_lock an u32 instead. Cc: #

[PATCH 15/17] ir-hix5hd2: drop the use of IRQF_NO_SUSPEND

2015-09-21 Thread Sudeep Holla
o.org> Cc: Patrice Chotard <patrice.chot...@st.com> Cc: Fabio Estevam <fabio.este...@freescale.com> Cc: Guoxiong Yan <yanguoxi...@huawei.com> Cc: linux-me...@vger.kernel.org Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- drivers/media/rc/ir-hix5hd2.c | 2 +- 1 file

[PATCH 17/17] mfd : ab8500/db8500: fix the abuse of IRQF_NO_SUSPEND

2015-09-21 Thread Sudeep Holla
. This patch removes the use of IRQF_NO_SUSPEND flags replacing it with enable_irq_wake instead. Cc: Linus Walleij <linus.wall...@linaro.org> Cc: linux-arm-ker...@lists.infradead.org Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- drivers/mfd/ab8500-core.c| 11 +--

[PATCH 12/17] ARM: OMAP2+: remove misuse of IRQF_NO_SUSPEND flag

2015-09-21 Thread Sudeep Holla
. This patch removes the use of IRQF_NO_SUSPEND flags replacing it with enable_irq_wake instead. Cc: Tony Lindgren <t...@atomide.com> Cc: Kevin Hilman <khil...@deeprootsystems.com> Cc: linux-o...@vger.kernel.org Cc: linux-arm-ker...@lists.infradead.org Signed-off-by: Sudeep Holla <sudee

[PATCH 13/17] net: gianfar: remove misuse of IRQF_NO_SUSPEND flag

2015-09-21 Thread Sudeep Holla
davemloft.net> Cc: Claudiu Manoil <claudiu.man...@freescale.com> Cc: Kevin Hao <haoke...@gmail.com> Cc: net...@vger.kernel.org Signed-off-by: Sudeep Holla <sudeep.ho...@arm.com> --- drivers/net/ethernet/freescale/gianfar.c | 9 + 1 file changed, 5 insertions(+), 4 dele

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