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:   Sat, 29 Oct 2022 09:28:26 +0700
From:   Bagas Sanjaya <bagasdotme@gmail.com>
To:     Tanjuate Brunostar <tanjubrunostar0@gmail.com>
Cc:     gregkh@linuxfoundation.org, linux-staging@lists.linux.dev,
        linux-kernel@vger.kernel.org, outreachy@lists.linux.dev
Subject: Re: [PATCH v8 0/6] staging: vt6655: a series of checkpatch fixes on
 the file: rxtx.c
Message-ID: <Y1yPyt5NsvfSL3UC@debian.me>
References: <cover.1666978292.git.tanjubrunostar0@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="ADH5S7zY59Gn+n8g"
Content-Disposition: inline
In-Reply-To: <cover.1666978292.git.tanjubrunostar0@gmail.com>
Precedence: bulk


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

On Fri, Oct 28, 2022 at 05:40:52PM +0000, Tanjuate Brunostar wrote:
> The fixes are similar, mostly long lines splitting. I had to make
> serveral patches to ease the work of inspectors
>=20
> v2: fixed a compilation error found by the kernel test robot and
> recompiled the code
>=20
> v3: tends out the error persisted in the second version. this version is
> a correction of that
>=20
> v4: did some corrections as recommended by Greg KH
>=20
> v5: shortend changelog comments as recommended by Greg KH
>=20
> v6: did some corrections as recommended by Greg KH
>=20
> v7: fixed some errors on my changelog comments
>=20
> v8: fixed some errors pointed out by Philipp Hortmann
>=20
> Tanjuate Brunostar (6):
>   staging: vt6655: fix lines ending in a '('
>   staging: vt6655: fix long lines of code in s_uGetRTSCTSDuration
>   staging: vt6655: fix long lines of code in s_uFillDataHead
>   staging: vt6655: fix long lines of code in s_vGenerateTxParamete
>   staging: vt6655: fix long lines of code in the rest of the file
>   staging: vt6655: fix lines of code ending in a '('
>=20

You messed up your patchset...; the shortlog above says "fix longlines"
but individual patch subject says "refactor". Well, refactoring
means resturcturing and improving it without introducing new
functionality or changing behavior, which is really changing existing
code. However, your patchset only do visual formatting, right?

Also, have you ever at least compile-test your patchset (make W=3D1 and
enabling CONFIG_WERROR)?

Thanks.

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

--ADH5S7zY59Gn+n8g
Content-Type: application/pgp-signature; name="signature.asc"

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

iHUEABYIAB0WIQSSYQ6Cy7oyFNCHrUH2uYlJVVFOowUCY1yPwAAKCRD2uYlJVVFO
oydYAQDtxv1qvxNsp2r080rxws4fOrvmdai2HM6FMB9NuFZJBQEA2zlbqqEl8/+q
1qKuGuRxbHVjustXYCAhX+veICs/XAw=
=KkOZ
-----END PGP SIGNATURE-----

--ADH5S7zY59Gn+n8g--