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 18F30D1CDD1 for ; Tue, 22 Oct 2024 10:16:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Type:MIME-Version: Message-ID:Date:References:In-Reply-To:Subject:Cc:To:From:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=O9uhSsPjuLDUWqYOIT4n8318oMtxdXxpij17XIrTsXE=; b=PEG/x2Eelw1e6YZG/7Rkd6K4Cy HeO6dF64xpZVlUIj96Pu6vr+tzs38NXDcOzVA/wF4bBzl3g7EOsnBc1X3L94UMxI/zt6niyngqLE+ DAj9AX3yHyK3mxrHTXivMH28aQrG8brPyZyCl3qep0vH/auspVCKvPQGZstdAfJqg/fQkPYpku5Zh QCsphQ6+oTQpWzjeYQKR3h3HBx269y8czCyOLQDjExBudS9BINcrjP0GT5J1/liEt1/jqvZJ8ddC0 ijivy/dhIM/Q7Mw+JQ0xNwSwTE/2KmP3spFSj/ivoBP50yoe+SIkZyid2ya2R9x92qzjBKRaxn0nP mSYMphlA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1t3Bve-0000000AUrM-0ucR; Tue, 22 Oct 2024 10:15:54 +0000 Received: from mail-wm1-x330.google.com ([2a00:1450:4864:20::330]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1t3BXz-0000000AQpT-0cXt for linux-arm-kernel@lists.infradead.org; Tue, 22 Oct 2024 09:51:28 +0000 Received: by mail-wm1-x330.google.com with SMTP id 5b1f17b1804b1-43168d9c6c9so33908145e9.3 for ; Tue, 22 Oct 2024 02:51:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20230601.gappssmtp.com; s=20230601; t=1729590685; x=1730195485; darn=lists.infradead.org; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:from:to:cc:subject:date:message-id:reply-to; bh=O9uhSsPjuLDUWqYOIT4n8318oMtxdXxpij17XIrTsXE=; b=IBLsnWKGciUmNJn1qtxqhwsIhgk2LA5qPA7ESyDHjlocARVRMBUfB58uFPLytY6B/4 BjroekuQo83zcMhAxIjOdGpPfGefyL+LGNq9ZsfButFCLgNL1/lSsjrHFvma7SVMZpSi pFWCK8ojRa+ibR6sEb4qAjWKo3MHA71fCQNjVFmKYLsAdwA6CTp+TelfuEb6Ey+JluBi Jh7FeyvYxZp4vgiMAvKHJZFR5ZPthVATNddnLz2KkcbRTEanB9Wgb+vE8hRWx5mKcvmn en2tlwUVh2FKOjsMd1RNXQCc9gl8Swz3ONtO685UZzSzQyiOB3+gRSNN272Ul/lqb171 oinA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1729590685; x=1730195485; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=O9uhSsPjuLDUWqYOIT4n8318oMtxdXxpij17XIrTsXE=; b=ADbaoJd10AaF+zhEQ+nUB0ig2bN6NDq811rPve1mPJIz2VnlBJ6IkfvO6G3pvVLp4P uU6D5htygAIrqDy7P03sGa1T0FoHU5X3w4YKHhS44EbcYDNYoDbTrJl2Yaiby0zzPmn7 cH2QP0nr1T9eZ7cteFoXgobHnFkxmi/9LjQ/2WoZKN6G7IWk0aMXpBjqLqvJCJrz9d6y iCJcu0OWdtjT/uX9mRSTTbs/YKYGVYsPerCIRxcqb6g9jKnj5e/RqWsfNcqwh4eJ0Hrk kmld7sNNK/BJxQJww/LLi9UZIJLsrjbu5kujbfDcnOyRWnAzJeyKevR7EkV5/x+W/Qct +lbQ== X-Forwarded-Encrypted: i=1; AJvYcCV81v1TMcNEjBNykD+o1BG5hJMMwn25rNkjybdAb/+q+fk5GbqJwF+FFaVbKU0p8ZYd6UJdWDXeMSX2jn3cPqbc@lists.infradead.org X-Gm-Message-State: AOJu0YzWDvb9oSkaIJbGMKU6+jIPRHdOoHx2TbP9AtKxGzjpc9UEYRkN p6YijeHWxzdB1PfB3Q7EjhiK7CVvky/z1ts2thwXawAzYyW5gX8Ne6RRdIGsMXc= X-Google-Smtp-Source: AGHT+IG4BjZjQ2Z2dqGHg8e13po3OOXPlOkZEZh2gmQz0kEl9ezHR/+UkwbK17+DbKxR73ieabZKcw== X-Received: by 2002:a05:600c:6747:b0:430:5356:ac92 with SMTP id 5b1f17b1804b1-4316161fb1dmr109088635e9.7.1729590685185; Tue, 22 Oct 2024 02:51:25 -0700 (PDT) Received: from localhost ([2a01:e0a:3c5:5fb1:ef1c:ae40:1300:20c6]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-4316f570de0sm83095085e9.6.2024.10.22.02.51.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 22 Oct 2024 02:51:24 -0700 (PDT) From: Jerome Brunet To: Jan Dakinevich Cc: Conor Dooley , , Kevin Hilman , "Krzysztof Kozlowski" , , , , , Martin Blumenstingl , Michael Turquette , Neil Armstrong , Philipp Zabel , Rob Herring , Stephen Boyd Subject: Re: [RFC PATCH v4 0/5] Add A1 Soc audio clock controller driver In-Reply-To: <20240913121152.817575-1-jan.dakinevich@salutedevices.com> (Jan Dakinevich's message of "Fri, 13 Sep 2024 15:11:47 +0300") References: <20240913121152.817575-1-jan.dakinevich@salutedevices.com> Date: Tue, 22 Oct 2024 11:51:24 +0200 Message-ID: <1jplnsjwir.fsf@starbuckisacylon.baylibre.com> MIME-Version: 1.0 Content-Type: text/plain X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20241022_025127_205313_418CD8A4 X-CRM114-Status: GOOD ( 17.94 ) 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 On Fri 13 Sep 2024 at 15:11, Jan Dakinevich wrote: > This series adds support for audio clock and reset controllers on A1 SoC family. > Split the reset part out of the series (I'd suggest adding VAD reset support while at it). Also remove the DT patch, since it will depends on both patchset. Drop the RFC tag, at v4 I think that phase is over. > Dependency: [4] > > Changes v3 [3] -> v4 > - Use auxiliary reset device implemented in [4] > - Split the driver into files > - Use common with axg-audio yaml schema > - Unify clock-names with axg-audio > > Changes v2 [2] -> v3 > - reset: > * added auxiliary device > - yaml: > * added declaration of optional clocks > * fixed names in example and another cosmetics > - clocks: > * reworked naming > * stop using of "core" clock name > * fixed wrong parenting > > Changes v1 [1] -> v2: > - Detached from v1's series (patch 2, 3, 4, 25). > - Reuse some of defines from axg-audio; > - Split the controller into two memory regions. > > Links: > [1] https://lore.kernel.org/lkml/20240314232201.2102178-1-jan.dakinevich@salutedevices.com/ > [2] https://lore.kernel.org/lkml/20240328010831.884487-1-jan.dakinevich@salutedevices.com/ > [3] https://lore.kernel.org/lkml/20240419125812.983409-1-jan.dakinevich@salutedevices.com/ > [4] https://lore.kernel.org/lkml/9a4377fe27d8eb940399e452b68fb5a6d678929f.camel@pengutronix.de/ > > Jan Dakinevich (5): > reset: amlogic: add support for A1 SoC in auxiliary reset driver > clk: meson: axg: share the set of audio helper macro > dt-bindings: clock: axg-audio: document A1 SoC audio clock controller > driver > clk: meson: a1: add the audio clock controller driver > arm64: dts: meson: a1: add the audio clock controller > > .../clock/amlogic,axg-audio-clkc.yaml | 3 + > arch/arm64/boot/dts/amlogic/meson-a1.dtsi | 48 +++ > drivers/clk/meson/Kconfig | 14 + > drivers/clk/meson/Makefile | 3 + > drivers/clk/meson/a1-audio-clkc.c | 359 ++++++++++++++++++ > drivers/clk/meson/a1-audio-drv.c | 104 +++++ > drivers/clk/meson/a1-audio-vad-clkc.c | 85 +++++ > drivers/clk/meson/a1-audio.h | 131 +++++++ > drivers/clk/meson/axg-audio.c | 138 +------ > drivers/clk/meson/meson-audio.h | 143 +++++++ > drivers/reset/amlogic/reset-meson-aux.c | 9 + > .../dt-bindings/clock/amlogic,a1-audio-clkc.h | 122 ++++++ > .../reset/amlogic,meson-a1-audio-reset.h | 29 ++ > 13 files changed, 1051 insertions(+), 137 deletions(-) > create mode 100644 drivers/clk/meson/a1-audio-clkc.c > create mode 100644 drivers/clk/meson/a1-audio-drv.c > create mode 100644 drivers/clk/meson/a1-audio-vad-clkc.c > create mode 100644 drivers/clk/meson/a1-audio.h > create mode 100644 drivers/clk/meson/meson-audio.h > create mode 100644 include/dt-bindings/clock/amlogic,a1-audio-clkc.h > create mode 100644 include/dt-bindings/reset/amlogic,meson-a1-audio-reset.h -- Jerome