[PATCH] ACPI: actypes.h: drop a duplicated word

[PATCH] ACPI: actypes.h: drop a duplicated word

[Randy Dunlap]

Drop the repeated word "an" in a comment.

Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
Cc: Len Brown <lenb@kernel.org>
Cc: linux-acpi@vger.kernel.org
---
 include/acpi/actypes.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--- linux-next-20200717.orig/include/acpi/actypes.h
+++ linux-next-20200717/include/acpi/actypes.h
@@ -824,7 +824,7 @@ typedef u8 acpi_adr_space_type;
  *
  * Note: A Data Table region is a special type of operation region
  * that has its own AML opcode. However, internally, the AML
- * interpreter simply creates an operation region with an an address
+ * interpreter simply creates an operation region with an address
  * space type of ACPI_ADR_SPACE_DATA_TABLE.
  */
 #define ACPI_ADR_SPACE_DATA_TABLE       (acpi_adr_space_type) 0x7E	/* Internal to ACPICA only */

Re: [PATCH] ACPI: actypes.h: drop a duplicated word

[Rafael J. Wysocki]

On Sun, Jul 19, 2020 at 2:27 AM Randy Dunlap <rdunlap@infradead.org> wrote:
>
> Drop the repeated word "an" in a comment.
>
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
> Cc: Len Brown <lenb@kernel.org>
> Cc: linux-acpi@vger.kernel.org
> ---
>  include/acpi/actypes.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- linux-next-20200717.orig/include/acpi/actypes.h
> +++ linux-next-20200717/include/acpi/actypes.h
> @@ -824,7 +824,7 @@ typedef u8 acpi_adr_space_type;
>   *
>   * Note: A Data Table region is a special type of operation region
>   * that has its own AML opcode. However, internally, the AML
> - * interpreter simply creates an operation region with an an address
> + * interpreter simply creates an operation region with an address
>   * space type of ACPI_ADR_SPACE_DATA_TABLE.
>   */
>  #define ACPI_ADR_SPACE_DATA_TABLE       (acpi_adr_space_type) 0x7E     /* Internal to ACPICA only */

This is an ACPICA header file, so it's better to route the change
through the upstream.

Erik, can you pick up this one, please?

RE: [PATCH] ACPI: actypes.h: drop a duplicated word

[Kaneda, Erik]

> -----Original Message-----
> From: Rafael J. Wysocki <rafael@kernel.org>
> Sent: Monday, July 27, 2020 5:44 AM
> To: Randy Dunlap <rdunlap@infradead.org>; Kaneda, Erik
> <erik.kaneda@intel.com>
> Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>; Rafael J.
> Wysocki <rjw@rjwysocki.net>; Len Brown <lenb@kernel.org>; ACPI Devel
> Maling List <linux-acpi@vger.kernel.org>; Moore, Robert
> <robert.moore@intel.com>
> Subject: Re: [PATCH] ACPI: actypes.h: drop a duplicated word
> 
> On Sun, Jul 19, 2020 at 2:27 AM Randy Dunlap <rdunlap@infradead.org>
> wrote:
> >
> > Drop the repeated word "an" in a comment.
> >
> > Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> > Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>
> > Cc: Len Brown <lenb@kernel.org>
> > Cc: linux-acpi@vger.kernel.org
> > ---
> >  include/acpi/actypes.h |    2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > --- linux-next-20200717.orig/include/acpi/actypes.h
> > +++ linux-next-20200717/include/acpi/actypes.h
> > @@ -824,7 +824,7 @@ typedef u8 acpi_adr_space_type;
> >   *
> >   * Note: A Data Table region is a special type of operation region
> >   * that has its own AML opcode. However, internally, the AML
> > - * interpreter simply creates an operation region with an an address
> > + * interpreter simply creates an operation region with an address
> >   * space type of ACPI_ADR_SPACE_DATA_TABLE.
> >   */
> >  #define ACPI_ADR_SPACE_DATA_TABLE       (acpi_adr_space_type) 0x7E
> /* Internal to ACPICA only */
> 
> This is an ACPICA header file, so it's better to route the change
> through the upstream.
> 
> Erik, can you pick up this one, please?

Done.

I made a pull request here: https://github.com/acpica/acpica/pull/618
It will be in the next ACPICA release.

Thanks!
Erik