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 01F2EC433EF for ; Mon, 9 May 2022 05:36:41 +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:MIME-Version:References:In-Reply-To: Date:CC:To:From:Subject:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=HP8Ntdy5pxV+Mh69xYTmP3aVEwov4Y5ghZanKR3ohHM=; b=Isc4SLtZdhJ4AP a4d6vZbtdNVjuQCYhbr23BCWazvOGIheEiuwJulwHfQ/1G6zWFk0Vde++PALf7BX5x9uCZFvrGzY3 5oflLWrAmBnGCcGLiOALO+QTgsCzGYa2q69trIPZYrMOc1cE1erVf9yX+UttrgUSpdY/Y9rPFalA8 MaD4FxhdFmhD/R5QiqCeAJNIhc7wYSZ9Ghl2zRVBFEEfFcMmUT/1exhvEfY5a6d0C/vdC6NHbFzDZ rSKBU5u8T5l+Lr23jKfoN+aGPOruEUSNNMdwD5fS/VWwoqamfJG4yCHM5XOitIfD6xROx7Bvgyonk 2mG7AbqM8nJHPUEkRbuA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nnw4S-00CXtl-0g; Mon, 09 May 2022 05:36:36 +0000 Received: from mailgw01.mediatek.com ([216.200.240.184]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nnw4F-00CXoV-Rc; Mon, 09 May 2022 05:36:25 +0000 X-UUID: 9bfde6b0ee8b48ee92a788fc9fe1a21c-20220508 X-CID-P-RULE: Release_Ham X-CID-O-INFO: VERSION:1.1.4, REQID:366b0dd2-355f-434b-a03e-08fb7414a25a, OB:0, LO B:0,IP:0,URL:0,TC:0,Content:0,EDM:0,RT:0,SF:0,FILE:0,RULE:Release_Ham,ACTI ON:release,TS:0 X-CID-META: VersionHash:faefae9, CLOUDID:58f711b3-56b5-4c9e-8d83-0070b288eb6a, C OID:IGNORED,Recheck:0,SF:nil,TC:nil,Content:0,EDM:-3,File:nil,QS:0,BEC:nil X-UUID: 9bfde6b0ee8b48ee92a788fc9fe1a21c-20220508 Received: from mtkcas66.mediatek.inc [(172.29.193.44)] by mailgw01.mediatek.com (envelope-from ) (musrelay.mediatek.com ESMTP with TLSv1.2 ECDHE-RSA-AES256-SHA384 256/256) with ESMTP id 2142957208; Sun, 08 May 2022 22:36:15 -0700 Received: from mtkmbs10n1.mediatek.inc (172.21.101.34) by MTKMBS62DR.mediatek.inc (172.29.94.18) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Sun, 8 May 2022 22:35:57 -0700 Received: from mtkcas11.mediatek.inc (172.21.101.40) by mtkmbs10n1.mediatek.inc (172.21.101.34) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.2.792.15; Mon, 9 May 2022 13:35:55 +0800 Received: from mtksdccf07 (172.21.84.99) by mtkcas11.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Mon, 9 May 2022 13:35:55 +0800 Message-ID: <445ef1e3ef9d62934e1b84d4fb207705a90d4f34.camel@mediatek.com> Subject: Re: [PATCH v6 00/16] Cleanup MediaTek clk reset drivers and support MT8192/MT8195 From: Rex-BC Chen To: "mturquette@baylibre.com" , "sboyd@kernel.org" , "matthias.bgg@gmail.com" , "robh+dt@kernel.org" , "krzysztof.kozlowski+dt@linaro.org" CC: "p.zabel@pengutronix.de" , "angelogioacchino.delregno@collabora.com" , Chun-Jie Chen =?UTF-8?Q?=28=E9=99=B3=E6=B5=9A=E6=A1=80=29?= , "wenst@chromium.org" , Runyang Chen =?UTF-8?Q?=28=E9=99=88=E6=B6=A6=E6=B4=8B=29?= , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-clk@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-mediatek@lists.infradead.org" , Project_Global_Chrome_Upstream_Group Date: Mon, 9 May 2022 13:35:55 +0800 In-Reply-To: <20220503093856.22250-1-rex-bc.chen@mediatek.com> References: <20220503093856.22250-1-rex-bc.chen@mediatek.com> X-Mailer: Evolution 3.28.5-0ubuntu0.18.04.2 MIME-Version: 1.0 X-MTK: N X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220508_223623_957219_B15AF7C5 X-CRM114-Status: GOOD ( 24.13 ) X-BeenThere: linux-mediatek@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-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org On Tue, 2022-05-03 at 17:38 +0800, Rex-BC Chen wrote: > In this series, we cleanup MediaTek clock reset drivers in > clk/mediatek > folder. MediaTek clock reset driver is used to provide reset control > of modules controlled in clk, like infra_ao. > > Changes for v6: > 1. Add a new patch to support inuput argument index mode. > 2. Revise definition in reset.h to index. > > Changes for V5: > 1. Add all infra reset bits for MT8192 and MT8195. > 2. Fix reviewers' comments. > > Changes for V4: > 1. Abandon the implementation of reset-cell = 2, and use reset index > to > determine which reset bit is used. > 2. Add documentation for enum/structure/function in reset.h. > 3. Combine binding/drvier support patch for MT8192 and MT8195. > 4. The MT8195 DTS is accepted by Matthias, and I add new DTS patch to > support infracfg_ao reset for MT8195. The DTS of MT8195 is still > not merged into mainline. Please refer to [1]. > > [1]: > https://git.kernel.org/pub/scm/linux/kernel/git/matthias.bgg/linux.git/commit/?h=for-next&id=37f2582883be7218dc69f9af135959a8e93de223 > > Changes for V3: > 1. Modify drivers for reviewers' comments. > 2. Add dt-binding patch for MT8192/MT8195 infra. > 3. Add reset property of infra node for MT8192. > 4. Use original function for simple operation. > > Changes for V2: > 1. Modify drivers for reviewers' comments. > 2. Use simple reset to replace v1. > 3. Recover v2 to set_clr. > 4. Separate error handling to another patch. > 5. Add support for input offset and bit from DT. > 6. Add support for MT8192 and MT8195. > > Rex-BC Chen (16): > clk: mediatek: reset: Add reset.h > clk: mediatek: reset: Fix written reset bit offset > clk: mediatek: reset: Refine and reorder functions in reset.c > clk: mediatek: reset: Extract common drivers to update function > clk: mediatek: reset: Merge and revise reset register function > clk: mediatek: reset: Revise structure to control reset register > clk: mediatek: reset: Support nonsequence base offsets of reset > registers > clk: mediatek: reset: Support inuput argument index mode > clk: mediatek: reset: Change return type for clock reset register > function > clk: mediatek: reset: Add new register reset function with device > clk: mediatek: reset: Add reset support for simple probe > dt-bindings: arm: mediatek: Add #reset-cells property for > MT8192/MT8195 > dt-bindings: reset: mediatek: Add infra_ao reset index for > MT8192/MT8195 > clk: mediatek: reset: Add infra_ao reset support for MT8192/MT8195 > arm64: dts: mediatek: Add infra #reset-cells property for MT8192 > arm64: dts: mediatek: Add infra #reset-cells property for MT8195 > > .../mediatek/mediatek,mt8192-sys-clock.yaml | 3 + > .../mediatek/mediatek,mt8195-sys-clock.yaml | 3 + > arch/arm64/boot/dts/mediatek/mt8192.dtsi | 1 + > arch/arm64/boot/dts/mediatek/mt8195.dtsi | 13 +- > drivers/clk/mediatek/clk-mt2701-eth.c | 10 +- > drivers/clk/mediatek/clk-mt2701-g3d.c | 10 +- > drivers/clk/mediatek/clk-mt2701-hif.c | 10 +- > drivers/clk/mediatek/clk-mt2701.c | 22 +- > drivers/clk/mediatek/clk-mt2712.c | 22 +- > drivers/clk/mediatek/clk-mt7622-eth.c | 10 +- > drivers/clk/mediatek/clk-mt7622-hif.c | 12 +- > drivers/clk/mediatek/clk-mt7622.c | 22 +- > drivers/clk/mediatek/clk-mt7629-eth.c | 10 +- > drivers/clk/mediatek/clk-mt7629-hif.c | 12 +- > drivers/clk/mediatek/clk-mt8135.c | 22 +- > drivers/clk/mediatek/clk-mt8173.c | 22 +- > drivers/clk/mediatek/clk-mt8183.c | 18 +- > drivers/clk/mediatek/clk-mt8192.c | 29 +++ > drivers/clk/mediatek/clk-mt8195-infra_ao.c | 24 +++ > drivers/clk/mediatek/clk-mtk.c | 7 + > drivers/clk/mediatek/clk-mtk.h | 9 +- > drivers/clk/mediatek/reset.c | 198 +++++++++++++--- > -- > drivers/clk/mediatek/reset.h | 82 ++++++++ > include/dt-bindings/reset/mt8192-resets.h | 8 + > include/dt-bindings/reset/mt8195-resets.h | 6 + > 25 files changed, 491 insertions(+), 94 deletions(-) > create mode 100644 drivers/clk/mediatek/reset.h > > -- > 2.18.0 > Hello Stephen and Michael, The drivers of this series are reviewed. The binding of this series are also acked. Could you spare some time and give us some suggestion? Thanks! BRs, Rex _______________________________________________ Linux-mediatek mailing list Linux-mediatek@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-mediatek