From: Arnd Bergmann <arnd@arndb.de>
To: Paul Osmialowski <pawelo@king.net.pl>
Cc: Andrew Morton <akpm@linux-foundation.org>,
Anson Huang <b20788@freescale.com>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
Bhupesh Sharma <bhupesh.sharma@freescale.com>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Frank Li <Frank.Li@freescale.com>,
Geert Uytterhoeven <geert+renesas@glider.be>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Guenter Roeck <linux@roeck-us.net>,
Haojian Zhuang <haojian.zhuang@gmail.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Jingchang Lu <jingchang.lu@freescale.com>,
Jiri Slaby <jslaby@suse.cz>, Kees Cook <keescook@chromium.org>,
Kumar Gala <galak@codeaurora.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Linus Walleij <linus.walleij@linaro.org>,
Magnus Damm <damm+renesas@opensource.se>,
Michael Turquette <mturquette@baylibre.com>,
Nathan Lynch <nathan_lynch@mentor.com>,
Nicolas Pitre <nico@linaro.org>,
Maxime Coquelin stm32 <mcoquelin.stm32@gmail.com>,
Olof Johansson <olof@lixom.net>, Paul Bolle <pebolle@tiscali.nl>,
Rob Herring <r.herring@freescale.com>,
Rob Herring <robh+dt@kernel.org>,
Russell King <linux@arm.linux.org.uk>,
Sergey Senozhatsky <sergey.senozhatsky@gmail.com>,
Shawn Guo <shawn.guo@linaro.org>,
Simon Horman <horms+renesas@verge.net.au>,
Stefan Agner <stefan@agner.ch>,
Stephen Boyd <sboyd@codeaurora.org>,
Thomas Gleixner <tglx@linutronix.de>,
Uwe Kleine-Koenig <u.kleine-koenig@pengutronix.de>,
Catalin Marinas <catalin.marinas@arm.com>,
Dave Martin <Dave.Martin@arm.com>,
Mark Rutland <mark.rutland@arm.com>,
Pawel Moll <pawel.moll@arm.com>,
linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org,
linux-gpio@vger.kernel.org, linux-serial@vger.kernel.org,
devicetree@vger.kernel.org, dmaengine@vger.kernel.org,
Yuri Tikhonov <yur@emcraft.com>,
Sergei Poselenov <sposelenov@emcraft.com>,
Dmitry Cherkassov <d_cherkasov@emcraft.com>,
Alexander Potashev <aspotashev@emcraft.com>
Subject: Re: [PATCH 3/9] arm: add call to CPU idle quirks handler
Date: Tue, 23 Jun 2015 23:59 +0200 [thread overview]
Message-ID: <2235698.8UjxLOMCMb@wuerfel> (raw)
In-Reply-To: <1435094387-20146-4-git-send-email-pawelo@king.net.pl>
On Tuesday 23 June 2015 23:19:41 Paul Osmialowski wrote:
> Some SoCs need additional actions to be performed after arch idle,
> e.g. Kinetis requires invalidation of the I/D bus cache.
>
> Such handler could be held in provided <mach/idle.h> header file.
>
> Signed-off-by: Paul Osmialowski <pawelo@king.net.pl>
> ---
> arch/arm/Kconfig | 7 +++++++
> arch/arm/kernel/process.c | 7 +++++++
> 2 files changed, 14 insertions(+)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 8e3a833..8ef8f8f 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -98,6 +98,13 @@ config ARM_HAS_SG_CHAIN
> config NEED_SG_DMA_LENGTH
> bool
>
> +config NEED_MACH_IDLE_H
> + bool
> +
> +config ARM_CPU_IDLE_QUIRKS
> + bool
> + select NEED_MACH_IDLE_H
> +
We're not adding header files like this, please come up
with another solution. How about a cpuidle driver, or
possibly just overriding arm_pm_idle()?
Arnd
WARNING: multiple messages have this Message-ID (diff)
From: Arnd Bergmann <arnd@arndb.de>
To: Paul Osmialowski <pawelo@king.net.pl>
Cc: Andrew Morton <akpm@linux-foundation.org>,
Anson Huang <b20788@freescale.com>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
Bhupesh Sharma <bhupesh.sharma@freescale.com>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Frank Li <Frank.Li@freescale.com>,
Geert Uytterhoeven <geert+renesas@glider.be>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Guenter Roeck <linux@roeck-us.net>,
Haojian Zhuang <haojian.zhuang@gmail.com>,
Ian Campbell <ijc+devicetree@hellion.org.uk>,
Jingchang Lu <jingchang.lu@freescale.com>,
Jiri Slaby <jslaby@suse.cz>, Kees Cook <keescook@chromium.org>,
Kumar Gala <galak@codeaurora.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Linus Walleij <linus.walleij@linaro.org>,
Magnus Damm <damm+renesas@opensource.se>,
Michael Turquette <mturquette@baylibre.com>,
Nathan Lynch <nathan_lynch@mentor.com>,
Nicolas Pitre <nico@linaro.org>,
Maxime
Subject: Re: [PATCH 3/9] arm: add call to CPU idle quirks handler
Date: Tue, 23 Jun 2015 23:59 +0200 [thread overview]
Message-ID: <2235698.8UjxLOMCMb@wuerfel> (raw)
In-Reply-To: <1435094387-20146-4-git-send-email-pawelo@king.net.pl>
On Tuesday 23 June 2015 23:19:41 Paul Osmialowski wrote:
> Some SoCs need additional actions to be performed after arch idle,
> e.g. Kinetis requires invalidation of the I/D bus cache.
>
> Such handler could be held in provided <mach/idle.h> header file.
>
> Signed-off-by: Paul Osmialowski <pawelo@king.net.pl>
> ---
> arch/arm/Kconfig | 7 +++++++
> arch/arm/kernel/process.c | 7 +++++++
> 2 files changed, 14 insertions(+)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 8e3a833..8ef8f8f 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -98,6 +98,13 @@ config ARM_HAS_SG_CHAIN
> config NEED_SG_DMA_LENGTH
> bool
>
> +config NEED_MACH_IDLE_H
> + bool
> +
> +config ARM_CPU_IDLE_QUIRKS
> + bool
> + select NEED_MACH_IDLE_H
> +
We're not adding header files like this, please come up
with another solution. How about a cpuidle driver, or
possibly just overriding arm_pm_idle()?
Arnd
next prev parent reply other threads:[~2015-06-23 21:59 UTC|newest]
Thread overview: 83+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-23 21:19 [PATCH 0/9] [New BSP] Add initial support for Freescale Kinetis TWR-K70F120M development kit Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 1/9] arm: select different compiler flags for ARM CortexM3 Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:48 ` Russell King - ARM Linux
2015-06-23 21:48 ` Russell King - ARM Linux
2015-06-24 4:22 ` Paul Osmialowski
2015-06-24 4:22 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 2/9] arm: do not place huge encoder tables on stack when it is too small Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-24 7:10 ` Geert Uytterhoeven
2015-06-24 7:17 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 3/9] arm: add call to CPU idle quirks handler Paul Osmialowski
2015-06-23 21:51 ` Russell King - ARM Linux
2015-06-23 21:51 ` Russell King - ARM Linux
2015-06-23 21:59 ` Arnd Bergmann [this message]
2015-06-23 21:59 ` Arnd Bergmann
2015-06-25 16:42 ` Nicolas Pitre
2015-06-25 16:42 ` Nicolas Pitre
2015-06-26 5:30 ` Paul Osmialowski
2015-06-26 5:30 ` Paul Osmialowski
2015-06-26 7:40 ` Arnd Bergmann
2015-06-26 7:40 ` Arnd Bergmann
2015-06-26 21:52 ` Paul Osmialowski
2015-06-26 21:52 ` Paul Osmialowski
2015-06-26 22:27 ` Russell King - ARM Linux
2015-06-26 22:27 ` Russell King - ARM Linux
2015-06-23 21:19 ` Paul Osmialowski
[not found] ` <1435094387-20146-1-git-send-email-pawelo-rhuoMcPwk82rDJvtcaxF/A@public.gmane.org>
2015-06-23 21:19 ` [PATCH 1/9] arm: select different compiler flags for ARM CortexM3 Paul Osmialowski
2015-06-23 21:19 ` [PATCH 2/9] arm: do not place huge encoder tables on stack when it is too small Paul Osmialowski
2015-06-23 21:19 ` [PATCH 3/9] arm: add call to CPU idle quirks handler Paul Osmialowski
2015-06-23 21:19 ` [PATCH 4/9] arm: allow copying of vector table to internal SRAM memory Paul Osmialowski
2015-06-23 21:19 ` [PATCH 6/9] arm: twr-k70f120m: clock source drivers for Kinetis SoC Paul Osmialowski
2015-06-23 21:19 ` [PATCH 7/9] arm: twr-k70f120m: IOMUX driver " Paul Osmialowski
2015-06-23 21:19 ` [PATCH 4/9] arm: allow copying of vector table to internal SRAM memory Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 5/9] arm: twr-k70f120m: basic support for Kinetis TWR-K70F120M Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 22:05 ` Arnd Bergmann
2015-06-23 22:05 ` Arnd Bergmann
2015-06-23 22:33 ` Russell King - ARM Linux
2015-06-23 22:33 ` Russell King - ARM Linux
2015-06-24 4:42 ` Paul Osmialowski
2015-06-24 4:42 ` Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 6/9] arm: twr-k70f120m: clock source drivers for Kinetis SoC Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 22:24 ` Stephen Boyd
2015-06-23 22:24 ` Stephen Boyd
2015-06-24 5:09 ` Paul Osmialowski
2015-06-24 5:09 ` Paul Osmialowski
2015-06-23 22:24 ` Stephen Boyd
2015-06-23 22:25 ` Arnd Bergmann
2015-06-23 22:25 ` Arnd Bergmann
2015-06-24 7:53 ` Thomas Gleixner
2015-06-24 7:53 ` Thomas Gleixner
2015-06-23 21:19 ` [PATCH 7/9] arm: twr-k70f120m: IOMUX driver " Paul Osmialowski
2015-06-24 10:21 ` Paul Bolle
2015-06-24 10:21 ` Paul Bolle
2015-06-24 17:44 ` Paul Osmialowski
2015-06-24 17:44 ` Paul Osmialowski
[not found] ` <1435094387-20146-8-git-send-email-pawelo-rhuoMcPwk82rDJvtcaxF/A@public.gmane.org>
2015-06-24 10:21 ` Paul Bolle
2015-07-14 8:53 ` Linus Walleij
2015-07-14 8:53 ` Linus Walleij
2015-07-14 8:53 ` Linus Walleij
2015-09-08 8:04 ` Paul Osmialowski
2015-09-08 8:04 ` Paul Osmialowski
2015-09-08 8:04 ` Paul Osmialowski
2015-09-08 8:04 ` Paul Osmialowski
2015-09-08 14:28 ` Linus Walleij
2015-09-08 14:28 ` Linus Walleij
2015-09-08 14:28 ` Linus Walleij
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 8/9] arm: twr-k70f120m: extend Freescale eDMA driver with ability to support " Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-24 16:14 ` Vinod Koul
2015-06-24 16:14 ` Vinod Koul
2015-06-24 17:43 ` Paul Osmialowski
2015-06-24 17:43 ` Paul Osmialowski
2015-06-23 21:19 ` [PATCH 9/9] arm: twr-k70f120m: extend Freescale lpuart " Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
2015-06-23 21:19 ` Paul Osmialowski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=2235698.8UjxLOMCMb@wuerfel \
--to=arnd@arndb.de \
--cc=Dave.Martin@arm.com \
--cc=Frank.Li@freescale.com \
--cc=akpm@linux-foundation.org \
--cc=ard.biesheuvel@linaro.org \
--cc=aspotashev@emcraft.com \
--cc=b20788@freescale.com \
--cc=bhupesh.sharma@freescale.com \
--cc=catalin.marinas@arm.com \
--cc=d_cherkasov@emcraft.com \
--cc=damm+renesas@opensource.se \
--cc=daniel.lezcano@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=dmaengine@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=geert+renesas@glider.be \
--cc=gregkh@linuxfoundation.org \
--cc=haojian.zhuang@gmail.com \
--cc=horms+renesas@verge.net.au \
--cc=ijc+devicetree@hellion.org.uk \
--cc=jingchang.lu@freescale.com \
--cc=jslaby@suse.cz \
--cc=keescook@chromium.org \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=linux@roeck-us.net \
--cc=mark.rutland@arm.com \
--cc=mcoquelin.stm32@gmail.com \
--cc=mturquette@baylibre.com \
--cc=nathan_lynch@mentor.com \
--cc=nico@linaro.org \
--cc=olof@lixom.net \
--cc=pawel.moll@arm.com \
--cc=pawelo@king.net.pl \
--cc=pebolle@tiscali.nl \
--cc=r.herring@freescale.com \
--cc=robh+dt@kernel.org \
--cc=sboyd@codeaurora.org \
--cc=sergey.senozhatsky@gmail.com \
--cc=shawn.guo@linaro.org \
--cc=sposelenov@emcraft.com \
--cc=stefan@agner.ch \
--cc=tglx@linutronix.de \
--cc=u.kleine-koenig@pengutronix.de \
--cc=yur@emcraft.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.