From: Kukjin Kim <kgene.kim@samsung.com>
To: Olof Johansson <olof@lixom.net>
Cc: Arnd Bergmann <arnd@arndb.de>, Kukjin Kim <kgene.kim@samsung.com>,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org
Subject: Re: [PATCH v3 00/12] ARM: EXYNOS: add support new EXYNOS5250
Date: Wed, 14 Mar 2012 00:08:17 -0700 [thread overview]
Message-ID: <4F6043E1.2090507@samsung.com> (raw)
In-Reply-To: <20120314035728.GC2531@quad.lixom.net>
On 03/13/12 20:57, Olof Johansson wrote:
> On Tue, Mar 13, 2012 at 04:22:08PM +0000, Arnd Bergmann wrote:
>> On Tuesday 13 March 2012, Kukjin Kim wrote:
>>>
>>> This patch adds support EXYNOS5250 SoC which has two A15 cores
>>> can be used on mobile and tablet devices.
>>
>> Looks basically ok to me. I've commented on patches 5 and 9 and
>> in both cases, I would either like to see the patch get dropped
>> or the changeset description expanded to give a good explanation
>> why the change is actually needed.
>
> Agree, looks quite reasonable. I had a few nits as well but nothing
> major.
>
Hi Arnd, Olof,
Thank you for kindly comments on this.
Let me address comments from you and re-submit in a couple of hours.
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
WARNING: multiple messages have this Message-ID (diff)
From: kgene.kim@samsung.com (Kukjin Kim)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3 00/12] ARM: EXYNOS: add support new EXYNOS5250
Date: Wed, 14 Mar 2012 00:08:17 -0700 [thread overview]
Message-ID: <4F6043E1.2090507@samsung.com> (raw)
In-Reply-To: <20120314035728.GC2531@quad.lixom.net>
On 03/13/12 20:57, Olof Johansson wrote:
> On Tue, Mar 13, 2012 at 04:22:08PM +0000, Arnd Bergmann wrote:
>> On Tuesday 13 March 2012, Kukjin Kim wrote:
>>>
>>> This patch adds support EXYNOS5250 SoC which has two A15 cores
>>> can be used on mobile and tablet devices.
>>
>> Looks basically ok to me. I've commented on patches 5 and 9 and
>> in both cases, I would either like to see the patch get dropped
>> or the changeset description expanded to give a good explanation
>> why the change is actually needed.
>
> Agree, looks quite reasonable. I had a few nits as well but nothing
> major.
>
Hi Arnd, Olof,
Thank you for kindly comments on this.
Let me address comments from you and re-submit in a couple of hours.
Thanks.
Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.
next prev parent reply other threads:[~2012-03-14 7:08 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-13 15:30 [PATCH v3 00/12] ARM: EXYNOS: add support new EXYNOS5250 Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 01/12] ARM: EXYNOS: to declare static for mach-exynos/common.c Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 02/12] ARM: EXYNOS: use exynos_init_uarts() instead of exynos4_init_uarts() Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 03/12] ARM: EXYNOS: add clock part for EXYNOS5250 SoC Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 04/12] ARM: EXYNOS: add initial setup-i2c0 for EXYNOS5 Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 05/12] ARM: EXYNOS: add support uart for EXYNOS4 and EXYNOS5 Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-13 16:13 ` Arnd Bergmann
2012-03-13 16:13 ` Arnd Bergmann
2012-03-14 7:06 ` Kukjin Kim
2012-03-14 7:06 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 06/12] ARM: EXYNOS: add support for EXYNOS5250 SoC Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 07/12] ARM: EXYNOS: add interrupt definitions for EXYNOS5250 Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-14 3:42 ` Olof Johansson
2012-03-14 3:42 ` Olof Johansson
2012-03-14 6:30 ` Kukjin Kim
2012-03-14 6:30 ` Kukjin Kim
2012-03-14 5:27 ` Thomas Abraham
2012-03-14 5:27 ` Thomas Abraham
2012-03-14 6:32 ` Kukjin Kim
2012-03-14 6:32 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 08/12] ARM: EXYNOS: support EINT for EXYNOS4 and EXYNOS5 Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-14 3:52 ` Olof Johansson
2012-03-14 3:52 ` Olof Johansson
2012-03-14 6:33 ` Kukjin Kim
2012-03-14 6:33 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 09/12] ARM: EXYNOS: add support get_core_count() for EXYNOS5250 Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-13 16:15 ` Arnd Bergmann
2012-03-13 16:15 ` Arnd Bergmann
2012-03-14 6:49 ` Kukjin Kim
2012-03-14 6:49 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 10/12] ARM: EXYNOS: add support ARCH_EXYNOS5 for EXYNOS5 SoCs Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 11/12] ARM: EXYNOS: add support device tree enabled board file for EXYNOS5 Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-13 15:30 ` [PATCH v3 12/12] ARM: dts: add initial dts file for EXYNOS5250, SMDK5250 Kukjin Kim
2012-03-13 15:30 ` Kukjin Kim
2012-03-14 3:56 ` Olof Johansson
2012-03-14 3:56 ` Olof Johansson
2012-03-14 6:43 ` Kukjin Kim
2012-03-14 6:43 ` Kukjin Kim
2012-03-13 16:22 ` [PATCH v3 00/12] ARM: EXYNOS: add support new EXYNOS5250 Arnd Bergmann
2012-03-13 16:22 ` Arnd Bergmann
2012-03-14 3:57 ` Olof Johansson
2012-03-14 3:57 ` Olof Johansson
2012-03-14 7:08 ` Kukjin Kim [this message]
2012-03-14 7:08 ` 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=4F6043E1.2090507@samsung.com \
--to=kgene.kim@samsung.com \
--cc=arnd@arndb.de \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=olof@lixom.net \
/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.