From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH net-next] net: core: change bool members of struct net_device to bitfield members Date: Mon, 8 Oct 2018 14:20:09 -0700 Message-ID: <20181008142009.7b32df14@xeon-e3> References: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: David Miller , "netdev@vger.kernel.org" To: Heiner Kallweit Return-path: Received: from mail-pg1-f193.google.com ([209.85.215.193]:34931 "EHLO mail-pg1-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725749AbeJIEeA (ORCPT ); Tue, 9 Oct 2018 00:34:00 -0400 Received: by mail-pg1-f193.google.com with SMTP id v133-v6so8398004pgb.2 for ; Mon, 08 Oct 2018 14:20:18 -0700 (PDT) In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Mon, 8 Oct 2018 22:00:51 +0200 Heiner Kallweit wrote: > * > + * @uc_promisc: Counter that indicates promiscuous mode > + * has been enabled due to the need to listen to > + * additional unicast addresses in a device that > + * does not implement ndo_set_rx_mode() > + * I see you just moved the pre-existing comment, but it the comment looks incorrect. uc_promisc is not a counter but a flag. A counter would have more than two states normally.