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

From: Kaneda, Erik
Date: Mon Jul 27 2020 - 12:49:36 EST




> -----Original Message-----
> From: Rafael J. Wysocki <rafael@xxxxxxxxxx>
> Sent: Monday, July 27, 2020 5:44 AM
> To: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>; Kaneda, Erik
> <erik.kaneda@xxxxxxxxx>
> Cc: Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>; Rafael J.
> Wysocki <rjw@xxxxxxxxxxxxx>; Len Brown <lenb@xxxxxxxxxx>; ACPI Devel
> Maling List <linux-acpi@xxxxxxxxxxxxxxx>; Moore, Robert
> <robert.moore@xxxxxxxxx>
> Subject: Re: [PATCH] ACPI: actypes.h: drop a duplicated word
>
> On Sun, Jul 19, 2020 at 2:27 AM Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> wrote:
> >
> > Drop the repeated word "an" in a comment.
> >
> > Signed-off-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
> > Cc: "Rafael J. Wysocki" <rjw@xxxxxxxxxxxxx>
> > Cc: Len Brown <lenb@xxxxxxxxxx>
> > Cc: linux-acpi@xxxxxxxxxxxxxxx
> > ---
> > 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