From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Sven Eckelmann Date: Sat, 24 Nov 2018 21:52:36 +0100 Message-ID: <2150224.WNsGD1XmFl@sven-desktop> In-Reply-To: <20181123161359.13342-12-sven@narfation.org> References: <20181123161359.13342-1-sven@narfation.org> <20181123161359.13342-12-sven@narfation.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [B.A.T.M.A.N.] [RFC v2 11/19] batman-adv: Add gateway mesh genl configuration List-Id: The list for a Better Approach To Mobile Ad-hoc Networking List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: b.a.t.m.a.n@lists.open-mesh.org Cc: Jiri Pirko , netdev@vger.kernel.org On Freitag, 23. November 2018 17:13:51 CET Sven Eckelmann wrote: > + /** @BATADV_GW_MODE_CLIENT: announce itself as gatway server */ > + BATADV_GW_MODE_SERVER, The kernel-doc should actually be for BATADV_GW_MODE_SERVER. [...] > + if (sel_class < 1 || sel_class > sel_class_max) { > + atomic_set(&bat_priv->gw.sel_class, sel_class); > + batadv_gw_reselect(bat_priv); > + } Should have been: if (sel_class >= 1 && sel_class <= sel_class_max) { Kind regards, Sven