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, 18 Sep 2022 21:03:46 +0700
From:   Bagas Sanjaya <bagasdotme@gmail.com>
To:     Jilin Yuan <yuanjilin@cdjrlc.com>
Cc:     robh+dt@kernel.org, frowand.list@gmail.com,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] of: base: fix repeated words in comments
Message-ID: <YyclQlws9d7vheDk@debian.me>
References: <20220918095957.24537-1-yuanjilin@cdjrlc.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="7yCBxU7llMplZvup"
Content-Disposition: inline
In-Reply-To: <20220918095957.24537-1-yuanjilin@cdjrlc.com>
Precedence: bulk


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

On Sun, Sep 18, 2022 at 05:59:57PM +0800, Jilin Yuan wrote:
>  /**
>   * of_match_node - Tell if a device_node has a matching of_match structu=
re
> - * @matches:	array of of device match structures to search in
> + * @matches:	array of device match structures to search in
>   * @node:	the of device structure to match against
>   *
>   * Low level utility function used by device matching.
> @@ -1175,7 +1175,7 @@ EXPORT_SYMBOL(of_match_node);
>   *		you pass will not be searched, only the next one
>   *		will; typically, you pass what the previous call
>   *		returned. of_node_put() will be called on it
> - * @matches:	array of of device match structures to search in
> + * @matches:	array of device match structures to search in
>   * @match:	Updated to point at the matches entry which matched

That second "of" is the device, so the fix doesn't make sense.

Also, PLEASE stop sending these mindless redundant word-stripping patches
as byproduct of whatever automated tool you use.

As Edward Cree said to one of your colleague [1]:

> #ifdef RANT
> NGL, getting kinda sick of these bogus comment text 'fixes' from people
>  who clearly don't have enough mastery of English to copyedit it.
> (Previous one from this author was actually wrong too but I didn't catch
>  it at the time.)
> English is a tricksy language, why would someone with a limited
>  understanding of it think that grammar fixes are the best use of their
>  time and energy?
> I can't help suspecting that this is a box-ticking exercise, where a
>  certain corporate culture has a standard requirement that engineers
>  must get X number of Linux / opensource commits in order to get
>  promoted, and this kind of mindless patch is the easiest way for them
>  to achieve that.
> #endif

If you really want to do real cleanup, consider joining kernel-janitors lis=
t.

Learn from above, thanks.

[1]: https://lore.kernel.org/netdev/30f2488d-9af3-fe8d-6e6f-713a7d38800b@gm=
ail.com/

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

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

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

iHUEABYIAB0WIQSSYQ6Cy7oyFNCHrUH2uYlJVVFOowUCYyclOgAKCRD2uYlJVVFO
o4ueAP9rOkS3/cc61IFsELqltLUWs+gQLDZrhG3lOmPXT1DkJwEAtQKtChFrPySL
n/Op1oMwsl+TgNxGjw/TNhQ1U/+tFwk=
=NZ1v
-----END PGP SIGNATURE-----

--7yCBxU7llMplZvup--