Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Sun, 9 Oct 2022 20:36:22 +0700
From:   Bagas Sanjaya <bagasdotme@gmail.com>
To:     Jiangshan Yi <13667453960@163.com>
Cc:     sudipm.mukherjee@gmail.com, teddy.wang@siliconmotion.com,
        gregkh@linuxfoundation.org, linux-fbdev@vger.kernel.org,
        linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org,
        Jiangshan Yi <yijiangshan@kylinos.cn>,
        k2ci <kernel-bot@kylinos.cn>
Subject: Re: [PATCH] staging: sm750fb: fix spelling typo in comment
Message-ID: <Y0LOVuL8uNJ19bjV@debian.me>
References: <20221009094809.3171319-1-13667453960@163.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="dZhQDIkS5gtuuN7K"
Content-Disposition: inline
In-Reply-To: <20221009094809.3171319-1-13667453960@163.com>
Precedence: bulk


--dZhQDIkS5gtuuN7K
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

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

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.
>=20
> This spelling typo was found using notes_check tool.
>=20

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(-)
>=20
> diff --git a/drivers/staging/sm750fb/ddk750_dvi.h b/drivers/staging/sm750=
fb/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__
> =20
> -/* dvi chip stuffs structros */
> +/* dvi chip stuffs structures */
> =20
>  typedef long (*PFN_DVICTRL_INIT)(unsigned char edge_select,
>  				 unsigned char bus_select,

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

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

--dZhQDIkS5gtuuN7K
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQSSYQ6Cy7oyFNCHrUH2uYlJVVFOowUCY0LOUgAKCRD2uYlJVVFO
o5abAQDCCPg+8fcvsOLmLonB7xsYw1IuPRjkPfOSqr/8ap6WEwEAn5L6/13EPykf
a2SW0VQuhr/iphjQnmqDUEb/ftTmHgM=
=zi0w
-----END PGP SIGNATURE-----

--dZhQDIkS5gtuuN7K--