From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kukjin Kim Subject: Re: [PATCH v3 00/12] ARM: EXYNOS: add support new EXYNOS5250 Date: Wed, 14 Mar 2012 00:08:17 -0700 Message-ID: <4F6043E1.2090507@samsung.com> References: <1331652643-12626-1-git-send-email-kgene.kim@samsung.com> <201203131622.09048.arnd@arndb.de> <20120314035728.GC2531@quad.lixom.net> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mail-gy0-f174.google.com ([209.85.160.174]:33666 "EHLO mail-gy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757594Ab2CNHIT (ORCPT ); Wed, 14 Mar 2012 03:08:19 -0400 Received: by ghrr11 with SMTP id r11so1465798ghr.19 for ; Wed, 14 Mar 2012 00:08:19 -0700 (PDT) In-Reply-To: <20120314035728.GC2531@quad.lixom.net> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Olof Johansson Cc: Arnd Bergmann , Kukjin Kim , linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org 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 , Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd. From mboxrd@z Thu Jan 1 00:00:00 1970 From: kgene.kim@samsung.com (Kukjin Kim) Date: Wed, 14 Mar 2012 00:08:17 -0700 Subject: [PATCH v3 00/12] ARM: EXYNOS: add support new EXYNOS5250 In-Reply-To: <20120314035728.GC2531@quad.lixom.net> References: <1331652643-12626-1-git-send-email-kgene.kim@samsung.com> <201203131622.09048.arnd@arndb.de> <20120314035728.GC2531@quad.lixom.net> Message-ID: <4F6043E1.2090507@samsung.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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 , Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd.