All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomasz Figa <tomasz.figa@gmail.com>
To: Viresh Kumar <viresh.kumar@linaro.org>, Tomasz Figa <t.figa@samsung.com>
Cc: linux-samsung-soc <linux-samsung-soc@vger.kernel.org>,
	"Kukjin Kim" <kgene.kim@samsung.com>,
	"Heiko Stübner" <heiko@sntech.de>,
	"Arnd Bergmann" <arnd@arndb.de>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Mark Brown" <broonie@kernel.org>,
	"Olof Johansson" <olof@lixom.net>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	"Marek Szyprowski" <m.szyprowski@samsung.com>,
	"Rafael J. Wysocki" <rjw@rjwysocki.net>
Subject: Re: [PATCH 00/19] ARM: SAMSUNG: S5PV210 platform clean-up
Date: Sat, 05 Jul 2014 13:00:43 +0200	[thread overview]
Message-ID: <53B7DADB.5080700@gmail.com> (raw)
In-Reply-To: <CAOh2x==dHrJkKw4h7y2nwYgFCv6s5ndU2DMiga302tzDaiaBWA@mail.gmail.com>

On 05.07.2014 06:46, Viresh Kumar wrote:
> On Fri, Jul 4, 2014 at 11:14 PM, Tomasz Figa <t.figa@samsung.com> 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
> 
>> Tomasz Figa (13):
>>   cpufreq: s3c24xx: Remove some dead code
>>   cpufreq: s5pv210: Make the driver multiplatform aware
> 
> Looks like you used --suppress-cc=all while sending these and none of
> CPUFreq/PM guys received it :(

Oops, I had a feeling that I has screwed up something with this series
and now we know what. Sorry for this. Will resend properly next Tuesday.

> 
> Anyways, for cpufreq part:
> 
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
> 

Thanks. Will add to the patches when resending.

Best regards,
Tomasz

WARNING: multiple messages have this Message-ID (diff)
From: tomasz.figa@gmail.com (Tomasz Figa)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 00/19] ARM: SAMSUNG: S5PV210 platform clean-up
Date: Sat, 05 Jul 2014 13:00:43 +0200	[thread overview]
Message-ID: <53B7DADB.5080700@gmail.com> (raw)
In-Reply-To: <CAOh2x==dHrJkKw4h7y2nwYgFCv6s5ndU2DMiga302tzDaiaBWA@mail.gmail.com>

On 05.07.2014 06:46, Viresh Kumar wrote:
> On Fri, Jul 4, 2014 at 11:14 PM, Tomasz Figa <t.figa@samsung.com> 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
> 
>> Tomasz Figa (13):
>>   cpufreq: s3c24xx: Remove some dead code
>>   cpufreq: s5pv210: Make the driver multiplatform aware
> 
> Looks like you used --suppress-cc=all while sending these and none of
> CPUFreq/PM guys received it :(

Oops, I had a feeling that I has screwed up something with this series
and now we know what. Sorry for this. Will resend properly next Tuesday.

> 
> Anyways, for cpufreq part:
> 
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
> 

Thanks. Will add to the patches when resending.

Best regards,
Tomasz

  reply	other threads:[~2014-07-05 11:01 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-04 17:44 [PATCH 00/19] ARM: SAMSUNG: S5PV210 platform clean-up Tomasz Figa
2014-07-04 17:44 ` Tomasz Figa
2014-07-05  4:46 ` Viresh Kumar
2014-07-05  4:46   ` Viresh Kumar
2014-07-05 11:00   ` Tomasz Figa [this message]
2014-07-05 11:00     ` Tomasz Figa
2014-07-05 11:05     ` Heiko Stübner
2014-07-05 11:05       ` Heiko Stübner
  -- strict thread matches above, loose matches on Subject: below --
2014-07-04 17:46 Tomasz Figa
2014-07-04 17:46 ` Tomasz Figa
2014-07-04 17:48 Tomasz Figa
2014-07-04 17:48 ` Tomasz Figa
2014-07-04 17:53 ` 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=53B7DADB.5080700@gmail.com \
    --to=tomasz.figa@gmail.com \
    --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=rjw@rjwysocki.net \
    --cc=t.figa@samsung.com \
    --cc=viresh.kumar@linaro.org \
    /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.