Re: [PATCH v1 3/4] ArmPkg: SpellCheck: Update valid acronyms in ExtendedWords

["Laszlo Ersek" <lersek@redhat.com>]

Hi,

On 06/12/21 05:44, Kun Qin wrote:
> From: Sean Brogan <sean.brogan@microsoft.com>
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3445
> 
> Spellcheck was not covering all specified files due to CSpell v5 and
> Node v10 incompatibility of current CI pipeline configuration.
> 
> This change updates ExtendedWords for ArmPkg with valid acronyms to avoid
> potential spell errors.
> 
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Sami Mujawar <sami.mujawar@arm.com>
> 
> Signed-off-by: Kun Qin <kuqin12@gmail.com>
> Signed-off-by: Sean Brogan <sean.brogan@microsoft.com>
> ---
>  ArmPkg/ArmPkg.ci.yaml | 19 +++++++++++++++++++
>  1 file changed, 19 insertions(+)

I'm not an ArmPkg reviewer, so I'm not sure about the Cc.

That said, the patch seems sensible and I guess the sorting should be
OK; the actual mnemonics should be checked by others however :)

Acked-by: Laszlo Ersek <lersek@redhat.com>

Thanks
Laszlo

> 
> diff --git a/ArmPkg/ArmPkg.ci.yaml b/ArmPkg/ArmPkg.ci.yaml
> index d91c03f2acb8..a0d6a75fe881 100644
> --- a/ArmPkg/ArmPkg.ci.yaml
> +++ b/ArmPkg/ArmPkg.ci.yaml
> @@ -94,13 +94,18 @@
>            "ackintid",
>            "actlr",
>            "aeabi",
> +          "asedis",
>            "ashldi",
>            "ashrdi",
> +          "baddr",
>            "ccidx",
>            "ccsidr",
>            "clidr",
>            "clrex",
>            "clzsi",
> +          "cnthctl",
> +          "cortexa",
> +          "cpacr",
>            "cpuactlr",
>            "csselr",
>            "ctzsi",
> @@ -116,6 +121,7 @@
>            "divdi",
>            "divsi",
>            "dmdepkg",
> +          "dpref",
>            "drsub",
>            "fcmpeq",
>            "fcmpge",
> @@ -125,17 +131,25 @@
>            "ffreestanding",
>            "frsub",
>            "hisilicon",
> +          "iccabpr",
>            "iccbpr",
>            "icciar",
>            "iccicr",
>            "icciidr",
> +          "iccpir",
>            "iccpmr",
> +          "iccrpr",
> +          "icdabr",
>            "icdicer",
>            "icdicfr",
> +          "icdicpr",
>            "icdictr",
> +          "icdiidr",
>            "icdiser",
>            "icdisr",
> +          "icdppisr",
>            "icdsgir",
> +          "icdspr",
>            "icenabler",
>            "intid",
>            "ipriority",
> @@ -160,6 +174,7 @@
>            "lshrdi",
>            "moddi",
>            "modsi",
> +          "mpcore",
>            "mpidr",
>            "muldi",
>            "mullu",
> @@ -168,6 +183,9 @@
>            "nsasedis",
>            "nuvia",
>            "oldit",
> +          "pcten",
> +          "plpis",
> +          "procno",
>            "readc",
>            "revsh",
>            "rfedb",
> @@ -189,6 +207,7 @@
>            "smmlsr",
>            "sourcery",
>            "srsdb",
> +          "ssacr",
>            "stmdb",
>            "stmia",
>            "strbt",
>