Received: by 2002:a25:f815:0:0:0:0:0 with SMTP id u21csp2020761ybd; Sun, 23 Jun 2019 22:37:33 -0700 (PDT) X-Google-Smtp-Source: APXvYqxaVCTPk/x7LSiu2SorPMLZLiavkUKQfx+goBl2Hp28tDtN7dP2mD/U9xBPmMXU2MWYpMAl X-Received: by 2002:a17:902:76c6:: with SMTP id j6mr120956780plt.263.1561354653856; Sun, 23 Jun 2019 22:37:33 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1561354653; cv=none; d=google.com; s=arc-20160816; b=eiWo/fBZULAJS6xIv9/KpgeI2cT2i6Fcz/tg/6W+OSM1a2q78USLuHif9wejje5tun 5+JzBjVKv49L2v5pZDBKcfxeyDeAI8Kf8a50blo+8hC8EZ/JZwH93eo3q6WNQVvP9zsZ S68pCyWMhBrO/R4OO9ZUSh8TUQEtn62twFfKc7le3ytHwTtYVoaFrbnInALHEFJsh4bv lu9iUdrEf/ShYOwF5J0jyCf0/VpjJFy93/9pyVqJi8m0IOr4q5lws3mUTaU/goyIqwAo +9KkPPQwFPzC28axRpt/i/mL9+K10S5BKUWKtasdv47aONZymR7fc56j9WDBW+r/ocXM f1qA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:to:from:cc:in-reply-to:subject:date; bh=DephS9aF1RqbAgEf2w/teieLK755TA9/UUG6864ldTk=; b=f3b4I38q6UhXfgtpYYWr67QS+0ktjULpbWnck9k0QdplQVoAj3Q5tRvbweitUtEMQn eKgbzZrR+f4Qvg190aqbSmPU0EdOBJJvjZgmn/FjUA7L7GC1JpVWP733VQXu+N7fiMHG +F2VkvL2hd4fRt5W/wZcUuR4J/F8AilWthCs9ZNicLukq4IyGXQ4A4aMlme7OKnZTNiZ pospknsZoIvPRYhvegHN01oKF9r873MLeVKOayN2J7jdRgYekZ8eWexFw8X4G/CUWQib yaM0QyVHx/EEHwvEWNprh6qoEjnWsAE6fDJHKxjHd4blO2EejB2epUhhb6HS35j+XKHU TnXA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id a11si9110741plm.168.2019.06.23.22.36.41; Sun, 23 Jun 2019 22:37:33 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727731AbfFXD7t (ORCPT + 99 others); Sun, 23 Jun 2019 23:59:49 -0400 Received: from mail-pf1-f195.google.com ([209.85.210.195]:43901 "EHLO mail-pf1-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726476AbfFXD7s (ORCPT ); Sun, 23 Jun 2019 23:59:48 -0400 Received: by mail-pf1-f195.google.com with SMTP id i189so6693863pfg.10 for ; Sun, 23 Jun 2019 20:59:48 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:subject:in-reply-to:cc:from:to:message-id :mime-version:content-transfer-encoding; bh=DephS9aF1RqbAgEf2w/teieLK755TA9/UUG6864ldTk=; b=jT/wq8XmUCz8Im8200YDWVs5dje0T+QGCmO5uSugbAC1PeL/PQgjnKSfwebDtHvf3n YldtzdHrE/5KKVZmbJziVLUtAx6SVLxutPXLLF+MHR3fvJbd/QJWTueh0pzR4t5AkQGF WRHi7MxYHtca7JNfQBUaViLHjuBQ3BJ/iei2xoOztQMKceDgC5vHBxMyAWD7ztp+2s3Q eQdQUiyk4+Q8xOP2PkDxfOdd4JKnreswB7Lr92GC+u7+SJL73h0WQsxFI0/bO5TP20pk bCavE76fh34nKkUhNZq7JkCV2cTGThiNzeopAVWjO+BqPYSs3OH7k3IU6xkjDNJ/70+J 7Dyg== X-Gm-Message-State: APjAAAWUO8+uaTT7FgS5j1eHN7qDMOTPApfS2DHx17iShTUwErTFn04N DOp3kbfCTJgdVnheVPmH+KelSw== X-Received: by 2002:a63:3d09:: with SMTP id k9mr24980118pga.321.1561348787037; Sun, 23 Jun 2019 20:59:47 -0700 (PDT) Received: from localhost (220-132-236-182.HINET-IP.hinet.net. [220.132.236.182]) by smtp.gmail.com with ESMTPSA id t24sm9569893pfh.113.2019.06.23.20.59.46 (version=TLS1_3 cipher=AEAD-AES256-GCM-SHA384 bits=256/256); Sun, 23 Jun 2019 20:59:46 -0700 (PDT) Date: Sun, 23 Jun 2019 20:59:46 -0700 (PDT) X-Google-Original-Date: Sun, 23 Jun 2019 20:59:43 PDT (-0700) Subject: Re: linux-next: build failure after merge of the net-next tree In-Reply-To: <20190624131245.359e59a4@canb.auug.org.au> CC: davem@davemloft.net, netdev@vger.kernel.org, linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, yash.shah@sifive.com From: Palmer Dabbelt To: Stephen Rothwell Message-ID: Mime-Version: 1.0 (MHng) Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sun, 23 Jun 2019 20:12:45 PDT (-0700), Stephen Rothwell wrote: > Hi all, > > On Thu, 20 Jun 2019 19:13:48 +1000 Stephen Rothwell wrote: >> >> After merging the net-next tree, today's linux-next build (powerpc >> allyesconfig) failed like this: >> >> drivers/net/ethernet/cadence/macb_main.c:48:16: error: field 'hw' has incomplete type >> struct clk_hw hw; >> ^~ >> drivers/net/ethernet/cadence/macb_main.c:4003:21: error: variable 'fu540_c000_ops' has initializer but incomplete type >> static const struct clk_ops fu540_c000_ops = { >> ^~~~~~~ >> drivers/net/ethernet/cadence/macb_main.c:4004:3: error: 'const struct clk_ops' has no member named 'recalc_rate' >> .recalc_rate = fu540_macb_tx_recalc_rate, >> ^~~~~~~~~~~ >> drivers/net/ethernet/cadence/macb_main.c:4004:17: warning: excess elements in struct initializer >> .recalc_rate = fu540_macb_tx_recalc_rate, >> ^~~~~~~~~~~~~~~~~~~~~~~~~ >> drivers/net/ethernet/cadence/macb_main.c:4004:17: note: (near initialization for 'fu540_c000_ops') >> drivers/net/ethernet/cadence/macb_main.c:4005:3: error: 'const struct clk_ops' has no member named 'round_rate' >> .round_rate = fu540_macb_tx_round_rate, >> ^~~~~~~~~~ >> drivers/net/ethernet/cadence/macb_main.c:4005:16: warning: excess elements in struct initializer >> .round_rate = fu540_macb_tx_round_rate, >> ^~~~~~~~~~~~~~~~~~~~~~~~ >> drivers/net/ethernet/cadence/macb_main.c:4005:16: note: (near initialization for 'fu540_c000_ops') >> drivers/net/ethernet/cadence/macb_main.c:4006:3: error: 'const struct clk_ops' has no member named 'set_rate' >> .set_rate = fu540_macb_tx_set_rate, >> ^~~~~~~~ >> drivers/net/ethernet/cadence/macb_main.c:4006:14: warning: excess elements in struct initializer >> .set_rate = fu540_macb_tx_set_rate, >> ^~~~~~~~~~~~~~~~~~~~~~ >> drivers/net/ethernet/cadence/macb_main.c:4006:14: note: (near initialization for 'fu540_c000_ops') >> drivers/net/ethernet/cadence/macb_main.c: In function 'fu540_c000_clk_init': >> drivers/net/ethernet/cadence/macb_main.c:4013:23: error: storage size of 'init' isn't known >> struct clk_init_data init; >> ^~~~ >> drivers/net/ethernet/cadence/macb_main.c:4032:12: error: implicit declaration of function 'clk_register'; did you mean 'sock_register'? [-Werror=implicit-function-declaration] >> *tx_clk = clk_register(NULL, &mgmt->hw); >> ^~~~~~~~~~~~ >> sock_register >> drivers/net/ethernet/cadence/macb_main.c:4013:23: warning: unused variable 'init' [-Wunused-variable] >> struct clk_init_data init; >> ^~~~ >> drivers/net/ethernet/cadence/macb_main.c: In function 'macb_probe': >> drivers/net/ethernet/cadence/macb_main.c:4366:2: error: implicit declaration of function 'clk_unregister'; did you mean 'sock_unregister'? [-Werror=implicit-function-declaration] >> clk_unregister(tx_clk); >> ^~~~~~~~~~~~~~ >> sock_unregister >> drivers/net/ethernet/cadence/macb_main.c: At top level: >> drivers/net/ethernet/cadence/macb_main.c:4003:29: error: storage size of 'fu540_c000_ops' isn't known >> static const struct clk_ops fu540_c000_ops = { >> ^~~~~~~~~~~~~~ >> >> Caused by commit >> >> c218ad559020 ("macb: Add support for SiFive FU540-C000") >> >> CONFIG_COMMON_CLK is not set for this build. >> >> I have reverted that commit for today. > > I am still reverting that commit. Has this problem been fixed in some > subtle way? I don't think so. I'm assuming something like this is necessary diff --git a/drivers/net/ethernet/cadence/Kconfig b/drivers/net/ethernet/cadence/Kconfig index 1766697c9c5a..d13db9e9c818 100644 --- a/drivers/net/ethernet/cadence/Kconfig +++ b/drivers/net/ethernet/cadence/Kconfig @@ -23,6 +23,7 @@ config MACB tristate "Cadence MACB/GEM support" depends on HAS_DMA select PHYLIB + depends on COMMON_CLK ---help--- The Cadence MACB ethernet interface is found on many Atmel AT32 and AT91 parts. This driver also supports the Cadence GEM (Gigabit @@ -42,7 +43,7 @@ config MACB_USE_HWSTAMP config MACB_PCI tristate "Cadence PCI MACB/GEM support" - depends on MACB && PCI && COMMON_CLK + depends on MACB && PCI ---help--- This is PCI wrapper for MACB driver. at a minimum, though it may be saner to #ifdef support for the SiFive clock driver as that's only useful on some systems. Assuming I can reproduce the build failure (which shouldn't be too hard), I'll send out a patch that adds a Kconfig for the FU540 clock driver to avoid adding a COMMON_CLK dependency for all MACB systems.