Received-SPF: pass (google.com: domain of linux-kernel+bounces-61066-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Precedence: bulk
MIME-Version: 1.0
References: <20240206044217.1283849-1-songshuaishuai@tinylab.org>
In-Reply-To: <20240206044217.1283849-1-songshuaishuai@tinylab.org>
From: Andy Chiu <andy.chiu@sifive.com>
Date: Mon, 12 Feb 2024 10:13:18 +0800
Message-ID: <CABgGipW_U2tx4FKeZ+SuS=oKas13KEoJH1-TQUNF22B2p0tGuQ@mail.gmail.com>
Subject: Re: [PATCH] riscv: vector: Fix a typo of preempt_v
To: Song Shuai <songshuaishuai@tinylab.org>
Cc: paul.walmsley@sifive.com, palmer@dabbelt.com, aou@eecs.berkeley.edu, 
	ebiggers@google.com, vincent.chen@sifive.com, greentime.hu@sifive.com, 
	linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Tue, Feb 6, 2024 at 12:42=E2=80=AFPM Song Shuai <songshuaishuai@tinylab.=
org> wrote:
>
> The term "preempt_v" represents the RISCV_PREEMPT_V field of riscv_v_flag=
s
> and is used in lots of comments.
>
> Here corrects the miss-spelling "prempt_v".
>
> Signed-off-by: Song Shuai <songshuaishuai@tinylab.org>
> ---
>  arch/riscv/include/asm/simd.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/riscv/include/asm/simd.h b/arch/riscv/include/asm/simd.=
h
> index 54efbf523d49..04baee713eff 100644
> --- a/arch/riscv/include/asm/simd.h
> +++ b/arch/riscv/include/asm/simd.h
> @@ -36,7 +36,7 @@ static __must_check inline bool may_use_simd(void)
>         /*
>          * Nesting is acheived in preempt_v by spreading the control for
>          * preemptible and non-preemptible kernel-mode Vector into two fi=
elds.
> -        * Always try to match with prempt_v if kernel V-context exists. =
Then,
> +        * Always try to match with preempt_v if kernel V-context exists.=
 Then,
>          * fallback to check non preempt_v if nesting happens, or if the =
config
>          * is not set.
>          */
> --
> 2.39.2
>

Please also do this:

s/acheived/achieved

With that,

Reviewed-by: Andy Chiu <andybnac@gmail.com>

Thanks!
Andy