From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8E126C77B60 for ; Wed, 26 Apr 2023 13:10:14 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240505AbjDZNKN (ORCPT ); Wed, 26 Apr 2023 09:10:13 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44162 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229744AbjDZNKM (ORCPT ); Wed, 26 Apr 2023 09:10:12 -0400 Received: from fudo.makrotopia.org (fudo.makrotopia.org [IPv6:2a07:2ec0:3002::71]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3C77BAF; Wed, 26 Apr 2023 06:10:11 -0700 (PDT) Received: from local by fudo.makrotopia.org with esmtpsa (TLS1.3:TLS_AES_256_GCM_SHA384:256) (Exim 4.96) (envelope-from ) id 1pretn-0004ob-2w; Wed, 26 Apr 2023 15:09:32 +0200 Date: Wed, 26 Apr 2023 14:07:45 +0100 From: Daniel Golle To: =?utf-8?B?QXLEsW7DpyDDnE5BTA==?= Cc: Sean Wang , Landen Chao , DENG Qingfang , Andrew Lunn , Florian Fainelli , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Matthias Brugger , AngeloGioacchino Del Regno , Russell King , Richard van Schagen , Richard van Schagen , Frank Wunderlich , Bartel Eerdekens , erkin.bozoglu@xeront.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Subject: Re: [PATCH net-next 04/24] net: dsa: mt7530: properly support MT7531AE and MT7531BE Message-ID: References: <20230425082933.84654-1-arinc.unal@arinc9.com> <20230425082933.84654-5-arinc.unal@arinc9.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org On Wed, Apr 26, 2023 at 11:12:09AM +0300, Arınç ÜNAL wrote: > On 25.04.2023 18:04, Daniel Golle wrote: > > On Tue, Apr 25, 2023 at 11:29:13AM +0300, arinc9.unal@gmail.com wrote: > > > From: Arınç ÜNAL > > > > > > Introduce the p5_sgmii pointer to store the information for whether port 5 > > > has got SGMII or not. > > > > The p5_sgmii your are introducing to struct mt7530_priv is a boolean > > variable, and not a pointer. > > I must've meant to say field. Being just a single boolean variable also 'field' would not be the right word here. We use 'field' as in 'bitfield', ie. usually disguised integer types in which each bit has an assigned meaning. > > > > > > > > > Move the comment about MT7531AE and MT7531BE to mt7531_setup(), where the > > > switch is identified. > > > > > > Get rid of mt7531_dual_sgmii_supported() now that priv->p5_sgmii stores the > > > information. Address the code where mt7531_dual_sgmii_supported() is used. > > > > > > Get rid of mt7531_is_rgmii_port() which just prints the opposite of > > > priv->p5_sgmii. > > > > > > Remove P5_INTF_SEL_GMAC5_SGMII. The p5_interface_select enum is supposed to > > > represent the mode that port 5 is being used in, not the hardware > > > information of port 5. Set p5_intf_sel to P5_INTF_SEL_GMAC5 instead, if > > > port 5 is not dsa_is_unused_port(). > > > > > > Signed-off-by: Arınç ÜNAL > > > --- > > > > Other than the comment above this change makes sense and looks good to > > me, so once you correct the commit message, you may add my Acked-by. > > Will do, thanks. > > Arınç