From patchwork Fri Jun 16 09:29:54 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 13282401 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 8817CEB64D7 for ; Fri, 16 Jun 2023 09:30:41 +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:Message-Id:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=5QQy8YTh4k/gsT4zZNS+MB4GyY3UtCu7wEDKYUhn+2o=; b=zoUU+uQdXPidXF v49Uim9+L/jfi/QmHJTbJF9pmcGkpCJomhyOZRcHBXKXN4uU0W05BhEsLZqHzeJtsZVkueqo2OUOP r4z0IeiEgs8caZglr+DNuFLAnefmANnl58BWQ6TQmzCb+UBhywfXLyPXbSIlGTZSGAocaoLEhw4wU 29MSObKHBkjW9N//yTfMsafOjrEQaQT56kAOamAZ2B3GtMOEE4pNAMpkVqOfOlUV8J4ehQL7Bq/Jg UtUGFQ9B0oCCjeUTWqvFA3N3sQmSw5b1MjlG2gsfxXOlCqfWfHhWS4ffZM4YnKfwk6OYboCp7LUmj HiNg6kOZcUGj7FMA65WQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qA5mf-00HSfH-2g; Fri, 16 Jun 2023 09:30:21 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qA5md-00HSeX-0g; Fri, 16 Jun 2023 09:30:20 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 59C09622F4; Fri, 16 Jun 2023 09:30:18 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id CF97EC433CA; Fri, 16 Jun 2023 09:30:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1686907817; bh=0LCiNW47vz9emKUNT6ghLeNK5ZkmMsQ8k9618ObBVAY=; h=From:To:Cc:Subject:Date:From; b=RFV0xw4rvVEg653MXihwltNfQZDxkpwhr2f/xjdVLCt8vLzv0j2Kx7BJQZBcCMew8 jpi1fc4WiXoZMdDVhDV2ZA6zF2ybZxYKtrWyRnOYuG11gj8ojIUoul6c+BaIcDN5ed 2CSLn8KDPvgytauycmr4EiD7bUB4gxFYyvc02Yij5ROssojdhQh0tReT+o0Cu8gV6Z 7xOLWD64XznEkiOolp5NfjxqQ6PgaLIGR+/hIzc6TLd/k3rQ5G7+lCYBrfX/AFcKbj AhqtzTzv1a+pwziGilkVzyk+WQN9knrCIUdprSrdstxLQX4EoW8LZWf7GJV5gRfKih STgq8ZDc0xN5A== From: Arnd Bergmann To: Andrew Lunn , Heiner Kallweit Cc: Arnd Bergmann , Russell King , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Matthias Brugger , AngeloGioacchino Del Regno , =?utf-8?q?Ram=C3=B3n_Nordin_Rodriguez?= , Parthiban Veerasooran , Frank Sae , Michael Walle , Daniel Golle , Piergiorgio Beruto , Vladimir Oltean , Jonathan Lemon , Randy Dunlap , netdev@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Subject: [PATCH] net: phy: mediatek: fix compile-test dependencies Date: Fri, 16 Jun 2023 11:29:54 +0200 Message-Id: <20230616093009.3511692-1-arnd@kernel.org> X-Mailer: git-send-email 2.39.2 MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230616_023019_312296_97525A72 X-CRM114-Status: GOOD ( 17.66 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org From: Arnd Bergmann The new phy driver attempts to select a driver from another subsystem, but that fails when the NVMEM subsystem is disabled: WARNING: unmet direct dependencies detected for NVMEM_MTK_EFUSE Depends on [n]: NVMEM [=n] && (ARCH_MEDIATEK [=n] || COMPILE_TEST [=y]) && HAS_IOMEM [=y] Selected by [y]: - MEDIATEK_GE_SOC_PHY [=y] && NETDEVICES [=y] && PHYLIB [=y] && (ARM64 && ARCH_MEDIATEK [=n] || COMPILE_TEST [=y]) I could not see an actual compile time dependency, so presumably this is only needed for for working correctly but not technically a dependency on that particular nvmem driver implementation, so it would likely be safe to remove the select for compile testing. To keep the spirit of the original 'select', just replace this with a 'depends on' that ensures that the driver will work but does not get in the way of build testing. Fixes: 98c485eaf509b ("net: phy: add driver for MediaTek SoC built-in GE PHYs") Signed-off-by: Arnd Bergmann Reviewed-by: Simon Horman Acked-by: Randy Dunlap Tested-by: Randy Dunlap # build-tested Reviewed-by: Daniel Golle --- drivers/net/phy/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/phy/Kconfig b/drivers/net/phy/Kconfig index a40269c175974..78e6981650d94 100644 --- a/drivers/net/phy/Kconfig +++ b/drivers/net/phy/Kconfig @@ -239,7 +239,7 @@ config MEDIATEK_GE_PHY config MEDIATEK_GE_SOC_PHY tristate "MediaTek SoC Ethernet PHYs" depends on (ARM64 && ARCH_MEDIATEK) || COMPILE_TEST - select NVMEM_MTK_EFUSE + depends on NVMEM_MTK_EFUSE help Supports MediaTek SoC built-in Gigabit Ethernet PHYs.