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 C6B98C77B7A for ; Tue, 30 May 2023 16:50:03 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:CC:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=V3x0rPa9OXoHh2NuzbOpyQ9oxvzAosN5KnF5lBdEyEw=; b=3D9HK74SUeLfuc OF8cHg8y1DoW9Xo8FGxyOOpxqHgNkdshP2Ub/mIT9VoH7Y2tL/TxvVjtljQY+5ryoi4xhpIthsDO5 WtRpd6Zr3yH5ctiHh8gh3uXmB2pQZbqCTWEWaQ6RbRloUbjO+l0pUKb26MFbXaxdVIo/mp3/eSBtP ms1lvsc94h+eHjGFkeA7EFycRIbJfxcRn9uXAZrj0m2w8N+CH+i7u8nTYC+mdA6t6tlAwNLXeDLoJ UfiEitORmJeun55J2iCx+FqCIBmccii3vqn3EJ3LE+aynM2FED7nU9Xhw83DJGvgcg7YOt3RhL+E1 5LZ37+EUCZD3wacznADw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1q42XV-00Edot-1R; Tue, 30 May 2023 16:49:41 +0000 Received: from mx.sberdevices.ru ([45.89.227.171]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1q42XR-00Edn0-1p; Tue, 30 May 2023 16:49:39 +0000 Received: from s-lin-edge02.sberdevices.ru (localhost [127.0.0.1]) by mx.sberdevices.ru (Postfix) with ESMTP id 9DEDB5FD3D; Tue, 30 May 2023 19:49:34 +0300 (MSK) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sberdevices.ru; s=mail; t=1685465374; bh=6QfHdz+QFGSro1lLX069r4LLDn31Yt+etPSZIg88FW0=; h=Date:From:To:Subject:Message-ID:MIME-Version:Content-Type; b=dpGK3rAAHlcU7PfwWR89EoEznPx/N8kZhkidmhRJz6yqgZGAR4nH7fsRgxqWHEqSd bzX1QR3+HMqMpVsR+0KL4BnbBKsGgfP65KUq+6QDdgCRC2ekIGjKRF2VesXq4r/WJP mbdSRqdI23TBXABczaqRHl68bhmYrbN9zydeDRnSPqVoEZPCOu+A9lU3Yc0Bj5FcZZ JxiYnhq7hO6A4sepY9aC5GjJJWdvoYuuhG/2myN+B17z28Nti/GJcNJRpRksPaZfbc tZq0W2akM9DSEXckGogDnR9Dt029wH6NGsSF7N6YYg5d0vBh+AS9lXYocbupn+K2Ew mBZXBXGllW4CA== Received: from S-MS-EXCH01.sberdevices.ru (S-MS-EXCH01.sberdevices.ru [172.16.1.4]) by mx.sberdevices.ru (Postfix) with ESMTP; Tue, 30 May 2023 19:49:33 +0300 (MSK) Date: Tue, 30 May 2023 19:49:33 +0300 From: Dmitry Rokosov To: Jerome Brunet CC: , ,, ,, ,, ,, ,, ,, , Subject: Re: [PATCH v15 0/6] add Amlogic A1 clock controller drivers Message-ID: <20230530164933.m42ojv357atxd4gc@CAB-WSD-L081021> References: <20230517133309.9874-1-ddrokosov@sberdevices.ru><1jilc94x0d.fsf@starbuckisacylon.baylibre.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <1jilc94x0d.fsf@starbuckisacylon.baylibre.com> User-Agent: NeoMutt/20220415 X-Originating-IP: [172.16.1.6] X-ClientProxiedBy: S-MS-EXCH02.sberdevices.ru (172.16.1.5) To S-MS-EXCH01.sberdevices.ru (172.16.1.4) X-KSMG-Rule-ID: 4 X-KSMG-Message-Action: clean X-KSMG-AntiSpam-Status: not scanned, disabled by settings X-KSMG-AntiSpam-Interceptor-Info: not scanned X-KSMG-AntiPhishing: not scanned, disabled by settings X-KSMG-AntiVirus: Kaspersky Secure Mail Gateway, version 1.1.2.30, bases: 2023/05/30 11:20:00 #21377521 X-KSMG-AntiVirus-Status: Clean, skipped X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230530_094937_986300_F28F0275 X-CRM114-Status: GOOD ( 26.99 ) 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: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Jerome, On Tue, May 30, 2023 at 06:14:01PM +0200, Jerome Brunet wrote: > On Wed 17 May 2023 at 16:33, Dmitry Rokosov wrote: > > > A1 SoC has four clock controllers on the board: PLL, Peripherals, CPU, > > and Audio. The audio clock controller is different from others, but the > > rest are very similar from a functional and regmap point of view. > > This patch series add support for Amlogic A1 PLL and Peripherals clock > > drivers. > > It blocks all A1 peripherals mainline support and a couple of patch series, > > which were already reviewed and acked, but weren't merged due to pending > > clock controller drivers series, e.g. > > https://lore.kernel.org/all/20230418111612.19479-1-ddrokosov@sberdevices.ru/ > > > > TODO: CPU and Audio clock controllers are not included in this patch > > series, it will be sent later. The following clks from these controllers > > are not supported for now: > > * Audio clks - vad, mclk_vad, mclk_d, resample_a, locker_in, mclk_b, > > pdmdclk, pdmsysclk, eqdrc, spdifin, mclk_a, audio2_toaudiotop, > > audio2_tovad, audio2_toddr_vad, audio2_tdmin_vad, audio2_pdm, > > audio2_ddr_arb, audio_audiolocker, audio_eqdrc, audio_resamplea, > > audio_spdifin, audio_toddrb, audio_toddra, audio_frddrb, audio_frddra, > > audio_tdmoutb, audio_tdmouta, audio_loopbacka, audio_tdminlb, > > audio_tdminb, audio_tdmina, audio_ddr_arb, mclk_c > > > > * CPU clks: cpu_fixed_source_sel0, cpu_fixed_source_div0, > > cpu_fixed_source_sel1, cpu_fixed_source_div1, cpu_clk > > > > Validation: > > * to double check all clk flags run below helper script: > > pushd /sys/kernel/debug/clk > > for f in *; do > > if [[ -f "$f/clk_flags" ]]; then > > flags="$(cat $f/clk_flags | awk '{$1=$1};1' | sed ':a;N;$!ba;s/\n/ | /g')" > > echo -e "$f: $flags" > > fi > > done > > popd > > > > * to trace current clks state use '/sys/kernel/debug/clk/clk_dump' node > > with jq post-processing: > > $ cat /sys/kernel/debug/clk/clk_dump | jq '.' > clk_dump.json > > > > * to debug clk rate propagation, compile kernel with the following > > definition: > > $ sed -i "s/undef CLOCK_ALLOW_WRITE_DEBUGFS/define CLOCK_ALLOW_WRITE_DEBUGFS/g" drivers/clk/clk.c > > after that, clk_rate debug node for each clock will be available for > > write operation > > > > Applied, Thx Thank you very much! I noticed that you have applied version 15 of the patch. If I understand correctly, this is because CLK_SET_RATE_NO_REPARENT for all RTC children is preferred from your perspective. I agree with this decision. However, version 16 includes a small refactoring patch that corrects a misstyping by changing all sys_* related dividers and gates to read-only operations, since they are set up from BootROM. Should I submit this patch separately at a later time? Also could you please clarify the official process for handling these patches? Currently, I don't see them in the linux-amlogic for-next branches. Is there a separate repository for clock changes? I believe the next repository is a fork of the clock framework, but unfortunately, I cannot seem to locate it on git.kernel.org... Apologies for the potentially simple questions... -- Thank you, Dmitry   :      ,   ,   .     ,       , , ,  ,    ,         ,  .       , ,           . CONFIDENTIALITY NOTICE: This email and any files attached to it are confidential. If you are not the intended recipient you are notified that using, copying, distributing or taking any action in reliance on the contents of this information is strictly prohibited. If you have received this email in error please notify the sender and delete this email. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel