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 59223C77B7A for ; Tue, 6 Jun 2023 10:13:48 +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:MIME-Version:References:In-Reply-To: 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=1Qc6Wxx37BHdykki+L1tpqWuQ4suaIy3dx38Edke5oQ=; b=1dORtwDlsQ2Kap eiO+3qS2TE4Y4YTjPsSQyxcL25qWFjDRyWybpQL9tWww/TABng2/dG1IowFX2M8ck5R7gxI1U3qMd NOpqEHjcqcQbbVsgR02roGBQEcQxJniwdSy46DovrAMNOx7oyayUMIErOJGsEkyRmoxR1sx42l70M dlwydvcd84xTd6474luQ1Z4okck2qZzgZMndBazGadzi0okJNDaOtDYYEQWgDqpvNYY4R8Y3sUVJD rehqHKNUitGg/u03AigN8akGU6rf5ibctZci2x7qKtUZA/E2lSufjiynmd7NGdPLWyoVBVWbfNYFr ABUqRUBvTU18Se/EJO7g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q6Tgq-0018j2-0H; Tue, 06 Jun 2023 10:13:24 +0000 Received: from relay3-d.mail.gandi.net ([217.70.183.195]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q6Tgm-0018hp-2u for linux-arm-kernel@lists.infradead.org; Tue, 06 Jun 2023 10:13:22 +0000 X-GND-Sasl: maxime.chevallier@bootlin.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1686046395; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=RkbzNl9IfYOosD0T4TQy+60yjqN1tYHVKVHSGmiLV0c=; b=aMk1sVX/fUIK+Wi2jBs8zqKYJ/ZbN5EEiZR6MgBrlhP6ptv1tRLLheFfxr5A1NmcFN+S1q zCKpy2hNtEaMPiOiZfmq9nfxYUsrimD8UNSTtCwFnn5JByokY+YkdYNfgSqRBCLhI5DHqI ZidvgX5zn77fn3WLdZeY8HGDvsIXLsocSGL8Evunv1itRRlhr4GLLqWSoKXTvb4USc6awr vdwhf2lLhAkyQZgggss9qoyHSdOWH/tnPMeRN2RK9b5MTUkY5jqhJ6rfL/BYEwYZAhTaob 8Z/vrpbdMEgr/pb4R8igKXW6CDD+TZItBVJj+ll7mjKa9vEGSEJztIL0gXYd+Q== X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com X-GND-Sasl: maxime.chevallier@bootlin.com Received: by mail.gandi.net (Postfix) with ESMTPSA id 840E56000A; Tue, 6 Jun 2023 10:13:12 +0000 (UTC) Date: Tue, 6 Jun 2023 12:13:11 +0200 From: Maxime Chevallier To: "Russell King (Oracle)" Cc: Geert Uytterhoeven , davem@davemloft.net, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, alexis.lothore@bootlin.com, thomas.petazzoni@bootlin.com, Andrew Lunn , Jakub Kicinski , Eric Dumazet , Paolo Abeni , Florian Fainelli , Heiner Kallweit , Vladimir Oltean , Ioana Ciornei , linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, Maxime Coquelin , Jose Abreu , Alexandre Torgue , Giuseppe Cavallaro , Simon Horman Subject: Re: [PATCH net-next 1/2] net: stmmac: Add PCS_LYNX as a dependency for the whole driver Message-ID: <20230606121311.3cc5aa78@pc-7.home> In-Reply-To: References: <20230606064914.134945-1-maxime.chevallier@bootlin.com> <20230606064914.134945-2-maxime.chevallier@bootlin.com> <889297a0-88c3-90df-7752-efa00184859@linux-m68k.org> Organization: Bootlin X-Mailer: Claws Mail 4.1.1 (GTK 3.24.37; x86_64-redhat-linux-gnu) MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230606_031321_209555_6FC43CEC X-CRM114-Status: GOOD ( 18.86 ) X-BeenThere: linux-arm-kernel@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-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello Geert, Russell, On Tue, 6 Jun 2023 10:30:32 +0100 "Russell King (Oracle)" wrote: > On Tue, Jun 06, 2023 at 10:29:20AM +0200, Geert Uytterhoeven wrote: > > Hi Maxime, > > > > On Tue, 6 Jun 2023, Maxime Chevallier wrote: > > > Although pcs_lynx is only used on dwmac_socfpga for now, the cleanup > > > path is in the generic driver, and triggers build issues for other > > > stmmac variants. Make sure we build pcs_lynx in all cases too, as for > > > XPCS. > > > > That seems suboptimal to me, as it needlesly increases kernel size for > > people who do not use dwmac_socfpga. Hence I made an alternative patch: > > https://lore.kernel/org/7b36ac43778b41831debd5c30b5b37d268512195.1686039915.git.geert+renesas@glider.be > > A better solution would be to re-architect the removal code so that > whatever creates the PCS is also responsible for removing it. > > Also, dwmac_socfpga nees to be reorganised anyway, because it calls > stmmac_dvr_probe() which then goes on to call register_netdev(), > publishing the network device, and then after stmmac_dvr_probe(), > further device setup is done. As the basic driver probe flow should > be setup and then publish, the existing code structure violates that. > I agree that this solution is definitely suboptimal, I wanted mostly to get it fixed quickly as this breaks other stmmac variants. Do we still go on with the current patch (as Geert's has issues) and then consider reworking dwmac_socfpga ? Maxime _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel