From: "Heiko Stübner" <heiko@sntech.de>
To: Tomasz Figa <tomasz.figa@gmail.com>
Cc: linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org, linux-pwm@vger.kernel.org,
Kukjin Kim <kgene.kim@samsung.com>,
thierry.reding@gmail.com, Arnd Bergmann <arnd@arndb.de>,
Olof Johansson <olof@lixom.net>,
Sylwester Nawrocki <sylvester.nawrocki@gmail.com>,
Mark Brown <broonie@kernel.org>,
Thomas Abraham <thomas.abraham@linaro.org>
Subject: Re: [PATCH 14/15] ARM: SAMSUNG: Remove remaining uses of plat/regs-timer.h header
Date: Thu, 13 Jun 2013 22:24:33 +0200 [thread overview]
Message-ID: <201306132224.33866.heiko@sntech.de> (raw)
In-Reply-To: <1370467100-10820-15-git-send-email-tomasz.figa@gmail.com>
Am Mittwoch, 5. Juni 2013, 23:18:19 schrieb Tomasz Figa:
> This patch removes remaining inclusions of plat/regs-timer.h as a
> preparation to remove the header.
hmm, the patch does a bit more than that, as it also removes the saving of the
registers from the suspend calls, which is most likely handled by the driver
now. But the description should possibly reflect that.
> Signed-off-by: Tomasz Figa <tomasz.figa@gmail.com>
> ---
> arch/arm/mach-s3c64xx/irq-pm.c | 2 --
> arch/arm/mach-s5p64x0/pm.c | 3 ---
> arch/arm/mach-s5pv210/pm.c | 10 ----------
> arch/arm/plat-samsung/s5p-irq.c | 1 -
> 4 files changed, 16 deletions(-)
>
> diff --git a/arch/arm/mach-s3c64xx/irq-pm.c
> b/arch/arm/mach-s3c64xx/irq-pm.c index 0c7e1d9..c3da1b6 100644
> --- a/arch/arm/mach-s3c64xx/irq-pm.c
> +++ b/arch/arm/mach-s3c64xx/irq-pm.c
> @@ -22,7 +22,6 @@
> #include <mach/map.h>
>
> #include <plat/regs-serial.h>
> -#include <plat/regs-timer.h>
> #include <mach/regs-gpio.h>
> #include <plat/cpu.h>
> #include <plat/pm.h>
> @@ -43,7 +42,6 @@ static struct sleep_save irq_save[] = {
> SAVE_ITEM(S3C64XX_EINT0FLTCON2),
> SAVE_ITEM(S3C64XX_EINT0FLTCON3),
> SAVE_ITEM(S3C64XX_EINT0MASK),
> - SAVE_ITEM(S3C64XX_TINT_CSTAT),
> };
>
> static struct irq_grp_save {
> diff --git a/arch/arm/mach-s5p64x0/pm.c b/arch/arm/mach-s5p64x0/pm.c
> index 97c2a08a..861e15c 100644
> --- a/arch/arm/mach-s5p64x0/pm.c
> +++ b/arch/arm/mach-s5p64x0/pm.c
> @@ -18,7 +18,6 @@
>
> #include <plat/cpu.h>
> #include <plat/pm.h>
> -#include <plat/regs-timer.h>
> #include <plat/wakeup-mask.h>
>
> #include <mach/regs-clock.h>
> @@ -48,8 +47,6 @@ static struct sleep_save s5p64x0_misc_save[] = {
> SAVE_ITEM(S5P64X0_MEM0CONSLP1),
> SAVE_ITEM(S5P64X0_MEM0DRVCON),
> SAVE_ITEM(S5P64X0_MEM1DRVCON),
> -
> - SAVE_ITEM(S3C64XX_TINT_CSTAT),
> };
>
> /* DPLL is present only in S5P6450 */
> diff --git a/arch/arm/mach-s5pv210/pm.c b/arch/arm/mach-s5pv210/pm.c
> index 2b68a67..3cf3f9c 100644
> --- a/arch/arm/mach-s5pv210/pm.c
> +++ b/arch/arm/mach-s5pv210/pm.c
> @@ -21,7 +21,6 @@
>
> #include <plat/cpu.h>
> #include <plat/pm.h>
> -#include <plat/regs-timer.h>
>
> #include <mach/regs-irq.h>
> #include <mach/regs-clock.h>
> @@ -77,15 +76,6 @@ static struct sleep_save s5pv210_core_save[] = {
> /* Clock ETC */
> SAVE_ITEM(S5P_CLK_OUT),
> SAVE_ITEM(S5P_MDNIE_SEL),
> -
> - /* PWM Register */
> - SAVE_ITEM(S3C2410_TCFG0),
> - SAVE_ITEM(S3C2410_TCFG1),
> - SAVE_ITEM(S3C64XX_TINT_CSTAT),
> - SAVE_ITEM(S3C2410_TCON),
> - SAVE_ITEM(S3C2410_TCNTB(0)),
> - SAVE_ITEM(S3C2410_TCMPB(0)),
> - SAVE_ITEM(S3C2410_TCNTO(0)),
> };
>
> static int s5pv210_cpu_suspend(unsigned long arg)
> diff --git a/arch/arm/plat-samsung/s5p-irq.c
> b/arch/arm/plat-samsung/s5p-irq.c index 6729cb2..ddfaca9 100644
> --- a/arch/arm/plat-samsung/s5p-irq.c
> +++ b/arch/arm/plat-samsung/s5p-irq.c
> @@ -17,7 +17,6 @@
>
> #include <mach/irqs.h>
> #include <mach/map.h>
> -#include <plat/regs-timer.h>
> #include <plat/cpu.h>
>
> void __init s5p_init_irq(u32 *vic, u32 num_vic)
WARNING: multiple messages have this Message-ID (diff)
From: heiko@sntech.de (Heiko Stübner)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 14/15] ARM: SAMSUNG: Remove remaining uses of plat/regs-timer.h header
Date: Thu, 13 Jun 2013 22:24:33 +0200 [thread overview]
Message-ID: <201306132224.33866.heiko@sntech.de> (raw)
In-Reply-To: <1370467100-10820-15-git-send-email-tomasz.figa@gmail.com>
Am Mittwoch, 5. Juni 2013, 23:18:19 schrieb Tomasz Figa:
> This patch removes remaining inclusions of plat/regs-timer.h as a
> preparation to remove the header.
hmm, the patch does a bit more than that, as it also removes the saving of the
registers from the suspend calls, which is most likely handled by the driver
now. But the description should possibly reflect that.
> Signed-off-by: Tomasz Figa <tomasz.figa@gmail.com>
> ---
> arch/arm/mach-s3c64xx/irq-pm.c | 2 --
> arch/arm/mach-s5p64x0/pm.c | 3 ---
> arch/arm/mach-s5pv210/pm.c | 10 ----------
> arch/arm/plat-samsung/s5p-irq.c | 1 -
> 4 files changed, 16 deletions(-)
>
> diff --git a/arch/arm/mach-s3c64xx/irq-pm.c
> b/arch/arm/mach-s3c64xx/irq-pm.c index 0c7e1d9..c3da1b6 100644
> --- a/arch/arm/mach-s3c64xx/irq-pm.c
> +++ b/arch/arm/mach-s3c64xx/irq-pm.c
> @@ -22,7 +22,6 @@
> #include <mach/map.h>
>
> #include <plat/regs-serial.h>
> -#include <plat/regs-timer.h>
> #include <mach/regs-gpio.h>
> #include <plat/cpu.h>
> #include <plat/pm.h>
> @@ -43,7 +42,6 @@ static struct sleep_save irq_save[] = {
> SAVE_ITEM(S3C64XX_EINT0FLTCON2),
> SAVE_ITEM(S3C64XX_EINT0FLTCON3),
> SAVE_ITEM(S3C64XX_EINT0MASK),
> - SAVE_ITEM(S3C64XX_TINT_CSTAT),
> };
>
> static struct irq_grp_save {
> diff --git a/arch/arm/mach-s5p64x0/pm.c b/arch/arm/mach-s5p64x0/pm.c
> index 97c2a08a..861e15c 100644
> --- a/arch/arm/mach-s5p64x0/pm.c
> +++ b/arch/arm/mach-s5p64x0/pm.c
> @@ -18,7 +18,6 @@
>
> #include <plat/cpu.h>
> #include <plat/pm.h>
> -#include <plat/regs-timer.h>
> #include <plat/wakeup-mask.h>
>
> #include <mach/regs-clock.h>
> @@ -48,8 +47,6 @@ static struct sleep_save s5p64x0_misc_save[] = {
> SAVE_ITEM(S5P64X0_MEM0CONSLP1),
> SAVE_ITEM(S5P64X0_MEM0DRVCON),
> SAVE_ITEM(S5P64X0_MEM1DRVCON),
> -
> - SAVE_ITEM(S3C64XX_TINT_CSTAT),
> };
>
> /* DPLL is present only in S5P6450 */
> diff --git a/arch/arm/mach-s5pv210/pm.c b/arch/arm/mach-s5pv210/pm.c
> index 2b68a67..3cf3f9c 100644
> --- a/arch/arm/mach-s5pv210/pm.c
> +++ b/arch/arm/mach-s5pv210/pm.c
> @@ -21,7 +21,6 @@
>
> #include <plat/cpu.h>
> #include <plat/pm.h>
> -#include <plat/regs-timer.h>
>
> #include <mach/regs-irq.h>
> #include <mach/regs-clock.h>
> @@ -77,15 +76,6 @@ static struct sleep_save s5pv210_core_save[] = {
> /* Clock ETC */
> SAVE_ITEM(S5P_CLK_OUT),
> SAVE_ITEM(S5P_MDNIE_SEL),
> -
> - /* PWM Register */
> - SAVE_ITEM(S3C2410_TCFG0),
> - SAVE_ITEM(S3C2410_TCFG1),
> - SAVE_ITEM(S3C64XX_TINT_CSTAT),
> - SAVE_ITEM(S3C2410_TCON),
> - SAVE_ITEM(S3C2410_TCNTB(0)),
> - SAVE_ITEM(S3C2410_TCMPB(0)),
> - SAVE_ITEM(S3C2410_TCNTO(0)),
> };
>
> static int s5pv210_cpu_suspend(unsigned long arg)
> diff --git a/arch/arm/plat-samsung/s5p-irq.c
> b/arch/arm/plat-samsung/s5p-irq.c index 6729cb2..ddfaca9 100644
> --- a/arch/arm/plat-samsung/s5p-irq.c
> +++ b/arch/arm/plat-samsung/s5p-irq.c
> @@ -17,7 +17,6 @@
>
> #include <mach/irqs.h>
> #include <mach/map.h>
> -#include <plat/regs-timer.h>
> #include <plat/cpu.h>
>
> void __init s5p_init_irq(u32 *vic, u32 num_vic)
next prev parent reply other threads:[~2013-06-13 20:24 UTC|newest]
Thread overview: 105+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-05 21:18 [PATCH 00/15] Final Samsung PWM support cleanup Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 01/15] ARM: SAMSUNG: Unify base address definitions of timer block Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 02/15] ARM: SAMSUNG: Add new PWM platform device Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 03/15] ARM: SAMSUNG: Set PWM platform data Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 04/15] ARM: SAMSUNG: Move all platforms to new clocksource driver Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 05/15] ARM: SAMSUNG: Remove old samsung-time driver Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 06/15] ARM: SAMSUNG: Remove unused PWM timer IRQ chip code Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 07/15] pwm: samsung: Rename to pwm-samsung-legacy Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 08/15] pwm: Add new pwm-samsung driver Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-13 20:14 ` Heiko Stübner
2013-06-13 20:14 ` Heiko Stübner
2013-06-13 20:18 ` Tomasz Figa
2013-06-13 20:18 ` Tomasz Figa
2013-06-16 17:19 ` Tomasz Figa
2013-06-16 17:19 ` Tomasz Figa
2013-06-16 20:45 ` Kukjin Kim
2013-06-16 20:45 ` Kukjin Kim
2013-06-17 20:29 ` Thierry Reding
2013-06-17 20:29 ` Thierry Reding
2013-06-17 20:50 ` Tomasz Figa
2013-06-17 20:50 ` Tomasz Figa
2013-06-18 22:17 ` Thierry Reding
2013-06-18 22:17 ` Thierry Reding
2013-06-18 22:45 ` Tomasz Figa
2013-06-18 22:45 ` Tomasz Figa
2013-06-19 9:43 ` Thierry Reding
2013-06-19 9:43 ` Thierry Reding
2013-06-19 10:23 ` Tomasz Figa
2013-06-19 10:23 ` Tomasz Figa
2013-06-24 17:52 ` Tomasz Figa
2013-06-24 17:52 ` Tomasz Figa
2013-06-24 19:50 ` Thierry Reding
2013-06-24 19:50 ` Thierry Reding
2013-06-24 20:03 ` Tomasz Figa
2013-06-24 20:03 ` Tomasz Figa
2013-06-24 20:28 ` Thierry Reding
2013-06-24 20:28 ` Thierry Reding
2013-06-24 20:55 ` Arnd Bergmann
2013-06-24 20:55 ` Arnd Bergmann
2013-06-18 17:59 ` Tomasz Figa
2013-06-18 17:59 ` Tomasz Figa
2013-06-18 18:06 ` Kukjin Kim
2013-06-18 18:06 ` Kukjin Kim
2013-06-18 18:13 ` Tomasz Figa
2013-06-18 18:13 ` Tomasz Figa
2013-06-18 21:33 ` Thierry Reding
2013-06-18 21:33 ` Thierry Reding
2013-06-18 21:35 ` Tomasz Figa
2013-06-18 21:35 ` Tomasz Figa
2013-06-18 19:41 ` Tomasz Figa
2013-06-18 19:41 ` Tomasz Figa
2013-06-18 21:40 ` Thierry Reding
2013-06-18 21:40 ` Thierry Reding
2013-06-18 21:42 ` Tomasz Figa
2013-06-18 21:42 ` Tomasz Figa
2013-06-18 22:20 ` Thierry Reding
2013-06-18 22:20 ` Thierry Reding
2013-06-05 21:18 ` [PATCH 09/15] ARM: SAMSUNG: Rework private data handling in dev-backlight Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 10/15] ARM: SAMSUNG: Modify board files to use new PWM platform device Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 11/15] pwm: Remove superseded pwm-samsung-legacy driver Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 12/15] ARM: SAMSUNG: Remove old PWM timer platform devices Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 13/15] ARM: SAMSUNG: Remove pwm-clock infrastructure Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 14/15] ARM: SAMSUNG: Remove remaining uses of plat/regs-timer.h header Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-13 20:24 ` Heiko Stübner [this message]
2013-06-13 20:24 ` Heiko Stübner
2013-06-13 20:38 ` Tomasz Figa
2013-06-13 20:38 ` Tomasz Figa
2013-06-05 21:18 ` [PATCH 15/15] ARM: SAMSUNG: Remove " Tomasz Figa
2013-06-05 21:18 ` Tomasz Figa
2013-06-12 21:48 ` [PATCH 00/15] Final Samsung PWM support cleanup Tomasz Figa
2013-06-12 21:48 ` Tomasz Figa
2013-06-12 23:00 ` Olof Johansson
2013-06-12 23:00 ` Olof Johansson
2013-06-12 23:13 ` Tomasz Figa
2013-06-12 23:13 ` Tomasz Figa
2013-06-12 23:38 ` Heiko Stübner
2013-06-12 23:38 ` Heiko Stübner
2013-06-12 23:52 ` Sylwester Nawrocki
2013-06-12 23:52 ` Sylwester Nawrocki
2013-06-12 23:52 ` Sylwester Nawrocki
2013-06-13 9:07 ` Tomasz Figa
2013-06-13 9:07 ` Tomasz Figa
2013-06-13 20:17 ` Heiko Stübner
2013-06-13 20:17 ` Heiko Stübner
2013-06-13 20:17 ` Heiko Stübner
2013-06-13 22:27 ` Heiko Stübner
2013-06-13 22:27 ` Heiko Stübner
2013-06-13 22:27 ` Heiko Stübner
2013-06-13 0:25 ` Kukjin Kim
2013-06-13 0:25 ` Kukjin Kim
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=201306132224.33866.heiko@sntech.de \
--to=heiko@sntech.de \
--cc=arnd@arndb.de \
--cc=broonie@kernel.org \
--cc=kgene.kim@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-pwm@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=olof@lixom.net \
--cc=sylvester.nawrocki@gmail.com \
--cc=thierry.reding@gmail.com \
--cc=thomas.abraham@linaro.org \
--cc=tomasz.figa@gmail.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.