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 BC0A6C001E0 for ; Mon, 23 Oct 2023 10:28: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:Subject:Cc:To:From:Date:References: In-Reply-To:Message-Id:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=XUf+J8ZQhf3dcnK33tMRhPKCx1ERQhf7c13nwoYs4lw=; b=F2Q0km/jb+J4nV PVouJi2aKrrlSmRsfjKWxRMQze6c8hbjGuRYczJQnhzwgfHdJW1Tbm9qlPSPsy5k8kRQWcP5bxsKX 0MciZ4aTcRx2Omi2LQGcnKlXSs1Umn0o+26pu9B4ulBpXqm5J+Qs0lwXmoEyfpZjnpCPTSGvvcuMl kReVx9lctFesY3oJAXRMMTNBn3dRhsEtZL6HZcdnU3wIRoIe6gBJkLDNl+HjSsDoIl88ZS5ToYNxa 6jpPqPZGuJHIN1FcN1Ooe+sQqg8+6LkkyoUvXSY8z3r4S1GISdwsuIEG27b/6v0sbmemmi7FjtYns TMtcpw/o/Kd705EIWPdg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qusAe-0071n4-07; Mon, 23 Oct 2023 10:28:28 +0000 Received: from out3-smtp.messagingengine.com ([66.111.4.27]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qusAa-0071lA-0T; Mon, 23 Oct 2023 10:28:26 +0000 Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.nyi.internal (Postfix) with ESMTP id 431875C027F; Mon, 23 Oct 2023 06:28:20 -0400 (EDT) Received: from imap51 ([10.202.2.101]) by compute5.internal (MEProxy); Mon, 23 Oct 2023 06:28:20 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-type:content-type:date:date:from:from:in-reply-to :in-reply-to:message-id:mime-version:references:reply-to:sender :subject:subject:to:to; s=fm2; t=1698056900; x=1698143300; bh=sl J4gsmLFLLG28yC7DauJDzx27bn8p/IiBTXbBZAE1A=; b=WYniU+v0LQ9c0HRur4 W10+/qiyVQOIdiVGnK1R2E9hrNKzDXTSU0hq8J4bC2cnYJcbRpRNlT3wl916l0K4 nLX4jRVrk496SlZ+bji5Yq5yeLalUVD2LQFfXoWEXLZ8LZOB3sWINDsnLL2sSQjb fm0lR5U5vj6M2clKJV0L3frCq/MbCPXHccMWGG7jYhnMDGH6T5+xHdisZ2EwQ33I K+FFjwgeGH0zl1BwLsxDV4O0X5oc3VArJB5pkCAUQ7QIgfcPmrJiOmsReHA6a/nW ifN+1Vi/bAS0Os93lQMoRhGy64uVeArjl+Pr6AroKuby3BvukV5QohvhRKvhjk30 95SQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-type:content-type:date:date :feedback-id:feedback-id:from:from:in-reply-to:in-reply-to :message-id:mime-version:references:reply-to:sender:subject :subject:to:to:x-me-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm3; t=1698056900; x=1698143300; bh=slJ4gsmLFLLG2 8yC7DauJDzx27bn8p/IiBTXbBZAE1A=; b=sacQjNjjZYyQ7+D6+xGCed3u3mK9m fYqOXgE9b8d8agnZbS8LXNv8upZhEMjYR10qc+lyTNZZPRUJjm6V0tG1SxKG4mPo TWJo3D5sXPCvqpn4ebNxF07vvoB1ee9MnD87npmBLRZIWWSF7xwjdglrsB3Rn67p 1iAHv9/xiVxV1ZWCw6zAckjMFCJh5gCQURLS0fA5QEnjRrvD2Ool9bVCyMWc7Hgf tMBUCzFiXQfJv4+VK9kF0r4O0ydFqEbThVcw+EnWzxHNanADToGvT49mmbv3+mRN U73AR3utLMJ9Uisf2jPacZh3IThl41en1CXROoxzDCsX9mMQp0mqfBKTA== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvkedrkeeigddvjecutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpqfgfvfdpuffrtefokffrpgfnqfghnecu uegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenuc fjughrpefofgggkfgjfhffhffvvefutgesthdtredtreertdenucfhrhhomhepfdetrhhn ugcuuegvrhhgmhgrnhhnfdcuoegrrhhnugesrghrnhgusgdruggvqeenucggtffrrghtth gvrhhnpeffheeugeetiefhgeethfejgfdtuefggeejleehjeeutefhfeeggefhkedtkeet ffenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmhgrihhlfhhrohhmpegrrh hnugesrghrnhgusgdruggv X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.nyi.internal (Postfix, from userid 501) id A5040B60089; Mon, 23 Oct 2023 06:28:19 -0400 (EDT) X-Mailer: MessagingEngine.com Webmail Interface User-Agent: Cyrus-JMAP/3.9.0-alpha0-1048-g9229b632c5-fm-20231019.001-g9229b632 MIME-Version: 1.0 Message-Id: In-Reply-To: <1jmswds9dn.fsf@starbuckisacylon.baylibre.com> References: <20231020132036.1181762-1-arnd@kernel.org> <1jmswds9dn.fsf@starbuckisacylon.baylibre.com> Date: Mon, 23 Oct 2023 12:27:56 +0200 From: "Arnd Bergmann" To: "Jerome Brunet" , "Arnd Bergmann" , "Neil Armstrong" , "Michael Turquette" , "Stephen Boyd" , "Kevin Hilman" , "Yu Tu" Cc: "Martin Blumenstingl" , "Dmitry Rokosov" , "Jian Hu" , linux-amlogic@lists.infradead.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] clk: meson: S4: select CONFIG_COMMON_CLK_MESON_CLKC_UTILS X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231023_032824_735214_F47B0477 X-CRM114-Status: GOOD ( 13.95 ) 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 On Fri, Oct 20, 2023, at 15:35, Jerome Brunet wrote: > On Fri 20 Oct 2023 at 15:19, Arnd Bergmann wrote: > >> From: Arnd Bergmann >> >> Without this, the newly added driver fails to link: >> >> aarch64-linux-ld: drivers/clk/meson/s4-pll.o: in function `meson_s4_pll_probe': >> s4-pll.c:(.text+0x13c): undefined reference to `meson_clk_hw_get' >> >> Fixes: e787c9c55edad ("clk: meson: S4: add support for Amlogic S4 SoC PLL clock driver") >> Signed-off-by: Arnd Bergmann >> --- >> drivers/clk/meson/Kconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/clk/meson/Kconfig b/drivers/clk/meson/Kconfig >> index c5303e4c16043..3c28560b0faba 100644 >> --- a/drivers/clk/meson/Kconfig >> +++ b/drivers/clk/meson/Kconfig >> @@ -149,6 +149,7 @@ config COMMON_CLK_S4_PLL >> tristate "S4 SoC PLL clock controllers support" >> depends on ARM64 >> default y >> + select COMMON_CLK_MESON_CLKC_UTILS >> select COMMON_CLK_MESON_MPLL >> select COMMON_CLK_MESON_PLL >> select COMMON_CLK_MESON_REGMAP > > Thx Arnd and sorry about this. > > Reviewed-by: Jerome Brunet > > Stephen, do you mind taking this directly ? I ran into the same bug with drivers/clk/meson/s4-peripherals.c, sending a v2 of my patch to fix both at the same time, assuming v1 has not been applied yet (it's not in today's linux-next). Arnd _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic