From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EF5C72209B for ; Tue, 22 Oct 2024 09:51:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729590689; cv=none; b=DfB84L/2JsrBjynRf5yYk8VpxJdH4z0ncqbv6yNXMir6lW0ox8tUtRQyqm3kJZIk36u7vOYrVjAwbeziRw3x6tgSwN7dY4erdw/tCiS2f6OiMpoiosjbLDy8mGQQWMv/utR5fg92au7I3/kQEWtjwliA9LjkXXYl5V9064rn/vc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729590689; c=relaxed/simple; bh=JsWWmLr80OvZZgtqzSLI+3NO59wkVjAXXzhBndm6rPs=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=YdHo+S7Vm2WvSH8mBQfVN5je8OCTH5QFeBWATsprFoGwLdYhbD/0xg3yVY7kiikXcQzkBAcSowhFVemdXj9XkQI8x1iqxmOcfx1RT2p0N4Ka079St0gUAmDSz/3a7MBT4H1Y2GThgxjPZ7yxffmZ/KAl4/gEBPhsOBya+aF/+rQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com; spf=pass smtp.mailfrom=baylibre.com; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b=fJbPglXI; arc=none smtp.client-ip=209.85.128.52 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=baylibre-com.20230601.gappssmtp.com header.i=@baylibre-com.20230601.gappssmtp.com header.b="fJbPglXI" Received: by mail-wm1-f52.google.com with SMTP id 5b1f17b1804b1-4316e9f4a40so28054115e9.2 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=vger.kernel.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=fJbPglXIIS2dknfgtbeunLcmjIXpFQlBmw2BrAwiOAVZu/rdZj3Y1BtqdRg7g9kyZ2 KhbNffz+hPm0J7WJIASZ6gyLBudGKgRGmBEHYcw0DaMlPYhGucp/SLET0JPqenbhLtQX qmU0f5Hh6tMoJFEY12pF2sTMa5ybFiiffOmCsMzR12mB6dvVSQVym8q8QjYFK6HjiB9R ZIi+pjGd6feUpyy9J15oOzryXiIffbLE+d+3Hh96C8UbUtykTGCBM3nw9gyOqW4BLk8l nm9zZzkdAus6FSw6wJHglp9sjPmrGF46cd2GugZ+7niwhSsjuFuc2Ya4Z6JMxorbjwnm +vyw== 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=BjLldqmnu61GnBHj/CvoA8CQirfMH7uoTkoXrGYClnKDTVOdmbYSGOYTeXxA9DGkiq oSlS8yRWBz9H7e8xf2qVBDLGI8CkdtrQnUlWV+xUwVzuzvev/KdldUr9+I0MPJXLC49G MPfqGK6MdxqDMzjVy1H6wQfm9HqJkz2OJDOQPp0fNjzqbTPajePsUUwheUlnf/XFy8hZ Q6cC8BUUhv51EV4Xt6NheKEcXXwukCLyGtNGrrGdS//PPynGWvzeCHdLNu7BfS3mn5Le 2JZYDV2/AS02Vc3W7SpHCqpmJ74yC/3kfzpD6IJdNYb3OVB4LW0eePNIxJavgxD7ZkAc aeog== X-Forwarded-Encrypted: i=1; AJvYcCXPF0dGo04QeMpJW16m+jf3NVIgekMrXusM0ybeR0IMDx6i4WNFYsTF+Q+N9JEF5XwFNj6M+HFjlWA=@vger.kernel.org X-Gm-Message-State: AOJu0YzH9AerJLniX0+3OWG1oxF/Z+raASfupfISxSNPL4oM8bcxCDfg u3rBQbwIKYR9O30ij4OcADj1/z50QPgj4LkRLnwdTLDLP4/GObfHeG3cVyggRaw= 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> Precedence: bulk X-Mailing-List: linux-clk@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain 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