linux-kernel - Re: [PATCH] staging: sm750fb: fix spelling typo in comment

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Y0LOVuL8uNJ19bjV@debian.me>
Date:   Sun, 9 Oct 2022 20:36:22 +0700
From:   Bagas Sanjaya <bagasdotme@...il.com>
To:     Jiangshan Yi <13667453960@....com>
Cc:     sudipm.mukherjee@...il.com, teddy.wang@...iconmotion.com,
        gregkh@...uxfoundation.org, linux-fbdev@...r.kernel.org,
        linux-staging@...ts.linux.dev, linux-kernel@...r.kernel.org,
        Jiangshan Yi <yijiangshan@...inos.cn>,
        k2ci <kernel-bot@...inos.cn>
Subject: Re: [PATCH] staging: sm750fb: fix spelling typo in comment

On Sun, Oct 09, 2022 at 05:48:09PM +0800, Jiangshan Yi wrote:
> From: Jiangshan Yi <yijiangshan@...inos.cn>
> 
> Fix spelling typo in comment.
> 

What comment are you trying to do this typofix? Even seeing the diff
below, I can't figure out that due to insufficient context.

> K2CI (Kylin Kernel Continuous Integration) is a code pre-verification tool
> independently developed by KylinSoft, which is used for ensuring the
> code quality of code submission. K2CI includes the comment check tool
> notes_check.
> 
> This spelling typo was found using notes_check tool.
> 

For CI bot like this, I'd like to see publicly-accessible reports (like
build status), just like what kernel test robot (aka LKP aka 01-DAY CI)
does (try searching LKML for recent report by it). Don't forget to add
Link: tag to the report.

> Reported-by: k2ci <kernel-bot@...inos.cn>
> Signed-off-by: Jiangshan Yi <yijiangshan@...inos.cn>
> ---
>  drivers/staging/sm750fb/ddk750_dvi.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/staging/sm750fb/ddk750_dvi.h b/drivers/staging/sm750fb/ddk750_dvi.h
> index c2518b73bdbd..afdfdb17ffce 100644
> --- a/drivers/staging/sm750fb/ddk750_dvi.h
> +++ b/drivers/staging/sm750fb/ddk750_dvi.h
> @@ -2,7 +2,7 @@
>  #ifndef DDK750_DVI_H__
>  #define DDK750_DVI_H__
>  
> -/* dvi chip stuffs structros */
> +/* dvi chip stuffs structures */
>  
>  typedef long (*PFN_DVICTRL_INIT)(unsigned char edge_select,
>  				 unsigned char bus_select,

Are the structures really plural (many instances of the struct)?

-- 
An old man doll... just what I always wanted! - Clara

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)