From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([140.186.70.92]:37765) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1QZhFy-0000r8-Ip for qemu-devel@nongnu.org; Thu, 23 Jun 2011 06:35:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1QZhFw-0001ly-IV for qemu-devel@nongnu.org; Thu, 23 Jun 2011 06:35:30 -0400 MIME-Version: 1.0 In-Reply-To: <57FD980B-E6A3-4E1A-A83B-062B9954DB5C@web.de> References: <57FD980B-E6A3-4E1A-A83B-062B9954DB5C@web.de> Date: Thu, 23 Jun 2011 20:35:26 +1000 Message-ID: From: Matthew Fernandez Content-Type: multipart/alternative; boundary=20cf30334f25a87b4c04a65ea4fb Subject: Re: [Qemu-devel] [PATCH trivial] Fix comment typos in hw/armv7m.c List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?ISO-8859-1?Q?Andreas_F=E4rber?= Cc: qemu-trivial@nongnu.org, qemu-devel Developers , Paul Brook --20cf30334f25a87b4c04a65ea4fb Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 23 June 2011 20:22, Andreas F=E4rber wrote: > Am 23.06.2011 um 02:59 schrieb Matthew Fernandez: > > > Fix a couple of typos in comments. >> >> Signed-off-by: Matthew Fernandez >> ---- >> This patch doesn't touch any code and should (hopefully) not be >> controversial. There doesn't seem to be a maintainer for this file, >> but changes are trivial. >> >> >> diff --git a/hw/armv7m.c b/hw/armv7m.c >> index 72d010a..83f3393 100644 >> --- a/hw/armv7m.c >> +++ b/hw/armv7m.c >> @@ -4,7 +4,7 @@ >> * Copyright (c) 2006-2007 CodeSourcery. >> * Written by Paul Brook >> * >> - * This code is licenced under the GPL. >> + * This code is licensed under the GPL. >> > > Nack. Licence is British English, license American English. It's not wron= g. Licence is the noun in British English, while license in the noun in American English. License is the correct verb in both. Nevertheless, I don'= t think this is worth debating. If there's dissent I'm happy to drop the firs= t part of the patch. > > > */ >> >> #include "sysbus.h" >> @@ -14,7 +14,7 @@ >> >> /* Bitbanded IO. Each word corresponds to a single bit. */ >> >> -/* Get the byte address of the real memory for a bitband acess. */ >> +/* Get the byte address of the real memory for a bitband access. */ >> > > This one's okay. > > Andreas > > > static inline uint32_t bitband_addr(void * opaque, uint32_t addr) >> { >> uint32_t res; >> >> > --20cf30334f25a87b4c04a65ea4fb Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable

On 23 June 2011 20:22, Andreas F=E4rber = <andreas.fae= rber@web.de> wrote:
Am 23.06.2011 um 02:59 schrieb Matthew Fernandez:


Fix a couple of typos in comments.

Signed-off-by: Matthew Fernandez <matthew.fernandez@gmail.com>
----
This patch doesn't touch any code and should (hopefully) not be
controversial. There doesn't seem to be a maintainer for this file,
but changes are trivial.


diff --git a/hw/armv7m.c b/hw/armv7m.c
index 72d010a..83f3393 100644
--- a/hw/armv7m.c
+++ b/hw/armv7m.c
@@ -4,7 +4,7 @@
=A0* Copyright (c) 2006-2007 CodeSourcery.
=A0* Written by Paul Brook
=A0*
- * This code is licenced under the GPL.
+ * This code is licensed under the GPL.

Nack. Licence is British English, license American English. It's not wr= ong.
Licence is the noun in British English, while license= in the noun in American English. License is the correct verb in both. Neve= rtheless, I don't think this is worth debating. If there's dissent = I'm happy to drop the first part of the patch.


=A0*/

#include "sysbus.h"
@@ -14,7 +14,7 @@

/* Bitbanded IO. =A0Each word corresponds to a single bit. =A0*/

-/* Get the byte address of the real memory for a bitband acess. =A0*/
+/* Get the byte address of the real memory for a bitband access. =A0*/

This one's okay.

Andreas


static inline uint32_t bitband_addr(void * opaque, uint32_t addr)
{
=A0 =A0uint32_t res;



--20cf30334f25a87b4c04a65ea4fb--