[Qemu-devel] [PATCH for-2.8] include: Fix typos found by codespell

[Qemu-devel] [PATCH for-2.8] include: Fix typos found by codespell

[Stefan Weil]

Add also a missing parenthesis in a comment.

Signed-off-by: Stefan Weil <sw@weilnetz.de>
---
 include/hw/dma/xlnx_dpdma.h    | 3 ++-
 include/hw/pci-host/q35.h      | 2 +-
 include/hw/register.h          | 2 +-
 include/io/task.h              | 2 +-
 include/qapi/dealloc-visitor.h | 2 +-
 include/qemu/qht.h             | 2 +-
 include/qemu/xattr.h           | 2 +-
 7 files changed, 8 insertions(+), 7 deletions(-)

diff --git a/include/hw/dma/xlnx_dpdma.h b/include/hw/dma/xlnx_dpdma.h
index 664df28..7a304a5 100644
--- a/include/hw/dma/xlnx_dpdma.h
+++ b/include/hw/dma/xlnx_dpdma.h
@@ -53,7 +53,8 @@ typedef struct XlnxDPDMAState XlnxDPDMAState;
  *                             data to the buffer specified by
  *                             dpdma_set_host_data_location().
  *
- * Returns The number of bytes transfered by the DPDMA or 0 if an error occured.
+ * Returns The number of bytes transferred by the DPDMA
+ *         or 0 if an error occurred.
  *
  * @s The DPDMA state.
  * @channel The channel to start.
