Re: [PATCH] cxl/ocxl: Possible repeated word
From: Michael Ellerman
Date: Tue Jun 20 2023 - 08:18:58 EST
zhumao001@xxxxxxxxxx writes:
> Delete repeated word in comment.
>
> Signed-off-by: Zhu Mao <zhumao001@xxxxxxxxxx>
> ---
> drivers/misc/cxl/native.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/misc/cxl/native.c b/drivers/misc/cxl/native.c
> index 50b0c44bb8d7..6957946a6463 100644
> --- a/drivers/misc/cxl/native.c
> +++ b/drivers/misc/cxl/native.c
> @@ -920,7 +920,7 @@ int cxl_attach_dedicated_process_psl9(struct
> cxl_context *ctx, u64 wed, u64 amr)
> * Ideally we should do a wmb() here to make sure the changes to
> the
> * PE are visible to the card before we call afu_enable.
> * On ppc64 though all mmios are preceded by a 'sync' instruction
> hence
> - * we dont dont need one here.
> + * we dont need one here.
I know the rule is to only do one change per patch, but if you're fixing
the wording of a comment you can absolutely fix the spelling at the same
time.
So please send a v2 with "dont" spelt correctly.
cheers