diff mbox series

staging: sm750fb: fix spelling typo in comment

Message ID 20221009094809.3171319-1-13667453960@163.com (mailing list archive)
State Not Applicable
Headers show
Series staging: sm750fb: fix spelling typo in comment | expand

Commit Message

Jiangshan Yi Oct. 9, 2022, 9:48 a.m. UTC
From: Jiangshan Yi <yijiangshan@kylinos.cn>

Fix spelling typo in comment.

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.

Reported-by: k2ci <kernel-bot@kylinos.cn>
Signed-off-by: Jiangshan Yi <yijiangshan@kylinos.cn>
---
 drivers/staging/sm750fb/ddk750_dvi.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Bagas Sanjaya Oct. 9, 2022, 1:36 p.m. UTC | #1
On Sun, Oct 09, 2022 at 05:48:09PM +0800, Jiangshan Yi wrote:
> From: Jiangshan Yi <yijiangshan@kylinos.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@kylinos.cn>
> Signed-off-by: Jiangshan Yi <yijiangshan@kylinos.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)?
Greg KH Oct. 9, 2022, 6:22 p.m. UTC | #2
On Sun, Oct 09, 2022 at 05:48:09PM +0800, Jiangshan Yi wrote:
> From: Jiangshan Yi <yijiangshan@kylinos.cn>

Again, why 163.com?

thanks,

greg k-h
Jiangshan Yi Oct. 10, 2022, 2:30 a.m. UTC | #3
At 2022-10-10 02:22:40, "Greg KH" <gregkh@linuxfoundation.org> wrote:
>On Sun, Oct 09, 2022 at 05:48:09PM +0800, Jiangshan Yi wrote:
>> From: Jiangshan Yi <yijiangshan@kylinos.cn>
>
>Again, why 163.com?
>



There is a problem with the company's mailbox, and sometimes I can't receive emails. So I temporarily use my personal email to send patches, so that I can receive feedback from the community in a timely manner.


thanks,


Jiangshan Yi
diff mbox series

Patch

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,