diff --git a/include/hw/pci-host/q35.h b/include/hw/pci-host/q35.h
index 94486fd..53b6760 100644
--- a/include/hw/pci-host/q35.h
+++ b/include/hw/pci-host/q35.h
@@ -180,7 +180,7 @@ typedef struct Q35PCIHost {
 uint64_t mch_mcfg_base(void);
 
 /*
- * Arbitary but unique BNF number for IOAPIC device.
+ * Arbitrary but unique BNF number for IOAPIC device.
  *
  * TODO: make sure there would have no conflict with real PCI bus
  */
diff --git a/include/hw/register.h b/include/hw/register.h
index 8c12233..5b6dc32 100644
--- a/include/hw/register.h
+++ b/include/hw/register.h
@@ -92,7 +92,7 @@ struct RegisterInfo {
  * This structure is used to group all of the individual registers which are
  * modeled using the RegisterInfo structure.
  *
- * @r is an aray containing of all the relevent RegisterInfo structures.
+ * @r is an array containing of all the relevant RegisterInfo structures.
  *
  * @num_elements is the number of elements in the array r
  *
diff --git a/include/io/task.h b/include/io/task.h
index 42028cb..6810842 100644
--- a/include/io/task.h
+++ b/include/io/task.h
@@ -231,7 +231,7 @@ void qio_task_complete(QIOTask *task);
  *
  * Mark the operation as failed, with @err providing
  * details about the failure. The @err may be freed
- * afer the function returns, as the notification
+ * after the function returns, as the notification
  * callback is invoked synchronously. The @task will
  * be freed when this call completes.
  */
diff --git a/include/qapi/dealloc-visitor.h b/include/qapi/dealloc-visitor.h
index b3e5c85..c36715f 100644
--- a/include/qapi/dealloc-visitor.h
+++ b/include/qapi/dealloc-visitor.h
@@ -19,7 +19,7 @@
 typedef struct QapiDeallocVisitor QapiDeallocVisitor;
 
 /*
- * The dealloc visitor is primarly used only by generated
+ * The dealloc visitor is primarily used only by generated
  * qapi_free_FOO() functions, and is the only visitor designed to work
  * correctly in the face of a partially-constructed QAPI tree.
  */
diff --git a/include/qemu/qht.h b/include/qemu/qht.h
index 311139b..56c2c77 100644
--- a/include/qemu/qht.h
+++ b/include/qemu/qht.h
@@ -72,7 +72,7 @@ void qht_destroy(struct qht *ht);
  * In case of successful operation, smp_wmb() is implied before the pointer is
  * inserted into the hash table.
  *
- * Returns true on sucess.
+ * Returns true on success.
  * Returns false if the @p-@hash pair already exists in the hash table.
  */
 bool qht_insert(struct qht *ht, void *p, uint32_t hash);
diff --git a/include/qemu/xattr.h b/include/qemu/xattr.h
index 83cf98c..a83fe8e 100644
--- a/include/qemu/xattr.h
+++ b/include/qemu/xattr.h
@@ -14,7 +14,7 @@
 #define QEMU_XATTR_H
 
 /*
- * Modern distributions (e.g. Fedora 15, have no libattr.so, place attr.h
+ * Modern distributions (e.g. Fedora 15), have no libattr.so, place attr.h
  * in /usr/include/sys, and don't have ENOATTR.
  */
 
-- 
2.10.2

Re: [Qemu-devel] [PATCH for-2.8] include: Fix typos found by codespell

[Alistair Francis]

On Sat, Nov 19, 2016 at 11:47 AM, Stefan Weil <sw@weilnetz.de> wrote:
> Add also a missing parenthesis in a comment.
>
> Signed-off-by: Stefan Weil <sw@weilnetz.de>

Acked-by: Alistair Francis <alistair.francis@xilinx.com>

Thanks,

Alistair

> ---
>  include/hw/dma/xlnx_dpdma.h    | 3 ++-
>  include/hw/pci-host/q35.h      | 2 +-
>  include/hw/register.h          | 2 +-
>  include/io/task.h              | 2 +-
>  include/qapi/dealloc-visitor.h | 2 +-
>  include/qemu/qht.h             | 2 +-
>  include/qemu/xattr.h           | 2 +-
>  7 files changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/include/hw/dma/xlnx_dpdma.h b/include/hw/dma/xlnx_dpdma.h
> index 664df28..7a304a5 100644
> --- a/include/hw/dma/xlnx_dpdma.h
> +++ b/include/hw/dma/xlnx_dpdma.h
> @@ -53,7 +53,8 @@ typedef struct XlnxDPDMAState XlnxDPDMAState;
>   *                             data to the buffer specified by
>   *                             dpdma_set_host_data_location().
>   *
> - * Returns The number of bytes transfered by the DPDMA or 0 if an error occured.
> + * Returns The number of bytes transferred by the DPDMA
> + *         or 0 if an error occurred.
>   *
>   * @s The DPDMA state.
>   * @channel The channel to start.
> diff --git a/include/hw/pci-host/q35.h b/include/hw/pci-host/q35.h
> index 94486fd..53b6760 100644
> --- a/include/hw/pci-host/q35.h
> +++ b/include/hw/pci-host/q35.h
> @@ -180,7 +180,7 @@ typedef struct Q35PCIHost {
>  uint64_t mch_mcfg_base(void);
>
>  /*
> - * Arbitary but unique BNF number for IOAPIC device.
> + * Arbitrary but unique BNF number for IOAPIC device.
>   *
>   * TODO: make sure there would have no conflict with real PCI bus
>   */
> diff --git a/include/hw/register.h b/include/hw/register.h
> index 8c12233..5b6dc32 100644
> --- a/include/hw/register.h
> +++ b/include/hw/register.h
> @@ -92,7 +92,7 @@ struct RegisterInfo {
>   * This structure is used to group all of the individual registers which are
>   * modeled using the RegisterInfo structure.
>   *
> - * @r is an aray containing of all the relevent RegisterInfo structures.
> + * @r is an array containing of all the relevant RegisterInfo structures.
>   *
>   * @num_elements is the number of elements in the array r
>   *
> diff --git a/include/io/task.h b/include/io/task.h
> index 42028cb..6810842 100644
> --- a/include/io/task.h
> +++ b/include/io/task.h
> @@ -231,7 +231,7 @@ void qio_task_complete(QIOTask *task);
>   *
>   * Mark the operation as failed, with @err providing
>   * details about the failure. The @err may be freed
> - * afer the function returns, as the notification
> + * after the function returns, as the notification
>   * callback is invoked synchronously. The @task will
>   * be freed when this call completes.
>   */
> diff --git a/include/qapi/dealloc-visitor.h b/include/qapi/dealloc-visitor.h
> index b3e5c85..c36715f 100644
> --- a/include/qapi/dealloc-visitor.h
> +++ b/include/qapi/dealloc-visitor.h
> @@ -19,7 +19,7 @@
>  typedef struct QapiDeallocVisitor QapiDeallocVisitor;
>
>  /*
> - * The dealloc visitor is primarly used only by generated
> + * The dealloc visitor is primarily used only by generated
>   * qapi_free_FOO() functions, and is the only visitor designed to work
>   * correctly in the face of a partially-constructed QAPI tree.
>   */
> diff --git a/include/qemu/qht.h b/include/qemu/qht.h
> index 311139b..56c2c77 100644
> --- a/include/qemu/qht.h
> +++ b/include/qemu/qht.h
> @@ -72,7 +72,7 @@ void qht_destroy(struct qht *ht);
>   * In case of successful operation, smp_wmb() is implied before the pointer is
>   * inserted into the hash table.
>   *
> - * Returns true on sucess.
> + * Returns true on success.
>   * Returns false if the @p-@hash pair already exists in the hash table.
>   */
>  bool qht_insert(struct qht *ht, void *p, uint32_t hash);
> diff --git a/include/qemu/xattr.h b/include/qemu/xattr.h
> index 83cf98c..a83fe8e 100644
> --- a/include/qemu/xattr.h
> +++ b/include/qemu/xattr.h
> @@ -14,7 +14,7 @@
>  #define QEMU_XATTR_H
>
>  /*
> - * Modern distributions (e.g. Fedora 15, have no libattr.so, place attr.h
> + * Modern distributions (e.g. Fedora 15), have no libattr.so, place attr.h
>   * in /usr/include/sys, and don't have ENOATTR.
>   */
>
> --
> 2.10.2
>
>

Re: [Qemu-devel] [PATCH for-2.8] include: Fix typos found by codespell

[Michael Tokarev]

19.11.2016 22:47, Stefan Weil wrote:
> Add also a missing parenthesis in a comment.
..

Applied to -trivial, thank you!

/mjt