Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp330746imm; Tue, 7 Aug 2018 20:10:00 -0700 (PDT) X-Google-Smtp-Source: AA+uWPykVhW+yqs2d71BfyXjFJK1waI28x8zUwJaG+AJjZByGmNrtcGS+DP1whCZAj83jkQ3h4OY X-Received: by 2002:a63:ec43:: with SMTP id r3-v6mr781847pgj.295.1533697800746; Tue, 07 Aug 2018 20:10:00 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1533697800; cv=none; d=google.com; s=arc-20160816; b=WuwCWiUnUBz7GSqOheBiV4xv2wtut5n8smx3w4WD21T4Y4jCXmVFLKni2FHlsmO8Ny jeLhtgpBtE0vQXlvF9GtdqKReZbcZEjheeOqr1HVf2SefZErJSxMaZTeyXlbG9XGfW24 vIP2Gu6c5uG3BOleQLh6ntW17w312dFm4v4m//Az3H5x4SYRQco5ye0AFLRrnNuov221 GayV3xhBuOY5w4DRh+DAX6C8hROeGDt1t9nSK/0WwcGcBl+Ddz37nJfhOrMctUz4mj2/ jCFVSBaWDsdP1SGEovv6aFVMz5fa3qSmf9lYZm2YLr1xt/q5a9xlfFUxAa3iKSfuqn24 xx9g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-language :content-transfer-encoding:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:arc-authentication-results; bh=clRNg9uqtWCChL/8tgI1+Vu1mrISOtc5Y6X0fTvP2eQ=; b=uiffczcoUiVYD6MzHSJA6jrqVHZvwo+RCi80KHjfHyqxBukUTLm6AQcBKCZVTyiasj 6GSeT/ZCALvjGLDbBbHWsz58FRZzRXo8oHdCHfMupRlanPTkDx2+JqsD+tAVdyQRADkt vV6xHYyn5I+Q2/9k8ZbS6w1Z5YiJFUiYfzctVTyT5ZycLyAYbulGiQvEQKEwrLW0QAI+ M813LxFZSfH4veQu0yKt1+hc+vm7wLMnqIDpyvEhfYdJE1SuuWYuOSxET0daXDLX0uNw r4ZOjqg4u/y3e5zFC+wLPi8f/ImpkL2+e2JqcbC6KFWdek1hiWoh9CoxgAMDuHRDbgCA babw== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id b33-v6si2835643pgb.467.2018.08.07.20.09.45; Tue, 07 Aug 2018 20:10:00 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726990AbeHHF0U (ORCPT + 99 others); Wed, 8 Aug 2018 01:26:20 -0400 Received: from mga03.intel.com ([134.134.136.65]:17443 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726409AbeHHF0U (ORCPT ); Wed, 8 Aug 2018 01:26:20 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 Aug 2018 20:08:52 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.51,456,1526367600"; d="scan'208";a="79457375" Received: from zhuyixin-mobl1.gar.corp.intel.com (HELO [10.226.38.39]) ([10.226.38.39]) by fmsmga001.fm.intel.com with ESMTP; 07 Aug 2018 20:08:49 -0700 Subject: Re: [PATCH v2 03/18] dt-bindings: clk: Add documentation of grx500 clock controller To: Rob Herring , Songjun Wu Cc: hua.ma@linux.intel.com, chuanhua.lei@linux.intel.com, qi-ming.wu@intel.com, Linux-MIPS , linux-clk , "open list:SERIAL DRIVERS" , devicetree@vger.kernel.org, Michael Turquette , Stephen Boyd , "linux-kernel@vger.kernel.org" , Mark Rutland References: <20180803030237.3366-1-songjun.wu@linux.intel.com> <20180803030237.3366-4-songjun.wu@linux.intel.com> From: yixin zhu Message-ID: <9c0cbdfa-8109-be0d-8e14-7d303c764f5c@linux.intel.com> Date: Wed, 8 Aug 2018 11:08:48 +0800 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 8/6/2018 11:18 PM, Rob Herring wrote: > On Thu, Aug 2, 2018 at 9:03 PM Songjun Wu wrote: >> From: Yixin Zhu >> >> This patch adds binding documentation for grx500 clock controller. >> >> Signed-off-by: YiXin Zhu >> Signed-off-by: Songjun Wu >> --- >> >> Changes in v2: >> - Rewrite clock driver's dt-binding document according to Rob Herring's >> comments. >> - Simplify device tree docoment, remove some clock description. >> >> .../devicetree/bindings/clock/intel,grx500-clk.txt | 39 ++++++++++++++++++++++ > Please match the compatible string: intel,grx500-cgu.txt Will update to use same name. > >> 1 file changed, 39 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/clock/intel,grx500-clk.txt >> >> diff --git a/Documentation/devicetree/bindings/clock/intel,grx500-clk.txt b/Documentation/devicetree/bindings/clock/intel,grx500-clk.txt >> new file mode 100644 >> index 000000000000..e54e1dad9196 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/clock/intel,grx500-clk.txt >> @@ -0,0 +1,39 @@ >> +Device Tree Clock bindings for grx500 PLL controller. >> + >> +This binding uses the common clock binding: >> + Documentation/devicetree/bindings/clock/clock-bindings.txt >> + >> +The grx500 clock controller supplies clock to various controllers within the >> +SoC. >> + >> +Required properties for clock node >> +- compatible: Should be "intel,grx500-cgu". >> +- reg: physical base address of the controller and length of memory range. >> +- #clock-cells: should be 1. >> + >> +Optional Propteries: >> +- intel,osc-frequency: frequency of the osc clock. >> +if missing, driver will use clock rate defined in the driver. > This should use a fixed-clock node instead. Yes, This is a fixed clock node registered in driver code. The frequency of the fixed clock is designed to be overwritten by device tree in case some one verify clock driver in the emulation platform or in some cases frequency other than driver defined one is preferred. These kinds of cases are very rare. But I feel it would be better to have a way to use customized frequency. The frequency defined in device tree will overwritten driver defined frequency before registering fixed-clock node. >> + >> +Example: Clock controller node: >> + >> + cgu: cgu@16200000 { >> + compatible = "intel,grx500-cgu", "syscon"; >> + reg = <0x16200000 0x200>; >> + #clock-cells = <1>; >> + }; >> + >> + >> +Example: UART controller node that consumes the clock generated by clock >> + controller. >> + >> + asc0: serial@16600000 { >> + compatible = "lantiq,asc"; >> + reg = <0x16600000 0x100000>; >> + interrupt-parent = <&gic>; >> + interrupts = , >> + , >> + ; >> + clocks = <&cgu CLK_SSX4>, <&cgu GCLK_UART>; >> + clock-names = "freq", "asc"; >> + }; >> -- >> 2.11.0 >>