From: Kukjin Kim <kgene.kim@samsung.com>
To: Arnd Bergmann <arnd@arndb.de>
Cc: linux-arm-kernel@lists.infradead.org,
"Tomasz Figa" <t.figa@samsung.com>,
linux-samsung-soc@vger.kernel.org,
"Kukjin Kim" <kgene.kim@samsung.com>,
"Heiko Stübner" <heiko@sntech.de>,
linux-kernel@vger.kernel.org,
"Tomasz Figa" <tomasz.figa@gmail.com>,
"Mark Brown" <broonie@kernel.org>,
"Olof Johansson" <olof@lixom.net>,
"Marek Szyprowski" <m.szyprowski@samsung.com>
Subject: Re: [PATCH 00/19] ARM: SAMSUNG: S5PV210 platform clean-up
Date: Sun, 13 Jul 2014 07:57:21 +0900 [thread overview]
Message-ID: <53C1BD51.9020500@samsung.com> (raw)
In-Reply-To: <12714831.uRzVr7yNEz@wuerfel>
On 07/05/14 05:15, Arnd Bergmann wrote:
> On Friday 04 July 2014 19:48:00 Tomasz Figa wrote:
>> This huge series is a (hopefully final) attempt to convert Samsung S5PV210
>> into a DT-only and multiplatform-aware platform. It consists of several
>> steps to gradually replace legacy code with one meeting current standards
>> and then finally remove unused remnants.
>>
>> Patches 1-6 migrate the platform to fully use the Common Clock Framework
>> and get rid of legacy private clock code, of which s5pv210 was the last
>> user. Then patch 7 adds generic PHY driver for USB PHY on S5PV210 to allow
>> USB to be supported when using DT. Further three patches (8-10) add DT
>> support for mach-s5pv210 and necessary DT sources for currently supported
>> boards. Patch 11 removes board files and code directly related to them,
>> effectively making s5pv210 a DT-only platform. In next step, patches 12-15
>> prepare remaining code for multiplatform enablement, which is finally done
>> in patch 16. Patches 17-19 are a final clean-up, which remove a lot of
>> unused code left after making the last S5P platform DT-only.
>>
>> Build tested patch by patch on following configs:
>> - s3c2410_defconfig (with DT support enabled),
>> - s3c6400_defconfig (with DT support enabled),
>> - s5pv210_defconfig (with DT support enabled after patches adding it),
>> - exynos_defconfig,
>> - exynos_defconfig with S5PV210 enabled in multiplatform configuration.
>>
>> Boot tested on s5pv210-goni board.
>>
>> Note that support for smdkc110, smdkv210 and torbreck boards is provided
>> by moving data from existing board files to new device tree sources. No
>> testing was performed due to mentioned board not being available anymore.
>> However I believe we agreed on this approach, because apparently there
>> are no active users of them. See the RFC from August 2013 asking for
>> removal of the whole platform [1].
>>
>> [1] http://www.mail-archive.com/linux-samsung-soc%40vger.kernel.org/msg21882.html
>>
>>
> Great work, everything
>
> Acked-by: Arnd Bergmann<arnd@arndb.de>
>
> I hope we can get this into linux-next soon so we can do some more
> cleanups on top.
Sorry for a little bit late applying, I just wanted to get ack from
driver maintainers but... anyway I'll apply this series so you may see
this on Monday in -next :)
Tomasz, many thanks for your effort and time.
- Kukjin
next prev parent reply other threads:[~2014-07-12 22:57 UTC|newest]
Thread overview: 72+ 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 ` [PATCH 01/19] clk: samsung: Add clock driver for S5PV210 and compatible SoCs 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 ` [PATCH 03/19] cpufreq: s3c24xx: Remove some dead code Tomasz Figa
2014-07-04 17:48 ` [PATCH 04/19] serial: samsung: Remove support for legacy clock code Tomasz Figa
2014-07-16 9:27 ` Paul Bolle
2014-07-16 12:43 ` Tomasz Figa
2014-07-16 14:26 ` Paul Bolle
2014-07-16 14:35 ` Paul Bolle
2014-07-16 14:43 ` Tomasz Figa
2014-09-04 9:39 ` 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 ` [PATCH 06/19] clk: samsung: Add S5PV210 Audio Subsystem clock driver 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 ` [PATCH 08/19] ARM: s5pv210: Add board file for boot using Device Tree 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 ` [PATCH 10/19] ARM: Samsung: DT: Add Device tree for S5PC110/S5PV210 Boards Tomasz Figa
2014-07-04 17:48 ` [PATCH 11/19] ARM: S5PV210: Remove support for board files Tomasz Figa
2014-07-16 10:04 ` Paul Bolle
2014-07-16 12:53 ` Tomasz Figa
2014-09-04 10:05 ` Paul Bolle
2014-09-04 16:02 ` Arnd Bergmann
2014-09-05 12:04 ` Paul Bolle
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 17:57 ` Mark Brown
2014-09-18 21:57 ` Paul Bolle
2014-09-23 1:04 ` Mark Brown
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 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 ` [PATCH 13/19] ARM: s5pv210: move debug-macro.S into the common space Tomasz Figa
2014-07-15 23:54 ` Kukjin Kim
2014-07-16 0:53 ` Kukjin Kim
2014-07-16 0:56 ` Tomasz Figa
2014-07-18 19:38 ` Kukjin Kim
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 ` [PATCH 15/19] cpufreq: s5pv210: Make the driver multiplatform aware Tomasz Figa
2014-07-04 17:48 ` [PATCH 16/19] ARM: s5pv210: Enable multi-platform build support Tomasz Figa
2014-07-04 17:48 ` [PATCH 17/19] gpio: samsung: Remove legacy support of S5PV210 Tomasz Figa
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 20:23 ` Arnd Bergmann
2014-07-08 14:10 ` Tomasz Figa
2014-07-16 9:47 ` Paul Bolle
2014-07-16 12:56 ` Tomasz Figa
2014-07-17 10:10 ` Paul Bolle
2014-07-17 10:12 ` Paul Bolle
2014-07-16 10:15 ` Paul Bolle
2014-07-16 12:58 ` Tomasz Figa
2014-09-04 10:16 ` Paul Bolle
2014-09-18 19:29 ` Paul Bolle
2014-09-18 19:33 ` Tomasz Figa
2014-07-16 10:24 ` Paul Bolle
2014-07-16 13:00 ` Tomasz Figa
2014-09-04 10:08 ` Paul Bolle
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:53 ` [PATCH 00/19] ARM: SAMSUNG: S5PV210 platform clean-up Tomasz Figa
2014-07-04 20:15 ` Arnd Bergmann
2014-07-12 22:57 ` Kukjin Kim [this message]
-- strict thread matches above, loose matches on Subject: below --
2014-07-04 17:46 Tomasz Figa
2014-07-04 17:44 Tomasz Figa
2014-07-05 4:46 ` Viresh Kumar
2014-07-05 11:00 ` Tomasz Figa
2014-07-05 11:05 ` Heiko Stübner
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=53C1BD51.9020500@samsung.com \
--to=kgene.kim@samsung.com \
--cc=arnd@arndb.de \
--cc=broonie@kernel.org \
--cc=heiko@sntech.de \
--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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox