From mboxrd@z Thu Jan 1 00:00:00 1970
Return-Path:
Date: Tue, 26 Jan 2016 20:53:14 +0800
From: Antonio Quartulli
Message-ID: <20160126125314.GA25892@prodigo>
References: <1453312110-32683-1-git-send-email-andrew@lunn.ch>
<1453312110-32683-4-git-send-email-andrew@lunn.ch>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
protocol="application/pgp-signature"; boundary="Q68bSM7Ycu6FN28Q"
Content-Disposition: inline
In-Reply-To: <1453312110-32683-4-git-send-email-andrew@lunn.ch>
Subject: Re: [B.A.T.M.A.N.] [PATCH 3/4] batman-adv: Handle parent interfaces
in a different netns
List-Id: The list for a Better Approach To Mobile Ad-hoc Networking
List-Unsubscribe: ,
List-Archive:
List-Post:
List-Help:
List-Subscribe: ,
To: Andrew Lunn
Cc: b.a.t.m.a.n@lists.open-mesh.org
--Q68bSM7Ycu6FN28Q
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
On Wed, Jan 20, 2016 at 06:48:29PM +0100, Andrew Lunn wrote:
> /**
> + * batadv_mutual_parents - check if two devices are each others parent
> + * @net_dev - 1st parent
> + * @parent_dev - 2nd parent
> + *
> + * veth devices come in pairs and each is the parent of the other!
> + *
> + * Return true if the devices are each others parent, otherwise false
Andrew, minor style issue: you should put the ':' after the word Return.
Please check the other comments in this file as example. Sven recently chan=
ged
all the comments to be coherent with this style.
Other than this, the patch looks good to me.
You can add my
Acked-by: Antonio Quartulli
--=20
Antonio Quartulli
--Q68bSM7Ycu6FN28Q
Content-Type: application/pgp-signature; name="signature.asc"
Content-Description: Digital signature
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2
iQIcBAEBCAAGBQJWp2w6AAoJENpFlCjNi1MRJ1UP/0E5B/pMK3N2so2GAFpm079U
UMz32dtqm/l5EQgyOQbtcKafc1MwyRzi7L3KDL8CMXrSREgoGmlasb7701SC39WO
9lbtlK0vfYR6A6CYzYHo3ACPe8zEVZGEgawY5vvJgqnNMeccpD786+Ju7PQ4R66d
0DEEFGUvx2WzHkVSCq9INd8+NlkeixLg6N3bKRPSjJi2nmGF0bsMI+AsHn7nZh7z
FMx+cRzKlm2wp5RuueFcjhGMaf+SXKhRNf0ArDvIArhQKlmzIYbA0yxDQUwybNwK
oednEavs6Wqg4OuitVvSGF/p3/GH9Rbsa3nhfaG+nkimnHx/3A4NnaJQvaKmDsvU
iBV1X7iKSimgKblWdaFku8qYeMtoPIo6Pksf9VUEBHq0t76B1pvma7Dfz8rEQZr9
OgUCAgODbsvnZfbdqUmZkNXmPGZxgMs5oahWNwLbVYmzv6DKho56jrhCmuGz4k/4
xsLTWqkHnltTH5kM+ICA0WRb1nXvmJ6EzME7LNfihzdANFKgaGuWfoPgUOgXSvWs
epGeW/PRxr1tYNWSDfC6zvf2/t/ylQ9VK5xMU2ITvHAIdETrwtV64ggcmeTRRirn
UQDPqBz1iBJ2WNzeVE2pRkVDXSvfIZ1PGQKi2Wfp1j0ynOg+coCZL0Aa7/uWgiHL
0Cizk3BfEnqiIoWQxs69
=iSzI
-----END PGP SIGNATURE-----
--Q68bSM7Ycu6FN28Q--