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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 48E87C48BC4 for ; Sat, 17 Feb 2024 23:49:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Qa+PGFp+fKpM1MJO+qOSb/tDzo4jK9IoRxfuZ8pd7SE=; b=NDIM2oMpUVNKAL YXg8yZ684pPzqB30XRmh8c68Va/679vfCFxVk0tTndbHv5OUu+pTgVuPuf7MXGbhbFE3VbaF4CSth n1sW0GopskEyllFlusb1TroNe8lyN/i4765pYQ1v7Z/7B9ysWvzxOIMlTSQYq+Z0xgkBw9cgqLMnB EEoF3HaMULzLdIDV8eGmRBRxNcM83Ynwxg9FpYK9dgl6Ks7ix6rRK+yamBbNwraF1PtyopEs0useO QvdFxttwH2WNJEwevX5V9/3hU3v12tgXzSqisMIH4WVXy7Ircy1LYKIMth0vbnnmPjx8bPKZvIxJn LW+tT9/Q9wy/exou0RQQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rbUQV-00000006ceU-2SDi; Sat, 17 Feb 2024 23:48:59 +0000 Received: from vps0.lunn.ch ([156.67.10.101]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rbUQS-00000006cdo-0lrK; Sat, 17 Feb 2024 23:48:57 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Disposition:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:From:Sender:Reply-To:Subject: Date:Message-ID:To:Cc:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Content-Disposition:In-Reply-To:References; bh=FMxvckYePbS1jnoZ5oG6Cm4EypY4Dw/pbZjxC0ZP4yU=; b=CPzKeDAmryQgULRVfeg9ZXkTij sV5K4ExtKLQtMWkXBHzzTZujVOaGPUXXKbK1euewjW9lDjsFSr2E4PYT7u1SBA71pkRVI3/BqoQ8P Yt5mKJuEZUeqa8K0hBbIQZ8PE6rcAY9Yy7FYlnkXwYiyuiFMNbNKcJjI+wuUfEVpT7gA=; Received: from andrew by vps0.lunn.ch with local (Exim 4.94.2) (envelope-from ) id 1rbUPw-0085vH-Pp; Sun, 18 Feb 2024 00:48:24 +0100 Date: Sun, 18 Feb 2024 00:48:24 +0100 From: Andrew Lunn To: Christian Marangi Cc: Michael Hennerich , Heiner Kallweit , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Florian Fainelli , Broadcom internal kernel review list , Ray Jui , Scott Branden , Richard Cochran , Marek =?iso-8859-1?Q?Beh=FAn?= , Daniel Golle , Qingfang Deng , SkyLake Huang , Neil Armstrong , Kevin Hilman , Jerome Brunet , Martin Blumenstingl , Arun Ramadoss , UNGLinuxDriver@microchip.com, Peter Geis , Frank , Xu Liang , Piergiorgio Beruto , Andrei Botila , Bjorn Andersson , Konrad Dybcio , Heiko Stuebner , Michal Simek , Miguel Ojeda , Alex Gaynor , Wedson Almeida Filho , Boqun Feng , Gary Guo , =?iso-8859-1?Q?Bj=F6rn?= Roy Baron , Benno Lossin , Andreas Hindborg , Alice Ryhl , Matthias Brugger , AngeloGioacchino Del Regno , Robert Marko , "Russell King (Oracle)" , Vladimir Oltean , David Epping , Harini Katakam , Simon Horman , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-amlogic@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-rockchip@lists.infradead.org, rust-for-linux@vger.kernel.org, linux-mediatek@lists.infradead.org Subject: Re: [net-next RFC PATCH 0/3] net: phy: detach PHY driver OPs from phy_driver struct Message-ID: References: <20240217194116.8565-1-ansuelsmth@gmail.com> <65d140fb.5d0a0220.81be3.7138@mx.google.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <65d140fb.5d0a0220.81be3.7138@mx.google.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240217_154856_250288_E47B3C27 X-CRM114-Status: GOOD ( 17.10 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org > Yes, it was done to limit the patch delta, if I had to account for the > tab for each new section we would be in the order of 2000+ changes I > think. > > > > 64 files changed, 737 insertions(+), 291 deletions(-) > > > > These statistics are not good. If you had deleted more lines than you > > added, then maybe it might be an O.K. idea. > > > > Sometimes KISS is best. > > > > Well IMHO these stats are a bit flawed, the additional code is really > just extra check if ops is defined and the new .ops variable in each > phy_driver. > > If you check patch 2 and 3 you can already see some code is removed. Yes, the problem is, it probably needs another 50 patches to remove all the duplication. I have to question, is that really going to happen? Are you going to keep working on this until every driver has its duplicates removed? It probably needs some tooling to help. Something which can decode the object file, and tell you which ops structures are identical. That can then guide you when editing all the PHY drivers. Andrew _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic