Message ID | 20231113154522.0bca3521@endymion.delvare |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b909:0:b0:403:3b70:6f57 with SMTP id t9csp1250471vqg; Mon, 13 Nov 2023 06:46:40 -0800 (PST) X-Google-Smtp-Source: AGHT+IFoXzSZyQNYcLxVzf0Vw3RctCQaOkwFcJ8ljhTzYsUIapjsLpEerfLBp189i0AgCJhY0xiP X-Received: by 2002:a17:90b:1b04:b0:280:9427:1e8f with SMTP id nu4-20020a17090b1b0400b0028094271e8fmr4323139pjb.46.1699886800707; Mon, 13 Nov 2023 06:46:40 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1699886800; cv=none; d=google.com; s=arc-20160816; b=l0UMGgL2QrCMvwWePU5qP8Mps82eddf4Rwwjf/ni4/vWY+GjfAbSwJjKYGmGrPYzgw C/AA8osHV4oBaHnkkTxMB/KEL4VHuswje/JwsPGUUmAn7z1eJB/rcDKsL3A95SfUXRmQ jSyQ2Qg8MyIRl9C7mdQkj54XjW+MRKchkP4kafT8Qp0u3aUiHfvntLmHFHTi3bwG58Dh 0TydrELsYluDxBStfI8SFqQSvtBn+fMLwYpX0wKDYO2Dt/mn45xQw6S0xBzu6SIBOq6d 0n4EgAztoXNT0RXjkleNOSF8kwaRoHI1wG8BQE3Psyn5dIPs0wj8sH91HI2kFAlLjSRT TJyA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :organization:message-id:subject:cc:to:from:date:dkim-signature :dkim-signature; bh=C55/Qkp749YOdFLW3Ql7AV3TyMx7ZOja0OEUL1O7cmw=; fh=CuF6RYiWquiuQh72A3GygaFseY6MPnO8X6klyTP/3O0=; b=G7p+v+wTIAUkGmWtSRNRG3t0IqFP5Lub6QYzlNMAr4ntGqMMMSmcneUjkFLfCE1ksR 4KR4Ma4auv2Q4JIC0MeoEyMBwb90/9iPvci/RWtthu/N68Tq4hh8z3yLT4KQ8qcXglfY iyLwaZTo7xEGA25tNd5/RwkmW0P5OfzRyxi0hZTZob7yFAXi5C03AAIVaIU0evXA3POu zAffgutG9w/F4j94ExS/1IkoRiiqWdzMuPVzR+LI/RFWshv9si0HJwBYr/mOrg0EgLzm +POcgQ3GnXyMVWS5zEIvXGERM4k0By4uuDGUCfBTtgLFE8HQ6R5qnadop62ZQhlz4rMu mI3Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@suse.de header.s=susede2_rsa header.b=qF9JMBBP; dkim=neutral (no key) header.i=@suse.de; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=suse.de Received: from groat.vger.email (groat.vger.email. [2620:137:e000::3:5]) by mx.google.com with ESMTPS id lk11-20020a17090b33cb00b002806cdeecc6si11446446pjb.35.2023.11.13.06.46.40 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 13 Nov 2023 06:46:40 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5; Authentication-Results: mx.google.com; dkim=pass header.i=@suse.de header.s=susede2_rsa header.b=qF9JMBBP; dkim=neutral (no key) header.i=@suse.de; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=suse.de Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by groat.vger.email (Postfix) with ESMTP id D324B807C650; Mon, 13 Nov 2023 06:46:36 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at groat.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231640AbjKMOqL (ORCPT <rfc822;heyuhang3455@gmail.com> + 29 others); Mon, 13 Nov 2023 09:46:11 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49854 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231633AbjKMOpy (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Mon, 13 Nov 2023 09:45:54 -0500 Received: from smtp-out1.suse.de (smtp-out1.suse.de [IPv6:2001:67c:2178:6::1c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B9B1A19AD; Mon, 13 Nov 2023 06:45:26 -0800 (PST) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out1.suse.de (Postfix) with ESMTPS id 314712187C; Mon, 13 Nov 2023 14:45:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1699886724; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=C55/Qkp749YOdFLW3Ql7AV3TyMx7ZOja0OEUL1O7cmw=; b=qF9JMBBPs6ELBULaJOXIQpMdwXEebNyty/ILRRrgEWaAkekhfn9984wQY7H6/cBJC+bZAE Zg4Fbai+9c8354J73virj+hNnWpkzFP4OQxSEINqwU+LfokvIO65p4+e6Vf2MmcGc2QENO 5A2JFHjJRc86q/DoCgcxY21Jo4DlYqw= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1699886724; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=C55/Qkp749YOdFLW3Ql7AV3TyMx7ZOja0OEUL1O7cmw=; b=5IDGdCbNOcZw4YEcqrXFOuDhcXXlLZTpdU/bEj2+Qwa1iJdVEBcvyWfBzZq5VYQwN0+vNW D6DGVF2logXv5JCA== Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id F2E5613398; Mon, 13 Nov 2023 14:45:23 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id oS+aOYM2UmX/MwAAMHmgww (envelope-from <jdelvare@suse.de>); Mon, 13 Nov 2023 14:45:23 +0000 Date: Mon, 13 Nov 2023 15:45:22 +0100 From: Jean Delvare <jdelvare@suse.de> To: Keguang Zhang <keguang.zhang@gmail.com> Cc: linux-mips@vger.kernel.org, netdev@vger.kernel.org, LKML <linux-kernel@vger.kernel.org> Subject: |PATCH] stmmac: dwmac-loongson: Add architecture dependency Message-ID: <20231113154522.0bca3521@endymion.delvare> Organization: SUSE Linux X-Mailer: Claws Mail 4.0.0 (GTK+ 3.24.34; x86_64-suse-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on groat.vger.email Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (groat.vger.email [0.0.0.0]); Mon, 13 Nov 2023 06:46:37 -0800 (PST) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1782460501635560728 X-GMAIL-MSGID: 1782460501635560728 |
Series |
|PATCH] stmmac: dwmac-loongson: Add architecture dependency
|
|
Commit Message
Jean Delvare
Nov. 13, 2023, 2:45 p.m. UTC
Only present the DWMAC_LOONGSON option on architectures where it can
actually be used.
This follows the same logic as the DWMAC_INTEL option.
Signed-off-by: Jean Delvare <jdelvare@suse.de>
Cc: Keguang Zhang <keguang.zhang@gmail.com>
---
I'm not familiar with the hardware, so please let me know if the
dependency needs to be adjusted somehow.
drivers/net/ethernet/stmicro/stmmac/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Comments
On Mon, Nov 13, 2023 at 03:45:22PM +0100, Jean Delvare wrote: > Only present the DWMAC_LOONGSON option on architectures where it can > actually be used. > > This follows the same logic as the DWMAC_INTEL option. > > Signed-off-by: Jean Delvare <jdelvare@suse.de> > Cc: Keguang Zhang <keguang.zhang@gmail.com> > --- > I'm not familiar with the hardware, so please let me know if the > dependency needs to be adjusted somehow. > > drivers/net/ethernet/stmicro/stmmac/Kconfig | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > --- linux-6.6.orig/drivers/net/ethernet/stmicro/stmmac/Kconfig > +++ linux-6.6/drivers/net/ethernet/stmicro/stmmac/Kconfig > @@ -269,7 +269,7 @@ config DWMAC_INTEL > config DWMAC_LOONGSON > tristate "Loongson PCI DWMAC support" > default MACH_LOONGSON64 > - depends on STMMAC_ETH && PCI > + depends on MACH_LOONGSON64 && STMMAC_ETH && PCI Hi Jean, Could we consider also allowing the build to occur if COMPILE_TEST is set? This would maintain the current level of build test coverage. Something line this (completely untested!): depends on (MACH_LOONGSON64 || COMPILE_TEST) && STMMAC_ETH && PCI > depends on COMMON_CLK > help > This selects the LOONGSON PCI bus support for the stmmac driver, > > > -- > Jean Delvare > SUSE L3 Support >
Hi Simon, On Mon, 13 Nov 2023 18:01:07 +0000, Simon Horman wrote: > On Mon, Nov 13, 2023 at 03:45:22PM +0100, Jean Delvare wrote: > > Only present the DWMAC_LOONGSON option on architectures where it can > > actually be used. > > > > This follows the same logic as the DWMAC_INTEL option. > > > > Signed-off-by: Jean Delvare <jdelvare@suse.de> > > Cc: Keguang Zhang <keguang.zhang@gmail.com> > > --- > > I'm not familiar with the hardware, so please let me know if the > > dependency needs to be adjusted somehow. > > > > drivers/net/ethernet/stmicro/stmmac/Kconfig | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > --- linux-6.6.orig/drivers/net/ethernet/stmicro/stmmac/Kconfig > > +++ linux-6.6/drivers/net/ethernet/stmicro/stmmac/Kconfig > > @@ -269,7 +269,7 @@ config DWMAC_INTEL > > config DWMAC_LOONGSON > > tristate "Loongson PCI DWMAC support" > > default MACH_LOONGSON64 > > - depends on STMMAC_ETH && PCI > > + depends on MACH_LOONGSON64 && STMMAC_ETH && PCI > > Could we consider also allowing the build to occur if COMPILE_TEST is set? > This would maintain the current level of build test coverage. > > Something line this (completely untested!): > > depends on (MACH_LOONGSON64 || COMPILE_TEST) && STMMAC_ETH && PCI Sure, that would be totally fine with me.
On Mon, Nov 13, 2023 at 07:33:47PM +0100, Jean Delvare wrote: > Hi Simon, > > On Mon, 13 Nov 2023 18:01:07 +0000, Simon Horman wrote: > > On Mon, Nov 13, 2023 at 03:45:22PM +0100, Jean Delvare wrote: > > > Only present the DWMAC_LOONGSON option on architectures where it can > > > actually be used. > > > > > > This follows the same logic as the DWMAC_INTEL option. > > > > > > Signed-off-by: Jean Delvare <jdelvare@suse.de> > > > Cc: Keguang Zhang <keguang.zhang@gmail.com> > > > --- > > > I'm not familiar with the hardware, so please let me know if the > > > dependency needs to be adjusted somehow. > > > > > > drivers/net/ethernet/stmicro/stmmac/Kconfig | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > --- linux-6.6.orig/drivers/net/ethernet/stmicro/stmmac/Kconfig > > > +++ linux-6.6/drivers/net/ethernet/stmicro/stmmac/Kconfig > > > @@ -269,7 +269,7 @@ config DWMAC_INTEL > > > config DWMAC_LOONGSON > > > tristate "Loongson PCI DWMAC support" > > > default MACH_LOONGSON64 > > > - depends on STMMAC_ETH && PCI > > > + depends on MACH_LOONGSON64 && STMMAC_ETH && PCI > > > > Could we consider also allowing the build to occur if COMPILE_TEST is set? > > This would maintain the current level of build test coverage. > > > > Something line this (completely untested!): > > > > depends on (MACH_LOONGSON64 || COMPILE_TEST) && STMMAC_ETH && PCI > > Sure, that would be totally fine with me. Thanks, I think that if no one objects within the next day or so then posting a v2 would be a good next step.
--- linux-6.6.orig/drivers/net/ethernet/stmicro/stmmac/Kconfig +++ linux-6.6/drivers/net/ethernet/stmicro/stmmac/Kconfig @@ -269,7 +269,7 @@ config DWMAC_INTEL config DWMAC_LOONGSON tristate "Loongson PCI DWMAC support" default MACH_LOONGSON64 - depends on STMMAC_ETH && PCI + depends on MACH_LOONGSON64 && STMMAC_ETH && PCI depends on COMMON_CLK help This selects the LOONGSON PCI bus support for the stmmac driver,