From: Paul Bolle <pebolle@tiscali.nl> To: Tomasz Figa <t.figa@samsung.com> Cc: linux-samsung-soc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, "Kukjin Kim" <kgene.kim@samsung.com>, "Arnd Bergmann" <arnd@arndb.de>, "Olof Johansson" <olof@lixom.net>, "Marek Szyprowski" <m.szyprowski@samsung.com>, "Mark Brown" <broonie@kernel.org>, "Heiko Stübner" <heiko@sntech.de>, "Tomasz Figa" <tomasz.figa@gmail.com> Subject: Re: [PATCH 18/19] ARM: SAMSUNG: Remove remaining legacy code Date: Wed, 16 Jul 2014 11:47:42 +0200 [thread overview] Message-ID: <1405504062.4408.11.camel@x220> (raw) In-Reply-To: <1404496099-26708-19-git-send-email-t.figa@samsung.com> On Fri, 2014-07-04 at 19:48 +0200, Tomasz Figa wrote: > After refactoring suspend/resume, which was last part with dependencies > on legacy code, all Kconfig symbols related to Samsung ATAGS support can > be deselected and more unused code removed. This includes most of s5p-* > code as well, as s5pv210 was their last user. > > Signed-off-by: Tomasz Figa <t.figa@samsung.com> > --- This one also landed in today's linux-next (ie, next-20140716). >[...] > diff --git a/arch/arm/mach-s5pv210/include/mach/gpio.h b/arch/arm/mach-s5pv210/include/mach/gpio.h > deleted file mode 100644 > index 6c8b903..0000000 This removed the single case were CONFIG_SAMSUNG_GPIO_EXTRA was used. >[...] > diff --git a/arch/arm/plat-samsung/Kconfig b/arch/arm/plat-samsung/Kconfig >[...] > @@ -64,31 +51,6 @@ config SAMSUNG_ATAGS > > if SAMSUNG_ATAGS > > -# options for IRQ support > - > -config S5P_IRQ > - def_bool ARCH_S5PV210 > - help > - Support common interrupt part for ARCH_S5P SoCs > - > -config S5P_EXT_INT > - bool > - help > - Use the external interrupts (other than GPIO interrupts.) > - > -config S5P_GPIO_INT > - bool > - help > - Common code for the GPIO interrupts (other than external interrupts.) > - > -# options for gpio configuration support > - > -config S5P_GPIO_DRVSTR > - bool > - help > - Internal configuration to get and set correct GPIO driver strength > - helper > - > config SAMSUNG_GPIO_EXTRA > int "Number of additional GPIO pins" > default 128 if SAMSUNG_GPIO_EXTRA128 So a second order effect is that SAMSUNG_GPIO_EXTRA can now be removed. Which implies that SAMSUNG_GPIO_EXTRA64 and SAMSUNG_GPIO_EXTRA128 can also be removed. Should I submit the trivial patch to do that or is a patch already queued somewhere? Paul Bolle
WARNING: multiple messages have this Message-ID (diff)
From: pebolle@tiscali.nl (Paul Bolle) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 18/19] ARM: SAMSUNG: Remove remaining legacy code Date: Wed, 16 Jul 2014 11:47:42 +0200 [thread overview] Message-ID: <1405504062.4408.11.camel@x220> (raw) In-Reply-To: <1404496099-26708-19-git-send-email-t.figa@samsung.com> On Fri, 2014-07-04 at 19:48 +0200, Tomasz Figa wrote: > After refactoring suspend/resume, which was last part with dependencies > on legacy code, all Kconfig symbols related to Samsung ATAGS support can > be deselected and more unused code removed. This includes most of s5p-* > code as well, as s5pv210 was their last user. > > Signed-off-by: Tomasz Figa <t.figa@samsung.com> > --- This one also landed in today's linux-next (ie, next-20140716). >[...] > diff --git a/arch/arm/mach-s5pv210/include/mach/gpio.h b/arch/arm/mach-s5pv210/include/mach/gpio.h > deleted file mode 100644 > index 6c8b903..0000000 This removed the single case were CONFIG_SAMSUNG_GPIO_EXTRA was used. >[...] > diff --git a/arch/arm/plat-samsung/Kconfig b/arch/arm/plat-samsung/Kconfig >[...] > @@ -64,31 +51,6 @@ config SAMSUNG_ATAGS > > if SAMSUNG_ATAGS > > -# options for IRQ support > - > -config S5P_IRQ > - def_bool ARCH_S5PV210 > - help > - Support common interrupt part for ARCH_S5P SoCs > - > -config S5P_EXT_INT > - bool > - help > - Use the external interrupts (other than GPIO interrupts.) > - > -config S5P_GPIO_INT > - bool > - help > - Common code for the GPIO interrupts (other than external interrupts.) > - > -# options for gpio configuration support > - > -config S5P_GPIO_DRVSTR > - bool > - help > - Internal configuration to get and set correct GPIO driver strength > - helper > - > config SAMSUNG_GPIO_EXTRA > int "Number of additional GPIO pins" > default 128 if SAMSUNG_GPIO_EXTRA128 So a second order effect is that SAMSUNG_GPIO_EXTRA can now be removed. Which implies that SAMSUNG_GPIO_EXTRA64 and SAMSUNG_GPIO_EXTRA128 can also be removed. Should I submit the trivial patch to do that or is a patch already queued somewhere? Paul Bolle
next prev parent reply other threads:[~2014-07-16 9:47 UTC|newest] Thread overview: 141+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-07-04 17:48 [PATCH 00/19] ARM: SAMSUNG: S5PV210 platform clean-up Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 01/19] clk: samsung: Add clock driver for S5PV210 and compatible SoCs Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 02/19] ARM: s5pv210: Migrate clock handling to Common Clock Framework Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 03/19] cpufreq: s3c24xx: Remove some dead code Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 04/19] serial: samsung: Remove support for legacy clock code Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-16 9:27 ` Paul Bolle 2014-07-16 9:27 ` Paul Bolle 2014-07-16 12:43 ` Tomasz Figa 2014-07-16 12:43 ` Tomasz Figa 2014-07-16 14:26 ` Paul Bolle 2014-07-16 14:26 ` Paul Bolle 2014-07-16 14:35 ` Paul Bolle 2014-07-16 14:35 ` Paul Bolle 2014-07-16 14:43 ` Tomasz Figa 2014-07-16 14:43 ` Tomasz Figa 2014-09-04 9:39 ` Paul Bolle 2014-09-04 9:39 ` Paul Bolle 2014-09-04 9:42 ` Paul Bolle 2014-09-04 9:42 ` Paul Bolle 2014-07-04 17:48 ` [PATCH 05/19] ARM: SAMSUNG: Remove " Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 06/19] clk: samsung: Add S5PV210 Audio Subsystem clock driver Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 07/19] phy: Add support for S5PV210 to the Exynos USB 2.0 PHY driver Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 08/19] ARM: s5pv210: Add board file for boot using Device Tree Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 09/19] ARM: Samsung: DT: Add Device tree for s5pv210 Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 10/19] ARM: Samsung: DT: Add Device tree for S5PC110/S5PV210 Boards Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 11/19] ARM: S5PV210: Remove support for board files Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-16 10:04 ` Paul Bolle 2014-07-16 10:04 ` Paul Bolle 2014-07-16 12:53 ` Tomasz Figa 2014-07-16 12:53 ` Tomasz Figa 2014-09-04 10:05 ` Paul Bolle 2014-09-04 10:05 ` Paul Bolle 2014-09-04 16:02 ` Arnd Bergmann 2014-09-04 16:02 ` Arnd Bergmann 2014-09-05 12:04 ` Paul Bolle 2014-09-05 12:04 ` Paul Bolle 2014-09-05 12:28 ` Arnd Bergmann 2014-09-05 12:28 ` Arnd Bergmann 2014-09-18 9:43 ` [PATCH] ASoC: samsung: Remove goni or aquila with the WM8994 Paul Bolle 2014-09-18 9:43 ` Paul Bolle 2014-09-18 17:57 ` Mark Brown 2014-09-18 17:57 ` Mark Brown 2014-09-18 21:57 ` Paul Bolle 2014-09-18 21:57 ` Paul Bolle 2014-09-23 1:04 ` Mark Brown 2014-09-23 1:04 ` Mark Brown 2014-09-24 20:27 ` Paul Bolle 2014-09-24 20:27 ` Paul Bolle 2014-09-18 10:42 ` [PATCH] ASoC: samsung: Remove PCM support for WM8580 on SMDK Paul Bolle 2014-09-18 10:42 ` Paul Bolle 2014-09-18 17:59 ` Mark Brown 2014-09-18 17:59 ` Mark Brown 2014-09-18 17:59 ` Mark Brown 2014-07-04 17:48 ` [PATCH 12/19] ARM: S5PV210: Untie PM support from legacy code Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 13/19] ARM: s5pv210: move debug-macro.S into the common space Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-15 23:54 ` Kukjin Kim 2014-07-15 23:54 ` Kukjin Kim 2014-07-16 0:53 ` Kukjin Kim 2014-07-16 0:53 ` Kukjin Kim 2014-07-16 0:56 ` Tomasz Figa 2014-07-16 0:56 ` Tomasz Figa 2014-07-18 19:38 ` Kukjin Kim 2014-07-18 19:38 ` Kukjin Kim 2014-07-18 23:25 ` Tomasz Figa 2014-07-18 23:25 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 14/19] ARM: s5pv210: Register cpufreq platform device Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 15/19] cpufreq: s5pv210: Make the driver multiplatform aware Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 16/19] ARM: s5pv210: Enable multi-platform build support Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` [PATCH 17/19] gpio: samsung: Remove legacy support of S5PV210 Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-15 23:52 ` Kukjin Kim 2014-07-15 23:52 ` Kukjin Kim 2014-07-04 17:48 ` [PATCH 18/19] ARM: SAMSUNG: Remove remaining legacy code Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 20:23 ` Arnd Bergmann 2014-07-04 20:23 ` Arnd Bergmann 2014-07-08 14:10 ` Tomasz Figa 2014-07-08 14:10 ` Tomasz Figa 2014-07-16 9:47 ` Paul Bolle [this message] 2014-07-16 9:47 ` Paul Bolle 2014-07-16 12:56 ` Tomasz Figa 2014-07-16 12:56 ` Tomasz Figa 2014-07-17 10:10 ` Paul Bolle 2014-07-17 10:10 ` Paul Bolle 2014-07-17 10:12 ` Paul Bolle 2014-07-17 10:12 ` Paul Bolle 2014-07-16 10:15 ` Paul Bolle 2014-07-16 10:15 ` Paul Bolle 2014-07-16 12:58 ` Tomasz Figa 2014-07-16 12:58 ` Tomasz Figa 2014-09-04 10:16 ` Paul Bolle 2014-09-04 10:16 ` Paul Bolle 2014-09-18 19:29 ` Paul Bolle 2014-09-18 19:29 ` Paul Bolle 2014-09-18 19:33 ` Tomasz Figa 2014-09-18 19:33 ` Tomasz Figa 2014-07-16 10:24 ` Paul Bolle 2014-07-16 10:24 ` Paul Bolle 2014-07-16 13:00 ` Tomasz Figa 2014-07-16 13:00 ` Tomasz Figa 2014-09-04 10:08 ` Paul Bolle 2014-09-04 10:08 ` Paul Bolle 2014-09-04 16:03 ` Arnd Bergmann 2014-09-04 16:03 ` Arnd Bergmann 2014-09-18 8:57 ` [PATCH] gpio: samsung: Remove remaining check for CONFIG_S5P_GPIO_DRVSTR Paul Bolle 2014-09-18 19:34 ` Tomasz Figa 2014-09-23 15:42 ` Linus Walleij 2014-07-04 17:48 ` [PATCH 19/19] clk: samsung: s5pv210: Remove legacy board support Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:48 ` Tomasz Figa 2014-07-04 17:53 ` [PATCH 00/19] ARM: SAMSUNG: S5PV210 platform clean-up Tomasz Figa 2014-07-04 17:53 ` Tomasz Figa 2014-07-04 20:15 ` Arnd Bergmann 2014-07-04 20:15 ` Arnd Bergmann 2014-07-12 22:57 ` Kukjin Kim 2014-07-12 22:57 ` 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=1405504062.4408.11.camel@x220 \ --to=pebolle@tiscali.nl \ --cc=arnd@arndb.de \ --cc=broonie@kernel.org \ --cc=heiko@sntech.de \ --cc=kgene.kim@samsung.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-samsung-soc@vger.kernel.org \ --cc=m.szyprowski@samsung.com \ --cc=olof@lixom.net \ --cc=t.figa@samsung.com \ --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: linkBe 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.