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 11402D69113 for ; Thu, 28 Nov 2024 14:13:15 +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:References:In-Reply-To: Message-Id:Cc:To:From:Date:MIME-Version:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=2F7sUnIKQgIAtV6z02DSRMVttSsy1hj5g57jCj35vRo=; b=b/cSTv3ZmydvZiOx8eY7FQl08J 4FEeAoCWINvLZB32PCw1PauPxa2aWZStnQgDxW7pF5v5QsJ9Y/yFgXgsCueQgbRDY1o3MC/EbABE4 pAB3RdL2zMwDhtUmyMpo/Uk24mvUnEJtyjh1HHHhXbpRiQu4N37vYF2pkHZZYbVvD5Sa9MSb8F/wI l+SSVL06TbamX5LAF59klZjXU1G3gdwRBmPIv+ANbkqZ/d3fuCDjExQrf0VKL3IHfjHu46UkAMtmC hlapP7V1YlWokx/Vox86Qa5ZhPIuRQgBxwsZXdcpwyGZ++PjxPNlCq0ZezeZspCk64C5/Q6Zz4L9D 9SkAeAPw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tGfGX-0000000FjGo-3q6Y; Thu, 28 Nov 2024 14:13:09 +0000 Received: from fout-b2-smtp.messagingengine.com ([202.12.124.145]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tGfFl-0000000Fj7X-0tuP; Thu, 28 Nov 2024 14:12:23 +0000 Received: from phl-compute-10.internal (phl-compute-10.phl.internal [10.202.2.50]) by mailfout.stl.internal (Postfix) with ESMTP id BDAFC114018B; Thu, 28 Nov 2024 09:12:17 -0500 (EST) Received: from phl-imap-11 ([10.202.2.101]) by phl-compute-10.internal (MEProxy); Thu, 28 Nov 2024 09:12:18 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=arndb.de; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:subject:subject:to:to; s=fm3; t=1732803137; x=1732889537; bh=53dfOzVmJQypdRBGQEw+/C6xquQgJGoz2MLq/Iv9RFQ=; b= cLj0RLHGjpFa5JldyNUe4RiLwdBQ+gSbloKo02q7zR9V3veBsqm9sC7x1yBrbUgB /U47zkwzcAEPnz2tAZ0ZGYqGV/ePwBwgvLWhE5CxcQwMZdyAeb7+K/Rq3c8OkWmU /NkqMzVpmyW1Y883daQXrRuPSWhc70sarEbsnDBx/wDdlElmN3ZKa+hm2XKkEYUz BDBusArMWZrcziEc4QrXa5FhkX38tO+CSBpJNNIsouCQY0XoifukOKWsdvZf+ZC7 CQ3KZwSC82yO2eSfyHyMZnevEXyrO7KWBNllVPiXc7P1RBMQiET+BotOaxQfKWDO AuCpI9Uu/y4oZ61Uvf+org== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :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:subject:subject:to:to:x-me-proxy :x-me-sender:x-me-sender:x-sasl-enc; s=fm1; t=1732803137; x= 1732889537; bh=53dfOzVmJQypdRBGQEw+/C6xquQgJGoz2MLq/Iv9RFQ=; b=c 7oCvGV3Q1h7CGzm+YVKdM608SLP54gfOuWP7MTPiUt2o+BVyspiCsJkR4ng0vYG/ 9747WXl/Iv08O//s69JBuHm1g5j5i5y6/jWunD5EZh6pzW586ej1fawW08izJR9i ujPNzbuK+UOD0hILKdqnP0DdvaVpSPVoNpgjHIb9G9wDw79i2niQIuA5DLRELqXs TWkUxUkZfMRoR6E50MaWsukZ5pl0NOvgpaYRZJNRkpJNGeWNCgv/I41Y9z1UgBgj bi8nWWBwlZnxVa301dsMgnw/0lXqOYWtToCk/2zS3K5vZXYH5kIV52YmlLRuSnJz KS4Ow9sM5x22PAySJGULw== X-ME-Sender: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeefuddrhedugdeitdcutefuodetggdotefrodftvf curfhrohhfihhlvgemucfhrghsthforghilhdpggftfghnshhusghstghrihgsvgdpuffr tefokffrpgfnqfghnecuuegrihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnth hsucdlqddutddtmdenucfjughrpefoggffhffvvefkjghfufgtgfesthejredtredttden ucfhrhhomhepfdetrhhnugcuuegvrhhgmhgrnhhnfdcuoegrrhhnugesrghrnhgusgdrug gvqeenucggtffrrghtthgvrhhnpefhtdfhvddtfeehudekteeggffghfejgeegteefgffg vedugeduveelvdekhfdvieenucevlhhushhtvghrufhiiigvpedtnecurfgrrhgrmhepmh grihhlfhhrohhmpegrrhhnugesrghrnhgusgdruggvpdhnsggprhgtphhtthhopeduuddp mhhouggvpehsmhhtphhouhhtpdhrtghpthhtohepjhgsrhhunhgvthessggrhihlihgsrh gvrdgtohhmpdhrtghpthhtohepkhhhihhlmhgrnhessggrhihlihgsrhgvrdgtohhmpdhr tghpthhtohepmhhtuhhrqhhuvghtthgvsegsrgihlhhisghrvgdrtghomhdprhgtphhtth hopehmrghrthhinhdrsghluhhmvghnshhtihhnghhlsehgohhoghhlvghmrghilhdrtgho mhdprhgtphhtthhopegsrhhoohhnihgvsehkvghrnhgvlhdrohhrghdprhgtphhtthhope hssghohigusehkvghrnhgvlhdrohhrghdprhgtphhtthhopehnvghilhdrrghrmhhsthhr ohhngheslhhinhgrrhhordhorhhgpdhrtghpthhtoheplhhinhhugidqrghmlhhoghhitg eslhhishhtshdrihhnfhhrrgguvggrugdrohhrghdprhgtphhtthhopehlihhnuhigqdgr rhhmqdhkvghrnhgvlheslhhishhtshdrihhnfhhrrgguvggrugdrohhrgh X-ME-Proxy: Feedback-ID: i56a14606:Fastmail Received: by mailuser.phl.internal (Postfix, from userid 501) id 4FE182220072; Thu, 28 Nov 2024 09:12:17 -0500 (EST) X-Mailer: MessagingEngine.com Webmail Interface MIME-Version: 1.0 Date: Thu, 28 Nov 2024 15:11:56 +0100 From: "Arnd Bergmann" To: "Jerome Brunet" Cc: "Neil Armstrong" , "Michael Turquette" , "Stephen Boyd" , "Kevin Hilman" , "Martin Blumenstingl" , linux-amlogic@lists.infradead.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, "Mark Brown" Message-Id: <306b0b30-5a32-4c7c-86b4-57d50e2307e8@app.fastmail.com> In-Reply-To: <1j4j3r32ld.fsf@starbuckisacylon.baylibre.com> References: <20241127-clk-audio-fix-rst-missing-v1-1-9f9d0ab98fce@baylibre.com> <12f29978-c8ce-4bee-a447-dcd086eb936d@app.fastmail.com> <1ja5dk2y5l.fsf@starbuckisacylon.baylibre.com> <1j4j3r32ld.fsf@starbuckisacylon.baylibre.com> Subject: Re: [PATCH] clk: amlogic: axg-audio: select RESET_MESON_AUX X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241128_061221_882083_F558F038 X-CRM114-Status: GOOD ( 20.68 ) 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 Thu, Nov 28, 2024, at 14:33, Jerome Brunet wrote: > On Wed 27 Nov 2024 at 22:23, "Arnd Bergmann" wrote: >> On Wed, Nov 27, 2024, at 21:56, Jerome Brunet wrote: >>> On Wed 27 Nov 2024 at 20:30, "Arnd Bergmann" wrote: >>>> >>>> It looks like RESET_MESON_AUX is a user-visible symbol, >>>> so you can simply ask users to turn it on, and add it to >>>> the defconfig. >>> >>> That would work yes but It's really something a user should not be >>> concerned with. I can follow-up with another change to remove the user >>> visibilty of RESET_MESON_AUX. It is always going to be something >>> requested by another driver. >> >> But that's true for all reset drivers, each one of them is >> only useful because it's going to be used by another driver, >> same for clk, pinctrl, regulator, ... >> > > All clk, pinctrl or regulator are used by other driver yes, this one as > well, sure. > > What special about this on is that it is an auxiliary bus driver. > It is directly instantiated by another driver. That's where > it differs. The axg-audio clock driver instantiate the auxiliary reset, > it does not use it, which is why it makes sense for it to select the > driver. Can you explain the logic behind this design? It seems that the entire problem here is the split into more drivers than it should be. It's common for clk drivers to also act as a reset driver, and my impression here is that you were trying too hard to split out the reset functionality into file in drivers/reset/ rather than to have it in drivers/clk/. Could you perhaps move the contents of drivers/reset/amlogic/reset-meson-aux.c into drivers/clk/meson/axg-audio.c, and change the exported symbol to a static function? This would still require a dependency on the exported meson_reset_toggle_ops, but that feels like a more natural interface here, since it's just a library module. Arnd _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic