From mboxrd@z Thu Jan 1 00:00:00 1970 From: Albert ARIBAUD Date: Sat, 1 Jun 2013 12:06:24 +0200 Subject: [U-Boot] [PATCH 1/6] ARM: add secure monitor handler to switch to non-secure state In-Reply-To: <20130531235009.GC17432@ubuntu> References: <1367846270-1827-1-git-send-email-andre.przywara@linaro.org> <1367846270-1827-2-git-send-email-andre.przywara@linaro.org> <20130531010213.GA3189@ubuntu> <51A86C04.3020500@linaro.org> <20130531235009.GC17432@ubuntu> Message-ID: <20130601120624.330cdfb0@lilith> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Christoffer, On Fri, 31 May 2013 16:50:09 -0700, Christoffer Dall wrote: > On Fri, May 31, 2013 at 11:23:16AM +0200, Andre Przywara wrote: > > On 05/31/2013 03:02 AM, Christoffer Dall wrote: > > This is added later in 5/6. For reviewing purposes I split the > > patches up to do the non-secure switch only first. Later I add the > > bits to actually go to HYP mode. > > The splitting of patches is fine, but it would be helpful to explain the > scope a little more in the commit text perhaps, maybe I'm just being > silly. That's the point where a cover letter will be very useful in V2. Also, speaking of V2 (and, presumably, later versions), that's where patman will be useful (see tools/patman/README) as it allows one's local branch to contain not only things like cover letter, but also patch history, which will be a must IMO for this series. Amicalement, -- Albert.