From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matt Fleming Subject: Re: [PATCH v3] efi/arm64: handle missing virtual mapping for UEFI System Table Date: Fri, 18 Jul 2014 22:18:20 +0100 Message-ID: <20140718211820.GC27099@console-pimps.org> References: <1404487509-26084-1-git-send-email-ard.biesheuvel@linaro.org> <1404492756.19665.57.camel@deneb.redhat.com> <20140704170110.GM16404@arm.com> <09C85B92-02BB-45C4-A461-D4E0CD4F7203@arm.com> <20140707201103.GB27474@console-pimps.org> <20140718193141.GA27099@console-pimps.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Ard Biesheuvel Cc: "matt.fleming-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org" , "linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "leif.lindholm-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org" List-Id: linux-efi@vger.kernel.org On Fri, 18 Jul, at 10:11:13PM, Ard Biesheuvel wrote: > > Thanks, looks lovely. The only thing I notice now is that the commit > message from the patch below is missing the second line, which used to > read > > "#including .c files to building the " > > or something to that effect. Obviously, the leading hash sign resulted > in this line having been dropped, so this is something that has been > in the patches all along. Oops, that's my over-zealous patch munging. Well spotted! > IMO there is really no point in bothering fixing this, but perhaps > your inner grammar nazi disagrees. He's a tyrant, I fixed it up, mainly because I broke it in the first place. -- Matt Fleming, Intel Open Source Technology Center