From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 444A93D75A7; Wed, 4 Mar 2026 18:17:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772648257; cv=none; b=lEc3wnLsbpkuyzNWLt9Y+dQ4NscwpnQLUu6EN1ARhVadU6Tqb29A0TZgwIYcqYpDrjdGJPHct7g5fF9CDPiKdqILmi1AD35BEwo/Xi4ytNgE5nEfF+Qk/oZoKZGBR/7UFy3NpBYCdghVp7lTDjI31H7PH1zlm4lvcD/kKQzOeII= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772648257; c=relaxed/simple; bh=9dyBZ5aCtMBIkcEwRcBVuwfowNsWWphrWBWKbim6R2g=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=LgyzloaCJDnjL8xBtJuFVppfD30yebIy+ND0uF4lNZ3eOJ1ymhqtEfA5DmJc7RIJqbpmarlcCnceNShESfn3iNLD2Of40On9rPbNdodx5HOCLkYK6HgIhvl6On07F7J61025gufNcGfSKitV5xhmmMmhbhf3IvF1mcfInL8R5t0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=l7igiJLU; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="l7igiJLU" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 80784C4CEF7; Wed, 4 Mar 2026 18:17:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1772648257; bh=9dyBZ5aCtMBIkcEwRcBVuwfowNsWWphrWBWKbim6R2g=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=l7igiJLUAMhc5GmhSG3HTFROQRaTif4Zw9BwBVKKt9k68jpgmQ52HgvM6TxIiJCYB MKqscKdwIG0PA2+zr6+qk0BtMDVqgOXo9tgwFnkiVrU2QWYTYdCikY+VsKqq0Ws0a6 GIlRXQuVIPUj5wB8e7YKdKva2uCtkphJ3hDC2bBoEM+0Yk+S4xTMX203l+b68Egjka rxspaHZYziPSpWxAtkDWKZ7Hpoj8yPIsiF4sYukgc/ZUd5JV1msg3qmhxSmSm2plSD YhAwLupah4U39OS2iE0sAkyEu59r/yyvu30hJPMXokMCZhRbGm0HNeWVJr8p+2sD7/ /7DU4F6xlUmhw== Date: Wed, 4 Mar 2026 10:17:35 -0800 From: Jakub Kicinski To: Bhargava Chenna Marreddy Cc: edumazet@google.com, andrew+netdev@lunn.ch, ajit.khaparde@broadcom.com, horms@kernel.org, pabeni@redhat.com, michael.chan@broadcom.com, linux-kernel@vger.kernel.org, vsrama-krishna.nemani@broadcom.com, davem@davemloft.net, netdev@vger.kernel.org, rajashekar.hudumula@broadcom.com, vikas.gupta@broadcom.com, pavan.chebbi@broadcom.com, rahul-rg.gupta@broadcom.com Subject: Re: [net-next,v3,3/8] bng_en: add ethtool link settings, get_link, and nway_reset Message-ID: <20260304101735.41b247eb@kernel.org> In-Reply-To: References: <20260228072715.1632280-4-bhargava.marreddy@broadcom.com> <20260303030029.1884718-1-kuba@kernel.org> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Wed, 4 Mar 2026 16:07:09 +0530 Bhargava Chenna Marreddy wrote: > The code was functionally safe, but I agree the inconsistency in the > array patterns was not ideal. Well TBH I think AI is being generous in how it phrases it's feedback. Assuming an arbitrary value in an enum is and always will be the last one is asking for bugs.