From: Alon Levy <alevy@redhat.com>
To: Brad Hards <bradh@frogmouth.net>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] Minor spelling mistakes in hw/usb-ccid.c
Date: Wed, 6 Apr 2011 11:15:33 +0300 [thread overview]
Message-ID: <20110406081533.GD2263@playa.tlv.redhat.com> (raw)
In-Reply-To: <201104050754.04561.bradh@frogmouth.net>
On Tue, Apr 05, 2011 at 07:54:04AM +1000, Brad Hards wrote:
> Hi,
>
> While looking at David Gibson's build-fix for hw/usb-ccid.c, I noticed a spello
> in a comment on the following (unchanged) line.
>
> This patch fixes that, and a couple of other spellos, in that file.
ACK. Thanks.
>
> Brad
>
> - -
>
> From e24a4cfce56799e091ce75aafa20c697ec5fe685 Mon Sep 17 00:00:00 2001
> From: Brad Hards <bradh@frogmouth.net>
> Date: Tue, 5 Apr 2011 07:39:10 +1000
> Subject: [PATCH] usb-ccid: spelling fixes.
>
> Signed-off-by: Brad Hards <bradh@frogmouth.net>
> ---
> hw/usb-ccid.c | 8 ++++----
> 1 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/hw/usb-ccid.c b/hw/usb-ccid.c
> index 44156cc..079b4a2 100644
> --- a/hw/usb-ccid.c
> +++ b/hw/usb-ccid.c
> @@ -22,7 +22,7 @@
> * "Universal Serial Bus, Device Class: Smart Card"
> * Specification for Integrated Circuit(s) Cards Interface Devices
> *
> - * Endianess note: from the spec (1.3)
> + * Endianness note: from the spec (1.3)
> * "Fields that are larger than a byte are stored in little endian"
> *
> * KNOWN BUGS
> @@ -172,7 +172,7 @@ enum {
> CLOCK_STATUS_RUNNING = 0,
> /*
> * 0 - Clock Running, 1 - Clock stopped in State L, 2 - H,
> - * 3 - unkonwn state. rest are RFU
> + * 3 - unknown state. rest are RFU
> */
> };
>
> @@ -1200,7 +1200,7 @@ void ccid_card_card_error(CCIDCardState *card, uint64_t error)
> s->bmCommandStatus = COMMAND_STATUS_FAILED;
> s->last_answer_error = error;
> DPRINTF(s, 1, "VSC_Error: %" PRIX64 "\n", s->last_answer_error);
> - /* TODO: these error's should be more verbose and propogated to the guest.*/
> + /* TODO: these errors should be more verbose and propagated to the guest.*/
> /*
> * We flush all pending answers on CardRemove message in ccid-card-passthru,
> * so check that first to not trigger abort
> @@ -1319,7 +1319,7 @@ static void ccid_pre_save(void *opaque)
> if (s->dev.attached) {
> /*
> * Migrating an open device, ignore reconnection CHR_EVENT to avoid an
> - * erronous detach.
> + * erroneous detach.
> */
> s->migration_state = MIGRATION_MIGRATED;
> }
> --
> 1.7.1
>
>
next prev parent reply other threads:[~2011-04-06 8:15 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-04-04 21:54 [Qemu-devel] [PATCH] Minor spelling mistakes in hw/usb-ccid.c Brad Hards
2011-04-06 8:15 ` Alon Levy [this message]
2011-04-06 13:42 ` Stefan Hajnoczi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110406081533.GD2263@playa.tlv.redhat.com \
--to=alevy@redhat.com \
--cc=bradh@frogmouth.net \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).