Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751284AbdGQGQm (ORCPT ); Mon, 17 Jul 2017 02:16:42 -0400 Received: from mailgw01.mediatek.com ([210.61.82.183]:1173 "EHLO mailgw01.mediatek.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1751253AbdGQGQk (ORCPT ); Mon, 17 Jul 2017 02:16:40 -0400 Message-ID: <1500272192.16278.44.camel@mtkswgap22> Subject: Re: [PATCH v4 0/3] some fixups for MediaTek cpufreq driver From: Sean Wang To: CC: , , , , , , , , , , , , Date: Mon, 17 Jul 2017 14:16:32 +0800 In-Reply-To: References: Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.2.3-0ubuntu6 Content-Transfer-Encoding: 7bit MIME-Version: 1.0 X-MTK: N Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2458 Lines: 60 Hi, Viresh I missed to add Acks from Rob for patch 2 and 3 since we sent out almost at the same time. Do I need to resend again for this or the series is okay for you? Sean On Mon, 2017-07-10 at 22:23 +0800, sean.wang@mediatek.com wrote: > From: Sean Wang > > Changes since v3: > - remove the redundant line depends on ARM || ARM64 || COMPILE_TEST > since the whole Kconfig.arm file depends on ARM || ARM64. > - renaming from mtk-cpufreq.c to mediatek-cpufreq.c avoiding > collide and cause confusion at some point in time. > > Changes since v2: > - correct the typo in the binding document > > Changes since v1: > - drop those patches already accepted > - refine the commit messages and Kconfig dependency > - Kconfig menu entry and file name itself are updated with more > generic name to drop "MT8173" since this driver actually supports > all MediaTek SoCs. > - generate patchset again with git format-patch -C -M --thread=shallow > - fix binding examples by replacing '@' with '-' as the OPP nodes will > never have a "reg" property. > > Hi, > > The purpose of the series is > - (patch 1 to 3) to fix up current Mediatek cpufreq driver > can't work with the latest tree since one required CPU clock muxer missing > would cause the driver getting the resource fails when driver probe gets > called. > - (patch 4) to enable cpufreq feature on MT2701/MT7623 platform. > - (patch 5 to 6) to update the binding document to reflect > latest driver logic and add more examples guiding people how to apply for > Mediatek cpufreq driver. > > Sean Wang (3): > cpufreq: mediatek: Add support of cpufreq to MT2701/MT7623 SoC > dt-bindings: cpufreq: move MediaTek cpufreq dt-bindings document to > proper place > dt-bindings: cpufreq: enhance MediaTek cpufreq dt-binding document > > .../devicetree/bindings/clock/mt8173-cpu-dvfs.txt | 83 ------- > .../bindings/cpufreq/cpufreq-mediatek.txt | 247 +++++++++++++++++++++ > drivers/cpufreq/Kconfig.arm | 7 +- > drivers/cpufreq/Makefile | 2 +- > .../{mt8173-cpufreq.c => mediatek-cpufreq.c} | 2 + > 5 files changed, 253 insertions(+), 88 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/clock/mt8173-cpu-dvfs.txt > create mode 100644 Documentation/devicetree/bindings/cpufreq/cpufreq-mediatek.txt > rename drivers/cpufreq/{mt8173-cpufreq.c => mediatek-cpufreq.c} (99%) >