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, 28 May 2023 00:31:11 +0200
From:   Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= <u.kleine-koenig@pengutronix.de>
To:     Prathu Baronia <prathubaronia2011@gmail.com>
Cc:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        "Fabio M. De Francesco" <fmdefrancesco@gmail.com>,
        Khadija Kamran <kamrankhadijadj@gmail.com>,
        linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org,
        dan.carpenter@linaro.org, error27@gmail.com, lkp@intel.com,
        oe-kbuild-all@lists.linux.dev, oe-kbuild@lists.linux.dev
Subject: Re: [PATCH v4 3/3] axis-fifo: cleanup space issues with fops struct
Message-ID: <20230527223111.jidik3ffcsxdkenb@pengutronix.de>
References: <ZHHE/H2p4Go/Igc/@redkillpc>
 <20230527115101.47569-1-prathubaronia2011@gmail.com>
 <20230527115101.47569-3-prathubaronia2011@gmail.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="cdvvvrjjz27c3mvb"
Content-Disposition: inline
In-Reply-To: <20230527115101.47569-3-prathubaronia2011@gmail.com>
Precedence: bulk


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

Hello,

On Sat, May 27, 2023 at 05:21:00PM +0530, Prathu Baronia wrote:
> Add required spaces for proper formatting of fops members for better
> readability.
>=20
> Signed-off-by: Prathu Baronia <prathubaronia2011@gmail.com>
> ---
>  drivers/staging/axis-fifo/axis-fifo.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>=20
> diff --git a/drivers/staging/axis-fifo/axis-fifo.c b/drivers/staging/axis=
-fifo/axis-fifo.c
> index d71bdc6dd961..59e962467a3d 100644
> --- a/drivers/staging/axis-fifo/axis-fifo.c
> +++ b/drivers/staging/axis-fifo/axis-fifo.c
> @@ -716,11 +716,11 @@ static int axis_fifo_close(struct inode *inod, stru=
ct file *f)
>  }
> =20
>  static const struct file_operations fops =3D {
> -	.owner =3D THIS_MODULE,
> -	.open =3D axis_fifo_open,
> +	.owner   =3D THIS_MODULE,
> +	.open    =3D axis_fifo_open,
>  	.release =3D axis_fifo_close,
> -	.read =3D axis_fifo_read,
> -	.write =3D axis_fifo_write
> +	.read    =3D axis_fifo_read,
> +	.write   =3D axis_fifo_write

Note this is only subjectively better. IMHO with just a single space
this is perfectly readable. Aligning the =3D might look nice, but it's
also annoying at times. When you add another member (e.g.
=2Eiterate_shared) you either add a line that doesn't match all others, or
you have to touch all other lines of that struct which (objectively?)
hurts readability of that patch. Also for generated patches this kind of
alignment yields extra work. (See for example
https://lore.kernel.org/lkml/20230525205840.734432-1-u.kleine-koenig@pengut=
ronix.de/
which required semi-manual fixup to keep the alignment after coccinelle
generated the patch.)

If you still think this is a good idea, I'd ask you to stick to one
style for the whole file. e.g. axis_fifo_driver uses inconsistent
and different indention.

A thing that IMHO is more useful to change here, is the name fops; I'd
suggest something like axis_fifo_fops (and also use prefixes for some
other symbols like "get_dts_property"). In 6.4-rc1 my ctags file knows
about 64 different places that define something called "fops".

Just my 0.02=E2=82=AC,
Uwe

--=20
Pengutronix e.K.                           | Uwe Kleine-K=C3=B6nig         =
   |
Industrial Linux Solutions                 | https://www.pengutronix.de/ |

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

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

iQEzBAABCgAdFiEEP4GsaTp6HlmJrf7Tj4D7WH0S/k4FAmRyhK4ACgkQj4D7WH0S
/k4GHQf/UqWfo79tCnSurnPKR+RHhrveNwWlX7kE0wdil5CyIe7xjiG/0TNrS86J
DBhUGdUczfaKmeuTG3sQKTJowMSmRQyuIxCM37lnJPvYWBAdkuhbsBgHaRNdR5fA
yIX9vrR9belBqHdMMl7sXdcLoIWdzsKUHL+8p2wiP5Qaz4X7WwyfPmkGLnk5Km49
k11VFJT2HAdq3zut/83XO4JNIeX0WL3Yzz4LQFVcmLaU6K19dvXXIV/bN0cQEXwR
hpUGiKWwWpEF9+JaFKIHfCsJ5luB9Nbb+HxU8ph84VOCgG8JFldIfylX1woz06R4
kQRnDIRnbbALLfda5tit7DxPVRV/OA==
=8i8o
-----END PGP SIGNATURE-----

--cdvvvrjjz27c3mvb--