From mboxrd@z Thu Jan  1 00:00:00 1970
From: kgene.kim@samsung.com (Kukjin Kim)
Date: Fri, 19 Aug 2011 14:24:09 +0900
Subject: [PATCH 0/5] Miscellaneous s3c64xx fixes.
In-Reply-To: <2042356.rlHOh83GOI@flatron>
References: <5661373.tZL1ZnPyVO@flatron> <2691749.ciX6KCIc5Y@flatron>
	<000801cc596f$a8435a50$f8ca0ef0$%kim@samsung.com>
	<2042356.rlHOh83GOI@flatron>
Message-ID: <000a01cc5e30$3a8508e0$af8f1aa0$%kim@samsung.com>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

Tomasz Figa wrote:
> 
> On Saturday 13 of August 2011 at 13:15:36, Kukjin Kim wrote:
> > Tomasz Figa wrote:
> > >
> > > Anyone?
> > >
> > Hi Tomasz Figa,
> >
> > Will be on this in the next week...
> >
> > Thanks.
> >
> > Best regards,
> > Kgene.
> > --
> > Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
> > SW Solution Development Team, Samsung Electronics Co., Ltd.
> >
> 
> Thanks a lot. I have plenty of other S3C64xx patches to be submitted, but
they
> are more feature additions not fixes, so I am waiting for these fixes to
be
> reviewed first.
> 
Hi,

Firstly, you have to re-submit whole series with your 'sign'. It is very
important.

And 2nd, 3rd and 4th patches look ok to me except missing your sign even
though I think, secure DMA mode should be setting in boot loader with trust
zone...

Please refer to my review on 5th patch.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.