Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Wed, 5 Dec 2018 22:09:51 +0100
From:   Sicilia Cristian <sicilia.cristian@gmail.com>
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc:     Bryan O Donoghue <pure.logic@nexus-software.ie>,
        Johan Hovold <johan@kernel.org>, Alex Elder <elder@kernel.org>,
        devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org,
        greybus-dev@lists.linaro.org
Subject: Re: [PATCH 1/2] staging: greybus: Align function call parameters to
 parenthesis
Message-ID: <20181205210951.GA24993@peano>
References: <cover.1543956271.git.sicilia.cristian@gmail.com>
 <6087f45a8a4544fa6c671d8b42928fb48194e66e.1543956271.git.sicilia.cristian@gmail.com>
 <20181205084603.GA4987@kroah.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="KsGdsel6WgEHnImy"
Content-Disposition: inline
In-Reply-To: <20181205084603.GA4987@kroah.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


--KsGdsel6WgEHnImy
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Dec 05, 2018 at 09:46:03AM +0100, Greg Kroah-Hartman wrote:
> On Tue, Dec 04, 2018 at 09:58:14PM +0100, Cristian Sicilia wrote:
> > Aligned some parameters to the previous parenthesis.
> >=20
> > Signed-off-by: Cristian Sicilia <sicilia.cristian@gmail.com>
> > ---
> >  drivers/staging/greybus/loopback.c | 10 ++++++----
> >  1 file changed, 6 insertions(+), 4 deletions(-)
> >=20
> > diff --git a/drivers/staging/greybus/loopback.c b/drivers/staging/greyb=
us/loopback.c
> > index e4d42c1..1085e06 100644
> > --- a/drivers/staging/greybus/loopback.c
> > +++ b/drivers/staging/greybus/loopback.c
> > @@ -638,7 +638,8 @@ static int gb_loopback_async_transfer(struct gb_loo=
pback *gb, u32 len)
> >  	retval =3D gb_loopback_async_operation(gb, GB_LOOPBACK_TYPE_TRANSFER,
> >  					     request, len + sizeof(*request),
> >  					     len + response_len,
> > -					     gb_loopback_async_transfer_complete);
> > +					     gb_loopback_async_transfer_complete
> > +					     );
>=20
> Ick, why do this?  That's not really needed.
>=20

Ok, better to leave it unchanged

> thanks,
>=20
> greg k-h

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

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iQIcBAABCAAGBQJcCD6eAAoJEAnJ2Lo5pAXyvhQP/16vTeHBKpmXlijHktcypkxp
wIR5wyZLlyiazKu5W1COZU5xrxDyWj3R8X+ViUn0MMGBQkDejYP6YvAm0VvgeKp3
Wr3kXlqkAj+5Q/GrRA1yX9UonxPyJhohYxW0OVzngr720xKg8WbTNRpVMBp2FVdD
OUNAJyB6+Vt+sEh0H2vJHlkH0+/cCQv0+orqQDpO1dfmswYmi2qExFH/EJ6x20qC
GIBLSqy9UR9T52dDjB7ehG8AupYefEX7HTyBfuCMn1y7YYGZWg6ajGwELlQKD9ir
AB8E5UIvA3a1sgRmSRcjgNKED4xLE7RSWqhRpX+Dg8QWtGnDucuG/bTY1pjd7mW8
LpFvsOppo0ojWeRnX+tWHj+mUzOGJxB1sjIGYkpp5eJRUe572/srPg+hZLWMgTen
lgRx6oAok8aQ0F8Gy3IhC2lFsZe7ULG7yjVjb1oUjxhZG3bE6seBEDU7JEscjv8U
en+qjML/zleJMMq90p0HaH7nuskGScdltvEnIMi+lQJ0rJnBJWPup692k2LAUbB4
DLYV2heK9I9YhKhF580MZWQoRoCQ1gjzUBURYLYxd26CtRE0pDld8VmHzhtB+t7f
5LfnifZz6ZY2TlHWjMe9V/O3UxSj3U1XmAteA/hPfif1I89lY2oss1qVdrjx4pOk
kE4huxRgl9aL4Mpk8Mt6
=8W48
-----END PGP SIGNATURE-----

--KsGdsel6WgEHnImy--